fix(init): respect CLAUDE_CONFIG_DIR for global paths#674
Open
TechWizard9999 wants to merge 1 commit intortk-ai:developfrom
Open
fix(init): respect CLAUDE_CONFIG_DIR for global paths#674TechWizard9999 wants to merge 1 commit intortk-ai:developfrom
TechWizard9999 wants to merge 1 commit intortk-ai:developfrom
Conversation
Signed-off-by: Roopesh <[email protected]>
157233e to
96e2620
Compare
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.
Description
rtk init -gignoredCLAUDE_CONFIG_DIRand always targeted~/.claude. This PR makes global Claude path resolution env-aware so installs/patches land in the intended config directory.Related Issue
Closes #633
Changes Made
CLAUDE_CONFIG_DIRwhen setCLAUDE_CONFIG_DIRsettings.jsonpath instead of always showing~/.claude/settings.jsonFiles Changed
src/init.rs- env-aware path resolution and testsCHANGELOG.md- unreleased bug-fix note forrtk init -gdoes not respect CLAUDE_CONFIG_DIR #633Testing
cargo fmt --all --checkrtk cargo clippy --all-targets(2 pre-existing warnings in untouched files)rtk cargo testrtk cargo test init::tests::test_resolve_claude_dir_Checklist
cc @pszymkowiak for review