Repository navigation
comms warm session: launch claude through PATH so cmux wraps it - #52
Merged
Merged
Conversation
The warm Slack session was launched as ~/.local/bin/claude, which skips cmux's per-surface claude shim, so cmux's wrapper never registered the session and only the cmux-auto-resume hook knew about it. The pane command is now `command claude` (PATH lookup, alias bypassed); with mukul-memory's ~/.zlogin shim fix the login shell resolves that to cmux's shim. CLAUDE_BIN still overrides it. Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
The warm Slack session ran
~/.local/bin/claudedirectly, which skips cmux's per-surfaceclaudeshim, so cmux's wrapper never registered it; onlycmux-auto-resumedid. The pane command is nowcommand claude: a PATH lookup that bypasses the shell alias. With mukul-memory#12's~/.zloginfix, the login shell resolves it to cmux's shim.CLAUDE_BINstill overrides it.This is a step toward retiring
cmux-auto-resume.pyand the registry fallback once every launch is wrapped.Tests: 2 new in
tests/test_comms_session.py(63 pass), each mutation-probed and caught. Verified in a real login shell: the PATH lookup reaches cmux's shim, andcommandskips aliases.🤖 Generated with Claude Code