Skip to content

fix: setting_sources=[] bug (issue #794)#806

Closed
nanookclaw wants to merge 1 commit intoanthropics:mainfrom
nanookclaw:fix/setting-sources-empty-list
Closed

fix: setting_sources=[] bug (issue #794)#806
nanookclaw wants to merge 1 commit intoanthropics:mainfrom
nanookclaw:fix/setting-sources-empty-list

Conversation

@nanookclaw
Copy link
Copy Markdown

Replace truthiness check with explicit is not None check, matching tools handling. When setting_sources is an empty list, pass --setting-sources "" to disable all setting sources.

This fixes issue #794 where setting_sources=[] was silently ignored due to a truthiness check.

Testing

All tests pass. Specifically:

  • still passes
  • updated and passes with new expected behavior
  • still passes
  • All other tests continue to pass

Replace truthiness check with explicit is not None check, matching tools handling. When setting_sources is an empty list, pass --setting-sources  to disable all setting sources.
IgorTavcar added a commit to IgorTavcar/claude-agent-sdk-python that referenced this pull request Apr 10, 2026
- anthropics#806: setting_sources=[] truthiness fix
- anthropics#803: betas=[]/plugins=[] truthiness fix
- anthropics#786: ThinkingBlock missing signature crash fix
- anthropics#790: suppress ProcessError when result already received
- anthropics#658: capture real stderr in ProcessError
- anthropics#791: suppress stale task notifications between turns
- anthropics#763: guard malformed CLAUDE_CODE_STREAM_CLOSE_TIMEOUT env var
- anthropics#805: delete_session() cascades subagent transcript dir
- anthropics#804: top-level skills option on ClaudeAgentOptions
- anthropics#691: PostCompact hook event type support

479 tests passing, mypy clean, ruff clean.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@nanookclaw
Copy link
Copy Markdown
Author

Closing — the upstream refactoring that introduced _apply_skills_defaults() resolves this bug. When setting_sources=[], the current main branch correctly produces --setting-sources= via f"--setting-sources={",".join(effective_setting_sources)}" (empty join → empty string). The test test_build_command_setting_sources_empty_list_disables_all already covers this case. No longer needed.

@nanookclaw nanookclaw closed this Apr 27, 2026
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