Skip to content

fix(agents): preserve queued prompt across bounded context#531

Merged
HomenShum merged 3 commits into
mainfrom
fix/fast-agent-empty-messages
Jul 14, 2026
Merged

fix(agents): preserve queued prompt across bounded context#531
HomenShum merged 3 commits into
mainfrom
fix/fast-agent-empty-messages

Conversation

@HomenShum

Copy link
Copy Markdown
Owner

Summary

  • validate the exact queued user prompt before routing, reservation, or provider execution
  • pass that validated prompt explicitly while keeping provider history bounded at zero
  • fail closed on stale or malformed prompt rows
  • redact SDK error-class wrappers and map internal empty-prompt failures to a safe retry message

Production evidence

The first bounded production dogfood after #529 selected the correct tier-eligible model and released its reservation, but the SDK received an empty provider message list because recentMessages was zero and only promptMessageId was supplied.

Verification

  • focused FastAgent/runtime tests: 63/63
  • full FastAgentPanel suite: 213 passed, 19 skipped
  • app TypeScript: green
  • Convex TypeScript: green
  • production build: green
  • git diff --check: green
  • independent prompt-correctness review: approve
  • independent accounting/lease/cancellation review: approve
  • independent error-redaction review: approve

Release

P0 CI-gated auto-merge only. No admin bypass. No manual Convex production deploy.

@vercel

vercel Bot commented Jul 14, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
nodebench-ai Ready Ready Preview, Comment Jul 14, 2026 4:44pm

Request Review

@HomenShum HomenShum enabled auto-merge (squash) July 14, 2026 16:39
@github-actions

Copy link
Copy Markdown

Tier B regression failed against preview

Preview URL: (preview URL unresolved)
Run: 29350564417

Likely a kit-parity DOM regression. Check the Playwright report artifact attached to the run for the failing selector + screenshot.

This check would have caught all 4 A9 sessionRows iterations as a single PR — that's why it exists.

@HomenShum HomenShum merged commit fa39758 into main Jul 14, 2026
15 of 16 checks passed
@HomenShum HomenShum deleted the fix/fast-agent-empty-messages branch July 14, 2026 16:49
@github-actions

Copy link
Copy Markdown

Demo: walkthrough of the surfaces this PR changed is available as a workflow artifact (pr-demo-531) at https://github.com/HomenShum/NodeBenchAI/actions/runs/29351247848

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.

2 participants