Skip to content

Worker name mismatch#9

Merged
kentcdodds merged 3 commits into
mainfrom
cursor/worker-name-mismatch-6d0b
Feb 7, 2026
Merged

Worker name mismatch#9
kentcdodds merged 3 commits into
mainfrom
cursor/worker-name-mismatch-6d0b

Conversation

@kentcdodds

@kentcdodds kentcdodds commented Feb 7, 2026

Copy link
Copy Markdown
Member

Set production worker name explicitly and target secrets to the production environment to resolve COOKIE_SECRET undefined error.

The previous configuration caused a mismatch where wrangler would automatically suffix the production worker name with -production (e.g., epicflare-production), but secrets were being put into the base worker name (e.g., epicflare). This resulted in the COOKIE_SECRET being unavailable to the deployed production worker. This change ensures the production worker name is consistent and secrets are correctly associated.


Open in Cursor Open in Web


Note

Low Risk
Workflow-only change that adjusts where a secret is written; limited blast radius but could still break deploys if the env name/config is incorrect.

Overview
Fixes production deploy secret scoping by updating the GitHub Actions workflow to write COOKIE_SECRET via wrangler secret put ... --env production, ensuring the deployed worker can read the secret.

Also tweaks docs/agents/setup.md formatting for the local .env setup instruction (line wrap only).

Written by Cursor Bugbot for commit 17a66da. This will update automatically on new commits. Configure here.

cursoragent and others added 2 commits February 7, 2026 07:22
Co-authored-by: Kent C. Dodds <me+github@kentcdodds.com>
Co-authored-by: Kent C. Dodds <me+github@kentcdodds.com>
@cursor

cursor Bot commented Feb 7, 2026

Copy link
Copy Markdown

Cursor Agent can help with this pull request. Just @cursor in comments and I'll start working on changes in this branch.
Learn more about Cursor Agents

@kentcdodds kentcdodds marked this pull request as ready for review February 7, 2026 07:27
Comment thread wrangler.jsonc Outdated
Comment on lines +19 to +20
// Keep this in sync with top-level "name" to avoid "-production".
"name": "epicflare",

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

@cursor, didn't realize this was how it's done by default with wrangler. Let's remove this line and update our workflow to handle the -production env.

@cursor cursor Bot Feb 7, 2026

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Summary

  • Removed the production name override so Wrangler uses the default {name}-production worker, while the deploy workflow continues to target production secrets via --env production.【F:wrangler.jsonc†L1-L46】【F:.github/workflows/deploy.yml†L134-L141】

Testing

  • bun run validate (full gate required by repo; fails in typecheck due to duplicate @modelcontextprotocol/sdk types in agents, unrelated to this change).【ca05b7†L1-L49】
  • Manual testing not run because this is a deploy config-only change with no UI/runtime behavior to exercise locally.

Open in Cursor Open in Web

Co-authored-by: Kent C. Dodds <me+github@kentcdodds.com>
@kentcdodds kentcdodds merged commit 0d0bf80 into main Feb 7, 2026
7 checks passed
@kody-bot kody-bot deleted the cursor/worker-name-mismatch-6d0b branch March 24, 2026 21:05
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.

2 participants