Skip to content

Feat/a2a call oauth#97

Merged
cuericlee merged 2 commits into
volcengine:mainfrom
songyichun1:feat/a2a-call-oauth
Jun 30, 2026
Merged

Feat/a2a call oauth#97
cuericlee merged 2 commits into
volcengine:mainfrom
songyichun1:feat/a2a-call-oauth

Conversation

@songyichun1

Copy link
Copy Markdown
Contributor

支持调用 OAuth2 client_credentials 鉴权的远程 Agent

@cuericlee cuericlee left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

/lgtm

@cuericlee cuericlee left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

/lgtm

@cuericlee cuericlee left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

/lgtm

@cuericlee cuericlee left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Overall Verdict: LGTM ✅
All changes are documentation-only — no code, no tests affected
Content is accurate, well-placed, and consistent with A2A AgentCard spec terminology
Deploy and invoke sections are kept in sync
Chinese prose flows naturally and matches the existing doc style
Non-blocking suggestion:
If REGISTRY_ID_ENDPOINT / AGENTKIT_ID_ENDPOINT support logic evolves, the long paragraph in 5.harness.md may benefit from being refactored into a dedicated reference section to avoid duplication across both files.

@cuericlee cuericlee merged commit e90efb7 into volcengine:main Jun 30, 2026
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