docs(sdk): describe chat.turn() in the advanced chat agent skill - #4902
docs(sdk): describe chat.turn() in the advanced chat agent skill#4902ericallam wants to merge 1 commit into
Conversation
The bundled skill still told an action to return a StreamTextResult, string or UIMessage to answer, an API #4816 removed. It now describes edit-only actions, chat.turn() for an answer, the action-turn trigger, persistence for both models, and sending actions through useChat.
|
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Repository UI Review profile: CHILL Plan: Team Run ID: 📒 Files selected for processing (1)
Included review availability: Your plan provides up to 10 included reviews per hour; 9 remain after this review. 📜 Recent review details⏰ Context from checks skipped due to timeout. (42)
🧰 Additional context used📓 Path-based instructions (1)Add crumbs as you write code — not just when debugging.📄 CodeRabbit inference engine (AGENTS.md) Files:
🔇 Additional comments (1)
WalkthroughThe actions documentation now separates edit-only actions from actions that call Merge Risk: ⚪ Minimal · up to This updates bundled chat-agent guidance to use chat.turn() for action-triggered responses and documents supported submission paths. The documented behavior is aligned with existing implementation coverage, with no remaining merge-blocking risk. 🚥 Pre-merge checks | ✅ 4 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (4 passed)
Full details: Description checkExplanation The description clearly explains the change and its purpose, but it does not follow the required template. It omits the issue-closing line, checklist, Testing section, Changelog section, and Screenshots section.
✨ 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 |
Summary
The bundled
trigger-chat-agent-advancedskill still told agents to answer from an action by returning a value, an API #4816 removed. Code generated from it fails at runtime with thechat.turn()error.Before, the skill said:
After:
The section now covers edit-only actions,
chat.turn()and theaction-turntrigger, persistence for both the platform-managed andhydrateMessagesmodels, and sending actions throughuseChat(body.actionoruseChatActions) so the answer renders, withtransport.sendActionnoted as the raw-stream path.Docs-only change to an SDK-bundled skill; no changeset, since the
chat.turn()release note from #4816 already covers the behavior.Raised by Devin on #4884 after merge.
🤖 Generated with Claude Code
https://claude.ai/code/session_01AxuSksX18bj1yhnLpkcQ6a