Skip to content

Support CLAUDE_CONFIG_DIR and checking multiple claude config dirs#51

Open
ndom91 wants to merge 1 commit into
Ataraxy-Labs:mainfrom
ndom91:ndom91/support-multiple-claude-code-project-dirs
Open

Support CLAUDE_CONFIG_DIR and checking multiple claude config dirs#51
ndom91 wants to merge 1 commit into
Ataraxy-Labs:mainfrom
ndom91:ndom91/support-multiple-claude-code-project-dirs

Conversation

@ndom91

@ndom91 ndom91 commented Jun 14, 2026

Copy link
Copy Markdown

@inspect-review inspect-review Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

inspect review

Triage: 11 entities analyzed | 0 critical, 0 high, 3 medium, 8 low
Verdict: standard_review

Findings (3)

  1. [low] In claude_code_projects_dirs_deduplicate_env_and_scan_matches test, the environment variable is set to an absolute path but the function expects it to be expanded. The test passes an absolute path home.join(".claude-personal") which won't match the tilde expansion logic in expand_home_path, so deduplication won't work as expected.
  2. [low] The expand_home_path function only handles tilde expansion for paths starting with ~ or ~/, but in the test claude_code_projects_dirs_deduplicate_env_and_scan_matches, an absolute path is passed. This means the path won't be expanded and deduplication won't occur, causing the test to fail because .claude-personal/projects will appear twice in the results.
  3. [low] The expand_home_path function only handles tilde expansion for paths that are valid UTF-8 strings (via path.to_str()). If the path contains non-UTF-8 characters, it returns the original path unchanged, which could lead to incorrect behavior when CLAUDE_CONFIG_DIR contains non-UTF-8 characters on Unix systems.

Reviewed by inspect | Entity-level triage found 0 high-risk changes

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Claude Code watcher hardcodes ~/.claude/projects/ — doesn't see sessions under $CLAUDE_CONFIG_DIR (e.g. ~/.claude-personal/)

1 participant