fix(cursor): bound capability reads and buffered tool budgets

[luvs01]

Summary

Combine #5233 and the buffered-tool budget correction in 799ebc5d5d6a8c75eeb99f358db5c64906117584 into one Cursor capability and tool-budget change on dev 482a3e0b6bd59b99276b163b0ceb3f1645c76035.

• Read installed Cursor capabilities through a bounded regular-file handle. Reuse the metadata cache while refreshing the installed product version on cache hits; preserve the platform-specific symlink and special-file protections.
• Reserve the translator's per-call and per-turn budget before retaining completed textual fallback arguments. Release that reservation when a real frame wins and transfer it when the fallback is emitted.
• Preserve both original fix(cursor): bound installed bundle reads #5233 commits, including its review-driven cache-version correction, and the buffered-tool contribution's author/provenance. Update the corresponding structure contracts.

This aggregate is independent of maintainer-owned #5507: it does not carry its quarantine/replay changes, alter its branch, or add a dependent chain. No GUI change.

Verification

• Exact head 799ebc5d5d6a8c75eeb99f358db5c64906117584: direct Bun focused run passed 141 tests / 648 assertions / 0 failures, covering cursor-effort-table, cursor-protobuf-events, cursor-effort-rows, cursor-effort-suffix, cursor-discovery, and the file-size ratchet.
• bun run typecheck, bun run structure:check, and git diff --check 482a3e0b6bd59b99276b163b0ceb3f1645c76035 HEAD passed.
• fix(cursor): bound installed bundle reads #5233's two original changed files match its final source head exactly. The buffered-tool contribution and carried aggregate commits match by stable Git patch ID.
• This Windows run does not establish execution coverage for POSIX symlink/FIFO branches. Full suite, exact-head cross-platform CI, and independent review remain pending; the PR stays draft. No limits, baselines, skips, or source timeout values were changed.

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.

[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]

리뷰 · 우선순위 54 / 80

이 PR은 Cursor 쪽에서 서로 다른 두 가지를 한 번에 고칩니다. 하나는 설치된 agent 번들(cursor-effort-table)을 읽을 때, 심볼릭 링크·특수 파일에 속지 않도록 같은 파일 핸들로 열고(O_NOFOLLOW/O_NONBLOCK + fstat), 크기 한도를 지키며, 캐시가 맞아도 설치 버전(install.version)만 새로 붙이는 쪽입니다. 다른 하나는 텍스트로 보이는 가짜 툴콜을 잠시 모아 둘 때, 나중에 버릴 수도 있는데도 인자 바이트를 translator 예산에 바로 잡아 두고, 진짜 프레임이 이기면 예약을 풀고, 턴이 끝날 때 가짜만 남으면 그 예약을 그대로 써서 내보내는 쪽입니다. base는 dev이고, 닫힌 #5233과 포크 쪽 예산 수정을 한데 모은 draft입니다. GUI는 건드리지 않았고, 작성자 말대로 #5507의 quarantine/replay와는 별 줄이라고 적혀 있습니다.

라인 - src/integrations/cursor-effort-table.ts readCursorBundle: Windows에서는 symlink/FIFO 거절이 실제로 되는지 이 PR·테스트가 증명하지 않습니다(win32면 테스트가 바로 return). POSIX만 믿고 머지할지, Windows에서도 “실패하면 null → 정적 테이블”로 충분한지 확인이 필요합니다.
라인 - src/adapters/cursor/protobuf-events.ts finalizeTurnEvents: 버퍼에 넣어 둔 callId로 예산을 이미 잡은 뒤, flush 때 recordToolCall이 같은 id로 openCall을 다시 하고 open.args = call.args로 예산을 우회한 다음 commitToolCall합니다. 지금 TranslatorBudget.openCall은 중복 id에 안전하고, args가 같아서 이중 청구는 피하지만, 이후 commitToolCall이 args를 바꾸거나 예산을 다시 셈하면 깨지기 쉬운 형태입니다.
라인 - 한 PR에 “번들 읽기 경계”와 “텍스트 툴콜 예산”이 같이 들어 있습니다. 되돌리기·충돌 해소·리뷰 책임이 한 덩어리로 커집니다. #5233은 이미 닫혔으니 중복 머지 위험은 낮지만, 비슷한 반쪽 PR이 남아 있으면 같이 닫는 편이 낫습니다.
라인 - 작성 시점 CI에 test shard·structure 등 실패/취소가 보입니다. 본문 검증(포커스 141 테스트)과 별개로, draft를 ready로 올리기 전에 head에서의 전체 게이트를 다시 보는 편이 안전합니다. preview deploy 이야기는 생략합니다.

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

한 덩어리로 dev에 넣을지, 번들 IO와 예산 회계를 커밋/PR 단위로 나눌지. Windows에서 symlink 추종을 “정적 폴백으로 충분”하다고 볼지. #5507과 같은 주(protobuf-events 근처)라 머지 순서를 어떻게 잡을지. types.ts/config.ts 분할 이슈는 이 변경에 해당 없음.

너의 추천

의도는 분명하고 테스트도 핵심(캐시+버전 갱신, symlink/FIFO 거절, 예산 선점·실프레임 승리 시 해제)을 짚습니다. draft 유지를 전제로, Windows 약속만 한 줄로 확정하고, flush 경로에 “예산 바이트가 commit 후에도 한 번만 잡혔다”는 회귀 테스트를 보강한 뒤 머지 후보로 두는 쪽을 추천합니다. #5507과 겹치는 충돌이 나면 이 PR의 예산/버퍼 쪽이 더 좁으니 그쪽을 기준으로 rebase하는 편이 낫습니다. 무효·중복 반쪽 PR이 열려 있으면 닫으세요.

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

[lidge-jun]

Carried into #5619 as a cherry-pick of this PR's net change at 799ebc5, with your authorship kept on the commit. Closing in favor of #5619. Thank you!