[Agents] Extract shared sections as partials for get-started pages#31276
Open
thomasgauvin wants to merge 4 commits into
Open
[Agents] Extract shared sections as partials for get-started pages#31276thomasgauvin wants to merge 4 commits into
thomasgauvin wants to merge 4 commits into
Conversation
- Created 4 partials: what-just-happened, common-integration-patterns, troubleshooting, next-steps - Unified troubleshooting across both pages (7 issues merged) - Added auth, custom prefix, server-side access, and multiple agent patterns to quick-start - Added deploy step and useAgent key points to add-to-existing-project - Removed duplicated content, both pages now share identical structure
github-actions
Bot
added
product:agents
github-actions
Bot
assigned
elithrar,
irvinebroque,
rita3ko,
threepointone and
whoiskatrin
Contributor
|
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:
|
Contributor
|
This pull request requires reviews from CODEOWNERS as it changes files that match the following patterns:
|
![ask-bonk[bot]](https://avatars.githubusercontent.com/in/2454580?s=60&v=4){kind=small}
This comment was marked as resolved.
This comment was marked as resolved.
Sorry, something went wrong.
Contributor
|
Posted a review on PR #31276 with suggested fixes. Labels applied: Issues flagged:
Validation results: |
Co-authored-by: ask-bonk[bot] <249159057+ask-bonk[bot]@users.noreply.github.com>
Co-authored-by: ask-bonk[bot] <249159057+ask-bonk[bot]@users.noreply.github.com>
thomasgauvin
requested review from
a team,
elithrar,
irvinebroque,
rita3ko,
threepointone,
vy-ton and
whoiskatrin
as code owners
Contributor
Review
Warnings (24)
CommandsOnly codeowners can run commands. Post a comment with the command to trigger it.
|
![devin-ai-integration[bot]](https://avatars.githubusercontent.com/in/811515?s=60&v=4){kind=small}
| @@ -1,5 +1,5 @@ | |||
| --- | |||
| title: Agent tools | |||
| title: Agents as tools | |||
Contributor
There was a problem hiding this comment.
🟡 Incomplete rename: page title says "Agents as tools" but body text still uses "Agent tools"
The PR renames the feature from "Agent tools" to "Agents as tools" — the title on line 2 was updated, and all cross-references in other files (concepts/tools.mdx, agents-api.mdx, sub-agents.mdx, think/tools.mdx, think/index.mdx, chat-agents.mdx, client-sdk.mdx) were updated to "Agents as tools". However, the body text of agent-tools.mdx itself still uses the old name "Agent tools" in 4 places: the opening paragraph (agent-tools.mdx:13), the second paragraph (agent-tools.mdx:15), a section heading ## Agent tools vs sub-agent RPC (agent-tools.mdx:17), and a later paragraph (agent-tools.mdx:210). This creates an inconsistency where the page title and all inbound links say "Agents as tools" but the content on the page says "Agent tools".
Prompt for agents
The title of agent-tools.mdx was changed from 'Agent tools' to 'Agents as tools', and all cross-references in other files were updated accordingly. However, the body text of agent-tools.mdx itself still uses the old name 'Agent tools' in 4 places that should be updated to match:
1. Line 13: 'Agent tools let one chat agent...' → 'Agents as tools let one chat agent...'
2. Line 15: 'Agent tools support...' → 'Agents as tools support...'
3. Line 17: '## Agent tools vs sub-agent RPC' → '## Agents as tools vs sub-agent RPC'
4. Line 210: 'Agent tools are normal sub-agents.' → 'Agents as tools are normal sub-agents.'
These are all in the file src/content/docs/agents/runtime/execution/agent-tools.mdx.
Was this helpful? React with 👍 or 👎 to provide feedback.
Contributor
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
7 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Extracts four shared sections from the Agents get-started pages into reusable partials, then applies them consistently to both the Quick Start and Add to Existing Project pages. This eliminates duplicated content and makes both pages structurally identical for the sections that follow the step-by-step instructions.
New partials (4):
what-just-happenedcommon-integration-patternstroubleshootingnext-stepsFixes inconsistencies:
useAgentkey points (was missing)LinkCardimport from both pages