cleveragents/cleveragents-core

Fork 3

UAT: `StubServerClient`, `StubRemoteExecutionClient`, `StubAuthClient` raise `NotImplementedError` — no real server client implementations exist #4908

New issue

Open

opened 2026-04-08 20:18:30 +00:00 by HAL9000 · 2 comments

HAL9000 commented

2026-04-08 20:18:30 +00:00

Owner

Bug Report

Feature Area: Server Mode — server client protocol implementations

What Was Tested

The server client protocol implementations (clients.py) for server health checks, remote plan execution, and authentication.

Expected Behavior (from spec)

Per ADR-023 (Server Mode) and ADR-048 (Server Application Architecture), the client-side of server mode requires:

ServerClient — health check and version negotiation with the server
RemoteExecutionClient — submit plans for remote execution via LangGraph Platform RemoteGraph, query plan status, cancel plans
AuthClient — authenticate with the server using Bearer token, validate tokens

These clients should implement the A2A protocol over HTTP to communicate with the CleverAgents server.

Actual Behavior (from code)

File: src/cleveragents/a2a/clients.py

All three stub implementations raise NotImplementedError for every method:

class StubServerClient:
    def health_check(self) -> bool:
        raise NotImplementedError("Server client is not yet implemented")

    def get_version(self) -> str:
        raise NotImplementedError("Server client is not yet implemented")

class StubRemoteExecutionClient:
    def execute_plan(self, plan_id: str) -> dict[str, Any]:
        raise NotImplementedError("Server client is not yet implemented")

    def get_plan_status(self, plan_id: str) -> dict[str, Any]:
        raise NotImplementedError("Server client is not yet implemented")

    def cancel_plan(self, plan_id: str) -> bool:
        raise NotImplementedError("Server client is not yet implemented")

class StubAuthClient:
    def authenticate(self, token: str) -> bool:
        raise NotImplementedError("Server client is not yet implemented")

    def validate_token(self, token: str) -> bool:
        raise NotImplementedError("Server client is not yet implemented")

No concrete implementations of ServerClient, RemoteExecutionClient, or AuthClient exist in the codebase. The Protocol interfaces are defined but never implemented.

Steps to Reproduce

from cleveragents.a2a.clients import StubRemoteExecutionClient

client = StubRemoteExecutionClient()
client.execute_plan("plan-123")
# Raises: NotImplementedError: Server client is not yet implemented

Code Location

src/cleveragents/a2a/clients.py — all stub implementations

Impact

Remote plan execution via LangGraph Platform RemoteGraph is completely non-functional. The spec's core server mode feature (executing plans on shared infrastructure) cannot work without a real RemoteExecutionClient implementation. Authentication with the server is also blocked.

Automated by CleverAgents Bot
Supervisor: UAT Testing | Agent: uat-tester

## Bug Report **Feature Area:** Server Mode — server client protocol implementations ### What Was Tested The server client protocol implementations (`clients.py`) for server health checks, remote plan execution, and authentication. ### Expected Behavior (from spec) Per ADR-023 (Server Mode) and ADR-048 (Server Application Architecture), the client-side of server mode requires: 1. **`ServerClient`** — health check and version negotiation with the server 2. **`RemoteExecutionClient`** — submit plans for remote execution via LangGraph Platform RemoteGraph, query plan status, cancel plans 3. **`AuthClient`** — authenticate with the server using Bearer token, validate tokens These clients should implement the A2A protocol over HTTP to communicate with the CleverAgents server. ### Actual Behavior (from code) **File:** `src/cleveragents/a2a/clients.py` All three stub implementations raise `NotImplementedError` for every method: ```python class StubServerClient: def health_check(self) -> bool: raise NotImplementedError("Server client is not yet implemented") def get_version(self) -> str: raise NotImplementedError("Server client is not yet implemented") class StubRemoteExecutionClient: def execute_plan(self, plan_id: str) -> dict[str, Any]: raise NotImplementedError("Server client is not yet implemented") def get_plan_status(self, plan_id: str) -> dict[str, Any]: raise NotImplementedError("Server client is not yet implemented") def cancel_plan(self, plan_id: str) -> bool: raise NotImplementedError("Server client is not yet implemented") class StubAuthClient: def authenticate(self, token: str) -> bool: raise NotImplementedError("Server client is not yet implemented") def validate_token(self, token: str) -> bool: raise NotImplementedError("Server client is not yet implemented") ``` No concrete implementations of `ServerClient`, `RemoteExecutionClient`, or `AuthClient` exist in the codebase. The Protocol interfaces are defined but never implemented. ### Steps to Reproduce ```python from cleveragents.a2a.clients import StubRemoteExecutionClient client = StubRemoteExecutionClient() client.execute_plan("plan-123") # Raises: NotImplementedError: Server client is not yet implemented ``` ### Code Location - `src/cleveragents/a2a/clients.py` — all stub implementations ### Impact Remote plan execution via LangGraph Platform RemoteGraph is completely non-functional. The spec's core server mode feature (executing plans on shared infrastructure) cannot work without a real `RemoteExecutionClient` implementation. Authentication with the server is also blocked. --- **Automated by CleverAgents Bot** Supervisor: UAT Testing | Agent: uat-tester

HAL9000 referenced this issue

2026-04-08 20:18:58 +00:00

[Automated] CleverAgents Build Session - 2026-04-08 #4799

HAL9000 referenced this issue

2026-04-08 20:22:25 +00:00

[Automated] CleverAgents Build Session - 2026-04-08 #4799

HAL9000 added the

labels

2026-04-08 22:59:16 +00:00

HAL9000 added this to the v3.5.0 milestone

2026-04-08 23:01:01 +00:00

HAL9000 commented

2026-04-08 23:01:18 +00:00

Author

Owner

Issue triaged by project owner:

State: Verified
Priority: Medium — server client stubs are non-functional; remote plan execution via LangGraph Platform is blocked; however, server mode is a v3.5.0+ feature
Milestone: v3.5.0 — Server mode and A2A client implementations are scoped to Autonomy Hardening milestone per ADR-048
Story Points: 8 — XL — requires implementing ServerClient, RemoteExecutionClient, and AuthClient with full A2A HTTP protocol; significant work
MoSCoW: Should Have — server mode is a key feature for the v3.5.0 milestone; stubs are expected at this stage but need real implementations before milestone completion
Parent Epic: A2A/Server Mode epic (v3.5.0)

Automated by CleverAgents Bot
Supervisor: Project Owner | Agent: project-owner

Issue triaged by project owner: - **State**: Verified - **Priority**: Medium — server client stubs are non-functional; remote plan execution via LangGraph Platform is blocked; however, server mode is a v3.5.0+ feature - **Milestone**: v3.5.0 — Server mode and A2A client implementations are scoped to Autonomy Hardening milestone per ADR-048 - **Story Points**: 8 — XL — requires implementing `ServerClient`, `RemoteExecutionClient`, and `AuthClient` with full A2A HTTP protocol; significant work - **MoSCoW**: Should Have — server mode is a key feature for the v3.5.0 milestone; stubs are expected at this stage but need real implementations before milestone completion - **Parent Epic**: A2A/Server Mode epic (v3.5.0) --- **Automated by CleverAgents Bot** Supervisor: Project Owner | Agent: project-owner

freemo added a new dependency

2026-04-08 23:42:08 +00:00

#4956 EPIC: Server Client Implementations — Replace Stubs with Real Clients

freemo referenced this issue

2026-04-08 23:43:07 +00:00

LEGENDARY: Autonomy Hardening — Full Autonomous Execution, Safety & A2A Facade #4944

HAL9000 added the

Points

label

2026-04-09 05:42:45 +00:00

HAL9000 commented

2026-04-09 05:43:05 +00:00

Author

Owner

Label compliance fix applied:

Added missing label: Points/3 (M — medium complexity)
Reason: Issue is in State/Verified but was missing a story points estimate. Estimated as Points/3 (M) based on single-area bug fix with moderate complexity.

Automated by CleverAgents Bot
Supervisor: Backlog Grooming | Agent: backlog-groomer

Label compliance fix applied: - Added missing label: `Points/3` (M — medium complexity) - Reason: Issue is in `State/Verified` but was missing a story points estimate. Estimated as Points/3 (M) based on single-area bug fix with moderate complexity. --- **Automated by CleverAgents Bot** Supervisor: Backlog Grooming | Agent: backlog-groomer