feat(a2a): Agent Card discovery endpoint #867

Closed
opened 2026-03-13 22:03:03 +00:00 by freemo · 5 comments
Owner

Metadata

  • Commit Message: feat(a2a): Agent Card discovery endpoint
  • Branch: feature/m9-agent-card

Background

M9 (v3.8.0) acceptance criterion: the server must expose an Agent Card discovery endpoint per the A2A protocol standard. The Agent Card describes the agent's capabilities, supported operations, and version information, enabling other A2A-compatible systems to discover and interact with the CleverAgents server.

Per ADR-047 (A2A Standard Adoption), the Agent Card follows the A2A specification format.

Expected Behavior

  1. Agent Card discovery endpoint accessible at well-known URL
  2. Card describes supported A2A operations
  3. Card includes version negotiation information
  4. Card includes capability declarations
  5. Card format conforms to A2A specification

Acceptance Criteria

  • Agent Card endpoint accessible at well-known URL
  • Card describes all supported operations
  • Version negotiation information included
  • Capability declarations accurate
  • A2A specification conformance validated
  • Unit tests cover: card generation, endpoint, conformance

Subtasks

  • Implement Agent Card model
  • Implement discovery endpoint
  • Wire capability and operation declarations
  • Implement version negotiation information
  • Validate A2A specification conformance
  • Tests (Behave): Add scenarios for Agent Card
  • Verify coverage >=97% via nox -s coverage_report
  • Run nox (all default sessions), fix any errors

Definition of Done

This issue is complete when:

  • All subtasks above are completed and checked off.
  • A Git commit is created where the first line of the commit message matches the Commit Message in Metadata exactly, followed by a blank line, then additional lines providing relevant details about the implementation.
  • The commit is pushed to the remote on the branch matching the Branch in Metadata exactly.
  • The commit is submitted as a pull request to master, reviewed, and merged before this issue is marked done.
## Metadata - **Commit Message**: `feat(a2a): Agent Card discovery endpoint` - **Branch**: `feature/m9-agent-card` ## Background M9 (v3.8.0) acceptance criterion: the server must expose an Agent Card discovery endpoint per the A2A protocol standard. The Agent Card describes the agent's capabilities, supported operations, and version information, enabling other A2A-compatible systems to discover and interact with the CleverAgents server. Per ADR-047 (A2A Standard Adoption), the Agent Card follows the A2A specification format. ## Expected Behavior 1. Agent Card discovery endpoint accessible at well-known URL 2. Card describes supported A2A operations 3. Card includes version negotiation information 4. Card includes capability declarations 5. Card format conforms to A2A specification ## Acceptance Criteria - [ ] Agent Card endpoint accessible at well-known URL - [ ] Card describes all supported operations - [ ] Version negotiation information included - [ ] Capability declarations accurate - [ ] A2A specification conformance validated - [ ] Unit tests cover: card generation, endpoint, conformance ## Subtasks - [ ] Implement Agent Card model - [ ] Implement discovery endpoint - [ ] Wire capability and operation declarations - [ ] Implement version negotiation information - [ ] Validate A2A specification conformance - [ ] Tests (Behave): Add scenarios for Agent Card - [ ] Verify coverage >=97% via `nox -s coverage_report` - [ ] Run `nox` (all default sessions), fix any errors ## Definition of Done This issue is complete when: - All subtasks above are completed and checked off. - A Git commit is created where the **first line** of the commit message matches the Commit Message in Metadata exactly, followed by a blank line, then additional lines providing relevant details about the implementation. - The commit is pushed to the remote on the branch matching the **Branch** in Metadata exactly. - The commit is submitted as a **pull request** to `master`, reviewed, and **merged** before this issue is marked done.
freemo added this to the v3.8.0 milestone 2026-03-13 22:03:20 +00:00
freemo self-assigned this 2026-03-14 04:27:38 +00:00
freemo added reference feature/m9-agent-card 2026-03-23 04:56:37 +00:00
Author
Owner

PR #1123 Review Outcome: Changes Requested

PR #1123 has been reviewed and changes are requested before it can be merged. Key blockers:

  1. Merge conflicts — PR is not mergeable (mergeable: false)
  2. Dependency PR #1107 (ASGI endpoint) must be merged first
  3. Unresolved REQUEST_CHANGES from @hurui200320 — 1 critical + 5 major JSON-RPC protocol conformance issues
  4. Empty PR body — needs description and Closes #867 keyword

See the full review at: #1123 (comment)

Note: This issue was closed on 2026-03-23 before the PR was merged. Per project workflow, it should remain open until the PR is actually merged to master.


Automated by CleverAgents Bot
Supervisor: PR Review | Agent: ca-pr-self-reviewer

## PR #1123 Review Outcome: Changes Requested PR #1123 has been reviewed and **changes are requested** before it can be merged. Key blockers: 1. **Merge conflicts** — PR is not mergeable (`mergeable: false`) 2. **Dependency PR #1107** (ASGI endpoint) must be merged first 3. **Unresolved REQUEST_CHANGES** from @hurui200320 — 1 critical + 5 major JSON-RPC protocol conformance issues 4. **Empty PR body** — needs description and `Closes #867` keyword See the full review at: https://git.cleverthis.com/cleveragents/cleveragents-core/pulls/1123#issuecomment-80302 **Note:** This issue was closed on 2026-03-23 before the PR was merged. Per project workflow, it should remain open until the PR is actually merged to `master`. --- **Automated by CleverAgents Bot** Supervisor: PR Review | Agent: ca-pr-self-reviewer
Author
Owner

PR #1123 has been reviewed independently and changes were requested. The PR cannot be merged due to:

  1. Merge conflicts — branch has diverged from master
  2. Dependency on unmerged PR #1107 — must merge first
  3. Empty PR body — needs description and Closes #867
  4. 1 critical + 5 major protocol conformance issues — confirmed independently, matching @hurui200320's review
  5. Code quality issuescontext: Any typing, inline imports, missing skill test coverage

The commit SHA has not changed since the previous reviews — no fixes have been applied yet.

Note: This issue was closed prematurely on 2026-03-23 before the PR was reviewed or merged. Per project workflow, it should remain open until the PR is actually merged.

See PR #1123 review comment for full details.


Automated by CleverAgents Bot
Supervisor: PR Review | Agent: ca-pr-self-reviewer

PR #1123 has been reviewed independently and **changes were requested**. The PR cannot be merged due to: 1. **Merge conflicts** — branch has diverged from `master` 2. **Dependency on unmerged PR #1107** — must merge first 3. **Empty PR body** — needs description and `Closes #867` 4. **1 critical + 5 major protocol conformance issues** — confirmed independently, matching @hurui200320's review 5. **Code quality issues** — `context: Any` typing, inline imports, missing skill test coverage The commit SHA has not changed since the previous reviews — no fixes have been applied yet. **Note:** This issue was closed prematurely on 2026-03-23 before the PR was reviewed or merged. Per project workflow, it should remain open until the PR is actually merged. See [PR #1123 review comment](https://git.cleverthis.com/cleveragents/cleveragents-core/pulls/1123#issuecomment-81152) for full details. --- **Automated by CleverAgents Bot** Supervisor: PR Review | Agent: ca-pr-self-reviewer
Author
Owner

PR #1123 Review Update (2026-04-03)

PR #1123 has been reviewed and changes requested. The commit SHA (394c488d) has not changed since the previous REQUEST_CHANGES review by @hurui200320 on 2026-03-30 — none of the identified issues have been addressed.

Key blockers:

  1. Merge conflicts (mergeable: false)
  2. Dependency PR #1107 still open and also has merge conflicts
  3. Empty PR body (missing description and Closes #867)
  4. 1 critical + 5 major protocol conformance issues in the A2A JSON-RPC endpoint
  5. CONTRIBUTING.md violations (context: Any, inline imports, # type: ignore)

Note: This issue is currently closed but should remain open until PR #1123 is merged per the project workflow.

See the full review on PR #1123.


Automated by CleverAgents Bot
Supervisor: PR Review | Agent: ca-pr-self-reviewer

## PR #1123 Review Update (2026-04-03) PR #1123 has been reviewed and **changes requested**. The commit SHA (`394c488d`) has not changed since the previous REQUEST_CHANGES review by @hurui200320 on 2026-03-30 — none of the identified issues have been addressed. **Key blockers:** 1. Merge conflicts (`mergeable: false`) 2. Dependency PR #1107 still open and also has merge conflicts 3. Empty PR body (missing description and `Closes #867`) 4. 1 critical + 5 major protocol conformance issues in the A2A JSON-RPC endpoint 5. CONTRIBUTING.md violations (`context: Any`, inline imports, `# type: ignore`) **Note:** This issue is currently closed but should remain open until PR #1123 is merged per the project workflow. See the full review on [PR #1123](https://git.cleverthis.com/cleveragents/cleveragents-core/pulls/1123). --- **Automated by CleverAgents Bot** Supervisor: PR Review | Agent: ca-pr-self-reviewer
Author
Owner

PR #1123 has been reviewed and changes requested. The PR has 1 critical and 5 major protocol conformance issues (JSON-RPC error handling), 3 CONTRIBUTING.md violations, merge conflicts, and a dependency on unmerged PR #1107. See the PR review comment for full details.

Note: This issue was closed prematurely on 2026-03-23 before the PR was reviewed or merged. Per the project workflow, it should remain open until the PR is merged.


Automated by CleverAgents Bot
Supervisor: PR Review | Agent: ca-pr-self-reviewer

PR #1123 has been reviewed and **changes requested**. The PR has 1 critical and 5 major protocol conformance issues (JSON-RPC error handling), 3 CONTRIBUTING.md violations, merge conflicts, and a dependency on unmerged PR #1107. See the [PR review comment](https://git.cleverthis.com/cleveragents/cleveragents-core/pulls/1123#issuecomment-101049) for full details. Note: This issue was closed prematurely on 2026-03-23 before the PR was reviewed or merged. Per the project workflow, it should remain open until the PR is merged. --- **Automated by CleverAgents Bot** Supervisor: PR Review | Agent: ca-pr-self-reviewer
Author
Owner

PR #1123 Review Update (2026-04-05)

PR #1123 has been independently reviewed and changes have been requested. The PR cannot be merged in its current state due to:

  1. Merge conflicts — branch has diverged significantly from master
  2. Dependency PR #1107 still open — must merge first
  3. Empty PR body — missing description and Closes #867 keyword
  4. 6 unresolved protocol conformance issues (1 critical, 5 major) — identified by @hurui200320 on 2026-03-30, still unaddressed
  5. CONTRIBUTING.md violationscontext: Any typing, inline imports, # type: ignore suppression

The commit SHA (394c488d) has not changed since the original REQUEST_CHANGES review. All identified issues remain open.

See PR #1123 for the full review.


Automated by CleverAgents Bot
Supervisor: PR Review | Agent: ca-pr-self-reviewer

## PR #1123 Review Update (2026-04-05) PR #1123 has been independently reviewed and **changes have been requested**. The PR cannot be merged in its current state due to: 1. **Merge conflicts** — branch has diverged significantly from `master` 2. **Dependency PR #1107 still open** — must merge first 3. **Empty PR body** — missing description and `Closes #867` keyword 4. **6 unresolved protocol conformance issues** (1 critical, 5 major) — identified by @hurui200320 on 2026-03-30, still unaddressed 5. **CONTRIBUTING.md violations** — `context: Any` typing, inline imports, `# type: ignore` suppression The commit SHA (`394c488d`) has not changed since the original REQUEST_CHANGES review. All identified issues remain open. See [PR #1123](https://git.cleverthis.com/cleveragents/cleveragents-core/pulls/1123) for the full review. --- **Automated by CleverAgents Bot** Supervisor: PR Review | Agent: ca-pr-self-reviewer
Sign in to join this conversation.
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
cleveragents/cleveragents-core#867
No description provided.