fix(setup): add Claude runtime alias for renamed installs#1885
fix(setup): add Claude runtime alias for renamed installs#1885maxpetrusenkoagent wants to merge 1 commit into
Conversation
|
Merging to
After your PR is submitted to the merge queue, this comment will be automatically updated with its status. If the PR fails, failure details will also be posted here |
|
Codex/autoreview: CLEAN Reviewed the exact PR diff for #1882. No blocking findings. Evidence checked:
Verification from PR body / agent run:
Residual risk: GitHub currently reports no checks on this fork draft PR; |
|
Possible overlap with #1819 (opened 2026-05-31, Fixes #1202 / addresses #1694) - flagging so the two can be coordinated. #1819 relocates a direct So the two are two halves of the same problem - a renamed or relocated install must still resolve Not a blocker on this PR's logic - the |
Summary
Fixes #1882.
Claude-host skill bodies still contain runtime references to
~/.claude/skills/gstack/.... When the checkout is installed as~/.claude/skills/<non-gstack>, setup links slash-command wrappers but leaves those runtime paths unresolved.This PR adds a narrow setup-time runtime alias:
~/.claude/skills/gstack -> ~/.claude/skills/<install-name>only for global Claude installs under$HOME/.claude/skills~/.claude/skills/gstackinstallsskills/directories to avoid scope creep~/.claude/skills/gstackinstallNo generated
SKILL.mdfiles are rewritten.Verification
bun test test/setup-claude-runtime-alias.test.ts test/setup-windows-fallback.test.ts test/setup-sections-linking.test.ts test/setup-conductor-worktree.test.tsbash -n setupgit diff --check$TMP_HOME/.claude/skills/gstack-dev$TMP_HOME/.claude/skills/gstack -> $TMP_HOME/.claude/skills/gstack-dev$TMP_HOME/.claude/skills/gstack/bin/gstack-config get skill_prefixruns through the hardcoded runtime pathReview
086d3663f24146281aa563d8eb29019fc1b56c21d73ef2450c97833227d15475CI / checks
GitHub currently reports no checks on the fork branch for this draft PR, so the exact-diff Codex review above was used as the fallback reviewer gate.
Notes
A broader
bun run test:windowsrun still has the pre-existing unrelatedbrowse/test/terminal-agent.test.tslazy-spawn failure. The touched files and this patch do not affect that path.