docs(training-agent): point examples at per-specialism tenant URLs#3714
Merged
docs(training-agent): point examples at per-specialism tenant URLs#3714
Conversation
bokelley
force-pushed
the
bokelley/training-agent-sdk-migrate
branch
from
3c04214 to
1a2d95a
Compare
bokelley
force-pushed
the
bokelley/training-agent-docs-per-tenant
branch
3 times, most recently
from
cfbe7e3 to
116a3ad
Compare
bokelley
force-pushed
the
bokelley/training-agent-sdk-migrate
branch
from
27e9916 to
527dc8b
Compare
bokelley
force-pushed
the
bokelley/training-agent-docs-per-tenant
branch
from
116a3ad to
15bbf9d
Compare
bokelley
force-pushed
the
bokelley/training-agent-sdk-migrate
branch
from
527dc8b to
df122fc
Compare
bokelley
force-pushed
the
bokelley/training-agent-docs-per-tenant
branch
from
15bbf9d to
f8dcba0
Compare
Follows multi-tenant split in 7974aec. Each doc's example URL routes to the tenant matching its content focus: - /sales/mcp — quickstart, media-buy task references, generic auth examples - /signals/mcp — signals specialist module + ecosystem reference - /governance/mcp — governance specialist module - /creative/mcp — creative specialist module (default, with notes for /creative-builder/mcp and /sales/mcp on lab exercises that span agents) - legacy /mcp — sponsored-intelligence specialist module (multi-specialism lab, no dedicated SI tenant; explanatory note added) specs/brand-protocol-sandbox-agent.md prose updated to describe the multi-tenant architecture (was still describing the single-URL legacy training agent). The legacy https://test-agent.adcontextprotocol.org/mcp URL keeps working via the back-compat alias from the same release; this PR only nudges docs at the per-tenant URLs that match each example's content. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
Address docs-expert review feedback on the per-tenant URL migration: - creative.mdx: replace inline-comment three-URL bash block with a single-default `export AGENT_URL=...` plus a per-exercise URL table. Coding agents and human readers both pick up which URL each lab exercise expects without parsing comments inside the export. - sponsored-intelligence.mdx: drop the word "legacy" from the multi-specialism endpoint description so it reads as a deliberate routing choice rather than an undocumented fallback. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
… agent DX-expert review noted that the quickstart is the entry path for every new developer, but it pointed at /sales/mcp without explaining that this is a per-specialism URL on a path-routed agent. A developer landing here, then trying a signals call against the same AGENT_URL, hit "Unknown tool" without understanding why. One-line callout naming the sibling URLs and pointing at /.well-known/adagents.json for the full tenant + tool registry. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
bokelley
force-pushed
the
bokelley/training-agent-docs-per-tenant
branch
from
f8dcba0 to
b194aa8
Compare
4 tasks
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
1 participant
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
Updates 13 docs/specs files to point training-agent examples at the per-specialism tenant URLs introduced in #3713. Stacked on top of #3713 so the URLs resolve when docs ship.
URL mapping by content focus
/sales/mcp/signals/mcp/governance/mcp/creative/mcpdefault, with a per-exercise URL table for the labs that hit/creative-builder/mcpand/sales/mcp/mcp(multi-specialism lab, no dedicated SI tenant)Quickstart callout — one-line note explaining the test agent is path-routed and pointing at
/.well-known/adagents.jsonfor the full tenant + tool registry. DX-expert review flagged this as the highest-impact change for new developers.Spec doc —
specs/brand-protocol-sandbox-agent.mdprose updated to describe the multi-tenant architecture (was still describing the single-URL legacy training agent).Why stacked on #3713
The docs URLs only resolve when the multi-tenant routing is deployed. Merging #3713 first ensures
https://test-agent.adcontextprotocol.org/sales/mcp(etc.) responds before docs reference it. The legacy/mcpURL keeps working via the back-compat alias from the same release for the SI doc that intentionally stays on it.Test plan
🤖 Generated with Claude Code