Files
cleveragents-core/features
HAL9000 f83708bb34
CI / load-versions (pull_request) Successful in 13s
CI / push-validation (pull_request) Successful in 28s
CI / lint (pull_request) Successful in 50s
CI / typecheck (pull_request) Successful in 1m4s
CI / quality (pull_request) Successful in 58s
CI / security (pull_request) Successful in 1m13s
CI / build (pull_request) Successful in 42s
CI / helm (pull_request) Successful in 39s
CI / unit_tests (pull_request) Successful in 4m40s
CI / docker (pull_request) Successful in 1m33s
CI / integration_tests (pull_request) Successful in 8m30s
CI / coverage (pull_request) Successful in 9m35s
CI / status-check (pull_request) Successful in 3s
fix(a2a): use feature data table as source of truth for symbol list
Reviewer HAL9001 noted that the 22-row data table in the ACP→A2A rename
feature was dead code because the step definition read from a hardcoded
`_ALL_SYMBOLS` constant instead of `context.table.rows`. Also, master
grew `cleveragents.a2a.__all__` to 43 exports (AgentCard*, Sync*,
ConflictResolution, VectorClock) unrelated to the ACP→A2A rename
contract, so the strict `len(__all__) == 22` assertion regressed.

* Add a `| symbol |` header row so Behave parses all 22 entries as data
  rows (the first row was previously being consumed as the header).
* Replace `_ALL_SYMBOLS` with `[row[0].strip() for row in context.table.rows]`
  so the feature file is the single source of truth.
* Change the exports assertion from `len(__all__) == 22` to a subset
  check (every listed symbol is present in `__all__`). Rename the
  scenario and the matching Then step to reflect the corrected intent.

ISSUES CLOSED: #8615
2026-06-15 02:59:42 -04:00
..