Skip to content

Fix error result message fallback#1036

Open
sanjitrajendiran42 wants to merge 1 commit into
anthropics:mainfrom
sanjitrajendiran42:fix-error-result-message
Open

Fix error result message fallback#1036
sanjitrajendiran42 wants to merge 1 commit into
anthropics:mainfrom
sanjitrajendiran42:fix-error-result-message

Conversation

@sanjitrajendiran42

Copy link
Copy Markdown

Summary

  • prefer the CLI result text when an error result has an empty errors list
  • keep the existing subtype fallback for minimal/older result payloads
  • add regression coverage for API-style error results with subtype=success

Fixes #1031.

Tests

  • .venv/bin/python -m pytest tests/test_query.py::TestProcessExitAfterErrorResult
  • .venv/bin/python -m ruff check src/claude_agent_sdk/_internal/query.py tests/test_query.py
  • .venv/bin/python -m ruff format --check src/claude_agent_sdk/_internal/query.py tests/test_query.py
  • .venv/bin/python -m pytest tests/test_query.py
  • .venv/bin/python -m mypy src/

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.

Misleading error message: 'Claude Code returned an error result: success' when errors[] is empty

1 participant