Skip to content

fix(cursor): combine envelope-echo diagnostics retention with bounded reasoning quarantine - #5476

Closed
luvs01 wants to merge 3 commits into
lidge-jun:devfrom
luvs01:fix/cursor-echo-combined
Closed

luvs01 wants to merge 3 commits into
lidge-jun:devfrom
luvs01:fix/cursor-echo-combined

Conversation

@luvs01

@luvs01 luvs01 commented Sep 21, 2026 •

Copy link
Copy Markdown
Collaborator

Summary

Combines two overlapping fork PRs that both rework src/adapters/cursor/envelope-echo.ts:

  • Retain adjacent mid-stream echo diagnostics: echo evidence adjacent to the mid-stream envelope is preserved instead of dropped.
  • Bound output-guard reasoning quarantine: reasoning quarantine in the output guard is bounded rather than unbounded.

Tests

  • bun test tests/providers/cursor/cursor-envelope-echo-retry.test.ts — 47 pass
  • cursor-adapter.test.ts + cursor-tool-continuation.test.ts — 54 pass

Summary by CodeRabbit

  • Bug Fixes

    • Improved handling of unusually large or closely spaced streamed responses.
    • Limited temporary retention of streamed output while preserving detection of relevant response markers.
    • Improved independent detection of multiple response integrity issues.
    • Ensured oversized responses are released and delivered normally when they exceed the safe processing limit.
  • Documentation

    • Documented bounded handling for streamed response output and oversized frames.

Loading
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants