Skip to content

fix(docs): use production worker name - #161

Merged
mynameistito merged 5 commits into
mainfrom
fix/docs-production-worker-name
Oct 2, 2026
Merged

mynameistito merged 5 commits into
mainfrom
fix/docs-production-worker-name

Conversation

@mynameistito

@mynameistito mynameistito commented Oct 2, 2026 •

Copy link
Copy Markdown
Owner

Summary

  • Set the production Worker name to opencode-plugins-docs-prod and keep preview names stage-specific.
  • Keep docs navigation relative so preview deployments stay on their own Worker.
  • Update the lychee v0.24.2 workflow, check homepage routes against the production base URL, and exclude only specific URLs that reject or intermittently fail lychee requests.
  • Add a patch Changeset for the docs package.

The production deploy has succeeded with the new name, and the site responds with HTTP 200. The apps/web deploy script is unchanged.

Checks

  • bun run check
  • bun run --filter @mynameistito/opencode-plugins-docs typecheck
  • bun run --filter @mynameistito/opencode-plugins-docs test
  • Lychee v0.24.2 checks for docs links and homepage routes
  • bunx changeset status

@changeset-bot

changeset-bot Bot commented Oct 2, 2026 •

Copy link
Copy Markdown

🦋 Changeset detected

Latest commit: 92a0643

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
@mynameistito/opencode-plugins-docs Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@coderabbitai

coderabbitai Bot commented Oct 2, 2026 •

Copy link
Copy Markdown

Review in Change Stack →

Navigate logical layers of code changes, visualize relationships, and explore their blast radius.

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: ASSERTIVE

Plan: Advanced

Run ID: dc95abc0-497b-467b-81b3-8f46b081afc5

📥 Commits

Reviewing files that changed from the base of the PR and between 2a46821 and 92a0643.

📒 Files selected for processing (6)
  • .changeset/d03cc118.md
  • .github/workflows/link-check.yml
  • .lycheeignore
  • README.md
  • apps/web/alchemy.run.ts
  • apps/web/docs/usage-limits.mdx

Included review availability: This review used your included allowance. Your plan provides up to 1 included review per hour; 0 remain after this review.


📝 Summary

Summary by CodeRabbit

  • Documentation
    • Updated the MiniMax Token Plan links in the README and usage limits guide to point to current destinations.
    • Refined documentation link checks to validate the homepage against the production site and skip selected external links that cannot be reliably checked.
  • Chores
    • Updated the production documentation site’s deployment name; non-production site names remain unchanged.

Walkthrough

The production docs Worker name changes. The link-check workflow adjusts its exclusions and adds a production homepage check. The README and usage-limits documentation use updated MiniMax Token Plan links.

Changes

Production Worker name

Layer / File(s) Summary
Production resource naming
apps/web/alchemy.run.ts, .changeset/d03cc118.md
The production resource name changes to opencode-plugins-docs-prod. The changeset records the Worker name update.

Documentation links and checks

Layer / File(s) Summary
Link-check workflow and exclusions
.github/workflows/link-check.yml, .lycheeignore
The workflow excludes the docs homepage from the pull-request check and adds a production homepage check for non-pull-request events. Six URL patterns are added to the Lychee ignore list.
MiniMax Token Plan links
README.md, apps/web/docs/usage-limits.mdx
The MiniMax Token Plan links point to updated URLs.

Priority: ⬇️ Low

Estimated code review effort: 2 (Simple) | ~10 minutes

Change: Bug fix

Merge Risk: ⚪ Minimal · up to 92a06

Preview and production deployments resolve their intended Worker names. No actionable merge-blocking risk is established by the reviewed changes.

Security Architecture Review

Security architecture risk: 🔵 Low · up to 92a06

The change is limited to the documentation site's production resource name. Preview naming remains compatible, and the public domain is unchanged. No introduced security issue was established, but cleanup of the previous resource and recovery from an interrupted deployment remain unverified.

Retained concerns
No architecture-level concerns identified.

Security review details

Security Blast Radius

  • inferred — The direct changed-resource scope is the production documentation Worker serving the existing public domain. Maximum account-wide deployment authority cannot be bounded without the effective Cloudflare token permissions.

Trust Boundaries and Controls

  • observed — Preview deployment crosses from pull-request-controlled code into Cloudflare-authorized execution. The untrusted-author job declares the docs-preview environment; the trusted-author job rechecks repository write-level access. Actual environment approval settings are not supplied.
  • observed — The pinned action passes Cloudflare credentials to the consumer command but clears GITHUB_TOKEN in that command's environment, separating deployment credentials from GitHub orchestration authority.

Hardening Proposals

  • proposed — Validate the rename against persisted deployment state and the live Worker inventory: confirm ownership of the production domain, disposition of the old Worker, and recovery after interrupted, repeated, or reversed deployment. Treat a successful HTTP response as a serving check, not proof of cleanup or rollback correctness.
🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly identifies the primary change: using the production Worker name for the documentation site.
Description check ✅ Passed The description clearly summarizes the production Worker rename, preview behavior, link-check updates, validation, and Changeset. It omits the template's Related issue, explicit docs-only compatibilit…
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 1…
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
✨ Finishing Touches
📝 Generate docstrings
  • Commit to this branch
  • Create a new PR
🧪 Generate unit tests (beta)
  • Commit to this branch
  • Create a new PR
  • Autopilot · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Autopilot is currently an internal CodeRabbit preview.


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

A rabbit checked each link at dawn
Then hopped to where the new names spawn
The homepage got its checking turn
The MiniMax links now point in turn
The burrow hums; the checks are done

Comment @coderabbitai help to get the list of available commands.

@github-actions github-actions Bot added docs Automatically managed pull request metadata release Automatically managed pull request metadata size/xs Automatically managed pull request metadata labels Oct 2, 2026
@codecov

codecov Bot commented Oct 2, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@cubic-dev-ai cubic-dev-ai Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No issues found across 2 files

Re-trigger cubic

@github-actions

github-actions Bot commented Oct 2, 2026 •

Copy link
Copy Markdown
Contributor

Deploying with Alchemy Alchemy

The latest deployment for this pull request.

Status Name Latest commit Updated (UTC)
Deployment successful
View Cloudflare logs
opencode-plugins-docs-pr-161 92a06430 2026-10-02T08:45:26.170Z

Diagnostics: View GitHub Actions run

@github-actions github-actions Bot added the github-actions Automatically managed pull request metadata label Oct 2, 2026
@github-actions github-actions Bot added size/s Automatically managed pull request metadata and removed size/xs Automatically managed pull request metadata labels Oct 2, 2026

@cubic-dev-ai cubic-dev-ai Bot left a comment •

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

All reported issues were addressed across 4 files (changes from recent commits).

Tip: Review your code locally with the cubic CLI to iterate faster.

Re-trigger cubic

Comment thread .github/workflows/link-check.yml Outdated
Comment thread apps/web/docs/index.mdx Outdated

@cubic-dev-ai cubic-dev-ai Bot left a comment •

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

All reported issues were addressed across 3 files (changes from recent commits).

Tip: Review your code locally with the cubic CLI to iterate faster.

Re-trigger cubic

Comment thread .github/workflows/link-check.yml
Comment thread .github/workflows/link-check.yml
@mynameistito
mynameistito merged commit 74d73bf into main Oct 2, 2026
18 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

docs Automatically managed pull request metadata github-actions Automatically managed pull request metadata release Automatically managed pull request metadata size/s Automatically managed pull request metadata

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant