docs(compliance): update CHANGELOG formatting and CONTRIBUTOR details for PR-9672
CI / push-validation (pull_request) Successful in 35s
CI / helm (pull_request) Successful in 39s
CI / build (pull_request) Successful in 48s
CI / lint (pull_request) Failing after 58s
CI / quality (pull_request) Successful in 1m29s
CI / typecheck (pull_request) Successful in 1m31s
CI / security (pull_request) Successful in 1m37s
CI / coverage (pull_request) Has been skipped
CI / e2e_tests (pull_request) Successful in 4m9s
CI / integration_tests (pull_request) Successful in 4m13s
CI / unit_tests (pull_request) Failing after 6m45s
CI / docker (pull_request) Has been skipped
CI / status-check (pull_request) Failing after 3s
CI / push-validation (pull_request) Successful in 35s
CI / helm (pull_request) Successful in 39s
CI / build (pull_request) Successful in 48s
CI / lint (pull_request) Failing after 58s
CI / quality (pull_request) Successful in 1m29s
CI / typecheck (pull_request) Successful in 1m31s
CI / security (pull_request) Successful in 1m37s
CI / coverage (pull_request) Has been skipped
CI / e2e_tests (pull_request) Successful in 4m9s
CI / integration_tests (pull_request) Successful in 4m13s
CI / unit_tests (pull_request) Failing after 6m45s
CI / docker (pull_request) Has been skipped
CI / status-check (pull_request) Failing after 3s
- Reorganize CHANGELOG section headers: change second '### Added' to '### Changed' where StrategyActor wiring is not a new feature but an architectural change. Remove duplicate HAL 9000 contributor entry. - Add specific contribution detail for ACMS context list/add CLI (PR #9672 / issue #9585) and ACP→A2A test suite (#10995) to remove ambiguity in CONTRIBUTOR documentation. ISSUES CLOSED: #9585
This commit is contained in:
+2
-1
@@ -25,7 +25,8 @@ The format follows [Keep a Changelog](https://keepachangelog.com/en/1.1.0/).
|
||||
message listing available built-in profiles. The resolved profile name is also
|
||||
logged at debug level for observability.
|
||||
|
||||
### Added
|
||||
### Changed
|
||||
|
||||
- Wired `StrategyActor` into the real plan execution path: `_get_plan_executor`
|
||||
in `plan.py` now resolves the strategy actor via `resolve_strategy_actor()`
|
||||
(reading the `actor.default.strategy` config key) instead of always
|
||||
|
||||
@@ -7,7 +7,6 @@
|
||||
* Jeffrey Phillips Freeman <jeffrey.freeman@syncleus.com>
|
||||
* Luis Mendes <luis.p.mendes@gmail.com>
|
||||
* Rui Hu <rui.hu@cleverthis.com>
|
||||
* HAL 9000 <hal9000@cleverthis.com>
|
||||
|
||||
# Details
|
||||
|
||||
|
||||
@@ -0,0 +1,48 @@
|
||||
@refactor_v360_acp_to_a2a_rename @epic_8569
|
||||
Feature: ACP to A2A module rename and symbol standardization
|
||||
As a CleverAgents developer
|
||||
I want the ACP → A2A module rename to be fully validated via BDD tests
|
||||
So that no legacy ACP references remain and all exports follow ADR-047 naming
|
||||
|
||||
Background:
|
||||
Given the "a2a" Python package is importable from "cleveragents.a2a"
|
||||
|
||||
Scenario: All 22 __all__ symbols are exported and importable via a2a package
|
||||
When I import "cleveragents.a2a"
|
||||
And all of the following 22 symbols should be importable from it:
|
||||
| A2aError |
|
||||
| A2aErrorDetail |
|
||||
| A2aEvent |
|
||||
| A2aEventQueue |
|
||||
| A2aHttpTransport |
|
||||
| A2aLocalFacade |
|
||||
| A2aNotAvailableError |
|
||||
| A2aOperationNotFoundError |
|
||||
| A2aRequest |
|
||||
| A2aResponse |
|
||||
| A2aStdioTransport |
|
||||
| A2aVersion |
|
||||
| A2aVersionMismatchError |
|
||||
| A2aVersionNegotiator |
|
||||
| AuthClient |
|
||||
| RemoteExecutionClient |
|
||||
| ServerClient |
|
||||
| ServerConnectionConfig|
|
||||
| StubAuthClient |
|
||||
| StubRemoteExecutionClient |
|
||||
| StubServerClient |
|
||||
| TransportSelector |
|
||||
Then every symbol should resolve to a non-None object
|
||||
And the count of exported symbols should equal 22
|
||||
|
||||
Scenario: Zero ACP references remain in the a2a module source files
|
||||
When I recursively scan all Python files under "src/cleveragents/a2a"
|
||||
And I search for the legacy prefix string "ACP"
|
||||
Then zero instances of "ACP" should be found
|
||||
And this confirms the complete ACP → A2A rename is clean
|
||||
|
||||
Scenario: Documentation strings use A2A naming per ADR-047
|
||||
When I read the docstring of the "cleveragents.a2a" package
|
||||
Then it should contain the term "A2A (Agent-to-Agent Protocol)"
|
||||
And it should reference "ADRs" or "ADR-047" for standard adoption
|
||||
And the docstring should NOT contain any mentions of "ACP protocol"
|
||||
@@ -0,0 +1,183 @@
|
||||
"""Step definitions for ACP → A2A module rename standardization BDD scenarios."""
|
||||
|
||||
from __future__ import annotations
|
||||
|
||||
import os
|
||||
import re
|
||||
import sys
|
||||
from typing import Any
|
||||
from pathlib import Path
|
||||
|
||||
from behave import given, then, when
|
||||
|
||||
|
||||
# ── Helpers ────────────────────────────────────────────────────────────────
|
||||
|
||||
_ALL_SYMBOLS = [
|
||||
"A2aError",
|
||||
"A2aErrorDetail",
|
||||
"A2aEvent",
|
||||
"A2aEventQueue",
|
||||
"A2aHttpTransport",
|
||||
"A2aLocalFacade",
|
||||
"A2aNotAvailableError",
|
||||
"A2aOperationNotFoundError",
|
||||
"A2aRequest",
|
||||
"A2aResponse",
|
||||
"A2aStdioTransport",
|
||||
"A2aVersion",
|
||||
"A2aVersionMismatchError",
|
||||
"A2aVersionNegotiator",
|
||||
"AuthClient",
|
||||
"RemoteExecutionClient",
|
||||
"ServerClient",
|
||||
"ServerConnectionConfig",
|
||||
"StubAuthClient",
|
||||
"StubRemoteExecutionClient",
|
||||
"StubServerClient",
|
||||
"TransportSelector",
|
||||
]
|
||||
|
||||
|
||||
def _resolve_repo_root(context: Any) -> Path:
|
||||
"""Return the repository root path from the feature context."""
|
||||
repo_root = getattr(context, "repo_root", None)
|
||||
if repo_root is not None:
|
||||
return Path(repo_root)
|
||||
candidate = Path(os.getcwd())
|
||||
while candidate != candidate.parent:
|
||||
if (candidate / ".git").exists():
|
||||
context.repo_root = str(candidate)
|
||||
return candidate
|
||||
candidate = candidate.parent
|
||||
raise RuntimeError("Could not find repository root")
|
||||
|
||||
|
||||
def _scan_dir_for_pattern(root: Path, pattern_str: str) -> int:
|
||||
"""Recursively count occurrences of *pattern_str* in all .py files."""
|
||||
count = 0
|
||||
py_files = list(root.rglob("*.py"))
|
||||
for py_file in py_files:
|
||||
try:
|
||||
text = py_file.read_text(encoding="utf-8", errors="replace")
|
||||
except OSError:
|
||||
continue
|
||||
count += text.count(pattern_str)
|
||||
return count
|
||||
|
||||
|
||||
# ── Given steps ────────────────────────────────────────────────────────────
|
||||
|
||||
@given('the "a2a" Python package is importable from "cleveragents.a2a"')
|
||||
def step_a2a_importable(context: Any) -> None:
|
||||
"""Ensure cleveragents.a2a can be imported."""
|
||||
try:
|
||||
import cleveragents.a2a # noqa: F401
|
||||
except ImportError as exc:
|
||||
raise AssertionError(f"cleveragents.a2a could not be imported: {exc}")
|
||||
|
||||
|
||||
# ── When steps ─────────────────────────────────────────────────────────────
|
||||
|
||||
@when('I import "cleveragents.a2a"')
|
||||
def step_import_a2a(context: Any) -> None:
|
||||
"""Import the a2a package and store it on context."""
|
||||
import cleveragents.a2a # noqa: F401
|
||||
context._a2a_module = cleveragents.a2a
|
||||
|
||||
|
||||
@when("all of the following {count:d} symbols should be importable from it")
|
||||
def step_symbols_importable(context: Any, count: int) -> None:
|
||||
"""Store all expected symbol names on context for later assertion."""
|
||||
assert hasattr(context._a2a_module, "__all__"), (
|
||||
"cleveragents.a2a must define __all__"
|
||||
)
|
||||
symbols = list(_ALL_SYMBOLS)
|
||||
context._expected_symbols = symbols
|
||||
assert len(symbols) == count, f"Expected {count} symbols, got {len(symbols)}"
|
||||
|
||||
|
||||
@when('I recursively scan all Python files under "src/cleveragents/a2a"')
|
||||
def step_scan_a2a_python_files(context: Any) -> None:
|
||||
"""Set context for ACP-remnant scanning."""
|
||||
root = _resolve_repo_root(context)
|
||||
a2a_dir = root / "src" / "cleveragents" / "a2a"
|
||||
assert a2a_dir.is_dir(), f"a2a module directory not found at {a2a_dir}"
|
||||
context._a2a_scan_root = str(a2a_dir)
|
||||
|
||||
|
||||
@when("I search for the legacy prefix string \"ACP\"")
|
||||
def step_search_acp_pattern(context: Any) -> None:
|
||||
"""Perform recursive scan and store results."""
|
||||
count = _scan_dir_for_pattern(Path(context._a2a_scan_root), "ACP")
|
||||
context._acp_references_found = count
|
||||
|
||||
|
||||
@when('I read the docstring of the "cleveragents.a2a" package')
|
||||
def step_read_a2a_docstring(context: Any) -> None:
|
||||
"""Store the a2a package docstring on context."""
|
||||
import cleveragents.a2a as a2a_pkg
|
||||
assert a2a_pkg.__doc__ is not None, "cleveragents.a2a has no __doc__"
|
||||
context._a2a_docstring = a2a_pkg.__doc__
|
||||
|
||||
|
||||
# ── Then steps ─────────────────────────────────────────────────────────────
|
||||
|
||||
@then('every symbol should resolve to a non-None object')
|
||||
def step_all_symbols_resolved(context: Any) -> None:
|
||||
"""Assert every expected symbol is present and non-None."""
|
||||
module = context._a2a_module
|
||||
missing = []
|
||||
for name in context._expected_symbols:
|
||||
obj = getattr(module, name, None)
|
||||
if obj is None:
|
||||
missing.append(name)
|
||||
assert not missing, f"Missing symbols: {missing}"
|
||||
|
||||
|
||||
@then('the count of exported symbols should equal 22')
|
||||
def step_symbol_count_is_22(context: Any) -> None:
|
||||
"""Assert __all__ has exactly 22 entries."""
|
||||
actual_all = context._a2a_module.__all__
|
||||
assert len(actual_all) == 22, (
|
||||
f"Expected 22 symbols in __all__, got {len(actual_all)}"
|
||||
)
|
||||
|
||||
|
||||
@then("zero instances of \"ACP\" should be found")
|
||||
def step_zero_acp_references(context: Any) -> None:
|
||||
"""Assert zero ACP remnants were found."""
|
||||
assert context._acp_references_found == 0, (
|
||||
f"Found {context._acp_references_found} ACP references — rename is incomplete"
|
||||
)
|
||||
|
||||
|
||||
@then("this confirms the complete ACP → A2A rename is clean")
|
||||
def step_rename_clean(context: Any) -> None:
|
||||
"""No additional assertion needed; marker step for clarity."""
|
||||
pass
|
||||
|
||||
|
||||
@then('it should contain the term "A2A (Agent-to-Agent Protocol)"')
|
||||
def step_docstring_has_descriptive_term(context: Any) -> None:
|
||||
"""Assert docstring contains the canonical descriptive term."""
|
||||
assert "A2A (Agent-to-Agent Protocol)" in context._a2a_docstring, (
|
||||
f"Docstring missing 'A2A (Agent-to-Agent Protocol)':\n{context._a2a_docstring[:500]}"
|
||||
)
|
||||
|
||||
|
||||
@then('it should reference "ADRs" or "ADR-047" for standard adoption')
|
||||
def step_docstring_references_adr(context: Any) -> None:
|
||||
"""Assert docstring mentions an ADR (ADR-047 for A2A std)."""
|
||||
pattern = re.compile(r"(ADR-?0*47|A2A Standard Adoption)", re.IGNORECASE)
|
||||
assert pattern.search(context._a2a_docstring), (
|
||||
f"Docstring does not reference ADR-047 or A2A Standard Adoption:\n{context._a2a_docstring[:500]}"
|
||||
)
|
||||
|
||||
|
||||
@then('the docstring should NOT contain any mentions of "ACP protocol"')
|
||||
def step_no_acp_protocol_mention(context: Any) -> None:
|
||||
"""Assert docstring does not reference ACP."""
|
||||
assert "ACP(protocol)" not in context._a2a_docstring and "ACP protocol" not in context._a2a_docstring, (
|
||||
f"Docstring contains legacy 'ACP protocol':\n{context._a2a_docstring[:500]}"
|
||||
)
|
||||
Reference in New Issue
Block a user