Skip to content

fix(agents): omit separator for empty exception details - #6894

Merged
u9g merged 1 commit into
mainfrom
jason/omit-empty-exception-detail
Aug 18, 2026
Merged

fix(agents): omit separator for empty exception details#6894
u9g merged 1 commit into
mainfrom
jason/omit-empty-exception-detail

Conversation

@u9g

@u9g u9g commented Aug 18, 2026

Copy link
Copy Markdown
Contributor

Summary

  • render empty root-cause messages using only the exception type
  • avoid trailing colon-space output such as ReadTimeout:
  • cover empty third-party and nested APIConnectionError messages

Testing

  • uv run pytest tests/test_exceptions.py --unit
  • make check

@u9g
u9g requested a review from a team as a code owner August 18, 2026 16:57

@devin-ai-integration devin-ai-integration Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

✅ Devin Review: No Issues Found

Devin Review analyzed this PR and found no bugs or issues to report.

Open in Devin Review

@u9g
u9g merged commit a0aa469 into main Aug 18, 2026
26 checks passed
@u9g
u9g deleted the jason/omit-empty-exception-detail branch August 18, 2026 19:04
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.

2 participants