Repository navigation
Release 1.3.1 - #12
Conversation
|
No actionable comments were generated in the recent review.
|
| Layer / File(s) | Summary |
|---|---|
Session switch settlement hooks/limits.ts, hooks/register.tsx |
Adds bounded session-ID polling and tracks pending switches. Settlement creates an empty inbox for clear or restores the retained inbox for resume, then coordinates mirror writes. |
Hook integration and documented behavior hooks/register.tsx, README.md, docs/usage.md, .claude-plugin/plugin.json |
Hooks settle the current session before processing messages, actions, commands, config changes, and rendering. Documentation describes inbox behavior and timing under managed settings and Team or Enterprise sign-ins. The plugin version changes to 1.3.1. |
Priority:
Estimated code review effort: 3 (Moderate) | ~25 minutes
Change: Bug fix
Sequence Diagram(s)
sequenceDiagram
participant EndHook as session.end hook
participant Poller as Session ID poller
participant Settlement as Inbox settlement
participant Mirror as Inbox mirror
participant CommandHook as command.run or config.set
EndHook->>Poller: Start polling after clear or resume
Poller->>Settlement: Report changed session ID
Settlement->>Mirror: Schedule or await inbox copy
CommandHook->>Settlement: Ensure current session and wait for copy
Possibly related PRs
- dot-agi/high-command#11: Introduced the prior conversation-switching implementation that this change extends.
- dot-agi/high-command#7: Established branch, resume, and clear inbox behavior that this change handles when the session-start event is blocked.
Merge Risk: 
Minimal · up to 9b4f5
-
The session-switching changes are mergeable after normal checks; no actionable issue was established.
Pre-merge checks: 
5
-
Full details: Docstring Coverage
-
Explanation
Docstring coverage is 100.00% which is sufficient. The required threshold is 80.00%. Docstring coverage is scoped to functions touched by this diff. Analyzed 13 functions across 2 files. (3 skipped: 3 unsupported.)
-
Finishing Touches
-
📝 Generate docstrings
-
- Commit to this branch
- Create a new PR
🧪 Generate unit tests (beta)
-
- Commit to this branch
- Create a new PR
-
- Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts
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 @coderabbitai help to get the list of available commands.
Fixes switches on Team and Enterprise sign-ins and on machines with managed settings.
There, Claude Code's built-in guard keeps the session-start event from installed mods. In 1.3.0,
/clearand an in-app/resumethen kept showing the inbox of the conversation that ended, and new messages went into it./clear, the inbox starts empty. After a/resumeof a conversation that ended in the same process, its inbox comes back. Any other/resumestarts empty./branchstarts with an empty inbox, because High Command cannot tell a branch from a resume there. Elsewhere, a branch still keeps its parent's inbox.Update:
claude plugin update high-command@high-command, then start a new session or run/reload-plugins.Summary by CodeRabbit
New Features
/branchnow starts with an empty inbox when managed settings or a Team or Enterprise sign-in is in use./resumecan still restore the parent conversation’s inbox.Documentation