Skip to content

Conversation

@PsiACE
Copy link
Member

@PsiACE PsiACE commented Dec 4, 2025

Summary

as title

Related issues

Testing

Docs & screenshots

Checklist

  • Conventional Commit title (e.g. feat:, fix:).
  • Tests cover the change or are not required (explain above).
  • Docs/examples updated when behaviour is user-facing.
  • Schema regenerations (make gen-all) are called out if applicable.

Signed-off-by: Chojan Shang <chojan.shang@vesoft.com>
Signed-off-by: Chojan Shang <chojan.shang@vesoft.com>
@PsiACE PsiACE marked this pull request as ready for review December 4, 2025 15:06
Copilot AI review requested due to automatic review settings December 4, 2025 15:06
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR upgrades the Agent Client Protocol (ACP) schema from version 0.8.0 to 0.9.1. The changes are primarily mechanical updates to auto-generated Python code based on the updated JSON schema, with the main enhancement being more precise type definitions and expanded documentation for the _meta extensibility property across all model classes.

Key Changes:

  • Updated schema version reference from v0.8.0 to v0.9.1 across all files
  • Enhanced _meta field type from Optional[Any] to Optional[Dict[str, Any]] with comprehensive documentation explaining ACP's extensibility mechanism
  • Added Dict to Python type imports to support the more specific type annotation
  • Removed discriminator-based union handling for MCP server types (schema simplification)

Reviewed changes

Copilot reviewed 6 out of 6 changed files in this pull request and generated no comments.

Show a summary per file
File Description
src/acp/schema.py Auto-generated Python models updated with precise _meta typing (Dict[str, Any]), expanded documentation, and removal of discriminator field from McpServerStdio
src/acp/meta.py Schema version reference updated to v0.9.1
schema/schema.json JSON schema source updated with detailed _meta property specifications including additionalProperties: true and comprehensive descriptions for all model definitions
schema/VERSION Version reference file updated to refs/tags/v0.9.1
examples/echo_agent.py Import statements alphabetically reordered (McpServerStdio moved to correct alphabetical position)
examples/agent.py Import statements alphabetically reordered (McpServerStdio moved to correct alphabetical position)

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Signed-off-by: Chojan Shang <chojan.shang@vesoft.com>
@PsiACE PsiACE merged commit 6f736db into 0.7.x Dec 4, 2025
7 checks passed
@PsiACE PsiACE deleted the bump-acp branch December 4, 2025 15:29
PsiACE added a commit that referenced this pull request Dec 4, 2025
* chore: make check happy

Signed-off-by: Chojan Shang <chojan.shang@vesoft.com>

* chore: gen schema with 0.9.1

Signed-off-by: Chojan Shang <chojan.shang@vesoft.com>

* chore: bump to 0.7.0

Signed-off-by: Chojan Shang <chojan.shang@vesoft.com>

---------

Signed-off-by: Chojan Shang <chojan.shang@vesoft.com>
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.

2 participants