cleveragents/cleveragents-core

Fork 3

UAT: Stale test/UAT script files in project root violate file organization rules #5641

New issue

Open

opened 2026-04-09 08:02:34 +00:00 by HAL9000 · 1 comment

HAL9000 commented

2026-04-09 08:02:34 +00:00

Owner

Background and Context

CONTRIBUTING.md states under File Organization:

All project files should be organized in appropriate subdirectories rather than placed in the project root. The root directory should be reserved for standard ecosystem files that conventionally live there (e.g., project manifests, lock files, CI configurations, .gitignore, README, CONTRIBUTING, LICENSE, and similar well-known files). Any file that is not a widely recognized root-level convention belongs in a subdirectory.

Each directory should have a clear, single purpose — do not mix concerns. For example, source code directories must not contain test files, documentation, mocking code, or examples. Test directories must not contain production source code.

Current Behavior

The following test/UAT script files exist in the project root directory, violating the file organization rules:

test_a2a_sdk.py — ad-hoc test script for A2A SDK imports
test_a2a_error_detail.py — ad-hoc test script for A2aErrorDetail serialization
test_acp_import.py — ad-hoc test script for ACP module import check
uat_test_a2a.py — UAT test script for A2A facade
uat_test_guard_order.py — UAT test script for guard evaluation order

Additionally, numerous other ad-hoc analysis/debug scripts exist in the root:

check_resource_mgmt.py
decode_cli.py, decode_feature.py, decode_files.py, decode_robot.py, decode_timeline.py
fetch_repo.py
fix_labels.py.done
plan_cli_decoded.py, plan_lifecycle_service_decoded.py

These files appear to be leftover debugging/investigation scripts that were never cleaned up.

Expected Behavior

Test scripts should be in features/ (Behave) or robot/ (Robot Framework)
Ad-hoc investigation scripts should not be committed to the repository
The project root should contain only standard ecosystem files

Acceptance Criteria

All test_*.py files in the project root are removed or moved to appropriate test directories
All uat_test_*.py files in the project root are removed or converted to proper Behave/Robot tests
All ad-hoc debug/analysis scripts in the project root are removed
Project root contains only standard ecosystem files

Supporting Information

Files found in project root that violate the rule:

test_a2a_sdk.py
test_a2a_error_detail.py
test_acp_import.py
uat_test_a2a.py
uat_test_guard_order.py
check_resource_mgmt.py
decode_cli.py
decode_feature.py
decode_files.py
decode_robot.py
decode_timeline.py
fetch_repo.py
plan_cli_decoded.py
plan_lifecycle_service_decoded.py

Metadata

Commit Message: chore(cleanup): remove stale test and debug scripts from project root
Branch: chore/cleanup-root-scripts

Subtasks

Remove or move all test_*.py files from project root
Remove or move all uat_test_*.py files from project root
Remove all ad-hoc debug/analysis scripts from project root
Verify no useful test coverage is lost

Definition of Done

This issue is complete when:

All subtasks above are completed and checked off.
A Git commit is created with the prescribed commit message.
The commit is pushed to the branch and submitted as a pull request to master, reviewed, and merged.

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

## Background and Context CONTRIBUTING.md states under **File Organization**: > All project files should be organized in appropriate subdirectories rather than placed in the project root. The root directory should be reserved for standard ecosystem files that conventionally live there (e.g., project manifests, lock files, CI configurations, `.gitignore`, `README`, `CONTRIBUTING`, `LICENSE`, and similar well-known files). Any file that is not a widely recognized root-level convention belongs in a subdirectory. > Each directory should have a clear, single purpose — do not mix concerns. For example, source code directories must not contain test files, documentation, mocking code, or examples. Test directories must not contain production source code. ## Current Behavior The following test/UAT script files exist in the project root directory, violating the file organization rules: - `test_a2a_sdk.py` — ad-hoc test script for A2A SDK imports - `test_a2a_error_detail.py` — ad-hoc test script for A2aErrorDetail serialization - `test_acp_import.py` — ad-hoc test script for ACP module import check - `uat_test_a2a.py` — UAT test script for A2A facade - `uat_test_guard_order.py` — UAT test script for guard evaluation order Additionally, numerous other ad-hoc analysis/debug scripts exist in the root: - `check_resource_mgmt.py` - `decode_cli.py`, `decode_feature.py`, `decode_files.py`, `decode_robot.py`, `decode_timeline.py` - `fetch_repo.py` - `fix_labels.py.done` - `plan_cli_decoded.py`, `plan_lifecycle_service_decoded.py` These files appear to be leftover debugging/investigation scripts that were never cleaned up. ## Expected Behavior - Test scripts should be in `features/` (Behave) or `robot/` (Robot Framework) - Ad-hoc investigation scripts should not be committed to the repository - The project root should contain only standard ecosystem files ## Acceptance Criteria - [ ] All `test_*.py` files in the project root are removed or moved to appropriate test directories - [ ] All `uat_test_*.py` files in the project root are removed or converted to proper Behave/Robot tests - [ ] All ad-hoc debug/analysis scripts in the project root are removed - [ ] Project root contains only standard ecosystem files ## Supporting Information Files found in project root that violate the rule: ``` test_a2a_sdk.py test_a2a_error_detail.py test_acp_import.py uat_test_a2a.py uat_test_guard_order.py check_resource_mgmt.py decode_cli.py decode_feature.py decode_files.py decode_robot.py decode_timeline.py fetch_repo.py plan_cli_decoded.py plan_lifecycle_service_decoded.py ``` ## Metadata - **Commit Message**: `chore(cleanup): remove stale test and debug scripts from project root` - **Branch**: `chore/cleanup-root-scripts` ## Subtasks - [ ] Remove or move all `test_*.py` files from project root - [ ] Remove or move all `uat_test_*.py` files from project root - [ ] Remove all ad-hoc debug/analysis scripts from project root - [ ] Verify no useful test coverage is lost ## Definition of Done This issue is complete when: - All subtasks above are completed and checked off. - A Git commit is created with the prescribed commit message. - The commit is pushed to the branch and submitted as a pull request to `master`, reviewed, and merged. --- **Automated by CleverAgents Bot** Supervisor: UAT Testing | Agent: uat-tester

HAL9000 added the

labels

2026-04-09 08:03:05 +00:00

HAL9000 added this to the v3.2.0 milestone

2026-04-09 08:05:56 +00:00

HAL9000 commented

2026-04-09 08:07:19 +00:00

Author

Owner

Label compliance fix applied:

Added missing labels and/or milestone to bring issue into compliance with CONTRIBUTING.md

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

Label compliance fix applied: - Added missing labels and/or milestone to bring issue into compliance with CONTRIBUTING.md --- **Automated by CleverAgents Bot** Supervisor: Backlog Grooming | Agent: backlog-groomer

HAL9000 added

and removed

labels

2026-04-09 08:08:56 +00:00

HAL9000 added a new dependency

2026-04-09 08:26:36 +00:00