docs(plan-execute): clarify Responses history requirement - #905
Conversation
Signed-off-by: Ryan Lempka <rlempka@nvidia.com>
|
|
Navigate logical layers of code changes, visualize relationships, and explore their blast radius. No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Repository: NVIDIA-NeMo/Switchyard/.coderabbit.yaml Review profile: CHILL Plan: Enterprise Run ID: 📒 Files selected for processing (2)
Included review availability: This review used your included allowance. Your plan provides up to 12 included reviews per hour; 10 remain after this review. WalkthroughThe documentation now describes the conversation history required for plan/execute routing, including tool calls and results. It explains that continuation IDs alone do not provide this history and updates the “Optional settings” label to a Markdown heading. ChangesPlan/execute conversation history
Priority: ⬇️ Low Estimated code review effort: 2 (Simple) | ~7 minutes Merge Risk: ⚪ Minimal · up to The documented full-history guidance matches how plan/execute detects tool activity. No issue identified here prevents merging after normal checks. 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
A rabbit reads the history trail, Comment |
Document that plan/execute needs conversation history, including tool calls and results. Explain that Responses requests using only
previous_response_idor a providerconversationID can miss the edit and stay on the planner.Apps embedding
libsymust supply that history before routing. This documents the limitation and the full-history workaround instead of adding automatic history replay in #897.This limitation affects Responses clients that send continuation IDs instead of conversation history. With Switchyard’s supplied Codex configuration, Codex includes the conversation history in each request, so this limitation does not apply.