Skip to content

fix: separate video agent phase and persistence deadlines - #28

Merged
devhims merged 2 commits into
mainfrom
fix/video-agent-phase-deadlines
Sep 5, 2026
Merged

fix: separate video agent phase and persistence deadlines#28
devhims merged 2 commits into
mainfrom
fix/video-agent-phase-deadlines

Conversation

@devhims

@devhims devhims commented Sep 5, 2026

Copy link
Copy Markdown
Owner

Four production research reruns completed every transcript analysis but lost synthesis at its 20-second ceiling. This gives routing/research an admission-based 40-second limit and finalization its own continuous 40-second limit starting at handoff. Synthesis starts immediately when research finishes.

Terminal saving leaves the model-phase timer and uses a separate 30-second persistence timeout. Validation failure restores the original finalization deadline so a repair cannot reset its budget. The Durable Object no longer rejects a ready answer merely because the old model deadline has passed; inactive/canceled runs remain protected. Billing settlement is bounded, idempotent, and retains watchdog retries. The processing watchdog now accounts for both 40-second phases. Total client-observed time can exceed 80 seconds while persistence finishes.

The external phase timer controls AgentCore, avoiding a second SDK loop timer cutting off saving. Clarification saving is bounded too. Persistence timeouts have their own diagnostic classification. Documentation and the local smoke-test polling window are updated. No change to the five-finding limit, model, output budget, or compact response schema.

Validation:

  • TypeScript build passed.
  • 369 unit tests passed across 52 files.
  • 17 local Workers integration tests passed.
  • 10 processor tests passed.
  • Regression coverage includes 40s research + 35s synthesis + 20s persistence completing at 95s, citation repair retaining its original deadline, persistence timeout, and Durable Object saving past the model deadline with idempotent settlement.

Started from main synced with origin/main at 68fab61. No production deployment or post-change production test was performed. Production replay follows merge and deployment.

@vercel

vercel Bot commented Sep 5, 2026

Copy link
Copy Markdown

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

Project Deployment Actions Updated
video2ctx-web Ready Ready Preview Sep 5, 2026 11:05pm UTC

@mintlify

mintlify Bot commented Sep 5, 2026

Copy link
Copy Markdown

Preview deployment for your docs. Learn more about Mintlify Previews.

Project Status Preview Updated
video2ctx 🟢 Ready View Preview Sep 5, 2026, 11:06 PM

💡 Tip: Enable Automations to automatically generate PRs for you.

@devhims
devhims merged commit f329ee6 into main Sep 5, 2026
7 checks passed
@devhims
devhims deleted the fix/video-agent-phase-deadlines branch September 5, 2026 23:13
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