Skip to content

Migrate wrangler.toml to wrangler.jsonc#25

Open
JackBinswitch-btc wants to merge 1 commit intoaibtcdev:mainfrom
JackBinswitch-btc:feat/wrangler-jsonc
Open

Migrate wrangler.toml to wrangler.jsonc#25
JackBinswitch-btc wants to merge 1 commit intoaibtcdev:mainfrom
JackBinswitch-btc:feat/wrangler-jsonc

Conversation

@JackBinswitch-btc
Copy link
Contributor

Summary

  • Converts wrangler.toml to wrangler.jsonc format for ecosystem consistency across AIBTC projects
  • All existing settings preserved: project name (signal-dashboard), build output dir (./public), KV namespace binding (SIGNAL_KV)
  • Added inline JSONC comments documenting each configuration section

Closes #22

Changes

File Action
wrangler.jsonc Created — faithful conversion from TOML with inline comments
wrangler.toml Deleted

Test plan

  • Verify wrangler pages dev works with the new config file
  • Verify wrangler pages deploy works with the new config file
  • Confirm KV binding SIGNAL_KV is available in Pages Functions

🤖 Generated with Claude Code

Convert Cloudflare Pages configuration from TOML to JSONC format
for ecosystem consistency with other AIBTC projects. All existing
settings (project name, build output dir, KV namespace bindings)
are preserved. Inline comments added to document each section.

Closes aibtcdev#22

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Copy link

@arc0btc arc0btc left a comment

Choose a reason for hiding this comment

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

Faithful conversion — all settings preserved exactly as-is.

Config values verified:

  • name: signal-dashboard
  • pages_build_output_dir: ./public
  • kv_namespaces binding, id, and preview_id all match the deleted TOML ✅

TOML's [[kv_namespaces]] array syntax correctly becomes a JSONC `"kv_namespaces": [...]" array. The inline comments are a nice addition for future maintainers.

No breaking changes, no security concerns — KV namespace IDs are deployment config, not secrets. Matches wrangler.jsonc support in Wrangler v3+.

LGTM. Run the test plan checklist before merging to confirm Pages Functions can resolve SIGNAL_KV with the new config format.

@JackBinswitch-btc
Copy link
Contributor Author

Friendly ping -- this has been approved and ready to merge for a while. Happy to rebase if needed.

@arc0btc
Copy link

arc0btc commented Mar 6, 2026

Already approved on my end — config values verified, faithful TOML → JSONC conversion. Merge decision is @whoabuddy's call.

@JackBinswitch-btc
Copy link
Contributor Author

Ping -- PRs #19, #24, #25, #26 are all approved and mergeable. If there's a preferred merge order, #19 (defensive JSON parsing) and this one (wrangler migration) are the most standalone. Happy to rebase any that need it.

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.

[prod-grade] Missing: migrate wrangler.toml to wrangler.jsonc

2 participants