Skip to content

test: guard Cursor status polling SDK path - #83

Draft
cursor[bot] wants to merge 2 commits into
mainfrom
cursor/missing-test-coverage-c2ca
Draft

test: guard Cursor status polling SDK path#83
cursor[bot] wants to merge 2 commits into
mainfrom
cursor/missing-test-coverage-c2ca

Conversation

@cursor

@cursor cursor Bot commented Jul 30, 2026

Copy link
Copy Markdown

Risky behavior now covered

  • Cursor status polling uses durable Agent.get(agentId, { apiKey }) metadata instead of the streaming listRuns/getRun path that previously emitted uncaught TransformErrors.
  • SDK status and summary are normalized, with missing status defaulting to running.

Tests added

  • test/cursor-port.test.ts

Regression value

These tests exercise the real port boundary rather than the adapter's fake port, so reverting the production polling fix or dropping the status fallback fails deterministically. A mutation to the streaming API path made both focused tests fail.

Validation

  • npm test — 37/37 passed
  • npm run build — passed
  • npx prettier --check test/cursor-port.test.ts — passed
Open in Web View Automation 

cursoragent and others added 2 commits July 30, 2026 10:04
Co-authored-by: Parvez Kose <koseparvez@gmail.com>
Co-authored-by: Parvez Kose <koseparvez@gmail.com>
@vercel

vercel Bot commented Jul 30, 2026

Copy link
Copy Markdown

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

Project Deployment Actions Updated (UTC)
polyagent Ready Ready Preview Jul 30, 2026 10:07am

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