Repository navigation
docs(relay): document the account-token creation permission - #14995
dannyclifford wants to merge 1 commit into
Conversation
Document the account-level permission needed to create the relay's runtime token. Explain why successful updates to an existing stage do not establish that the deployment token can provision a new stage. Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
ApprovabilityVerdict: Approved at Macroscope's review found this PR approvable — This PR only adds deployment-permission guidance to an existing relay README. It does not modify executable code, configuration, product defaults, or static-analysis behavior, so it has no runtime blast radius. Notes:
You can add or adjust custom eligibility rules. Learn more. |
Dismissing prior approval to re-evaluate 0f19e34
|
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
📒 Files selected for processing (1)
Included review availability: This review used your included allowance. Your plan provides up to 10 included reviews per hour; 9 remain after this review. 📝 WalkthroughWalkthroughThe relay deployment README now explains the Worker’s account-owned runtime token, the permission required to create it, and a possible ChangesRelay deployment guidance
Priority: ➖ Normal Estimated code review effort: 1 (Trivial) | ~3 minutes Change: Other Suggested reviewers: Merge Risk: ⚪ Minimal · up to The permission guidance can merge with normal documentation checks; no actionable deployment risk has been established. 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
Comment |
Problem
The relay deployment guide omits the permission needed to create its account-owned runtime
token. The resulting authorization failure was previously
reported in #10652.
Change
Name the account-level permission, distinguish it from the user-level permission, and explain
why successful updates to an existing stage do not establish that a new stage can be deployed.
Scope and approval
Small documentation fix for a missing prerequisite of the existing deployment workflow,
submitted under the obvious-fix exception. No runtime behavior changes.
Verification
Manually ran
alchemy deploy --stage prodon a separate Cloudflare account using Alchemy2.0.0-beta.79on macOS 26.6 arm64. The checkout was based on upstreamde34391427, withonly PlanetScale sizing changed in
src/db.tsto PS_5 without replicas; Worker and tokenbindings were unchanged.
ApiTokenfailed withUnauthorized.This run also had unrelated PlanetScale billing failures.
API Tokens: Edit: database provisioningsucceeded, but
ApiTokenstill failed.Account API Tokens: Edit: the token and Worker were created;deployment reported
Done: 5 succeeded.Traced token creation and reuse through the relay bindings and Alchemy's account-token
provider. Reuse was checked in source, not by a deployment test. The successful credential
retained user-level token-edit permission; this is not a complete or minimal permission audit.
No automated tests were run for this docs-only change.
Original draft: Claude Fable 5.1 in T3 Code.
Review and final wording: GPT-6-Astra (Codex) in T3 Code.
🤖 Generated with Claude Code