Load Claude user and project settings sources#1299
Load Claude user and project settings sources#1299Jeccoman wants to merge 1 commit intopingdotgg:mainfrom
Conversation
|
Important Review skippedAuto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Repository UI Review profile: CHILL Plan: Pro Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment Tip You can disable poems in the walkthrough.Disable the |
What Changed
This updates the Claude adapter to pass settingSources: ["user", "project"] when creating Claude SDK queries, and adds a regression test covering the query options.
Why
Claude settings from user and project config were not being loaded for these queries, so repo-level and user-level Claude configuration could be ignored. This keeps the adapter aligned with expected Claude config resolution and fixes the missing settings behavior with a minimal change.
UI Changes
Checklist
Note
Load user and project settings sources in Claude adapter queries
Adds
settingSources: ["user", "project"]to the options passed tocreateQueryin ClaudeAdapter.ts, ensuring Claude resolves settings from both user and project scopes. A corresponding test verifies the property is present on the query options.Macroscope summarized 8b1a0a2.