Skip to content

fix: discover Claude profile roots#126

Merged
graykode merged 2 commits into
mainfrom
fix/claude-profile-discovery
May 24, 2026
Merged

fix: discover Claude profile roots#126
graykode merged 2 commits into
mainfrom
fix/claude-profile-discovery

Conversation

@graykode
Copy link
Copy Markdown
Owner

Summary

  • Add claude_config_dirs support for explicit Claude Code profile roots.
  • Auto-discover ~/.claude and ~/.claude-* roots that contain both sessions/ and projects/.
  • Wire configured roots through the app/collector stack and document the config option.

Fixes #123

Test Plan

  • cargo test
  • cargo clippy -- -D warnings
  • cargo build

Copy link
Copy Markdown
Owner Author

@graykode graykode left a comment

Choose a reason for hiding this comment

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

lgtm — security is clean (no new network/shell/deps, only adds read-only home-dir scanning). Discovery flow reuses the existing BTreeSet dedup + ConfigDir wrapping cleanly, and the #[cfg(test)] gating on the now-test-only constructors avoids dead-code noise. Build/clippy/143 tests all green, including the four new ones.

one minor inconsistency, not blocking: configured claude_config_dirs entries are dropped unless they pass is_claude_config_root (need both sessions/ and projects/), while env-based CLAUDE_CONFIG_DIR only checks is_dir(). A typo'd configured path is silently ignored with no warning — fine as a deliberate choice, just worth a note in the docs if users hit it.

@graykode graykode merged commit 27f523e into main May 24, 2026
7 checks passed
@graykode graykode deleted the fix/claude-profile-discovery branch May 24, 2026 08:23
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 Personal Plan does not display

1 participant