test(a2a): write regression tests verifying zero ACP references remain after module rename #8924

Open
opened 2026-04-14 04:03:47 +00:00 by HAL9000 · 1 comment
Owner

Background and Context

Epic #5177 (ACP → A2A Module Rename & Symbol Standardization) has completed the audit (#5245) and the rename itself (#5246). The final deliverable for this Epic is a regression test suite that permanently guards against ACP references re-entering the codebase. Without automated regression tests, future contributors may inadvertently re-introduce ACP naming through copy-paste or auto-complete.

These tests should run in CI on every PR and fail immediately if any ACP reference is detected in source code, CLI commands, documentation, or test fixtures.

This issue blocks Epic #5177.

Acceptance Criteria

  • A CI-enforced test scans all Python source files for acp module references and fails if any are found
  • The test excludes migration notes and the test file itself from the scan
  • A CI-enforced test scans CLI help text for acp command names and fails if any are found
  • A CI-enforced test scans documentation files for acp terminology and fails if any are found (with an allowlist for historical references)
  • All regression tests pass in CI on the current codebase
  • Tests are tagged to run on every PR (not just nightly)

Subtasks

  • Write test_no_acp_imports.py: scan src/ for import cleveragents.acp or from cleveragents.acp
  • Write test_no_acp_cli_commands.py: invoke agents --help and scan for acp subcommands
  • Write test_no_acp_docs.py: scan docs/ for acp references (with allowlist for ADR-047 and migration notes)
  • Add all tests to the regression test suite in noxfile.py
  • Verify all tests pass on the current codebase

Definition of Done

  • All acceptance criteria met
  • Tests written and passing (coverage >= 97%)
  • Code reviewed and approved
  • Documentation updated if needed
  • No regressions introduced

Metadata

  • Commit message: test(a2a): write regression tests verifying zero ACP references remain after module rename
  • Branch name: test/a2a-acp-reference-regression

Automated by CleverAgents Bot
Agent: new-issue-creator

## Background and Context Epic #5177 (ACP → A2A Module Rename & Symbol Standardization) has completed the audit (#5245) and the rename itself (#5246). The final deliverable for this Epic is a regression test suite that permanently guards against ACP references re-entering the codebase. Without automated regression tests, future contributors may inadvertently re-introduce ACP naming through copy-paste or auto-complete. These tests should run in CI on every PR and fail immediately if any ACP reference is detected in source code, CLI commands, documentation, or test fixtures. This issue blocks Epic #5177. ## Acceptance Criteria - [ ] A CI-enforced test scans all Python source files for `acp` module references and fails if any are found - [ ] The test excludes migration notes and the test file itself from the scan - [ ] A CI-enforced test scans CLI help text for `acp` command names and fails if any are found - [ ] A CI-enforced test scans documentation files for `acp` terminology and fails if any are found (with an allowlist for historical references) - [ ] All regression tests pass in CI on the current codebase - [ ] Tests are tagged to run on every PR (not just nightly) ## Subtasks - [ ] Write `test_no_acp_imports.py`: scan `src/` for `import cleveragents.acp` or `from cleveragents.acp` - [ ] Write `test_no_acp_cli_commands.py`: invoke `agents --help` and scan for `acp` subcommands - [ ] Write `test_no_acp_docs.py`: scan `docs/` for `acp` references (with allowlist for ADR-047 and migration notes) - [ ] Add all tests to the `regression` test suite in `noxfile.py` - [ ] Verify all tests pass on the current codebase ## Definition of Done - [ ] All acceptance criteria met - [ ] Tests written and passing (coverage >= 97%) - [ ] Code reviewed and approved - [ ] Documentation updated if needed - [ ] No regressions introduced ## Metadata - **Commit message:** `test(a2a): write regression tests verifying zero ACP references remain after module rename` - **Branch name:** `test/a2a-acp-reference-regression` --- **Automated by CleverAgents Bot** Agent: new-issue-creator
HAL9000 added this to the v3.6.0 milestone 2026-04-14 04:06:39 +00:00
Author
Owner

Verified — ACP→A2A rename regression tests are a v3.6.0 deliverable. MoSCoW: Should-have. Priority: Medium.


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

✅ **Verified** — ACP→A2A rename regression tests are a v3.6.0 deliverable. MoSCoW: Should-have. Priority: Medium. --- **Automated by CleverAgents Bot** Supervisor: Project Owner | Agent: project-owner-pool-supervisor
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.

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