chore: migrate wrangler.toml to wrangler.jsonc#25
Conversation
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
|
@coderabbitai review |
|
Warning Rate limit exceeded
⌛ How to resolve this issue?After the wait time has elapsed, a review can be triggered using the We recommend that you space out your commits to avoid hitting the rate limit. 🚦 How do rate limits work?CodeRabbit enforces hourly rate limits for each developer per organization. Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout. Please see our FAQ for further information. ℹ️ Review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (6)
✨ Finishing Touches🧪 Generate unit tests (beta)
📝 Coding Plan
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. Comment |
There was a problem hiding this comment.
Pull request overview
Migrates Cloudflare Workers Wrangler configuration from TOML to JSONC to enable comment support while keeping the same deployment settings (routes, bindings, env vars) for the affected workers.
Changes:
- Replaced
wrangler.tomlwithwrangler.jsoncforproject-awareness/cloudflare-worker. - Replaced
wrangler.tomlwithwrangler.jsoncforchitty-marketplace-skillandchitty-marketplace-skill/backend. - Preserved existing routes, vars, and bindings in the new JSONC configs.
Reviewed changes
Copilot reviewed 6 out of 6 changed files in this pull request and generated 2 comments.
Show a summary per file
| File | Description |
|---|---|
| project-awareness/cloudflare-worker/wrangler.toml | Removed TOML config in favor of JSONC. |
| project-awareness/cloudflare-worker/wrangler.jsonc | Added JSONC worker configuration equivalent to prior TOML. |
| chitty-marketplace-skill/wrangler.toml | Removed TOML config in favor of JSONC. |
| chitty-marketplace-skill/wrangler.jsonc | Added JSONC worker configuration equivalent to prior TOML. |
| chitty-marketplace-skill/backend/wrangler.toml | Removed TOML config in favor of JSONC. |
| chitty-marketplace-skill/backend/wrangler.jsonc | Added JSONC worker configuration equivalent to prior TOML. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| // chittyops-project-awareness — Cloudflare Workers configuration | ||
| { | ||
| "name": "chittyops-project-awareness", | ||
| "main": "src/worker.js", |
| "name": "chittyops-project-awareness", | ||
| "main": "src/worker.js", | ||
| "compatibility_date": "2024-08-29", | ||
| "compatibility_flags": [ | ||
| "nodejs_compat" | ||
| ], |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: f65db26bac
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| @@ -0,0 +1,95 @@ | |||
| // chittyops-project-awareness — Cloudflare Workers configuration | |||
There was a problem hiding this comment.
Keep wrangler.toml until deploy script accepts JSONC
Switching this worker to wrangler.jsonc while deleting wrangler.toml breaks the existing deployment flow in project-awareness/deployment/deploy-cloudflare-optimized.sh: validate_prerequisites() exits if $WORKER_DIR/wrangler.toml is missing (lines 68–70), so deployments now fail before any setup runs. In the current repo state, this migration is incomplete unless that script is updated to accept wrangler.jsonc (or a compatibility wrangler.toml is retained).
Useful? React with 👍 / 👎.
Summary
wrangler.tomltowrangler.jsoncformatTest plan
npx wrangler deploy --dry-runsucceeds with new config🤖 Generated with Claude Code