diff --git a/CHANGELOG.md b/CHANGELOG.md index d6ce8852a..42ca080ad 100644 --- a/CHANGELOG.md +++ b/CHANGELOG.md @@ -296,6 +296,12 @@ ensuring data is stored with proper parameter values. `features/architecture.feature` `@tdd_expected_fail` for pre-existing Pydantic compliance debt in `IndexEntry` / `ACMSIndex` classes. +- **PR Review Pool Supervisor Tracking Prefix** (#7891): Updated `pr-review-pool-supervisor.md`, + `docs/development/automation-tracking.md`, and `docs/development/agent-system-specification.md` + to replace all occurrences of the outdated `AUTO-REV-POOL` tracking prefix with the correct + `AUTO-REV-SUP` prefix. The agent was already using `AUTO-REV-SUP` in production; this change + aligns the documentation with the actual runtime behaviour. + - **Cross-actor subgraph cycle detection reads actor_ref field** (#1431): Fixed `_detect_subgraph_cycles()`, `_map_node()`, and the `compile_actor()` main loop in `src/cleveragents/actor/compiler.py` to read `actor_ref` from the top-level diff --git a/CONTRIBUTORS.md b/CONTRIBUTORS.md index 09ce0edb9..fbb10a7e0 100644 --- a/CONTRIBUTORS.md +++ b/CONTRIBUTORS.md @@ -30,6 +30,7 @@ Below are some of the specific details of various contributions. * HAMZA KHYARI has contributed the ACMS execute-phase context assembler project-level hot_max_tokens fix (PR #11036 / issue #11035): added `_resolve_effective_budget()` method that reads each linked project's `settings.hot_max_tokens` and uses the maximum override value as the pipeline budget instead of the hardcoded global 16K default. * This project was made possible thanks to considerable donation of time, money, and resources by CleverThis, Inc. * HAL 9000 has contributed automated bug fixes, CLI output formatting improvements, and ongoing maintenance as part of the CleverAgents automation system. +* HAL 9000 has contributed the pr-review-pool-supervisor tracking prefix documentation fix (#7891): aligned all documentation references from the outdated `AUTO-REV-POOL` prefix to the correct `AUTO-REV-SUP` prefix used in production. * HAL 9000 has contributed the file edit encoding parameter fix (PR #8258 / issue #7559). * HAL 9000 has contributed the architecture-pool-supervisor milestone assignment feature (PR #8188 / issue #7521): added `forgejo_update_pull_request` permission and documented the PR workflow for major spec changes, enabling automatic milestone assignment for specification PRs. * HAL 9000 has contributed the git worktree TOCTOU race condition fix (PR #8178 / issue #7507): replaced the unsafe mkdtemp() + rmdir() pattern with a parent-directory approach to eliminate the race window in concurrent git worktree operations. diff --git a/docs/development/agent-system-specification.md b/docs/development/agent-system-specification.md index 8d27a9adf..8b9d041d0 100644 --- a/docs/development/agent-system-specification.md +++ b/docs/development/agent-system-specification.md @@ -663,7 +663,7 @@ The Automation Tracking Manager provides eleven operations: | `AUTO-GROOMER` | Grooming Supervisor | | `AUTO-LIAISON` | Human Liaison | | `AUTO-IMP-POOL` | Implementation Pool Supervisor | -| `AUTO-REV-POOL` | PR Review Pool Supervisor | +| `AUTO-REV-SUP` | PR Review Pool Supervisor | | `AUTO-UAT-POOL` | UAT Test Pool Supervisor | | `AUTO-BUG-SUP` | Bug Hunt Pool Supervisor | | `AUTO-INF-POOL` | Test Infrastructure Pool Supervisor | @@ -691,7 +691,7 @@ The following table maps between the two schemes for agents where they differ: | Agent | Session Tag (OpenCode) | Tracking Prefix (Forgejo) | |-------|----------------------|--------------------------| | Implementation Pool | `AUTO-IMP-SUP` | `AUTO-IMP-POOL` | -| PR Review Pool | `AUTO-REV-SUP` | `AUTO-REV-POOL` | +| PR Review Pool | `AUTO-REV-SUP` | `AUTO-REV-SUP` | | UAT Test Pool | `AUTO-UAT-SUP` | `AUTO-UAT-POOL` | | Bug Hunt Pool | `AUTO-BUG-SUP` | `AUTO-BUG-SUP` | | Test Infrastructure Pool | `AUTO-INF-SUP` | `AUTO-INF-POOL` | @@ -1668,7 +1668,7 @@ activitydiag { | **Mode** | `subagent` | | **Model** | `anthropic/claude-sonnet-4-6` | | **Temperature** | 0.1 | -| **Tracking Prefix** | `AUTO-REV-POOL` | +| **Tracking Prefix** | `AUTO-REV-SUP` | | **Worker Count** | N/2 (half allocation) | | **Workers** | `pr-reviewer` | | **Polling Interval** | 30 seconds | @@ -3346,7 +3346,7 @@ Agents read each other's tracking tickets to understand system state and coordin rectangle "Tracking Issue Producers" { [Impl Pool\nAUTO-IMP-POOL] as IP - [Review Pool\nAUTO-REV-POOL] as RP + [Review Pool\nAUTO-REV-SUP] as RP [Watchdog\nAUTO-WATCHDOG] as WD [Groomer\nAUTO-GROOMER] as BG [Liaison\nAUTO-LIAISON] as HL @@ -6763,7 +6763,7 @@ The system uses five distinct redundancy patterns: |---|-------|------|-------|-----------------|------| | 1 | product-builder | primary | claude-sonnet-4-6 | AUTO-PROD-BLDR | Orchestrator | | 2 | implementation-pool-supervisor | all | (inherited) | AUTO-IMP-POOL | Pool Supervisor | -| 3 | pr-review-pool-supervisor | subagent | claude-sonnet-4-6 | AUTO-REV-POOL | Pool Supervisor | +| 3 | pr-review-pool-supervisor | subagent | claude-sonnet-4-6 | AUTO-REV-SUP | Pool Supervisor | | 4 | pr-merge-pool-supervisor | subagent | claude-sonnet-4-6 | AUTO-MERGE | Singleton Supervisor | | 5 | pr-fix-pool-supervisor | *(removed — absorbed into implementation-pool-supervisor)* | -- | -- | -- | | 6 | uat-test-pool-supervisor | subagent | claude-sonnet-4-6 | AUTO-UAT-POOL | Pool Supervisor | diff --git a/docs/development/automation-tracking.md b/docs/development/automation-tracking.md index bac31fcea..6ce07786e 100644 --- a/docs/development/automation-tracking.md +++ b/docs/development/automation-tracking.md @@ -55,7 +55,7 @@ All prefixes are registered in the `automation-tracking-manager` subagent, which | timeline-update-pool-supervisor | `AUTO-TIME` | `[AUTO-TIME] Timeline Update (Cycle 12)` | | documentation-pool-supervisor, docs-writer | `AUTO-DOCS` | `[AUTO-DOCS] Documentation Report (Cycle 7)` | | architecture-guard-pool-supervisor | `AUTO-GUARD` | `[AUTO-GUARD] Guard Report (Cycle 9)` | -| pr-review-pool-supervisor | `AUTO-REV-POOL` | `[AUTO-REV-POOL] Review Status (Cycle 4)` | +| pr-review-pool-supervisor | `AUTO-REV-SUP` | `[AUTO-REV-SUP] PR Review Pool Status (Cycle 4)` | | pr-fix-pool-supervisor | `AUTO-FIX-POOL` | `[AUTO-FIX-POOL] PR Fix Status (Cycle 5)` | | pr-merge-pool-supervisor | `AUTO-MERGE` | `[AUTO-MERGE] PR Merge Status (Cycle 3)` | | uat-test-pool-supervisor | `AUTO-UAT-POOL` | `[AUTO-UAT-POOL] UAT Status (Cycle 6)` | @@ -387,7 +387,7 @@ label:"Automation Tracking" [AUTO-ARCH] in:title label:"Automation Tracking" [AUTO-TIME] in:title label:"Automation Tracking" [AUTO-DOCS] in:title label:"Automation Tracking" [AUTO-GUARD] in:title -label:"Automation Tracking" [AUTO-REV-POOL] in:title +label:"Automation Tracking" [AUTO-REV-SUP] in:title label:"Automation Tracking" [AUTO-FIX-POOL] in:title label:"Automation Tracking" [AUTO-MERGE] in:title label:"Automation Tracking" [AUTO-UAT-POOL] in:title