Open
Conversation
Two fixes in the ghostel backend that emerged from code review: 1. Pin the ghostel shell to /bin/sh with shell-integration disabled. Previously the user's $SHELL ran with full integration, which sourced bashrc/zshrc, emitted a prompt, and leaked GHOSTEL_* injection env vars into Claude before `exec claude-cmd' took over. A minimal sh doesn't source rc files and still `exec's cleanly. 2. Chain ghostel's native process sentinel. The wrapping sentinel in `claude-code-ide--start-session' replaced it unconditionally, skipping cleanup of ghostel's buffer-local redraw/input timers, the `after-focus-change-function' registration, and `ghostel-exit-functions'. Now we stash the original sentinel via `process-put' and invoke it first inside the wrapping sentinel. Also drop `ghostel-kill-buffer-on-exit' to nil so `claude-code-ide--cleanup-on-exit' is the single buffer-kill path, preventing a double cleanup-on-exit invocation (once via kill-buffer-hook, once via the sentinel).
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
1 participant
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.
codex yolo'd
resolves #187
I've given it brief usage this evening.. should be a good starting point.