pal-e-docs

Review: Commit vite.config.ts allowedHosts to repo

review-830-2026-04-05 Doc

review ready
Project

Verdict: READY

Template Completeness

Checked against template-issue-bug:
  • [x] Type -- Bug
  • [x] Lineage -- "Discovered during validation. Local edit not committed."
  • [x] Repo -- forgejo_admin/twitch-2k-wager
  • [x] What Broke -- vite.config.ts missing prod hostname in allowedHosts
  • [x] Repro Steps -- present
  • [x] Expected Behavior -- present
  • [x] Environment -- present (minimal but acceptable for a config-only bug)
  • [x] Acceptance Criteria -- present (1 criterion)
  • [x] Related -- references #47
Note: Issue includes extra sections from the feature template (File Targets, Test Expectations, Constraints, Checklist). These are harmless additions.

Traceability

  • [x] story:challenger-pay label -- challenger payment flow
  • [ ] story note NOT VERIFIED -- project-twitch-2k-wager has no TOC entries (project page appears empty or missing user-stories section). [SCOPE] Create user-stories section on project-twitch-2k-wager with challenger-pay story entry.
  • [x] arch:twitch-2k-wager-app label -- SvelteKit app component
  • [ ] arch note MISSING -- search for "arch-twitch-2k-wager-app" returned no results in pal-e-docs. [SCOPE] Create architecture note arch-twitch-2k-wager-app for the SvelteKit app component.
  • [x] Forgejo issue -- https://forgejo.tail5b443a.ts.net/forgejo_admin/twitch-2k-wager/issues/59, state: open

File Targets

  • [x] vite.config.ts -- verified exists on main branch. Currently contains allowedHosts: ['playme2k-dev.tail5b443a.ts.net']. Prod hostname playme2k.tail5b443a.ts.net is confirmed missing. Issue description is accurate.

Repo Placement

OK. Issue filed on forgejo_admin/twitch-2k-wager. Fix targets vite.config.ts in the same repo. Single-repo, single-file change.

Dependencies

  • [x] #47 (Fix: Stripe webhook delivery for dev) -- satisfied, board item #818 is in done column.
No unresolved dependencies. No in-progress items block this work.

Acceptance Criteria

Single AC: "Both hostnames in allowedHosts." Clear and programmatically verifiable -- agent can grep vite.config.ts for both hostnames after the change. The "Build passes" test expectation is also verifiable. Sufficient for a one-line config bug fix.

Blast Radius

Warning: westside-app has the identical pattern -- vite.config.js contains only ['westside-dev.tail5b443a.ts.net'] with no prod hostname. Same bug likely exists there.
Other repos checked: mcd-tracker-app and pal-e-playground-archived use allowedHosts: true (permissive), so they are unaffected.
Rollback is trivial -- single config line in a non-breaking direction (adding a hostname to an allowlist).

Decomposition Assessment

No decomposition needed. 1 file target, 1 repo, 1 acceptance criterion, estimated under 2 minutes agent time. Well within the three-thing limit and five-minute rule. No independent subtasks to parallelize.

Recommendation

  • [SCOPE] Create user-stories section on project-twitch-2k-wager with challenger-pay story entry.
  • [SCOPE] Create architecture note arch-twitch-2k-wager-app for the SvelteKit app component.
  • [SCOPE] Create Forgejo issue on forgejo_admin/westside-app for the same allowedHosts bug (only dev hostname, missing prod hostname in vite.config.js).
None of these block the fix itself. The ticket scope is correct and actionable as-is.