Skip to content

Adopt Wrangler secrets config property#63

Draft
kentcdodds wants to merge 1 commit intomainfrom
cursor/secrets-config-property-6252
Draft

Adopt Wrangler secrets config property#63
kentcdodds wants to merge 1 commit intomainfrom
cursor/secrets-config-property-6252

Conversation

@kentcdodds
Copy link
Copy Markdown
Member

Summary

  • adopt Cloudflare's secrets.required Wrangler config for the app worker
  • preserve local development by forwarding optional .env values through wrangler-env.ts
  • strengthen generated preview/production configs so CI deploy validation requires AI_GATEWAY_ID

Testing

  • bun test shared/ai-env-validation.test.ts worker/ai-runtime.test.ts
  • bun run typecheck
  • bun run generate-types
  • bun tools/ci/production-resources.ts ensure --out-config wrangler-production.generated.json --dry-run && bun tools/ci/preview-resources.ts ensure --worker-name epicflare-pr-999 --out-config wrangler-preview.generated.json --dry-run

Notes

  • checked-in wrangler.jsonc only declares COOKIE_SECRET as universally required to avoid forcing optional integrations during local dev
  • generated preview/production configs add AI_GATEWAY_ID so deploy-time validation matches remote AI requirements
Open in Web Open in Cursor 

Co-authored-by: Kent C. Dodds <me+github@kentcdodds.com>
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