Skip to content

fix(compressor): handle None LLM response content - #123

Merged
knight22-21 merged 2 commits into
knight22-21:mainfrom
Saksham1105:fix/compressor-none-content
Nov 1, 2026
Merged

knight22-21 merged 2 commits into
knight22-21:mainfrom
Saksham1105:fix/compressor-none-content

Conversation

@Saksham1105

Copy link
Copy Markdown
Contributor

Summary

  • Treat a missing LLM response body as an empty summary instead of raising AttributeError.
  • Preserve the existing truncated-event fallback when the provider returns no text.
  • Add a regression test asserting that content=None does not persist a [Compression failed ...] message.

Fixes #121

Testing

  • Added regression coverage in tests/test_phase8.py.
  • Tests have not been executed in this environment; please rely on CI for runtime verification.

@knight22-21
knight22-21 merged commit f638222 into knight22-21:main Nov 1, 2026
5 checks passed
@knight22-21

Copy link
Copy Markdown
Owner

Thanks for the fix, @Saksham1105! Handling None in response content during session compression cleanly avoids AttributeError and preserves the truncated-event fallback.

If you find DevAgent useful, a ⭐ on the repo would mean a lot — no pressure though!

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.

compress_session() crashes with AttributeError when LLM response content is None

2 participants