Repository navigation
Conversation
- Report totalKeys and totalMemoryBytes on session detail, measuring the keys the preset skips so the header matches the list - Label the read part as the latest checkpoint when keys were skipped - Parse __interrupt__ writes of the newest checkpoint and end the timeline with a "Waiting for input" entry Closes #595
|
Warning Review limit reachedYou've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository. Next included review available in 23 minutes. View limit details
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. Comment |
![cursor[bot]](https://avatars.githubusercontent.com/in/1210556?s=60&v=4){kind=small}
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes using default effort and found 1 potential issue.
❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.
Reviewed by Cursor Bugbot for commit 808a772. Configure here.
- Add skippedKeys to SessionDetail, set from the preset's unselected keys - Show the checkpoint note only when skippedKeys > 0, so missing, truncated or failed-node reads no longer trigger it
Closes #595
Changes
SessionDetailgainstotalKeysandtotalMemoryBytes. When a preset reads only some keys (agent-cache reads the newest checkpoint and its writes), the service runsMEMORY USAGEon the skipped keys so the totals match the list row.showing latest checkpoint (N keys · X)when keys were skipped.__interrupt__writes of the newest checkpoint intointerruptentries that end the timeline. The web shows them as a "Waiting for input" card.Test plan
src/session-explorer(365 pass):measureKeysrouting/unmeasured/empty, service totals with and without skipped keys, interrupt ordering and non-string values, fixturepending-writetscapi + web; eslint on changed sourcesNote
Medium Risk
Adds cluster MEMORY USAGE calls on session load when keys are skipped, and extends agent-cache merge/parse behavior; scope is session-explorer only but touches shared SessionDetail types.
Overview
Session detail now reports whole-session key count and memory even when a preset only loads a subset (e.g. agent-cache’s latest checkpoint). Skipped keys are sized via a new
measureKeyshelper (MEMORY USAGEon cluster primaries); the UI header usestotalMemoryBytes/totalKeysand adds a “showing latest checkpoint …” line whenskippedKeysis non-zero.The agent-cache preset treats
__interrupt__pending writes like messages: they are parsed intointerrupttimeline entries (after pendingmessageson the newest checkpoint). The timeline renders those as “Waiting for input” cards. Seeds and fixture expectations include a sample interrupt.Reviewed by Cursor Bugbot for commit daffc0f. Bugbot is set up for automated code reviews on this repo. Configure here.