Skip to content

fix(agents): honor CLAUDE_CONFIG_DIR for Claude hooks & MCPs - #333

Open
Fluzko wants to merge 4 commits into
symposium-dev:mainfrom
Fluzko:fix/claude-config-dir
Open

Fluzko wants to merge 4 commits into
symposium-dev:mainfrom
Fluzko:fix/claude-config-dir

Conversation

@Fluzko

@Fluzko Fluzko commented Oct 9, 2026 •

Copy link
Copy Markdown
Contributor

What does this PR do?

Part of #277.
Depends on #332 (until it merges, only the last commit is new here).
Needed by #345, #350, #355.

Claude Code reads its user settings from $CLAUDE_CONFIG_DIR when that variable is set and ignores ~/.claude/settings.json. Symposium still wrote its global hooks to ~/.claude/settings.json and reported success, so for anyone who relocated their Claude config the hooks never ran.

Global hooks now go to $CLAUDE_CONFIG_DIR/settings.json, next to the user MCP servers and the plugins from #332. Without the variable nothing changes. Tests no longer read CLAUDE_CONFIG_DIR or XDG_CONFIG_HOME from the developer's environment.

Disclosure questions

AI disclosure.

  • The AI tool authored large parts of the code

This branch has not been deployed

No deployments
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.

1 participant