(MOT-3875) feat: routed provider family + freeform apply_patch custom tool#416
(MOT-3875) feat: routed provider family + freeform apply_patch custom tool#416ytallo wants to merge 1 commit into
Conversation
…atch
- harness: a send (or parentless spawn) that omits provider resolves it
from the router's model catalog before options freeze, so prompt-family
selection and routing agree — headless codex sends now get the
apply_patch code identity without passing provider explicitly. Router
unreachable → warn and fall back to the previous default-family
behavior.
- provider-openai-codex: coder::apply-patch travels as a freeform custom
tool named exactly apply_patch — the trained name and shape. The wire
emits { type: custom }, and the SSE state machine accumulates
custom_tool_call raw input (delta + authoritative done) into
{ patch: <raw text> }, so patches cross the wire with no JSON escaping.
Harness and shell still see an ordinary coder::apply-patch call.
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
|
Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
skill-check — worker0 verified, 32 skipped (no docs/).
Four for four. Nicely done. |
|
Consolidated into #423. |
1 participant
from the router's model catalog before options freeze, so prompt-family
selection and routing agree — headless codex sends now get the
apply_patch code identity without passing provider explicitly. Router
unreachable → warn and fall back to the previous default-family
behavior.
tool named exactly apply_patch — the trained name and shape. The wire
emits { type: custom }, and the SSE state machine accumulates
custom_tool_call raw input (delta + authoritative done) into
{ patch: }, so patches cross the wire with no JSON escaping.
Harness and shell still see an ordinary coder::apply-patch call.
Refs MOT-3875