Skip to content

fix: track wrangler.jsonc in git and suppress Hono auto-detection - #14

Merged
tbjers merged 1 commit into
mainfrom
fix/track-wrangler-config
Jun 25, 2026
Merged

fix: track wrangler.jsonc in git and suppress Hono auto-detection#14
tbjers merged 1 commit into
mainfrom
fix/track-wrangler-config

Conversation

@tbjers

@tbjers tbjers commented Jun 25, 2026

Copy link
Copy Markdown
Contributor

Root cause

wrangler.jsonc was in .gitignore — a leftover from when CI only used wrangler pages deploy (which takes CLI flags, no config file needed). The new wrangler deploy requires wrangler.jsonc to exist. Without it, wrangler 4.105 falls back to framework auto-detection, detects hono in package.json, and fails:

✘ [ERROR] The detected framework ("Hono") cannot be automatically configured.

Changes

  • .gitignore — remove wrangler.jsonc (and the now-irrelevant !dashboard/wrangler.jsonc negation for the deleted file)
  • wrangler.jsonc — now tracked in git; also adds build.command pointing at the SvelteKit dashboard build, which gives wrangler an explicit build step and suppresses auto-detection
  • .github/workflows/deploy-dashboard.yml — remove the separate "Build dashboard" step; wrangler now invokes it via build.command; dashboard npm ci still runs first so deps are available

Test plan

  • CI deploy workflow passes on merge

🤖 Generated with Claude Code

@tbjers
tbjers force-pushed the fix/track-wrangler-config branch from 273d217 to 410780c Compare June 25, 2026 22:17
wrangler.jsonc was gitignored (leftover from when only wrangler pages
deploy was used in CI, which took CLI flags instead). Now that CI runs
wrangler deploy, the config must be in the repo — without it, wrangler
falls back to framework auto-detection and fails on Hono 4.105.

Also adds build.command to wrangler.jsonc pointing at the dashboard
SvelteKit build, which suppresses wrangler 4.105 auto-detection (Hono
is the Worker router, not a front-end framework). The separate "Build
dashboard" CI step is removed since wrangler now invokes the build
itself; dashboard npm ci still runs first so deps are in place.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@tbjers
tbjers merged commit 904c5f7 into main Jun 25, 2026
4 checks passed
@tbjers
tbjers deleted the fix/track-wrangler-config branch June 25, 2026 22:20
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