fix(responses): consolidate tool history and continuation boundaries

[luvs01]

Summary

Preserve tool history and current user scope across external-provider continuations by combining the unlanded changes in #5350 and #5420 with the stateful output-only corrections carried in 82a5f6da81807eee582d744ce0086f22b19bd021 and aac783fe8d4fd90667ccb8ea815d31cc014e798a.

• Keep Cursor continuation scope on the latest actual user input through compaction, preserve evidence roles and bounded checkpoint fallback, and make code-mode guidance distinguish emitted output from an empty completed cell.
• Lower valid historical custom calls for destinations that explicitly deny native custom tools without expanding the live tool catalog. Reject malformed, duplicate, mismatched, or colliding identities before dispatch.
• Preserve stateful output-only deltas when their calls remain upstream, while keeping replay-miss reasoning cleanup independent. Repair stable xAI custom item IDs only on the native-compatible path.
• Preserve current dev's stronger empty-catalog tools: [] authorization boundary. Its already-landed selector repair and role-fixture fixes are not reapplied. Source follow-ups, author identities, and test placement remain accounted for.

This is an integration proposal. The other contributors' original PRs remain open and retain their own review and ownership.

Verification

• Prepared head 2ec0cd12f5bb1567b3723dbf13555046604aa083 on dev 41ec40f7e31969d1ca7140e0d510077f74f4cbe3, with all three source heads rechecked before integration.
• New combined adapter matrix: 10 passed / 56 assertions, covering stateful outputs, independent reasoning cleanup, capability-driven lowering, missing-result synthesis, pairing identity, and deny-all preservation.
• Existing focused suites: 366 passed, with one file-size scan timing out at the unchanged 5-second limit. The unchanged standalone ratchet then passed 9/9 at its normal limit. The first failed run is retained in the validation record.
• Typecheck, structure SSOT, privacy scan, and git diff --check passed. Frozen-lockfile documentation install and the actual documentation build passed: 497 pages.
• Sixteen key implementation/regression files match their source or the protected dev implementation byte-for-byte. Seven carried commits have identical stable patch IDs; the latest Cursor follow-up differs only in patch context. JSON layout-marker cleanup is incorporated into the integration resolution.
• The broader local changed-import-graph run did not complete: exit 124 at its unchanged 900-second limit, with Windows cleanup/ACL/timeouts and an unresolved assertion in existing server tests. Its 224 failure lines are partial output, not a completed total or proven baseline failures. No named failure matched the focused passing cases, but broader attribution remains unresolved.
• Exact-head Cross-platform CI run 35740377784 passed, including four Linux test shards, both macOS shards, desktop Rust checks, documentation and the aggregate ci check. Optional Windows diagnostic shards and unsharded macOS control were skipped; this is not a full Windows-suite pass. Local broader validation, live-provider acceptance and independent security review remain pending. Keep this proposal draft.

Checklist

• Scope stays focused and avoids unrelated cleanup.
• Docs or release notes were updated when needed.
• Security-sensitive changes were reviewed for secrets, auth, and unsafe defaults.

Co-authored-by: Yeonwoo Choi 32544727+twoimo@users.noreply.github.com
Co-authored-by: maosisheng maosisheng@gmail.com
Co-authored-by: Cursor cursoragent@cursor.com
Co-authored-by: Epinephrine luvs01@hanmail.net
Co-authored-by: Devin AI <158243242+devin-ai-integration[bot]@users.noreply.github.com>

[coderabbitai]

Important

Draft PR not reviewed

Draft PRs are not automatically reviewed by default.

• Trigger a manual review

To automatically review draft PRs, update your CodeRabbit configuration:

reviews:
  auto_review:
    drafts: true

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands.}

[github-actions]

✅ Deterministic PR hygiene checks passed.

[lidge-jun]

리뷰 · 우선순위 58 / 80

이 PR은 Cursor·xAI로 이어지는 도구 대화가 끊기거나 잘못된 요청을 다시 잡으려다 깨지는 문제를 한곳에 모은 통합 수정입니다. 예전에 열려 있던 #5350(요청 범위·도구 결과 이어가기)과 #5420(과거 custom 도구를 목적지가 거부할 때 낮춰 보내기)의 미머지 내용에, stateful 출력만 오는 이어가기와 재현(replay) 놓침 때 추론 정리를 서로 섞지 않는 수정을 얹었습니다. 베이스는 dev입니다.

구체적으로는 (1) Cursor 외부 모델 이어가기에서 최신 실제 사용자 요청만 활성 범위로 두고, 요약·브라우저 주변 문맥은 역사로만 두며, Grok 4.6 코드 모드에서 빈 완료 셀을 “이미 출력했다”고 오해하지 않게 안내하고, 체크포인트가 봉투 한도를 넘으면 한 번 전체 재현으로 다시 짜고, (2) custom 도구를 거부하는 목적지에는 살아 있는 도구 목록을 키우지 않고도 과거 custom 호출·결과를 function 형태로 낮추며, 잘못된·중복·충돌 identity는 보내기 전에 막고 HTTP 400으로 돌리며, (3) stateful 목적지에서 호출이 서버 쪽에만 있을 수 있는 출력만 오는 델타를 지우지 않고, 빈 카탈로그의 tools: [] 거부 경계는 현재 dev 쪽을 지킨다고 합니다. 포커스 테스트와 합친 매트릭스는 통과했다고 적혀 있으나, 넓은 import-graph 검증은 900초에서 끊겼고, 보안 체크박스와 전체 CI·라이브 수락은 아직 비어 있습니다. Draft로 두는 판단은 본문과 맞습니다.

src/responses/custom-tool-compat.ts (rewriteHistoricalCustomItems / validateFinalCustomToolCompatibility) - 거부 목적지에서 과거 custom을 낮추고, 남는 native custom은 최종 가드에서 막습니다. 방향은 맞지만, 잘못 생긴 호출은 500이 아니라 400으로 떨어집니다. 클라이언트가 “재시도하면 고쳐진다”고 오해할 수 있어, 어떤 입력이 영구 거부인지를 응답 메시지·문서에 더 짧게 적어 두는 편이 좋습니다.

src/adapters/openai-responses/tool-output-recovery.ts (repairOrphanedInputItems의 repairOrphanOutputs) / passthrough.ts - stateful + 미전개 previous_response_id일 때 orphan 출력을 사용자 메시지로 바꾸지 않습니다. 의도(서버에 호출이 있을 수 있음)는 분명합니다. 반대로 전개 실패가 “진짜로 호출이 없는” 경우와 구분되는지는 넓은 스위트·실기기에서만 드러납니다. 지금 기록된 넓은 실행은 타임아웃이라 그 증거가 아직 없습니다.

src/adapters/cursor/protobuf-request.ts (체크포인트 envelope_exhausted 시 재호출) - 체크포인트 필드를 지우고 한 번 다시 짜는 형태라 무한 재귀는 막혀 보입니다. 다만 재현 경로에서도 같은 한도에 걸리면 그대로 에러입니다. 그 실패가 사용자에게 “이어가기 불가”로 읽히는지, 아니면 다른 복구로 넘어가는지는 이 diff만으로는 확인되지 않습니다.

본문 Verification - 포커스 366통과·합친 매트릭스 10통과는 적혀 있고, changed-import-graph는 exit 124·부분 실패 224줄로 미완료입니다. Cross-platform CI도 아직 큐/진행 중인 항목이 있습니다. Draft를 Ready로 올리기 전에 “넓은 실행이 무엇을 증명했는지”가 본문 주장과 같아야 합니다.

메인테이너의 판단이 필요한 지점

#5350과 #5420은 여전히 열려 있습니다. 이 통합 PR이 dev에 들어가면 원본 PR을 superseded로 닫을지, 원본만 남기고 이 PR을 닫을지, 아니면 일부를 되돌릴지 소유권 정리가 필요합니다. 같은 계약을 두 갈래로 머지하면 충돌·중복 수정이 납니다. 또한 빈 카탈로그 tools: [] 거부 경계를 “지킨다”는 주장과, historical lowering이 과거 이름을 현재 권한으로 키우지 않는다는 주장은 보안 리뷰 한 번의 대상입니다. 체크리스트의 보안 항목이 아직 비어 있습니다.

너의 추천

Draft는 유지한 채, Cross-platform CI와(가능하면) 끊긴 import-graph를 끝내거나 실패를 이번 변경과 무관하다고 이름 단위로 가른 뒤에만 Ready로 옮기세요. 머지 시 #5350·#5420은 이 PR에 흡수된 범위만큼 닫아 중복을 막는 편이 낫습니다. 보안은 custom-tool 낮추기·최종 가드·빈 카탈로그 거부 세 줄만 짧게 독립 확인하면 됩니다. 포커스 테스트 방향은 타당해 보이며, 넓은 검증이 비어 있는 상태에서는 머지하지 않는 것이 맞습니다.

이 댓글은 grok-bot이 작성했습니다

[lidge-jun]

Carried in #5608: nine commits are cherry-picked with their authors kept, and the first two land as their net change because the xAI empty-catalog selector part is already on dev in b20acc7 (#5376). A follow-up pins the exact host-wrapper classification in continuation scope. The source PRs #5350 and #5420 are credited in #5608 and remain with their authors. Closing as superseded. Thank you @luvs01.