Skip to content

Fix scheduled report delivery and chat UI regressions - #185

Merged
fmhall merged 1 commit into
mainfrom
mason/fix-production-chat-regressions
Nov 2, 2026
Merged

fmhall merged 1 commit into
mainfrom
mason/fix-production-chat-regressions

Conversation

@fmhall

@fmhall fmhall commented Nov 2, 2026 •

Copy link
Copy Markdown
Member

Scheduled browser runs completed after the Eve 1.0 upgrade, but web-chat reports stayed pending: /internal/scheduled-run/report reached Next's fallback page, whose HTTP 200 response was treated as success. Route both internal callbacks to the generated Eve service and require the handler's 202 acknowledgment for reports and answers.

The chat now renders native questions, approvals, sign-ins, and runtime failures in its default view. Cancelled calls show “Cancelled” with an explanation; failed calls open with recorded details or a fallback. Paginated history retains admission events for every displayed turn, including messages queued before the loaded tail, so user messages survive reload. Partial usage is labeled as saved or loaded usage and cannot replace a complete saved total.

Validation:

  • pnpm check: 890 tests across 96 files; lint, types, formatting, and Knip pass.
  • pnpm test:runtime: 2 evals and 16 gates pass.
  • pnpm build and pnpm build:eve pass.
  • Independent review and one re-review complete with no outstanding findings.
  • Hosted Vercel preview: both callbacks return 401 without authentication. Project OIDC authentication reaches the report handler (202) and answer handler (409 for a nonexistent run), instead of the previous HTML 200 fallback. No real run was dispatched or resumed.

Related to #184. Production code/configuration has not been changed outside this branch.

@fmhall
fmhall marked this pull request as ready for review November 2, 2026 19:31
@fmhall
fmhall merged commit d78cb0a into main Nov 2, 2026
2 checks passed

This branch was successfully deployed

1 active deployment
Preview — e243d362 Deployed Nov 2, 2026 by vercel[bot]
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