[Agents] Move Chat SDK to Runtime Communication and rename Agent tools#31272
[Agents] Move Chat SDK to Runtime Communication and rename Agent tools#31272thomasgauvin wants to merge 4 commits into
Conversation
github-actions
Bot
added
product:agents
|
This PR requires additional review attention because it affects the following areas: RedirectsThis PR changes current filenames or deletes current files. Make sure you have redirects set up to cover the following paths:
|
github-actions
Bot
assigned
elithrar,
irvinebroque,
rita3ko,
threepointone and
whoiskatrin
|
This pull request requires reviews from CODEOWNERS as it changes files that match the following patterns:
|
thomasgauvin
requested review from
a team,
elithrar,
irvinebroque,
rita3ko,
threepointone and
whoiskatrin
as code owners
Review⏸️ Automatic reviews for this PR are paused. This PR has already received 2 automatic reviews. To run another review, a codeowner can comment
✅ No style-guide issues found in commit CommandsOnly codeowners can run commands. Post a comment with the command to trigger it.
|
ask-bonk
Bot
added
documentation
![devin-ai-integration[bot]](https://avatars.githubusercontent.com/in/811515?s=60&v=4){kind=small}
![ask-bonk[bot]](https://avatars.githubusercontent.com/in/2454580?s=60&v=4){kind=small}
There was a problem hiding this comment.
2 issues need attention.
CRITICAL
- Missing redirect for the old Chat SDK direct path (/agents/runtime/execution/chat-sdk/).
MEDIUM
- agent-tools.mdx body text and heading still say "Agent tools" after the title was renamed to "Agents as tools". For consistency, also update lines 13, 15, 17, and 210.
Review complete for PR #31272Labels applied: Issues found: CRITICAL — Missing redirect for the moved Chat SDK page. The file moved from MEDIUM — Incomplete terminology update in
These should be updated for consistency with the new page title. What changed correctly:
|
Co-authored-by: devin-ai-integration[bot] <158243242+devin-ai-integration[bot]@users.noreply.github.com>
Co-authored-by: ask-bonk[bot] <249159057+ask-bonk[bot]@users.noreply.github.com>
- Remove duplicate /agents/runtime/execution/chat-sdk/ redirect in __redirects (fixes CI build failure) - Update remaining 'Agent tools' body text to 'Agents as tools' in agent-tools.mdx for consistency with page title
|
Manual review. LGTM |
|
/bonk rereview |
|
Both previously flagged issues have been resolved.
No new issues found. The PR is ready to merge. |
|
/bonk rereview and pls approved if good |
|
APPROVED ✅ Re-reviewed PR #31272. Both previously flagged issues are resolved:
All internal links are correctly updated across the 11 changed files. No new issues found. |
6 participants
Summary
Moves the Chat SDK documentation from the Runtime Execution section to the Runtime Communication section, since Chat SDK is a communication channel integration. Also renames "Agent tools" to "Agents as tools" for consistency.
chat-sdk.mdxfrom/agents/runtime/execution/to/agents/runtime/communication/agent-tools.mdxfrom "Agent tools" to "Agents as tools"public/__redirectsfor the old Chat SDK pathDocumentation checklist