cleveragents/cleveragents-core
4
0
Fork 3
Code Issues 6.0k Pull Requests 115 Actions Packages Projects Releases Wiki Activity

feat(agents): add mandatory PR compliance checklist to implementation-pool-supervisor #11052

Closed
HAL9000 wants to merge 3 commits from bugfix/pr-11015-pool-supervisor-checklist into master
pull from: bugfix/pr-11015-pool-supervisor-checklist
merge into: cleveragents:master
Conversation 7 Commits 3 Files Changed 5
+291 -21

3 Commits

Author SHA1 Message Date
HAL9000 307e6aafbc feat(agents): add mandatory PR compliance checklist to implementation-pool-supervisor
CI / benchmark-publish (pull_request) Has been skipped
Details
CI / lint (pull_request) Failing after 13s
Details
CI / quality (pull_request) Failing after 12s
Details
CI / unit_tests (pull_request) Failing after 11s
Details
CI / security (pull_request) Failing after 13s
Details
CI / typecheck (pull_request) Failing after 14s
Details
CI / build (pull_request) Failing after 10s
Details
CI / coverage (pull_request) Has been skipped
Details
CI / docker (pull_request) Has been skipped
Details
CI / e2e_tests (pull_request) Failing after 13s
Details
CI / integration_tests (pull_request) Failing after 15s
Details
CI / helm (pull_request) Failing after 9s
Details
CI / push-validation (pull_request) Failing after 3s
Details
CI / benchmark-regression (pull_request) Failing after 2m1s
Details
CI / status-check (pull_request) Failing after 3s
Details 
Add CHANGELOG.md and CONTRIBUTORS.md entries for the Implementation Pool
Supervisor PR Compliance Checklist feature. This ensures all 8 checklist
items are properly documented before workers create PRs.

ISSUES CLOSED: #11015
 2026-05-12 18:43:40 +00:00
HAL9000 d98221509c fix(agents): resolve unit test failures from duplicate PR compliance checklist step definitions
CI / benchmark-publish (pull_request) Has been skipped
Details
CI / helm (pull_request) Successful in 42s
Details
CI / push-validation (pull_request) Successful in 25s
Details
CI / build (pull_request) Successful in 54s
Details
CI / lint (pull_request) Failing after 1m10s
Details
CI / quality (pull_request) Successful in 1m15s
Details
CI / benchmark-regression (pull_request) Failing after 1m20s
Details
CI / typecheck (pull_request) Successful in 1m30s
Details
CI / security (pull_request) Successful in 1m44s
Details
CI / integration_tests (pull_request) Successful in 4m6s
Details
CI / e2e_tests (pull_request) Successful in 4m13s
Details
CI / unit_tests (pull_request) Successful in 6m19s
Details
CI / coverage (pull_request) Has been skipped
Details
CI / docker (pull_request) Has been skipped
Details
CI / status-check (pull_request) Failing after 3s
Details 
Remove the duplicate pr_compliance_checklist.feature and
pr_compliance_checklist_steps.py files that conflict with the
new pool supervisor test files (identical @then decorators).

Fix PROJECT_ROOT path resolution in pr_compliance_pool_supervisor_steps.py
(parents[3] -> parents[2]) which caused assertion failures when running
tests against the implementation-pool-supervisor.md agent definition.

Unit tests now pass cleanly after these two fixes.

ISSUES CLOSED: #11052
 2026-05-08 23:30:56 +00:00
HAL9000 0558724b5b feat(agents): add mandatory PR compliance checklist to implementation-pool-supervisor
CI / benchmark-publish (pull_request) Has been skipped
Details
CI / lint (pull_request) Failing after 1m13s
Details
CI / benchmark-regression (pull_request) Failing after 1m23s
Details
CI / push-validation (pull_request) Successful in 30s
Details
CI / build (pull_request) Successful in 1m0s
Details
CI / helm (pull_request) Successful in 41s
Details
CI / typecheck (pull_request) Successful in 1m45s
Details
CI / quality (pull_request) Successful in 1m34s
Details
CI / security (pull_request) Successful in 1m44s
Details
CI / unit_tests (pull_request) Failing after 2m55s
Details
CI / docker (pull_request) Has been skipped
Details
CI / coverage (pull_request) Has been skipped
Details
CI / integration_tests (pull_request) Successful in 4m4s
Details
CI / e2e_tests (pull_request) Successful in 4m26s
Details
CI / status-check (pull_request) Failing after 4s
Details 
Add a new implementation-pool-supervisor agent definition with an embedded
8-item PR Compliance Checklist. Workers dispatched by the pool supervisor
must complete all 8 items before creating any PR: CHANGELOG.md update,
CONTRIBUTORS.md update, commit footer (ISSUES CLOSED: #N), CI verification,
BDD tests, Epic reference, label application via forgejo-label-manager,
and milestone assignment. Includes concrete markdown examples for each
subsection and compliance verification pseudocode.

Also adds BDD test coverage (pr_compliance_pool_supervisor.feature + steps)
to verify the pool supervisor agent definition contains all 8 checklist items.

Parent Epic: #9779

ISSUES CLOSED: #9824
 2026-05-08 16:41:10 +00:00