Skip to content

fix(cli): load user and project settings in claude-sdk harness#70

Merged
josemontesdeoca merged 1 commit into
mainfrom
fix/claude-sdk-setting-sources
May 5, 2026
Merged

fix(cli): load user and project settings in claude-sdk harness#70
josemontesdeoca merged 1 commit into
mainfrom
fix/claude-sdk-setting-sources

Conversation

@josemontesdeoca
Copy link
Copy Markdown
Member

The Claude Agent SDK defaults settingSources to empty, so the claude-sdk harness silently ignored ~/.claude and /.claude: user skills, project skills, hooks, MCP servers, and project CLAUDE.md never reached the agent. Set ["user", "project"] so prose runs match the workspace a user already configured for Claude Code.

Adds three tests: settingSources is always forwarded; auth_status messages stream their output lines to stderr; auth_status with an error field exits 1 (the existing branch was untested).

The Claude Agent SDK defaults settingSources to empty, so the
claude-sdk harness silently ignored ~/.claude and <repo>/.claude:
user skills, project skills, hooks, MCP servers, and project
CLAUDE.md never reached the agent. Set ["user", "project"] so prose
runs match the workspace a user already configured for Claude Code.

Adds three tests: settingSources is always forwarded; auth_status
messages stream their output lines to stderr; auth_status with an
error field exits 1 (the existing branch was untested).
@josemontesdeoca josemontesdeoca requested a review from irl-dan May 5, 2026 17:00
@josemontesdeoca josemontesdeoca merged commit 540394c into main May 5, 2026
23 checks passed
@josemontesdeoca josemontesdeoca deleted the fix/claude-sdk-setting-sources branch May 5, 2026 17:02
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