Make profile launch setup explicit and reversible#5
Merged
Conversation
Add interactive shell initialization that detects zsh, bash, or PowerShell, prints the rc-file diff, requires confirmation unless --yes is passed, and backs up existing rc files before writing managed launcher blocks. Constraint: Claude Code docs identify CLAUDE_CONFIG_DIR, not CLAUDE_CONFIG_PATH, as the supported configuration-directory override; direnv output uses that variable. Rejected: Raw PowerShell aliases | PowerShell aliases cannot chain Set-Location and launcher args, so generated profile launchers are functions. Rejected: Editing shell rc files silently | violates the A16 safety contract and makes startup changes hard to audit. Confidence: high Scope-risk: moderate Directive: When A14 replaces the soul use stub, preserve the --direnv path or call buildDirenvPlan/writeDirenvFile from shell-init.ts. Tested: bun test bin/cli/lib bin/cli/commands; temp zsh alias generation for claude-/codex- profile launchers; temp bash rc backup; temp PowerShell function rendering; temp soul use --direnv .envrc with inherited profile.env; git diff --cached --check Not-tested: Full bun test still fails on existing missing supertest dependency in skills/skills/stripe/stripe-webhooks/examples/express/test/webhook.test.js; zsh and pwsh binaries are not installed in this environment, so zsh/pwsh verification was text-level.
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.
Automated by gx branch finish (PR flow).