Skip to content

Fix execute wait false timeouts and active HTTP session reaping#228

Draft
cursor[bot] wants to merge 4 commits into
mainfrom
cursor/critical-correctness-bugs-ed29
Draft

Fix execute wait false timeouts and active HTTP session reaping#228
cursor[bot] wants to merge 4 commits into
mainfrom
cursor/critical-correctness-bugs-ed29

Conversation

@cursor
Copy link
Copy Markdown
Contributor

@cursor cursor Bot commented May 19, 2026

Description

Fixes two high-severity execute-wait correctness issues:

  • harness_execute(..., wait=true) no longer reports execution_timed_out=true when polling gives up after five consecutive execution.get errors before the configured timeout expires. It now fails the wait path so callers see _wait.error while preserving the successful trigger response.
  • HTTP Streamable sessions now track active requests and the idle reaper skips sessions with in-flight POST/SSE requests, preventing long waits (up to the documented 7200s) from being closed by the 30-minute session TTL.

Added focused regression coverage for both behaviors and recorded the inspection in tasks/todo.md.

Type of Change

  • Bug fix
  • New feature
  • Refactor
  • Documentation
  • Other

Checklist

  • Tests pass
  • Typecheck passes

Validation:

  • pnpm test tests/utils/poll-execution.test.ts tests/integration/http-transport.test.ts
  • pnpm typecheck
  • pnpm build
  • pnpm test (59 files / 1400 tests)
  • Independent code review subagent: no blocking issues found
Open in Web View Automation 

cursoragent and others added 4 commits May 19, 2026 11:20
Co-authored-by: Rohan Gupta <thisrohangupta@users.noreply.github.com>
Co-authored-by: Rohan Gupta <thisrohangupta@users.noreply.github.com>
Co-authored-by: Rohan Gupta <thisrohangupta@users.noreply.github.com>
Co-authored-by: Rohan Gupta <thisrohangupta@users.noreply.github.com>
@CLAassistant
Copy link
Copy Markdown

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

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