Skip to content

docs: add TypeScript A2A support to agent-to-agent documentation#597

Merged
dbschmigelski merged 7 commits intostrands-agents:mainfrom
dbschmigelski:docs/a2a
Mar 12, 2026
Merged

docs: add TypeScript A2A support to agent-to-agent documentation#597
dbschmigelski merged 7 commits intostrands-agents:mainfrom
dbschmigelski:docs/a2a

Conversation

@dbschmigelski
Copy link
Copy Markdown
Contributor

Add Python/TypeScript tabs throughout the A2A docs to reflect the new A2A protocol support in sdk-typescript PR #601. Includes A2AAgent (consuming remote agents), A2AServer (exposing agents), and notes on TypeScript-specific differences like streaming limitations and Express middleware. Also adds a TypeScript note to graph.md for A2AAgent support status.

Description

Related Issues

Type of Change

  • New content
  • Content update/revision
  • Structure/organization improvement
  • Typo/formatting fix
  • Bug fix
  • Other (please describe):

Checklist

  • I have read the CONTRIBUTING document
  • My changes follow the project's documentation style
  • I have tested the documentation locally using mkdocs serve
  • Links in the documentation are valid and working

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Mar 5, 2026

Assessment: Approve

This PR effectively adds TypeScript documentation throughout the A2A docs, with clear explanations of TypeScript-specific differences and limitations. The code compiles successfully and follows established documentation patterns.

Review Details
  • Content Quality: Well-structured tabs, clear API configuration tables, and proper cross-referencing to related docs
  • TypeScript Patterns: Correct use of snippet markers, function scoping, and tabbed content per AGENTS.md guidelines
  • Feature Limitations: Streaming and Graph workflow limitations are clearly documented with appropriate notes

Minor suggestions in inline comments for improving the tool example and adding a link to the SDK PR. Nice work documenting the A2A TypeScript support! 🚀

@strands-agent
Copy link
Copy Markdown
Contributor

Documentation Deployment Complete

Your documentation preview has been successfully deployed!

Preview URL: https://d3ehv1nix5p99z.cloudfront.net/pr-597/

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Mar 5, 2026

Assessment: Approve ✅

Thanks for the update! The removal of the TypeScript note from graph.md is a good simplification—the limitation is already documented in agent-to-agent.md (lines 230-231), so readers will find it when following the cross-reference link.

Review Summary
  • TypeScript compilation: ✅ Passes (npm run test)
  • Documentation patterns: ✅ Follows AGENTS.md guidelines correctly
  • Content quality: ✅ Clear Python/TypeScript tabs with appropriate limitation notes
  • Link update: ✅ graph.md anchor updated to #consuming-remote-agents

The inline suggestions from the previous review (linking SDK PR #601, adding inputSchema) remain as optional enhancements but are not blocking.

@strands-agent
Copy link
Copy Markdown
Contributor

Documentation Deployment Complete

Your documentation preview has been successfully deployed!

Preview URL: https://d3ehv1nix5p99z.cloudfront.net/pr-597/

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Mar 5, 2026

Assessment: Request Changes

The latest commit adds a comprehensive A2AAgent section to graph.md which is great content, but there's a broken anchor link that needs to be fixed.

Review Details

Issue Found:

  • graph.md:200 links to #a2aagent-consuming-remote-agents but the correct anchor is #consuming-remote-agents

Validations Passed:

  • ✅ TypeScript compilation (npm run test)
  • ✅ Documentation patterns follow AGENTS.md guidelines
  • ✅ Content quality is good with clear examples

Once the anchor link is fixed, this PR is ready to merge.

@strands-agent
Copy link
Copy Markdown
Contributor

Documentation Deployment Complete

Your documentation preview has been successfully deployed!

Preview URL: https://d3ehv1nix5p99z.cloudfront.net/pr-597/

Add Python/TypeScript tabs throughout the A2A docs to reflect the new
A2A protocol support in sdk-typescript PR strands-agents#601. Includes A2AAgent
(consuming remote agents), A2AServer (exposing agents), and notes on
TypeScript-specific differences like streaming limitations and Express
middleware. Also adds a TypeScript note to graph.md for A2AAgent
support status.
@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Mar 12, 2026

Documentation Preview Ready

Your documentation preview has been successfully deployed!

Preview URL: https://d3ehv1nix5p99z.cloudfront.net/pr-cms-597/docs/user-guide/quickstart/overview/

Updated at: 2026-03-12T19:35:57.413Z

- A2AServer → A2AExpressServer for Express-based server examples
- A2AAgent config: only url and agentCardPath (no name/description)
- tool() uses callback with inputSchema, not handler
- createMiddleware() is synchronous (removed await)
- Streaming is real incremental via sendMessageStream
- Remove stale TypeScript Graph support note
- Remove import statements from inside function bodies
- Remove experimental frontmatter flag
zastrowm
zastrowm previously approved these changes Mar 12, 2026
…t.mdx

Co-authored-by: Mackenzie Zastrow <3211021+zastrowm@users.noreply.github.com>
…t.mdx

Co-authored-by: Mackenzie Zastrow <3211021+zastrowm@users.noreply.github.com>
Copy link
Copy Markdown
Contributor

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

Assessment: Comment

All tests pass (291/291), but there's a new open discussion that should be resolved before merge.

Status

Tests: ✅ All 291 passed

Open Discussion:

  • @zastrowm raised a valid question about whether the "A2A Client Examples" section (line 549+) should still be included, since A2AAgent is now the recommended approach. This needs author input.

Previously Addressed:

  • ✅ Anchor link fixed
  • ✅ Tool example includes Zod inputSchema
  • ✅ Uses correct A2AExpressServer class name

Waiting for author response on the open thread before final approval.

Remove A2A Client Examples and manual A2A Agent as a Tool sections.
These are superseded by A2AAgent and add unnecessary noise.
Keep the Strands A2A Tool section as it covers the tool provider.
@dbschmigelski dbschmigelski merged commit 70a9ea6 into strands-agents:main Mar 12, 2026
5 checks passed
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.

3 participants