Skip to content

fix(session): emit compaction summary as user role - #27

Open
jfenal wants to merge 2 commits into
bobbyjohnstx:mainfrom
jfenal:fix/no-user-query
Open

jfenal wants to merge 2 commits into
bobbyjohnstx:mainfrom
jfenal:fix/no-user-query

Conversation

@jfenal

@jfenal jfenal commented Oct 9, 2026

Copy link
Copy Markdown

Fixes #23.

After compaction the conversation started with a system-role summary message, and providers requiring a user-role message rejected the request with HTTP 500 'no user query found in messages'. Emit the summary as a user message instead.

Adds a regression test asserting the compacted conversation always contains a user-role first message.

…on valid

After compaction the conversation started with a system-role summary
message. Providers that require a user-role message reject the request
with HTTP 500 'no user query found in messages'. Emit the summary as a
user message instead. Adds a regression test asserting the compacted
conversation always contains a user-role user message.
@jfenal
jfenal requested a review from bobbyjohnstx as a code owner October 9, 2026 19:11
Co-Authored-By: Claude Sonnet 4.6 (1M context) <noreply@anthropic.com>
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.

fix(session): processor error — HTTP 500 "no user query found in messages"

1 participant