Skip to content

feat(cf-v2): Native Preview uses the shared CLOUDFLARE_API_TOKEN/ACCOUNT_ID - #87

Merged
ClaudiaFang merged 1 commit into
mainfrom
claude/cf-v2-drop-preview-token-65abc6
Oct 30, 2026
Merged

ClaudiaFang merged 1 commit into
mainfrom
claude/cf-v2-drop-preview-token-65abc6

Conversation

@ClaudiaFang

Copy link
Copy Markdown
Member

Removes the dedicated CLOUDFLARE_PREVIEW_API_TOKEN / CLOUDFLARE_PREVIEW_ACCOUNT_ID from _cf-worker-v2.yml and _cf-worker-preview-gc.yml. Preview (non-main) now uses CLOUDFLARE_API_TOKEN / CLOUDFLARE_ACCOUNT_ID, same as Production and Dev (precedent: #76). No deprecated alias, no fallback.

  • Migration / Deploy / Rollback env: plain secrets.CLOUDFLARE_* on every ref.
  • Preview GC: CLOUDFLARE_API_TOKEN / CLOUDFLARE_ACCOUNT_ID now required.
  • Missing-credential guard (engine.sh) unchanged in behaviour: fails closed with "credentials not provided (set CLOUDFLARE_API_TOKEN and CLOUDFLARE_ACCOUNT_ID)"; Smoke still must not receive credentials.
  • Docs, openspec and contract/job tests updated.

Breaking: callers that still pass the old CLOUDFLARE_PREVIEW_* secrets will be rejected by GitHub (undeclared secret) and must drop them; Preview GC callers must pass the account pair.

Tests: contract, test-jobs, deploy/migrate/smoke/parse-env/probe/gh, node unit tests pass locally. actionlint/shellcheck not installed locally; left to CI.

🤖 Generated with Claude Code

…UNT_ID

Remove the dedicated CLOUDFLARE_PREVIEW_API_TOKEN / CLOUDFLARE_PREVIEW_ACCOUNT_ID
from _cf-worker-v2 and _cf-worker-preview-gc (no deprecated alias, no fallback),
as PR #76 did for the Dev token. Docs, openspec and contract tests updated.

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
@ClaudiaFang
ClaudiaFang merged commit 5def01d into main Oct 30, 2026
1 check passed
@ClaudiaFang
ClaudiaFang deleted the claude/cf-v2-drop-preview-token-65abc6 branch October 30, 2026 04:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant