From ee051ca00d71fe134456243ab5675d7e8d27b8dd Mon Sep 17 00:00:00 2001 From: CleverThis Date: Tue, 5 May 2026 16:22:37 +0000 Subject: [PATCH] chore(agents): clean up CONTRIBUTORS.md duplicate entry and PR artifacts After the rebase onto master, the pr-review-pool-supervisor.md file was deleted (replaced with pr-review-supervisor.md which already uses correct AUTO-REV prefix). This commit cleans up remaining branch quality issues: - Remove duplicate HAL 9000 entry in CONTRIBUTORS.md from rebase conflict resolution - Clean up commit message artifacts - Update CHANGELOG entry to reference the actual file (pr-review-supervisor.md) ISSUES CLOSED: #7891 --- CHANGELOG.md | 5 +++++ CONTRIBUTORS.md | 1 - 2 files changed, 5 insertions(+), 1 deletion(-) diff --git a/CHANGELOG.md b/CHANGELOG.md index 7997c9251..31bec6f2d 100644 --- a/CHANGELOG.md +++ b/CHANGELOG.md @@ -491,6 +491,11 @@ The format follows [Keep a Changelog](https://keepachangelog.com/en/1.1.0/). ### Fixed +- **PR Review Pool Supervisor Tracking Prefix** (#7891): Fixed inconsistency in + the PR review supervisor agent definition where `AUTO-REV-POOL` was documented as + the tracking prefix but the running agent uses `AUTO-REV-SUP` in practice. Superseded by + the file rename and rewrite from `pr-review-pool-supervisor.md` → `pr-review-supervisor.md`. + - **Plan Concurrency Race Condition** (#7989): Fixed critical race condition in `execute_plan()` and `apply_plan()` where concurrent CLI/worker sessions could simultaneously modify the same plan, corrupting plan state. `LockService` is now wired into the plan lifecycle with plan-level advisory diff --git a/CONTRIBUTORS.md b/CONTRIBUTORS.md index 233e467bc..7f664231c 100644 --- a/CONTRIBUTORS.md +++ b/CONTRIBUTORS.md @@ -7,7 +7,6 @@ * Jeffrey Phillips Freeman * Luis Mendes * Rui Hu -* HAL 9000 # Details -- 2.52.0