Skip to content

feat: add Poolside reasoning_content echo-back detector#2

Draft
varunrandery wants to merge 1 commit into
mainfrom
varun/add-poolside-echo-back
Draft

feat: add Poolside reasoning_content echo-back detector#2
varunrandery wants to merge 1 commit into
mainfrom
varun/add-poolside-echo-back

Conversation

@varunrandery

Copy link
Copy Markdown
Collaborator

Poolside's Laguna models require reasoning_content on every assistant tool-call message when replaying history.

Adds _needs_poolside_tool_reasoning() following the DeepSeek/MiMo pattern (provider equality + model substring + host match), wired into the _needs_thinking_reasoning_pad() OR gate so the full echo-back machinery (pad at write time, re-pad on replay, strip on strict fallback) applies automatically.

Detection signals:

  • provider == "poolside"
  • "laguna" in model
  • host api.poolsi.de or inference.poolside.ai

Poolside's Laguna models require reasoning_content on every assistant
tool-call message when replaying history; omitting it causes HTTP 400.

Adds _needs_poolside_tool_reasoning() following the DeepSeek/MiMo
pattern (provider equality + model substring + host match), wired into
the _needs_thinking_reasoning_pad() OR gate so the full echo-back
machinery (pad at write time, re-pad on replay, strip on strict fallback)
applies automatically.

Detection signals:
- provider == "poolside"
- "laguna" in model
- host api.poolsi.de or inference.poolside.ai
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.

1 participant