Skip to content

docs: add creative agent setup verification#5881

Open
sangilish wants to merge 1 commit into
adcontextprotocol:mainfrom
sangilish:agent/docs-creative-agent-setup-verification
Open

docs: add creative agent setup verification#5881
sangilish wants to merge 1 commit into
adcontextprotocol:mainfrom
sangilish:agent/docs-creative-agent-setup-verification

Conversation

@sangilish

Copy link
Copy Markdown
Contributor

Summary

  • add a deployment-checklist assertion for non-empty creative.supported_formats
  • document a vendor-neutral MCP get_adcp_capabilities connectivity probe
  • show the canonical format declaration shape buyers should expect

Why

Creative-agent implementers and buyers have no runnable setup check in the deployment guide. The new probe verifies endpoint reachability and the canonical creative-format discovery surface without depending on a specific vendor or the deprecated list_creative_formats discovery path.

Closes #5877.

Validation

  • node tests/snippet-validation.test.cjs --file docs/creative/implementing-creative-agents.mdx
  • node tests/docs-nav-validation.test.cjs
  • npm run check:seo (passed with existing repository warnings)
  • git diff --check

No changeset: documentation-only, non-normative addition.

@sangilish
sangilish marked this pull request as ready for review July 18, 2026 21:25

@aao-secretariat aao-secretariat Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

⚠️ Argus review could not complete

The automated review encountered an issue (possibly reached max turns, timed out, or failed to post the final gh pr review). A human reviewer should take this PR.

View workflow run

This is an automated message from the Argus AI review workflow.

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 089723774b

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment on lines +446 to +452
curl -X POST "$AGENT_URL" \
-H "Content-Type: application/json" \
-H "Accept: application/json, text/event-stream" \
-d '{
"jsonrpc": "2.0",
"id": 1,
"method": "tools/call",

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

P2 Badge Initialize MCP sessions before probing tools

For MCP endpoints that implement the standard lifecycle or return an Mcp-Session-Id, this raw tools/call as the first request will be rejected or run before capabilities are negotiated; the MCP lifecycle spec says initialization is the first interaction and Streamable HTTP clients must carry the returned session ID on subsequent requests (see https://modelcontextprotocol.io/specification/2025-03-26/basic/lifecycle#initialization and https://modelcontextprotocol.io/specification/2025-03-26/basic/transports#session-management). Since this is presented as the runnable deployment check for remote MCP creative agents, compliant stateful servers can appear broken. Add the initialize/initialized handshake and session header steps, or point readers to an MCP client/CLI that performs them.

Useful? React with 👍 / 👎.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

docs: knowledge gap (critical) — Getting Started / Verifying Your Setup

1 participant