#4837
fix: guard .strip() on multimodal content and log prepare_call exceptions
Fixes #4800 and #4805.
Changes
- #4800: Guard
msg.content.strip()calls withisinstance()check inloop.py— prevents crash when multimodal content returns non-string content - #4805: Replace
suppress(Exception)with explicittry/except+logger.exception()inrunner.py— ensuresprepare_callfailures are logged instead of silently swallowed
(The aiohttp dependency fix for #4829 was removed from this PR — already addressed in #4830 by @alekwo.)
Verification
- Tests: 1243 pass | Lint: clean | Diff: 2 files, +7/-4
Generated with Claude Code via Happy
Co-Authored-By: Claude noreply@anthropic.com Co-Authored-By: Happy yesreply@happy.engineering
Activity
6Issue #4829 was already fixed in PR #4830 (note that it also required changes in the tests)
Issue #4829 was already fixed in PR #4830 (note that it also required changes in the tests)
chengyongrureviewed3mo ago
Xingkai98reviewed3mo ago
Issue #4829 was already fixed in PR #4830 (note that it also required changes in the tests)