Conversation
There was a problem hiding this comment.
Pull request overview
This PR fixes the Claude CLI tool configuration by correcting the configuration directory path from ~/.claude/ to ~/.config/claude/ (following XDG Base Directory specification) and restructuring the settings for better organization and maintainability.
Key changes:
- Corrected configuration directory path from
~/.claude/to~/.config/claude/in the installation script - Restructured
settings.jsonby moving environment variables and cleanup settings to the top level for clarity - Simplified sandbox configuration by disabling it and removing granular options, while maintaining security through the comprehensive permissions deny list
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.
| File | Description |
|---|---|
install/symlink.zsh |
Fixed symlink target path from ~/.claude/ to ~/.config/claude/, added directory creation check, and added symlink for statusline.sh |
.config/claude/settings.json |
Reorganized configuration structure by moving env and cleanupPeriodDays to top level, reordered permissions blocks for clarity, simplified sandbox settings, and added new configuration options (enableAllProjectMcpServers, forceLoginMethod, outputStyle) |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| @@ -1,5 +1,13 @@ | |||
| { | |||
There was a problem hiding this comment.
The "model" field (previously set to "claude-sonnet-4-5-20250929") has been removed from the configuration. If the Claude tool requires a model specification to function correctly, this could cause runtime errors or unexpected behavior. Please verify if the model field is still needed or if the tool now uses a default model.
| { | |
| { | |
| "model": "claude-sonnet-4-5-20250929", |
This pull request mainly updates configuration and setup for the Claude tool, focusing on improving permissions management, environment setup, and installation reliability. The most significant changes are in
.config/claude/settings.json, where permissions and environment variables are restructured, and in the installation script, which now ensures directories exist before creating symlinks.Claude configuration changes:
env) and cleanup settings (cleanupPeriodDays) to the top level of.config/claude/settings.json, simplifying configuration management.denylist higher in the permissions block for clarity and maintainability, without changing the actual denied permissions.enableAllProjectMcpServers,forceLoginMethod, andoutputStyle, enhancing integration and output customization.Installation script improvements:
install/symlink.zshto create the.config/claudedirectory if it doesn't exist before creating symlinks, preventing installation errors. Also added a symlink forstatusline.sh.