Skip to content

agents: review-warden, a profile agent for the review-speed watch (Closes #6088) - #6089

Merged
gHashTag merged 4 commits into
masterfrom
feat/review-warden-agent
Oct 4, 2026
Merged

gHashTag merged 4 commits into
masterfrom
feat/review-warden-agent

Conversation

@gHashTag

@gHashTag gHashTag commented Oct 4, 2026

Copy link
Copy Markdown
Owner

Closes #6088

What

.claude/agents/review-warden.md: a profile agent for the review-speed watch (#5776, epic #5906). It says where the plan, the reviewer, the Queen's status and the task-shape checker are, and gives one round:

  1. measure with read-only cards (tri review-log tri review-log: a health card from a window of the reviewer bee's own log (scripts/tri_loop/review_log.py) #6086, tri epic-ci tri epic-ci: which checks of an epic's pull requests turned red since the last snapshot (scripts/tri_loop/epic_ci.py) #6087, tri queen-log tri queen-log: a health card from a window of the Queen engine's own log (scripts/tri_loop/queen_log.py) #6056, reviewer.py tick/doctor);
  2. name a cause only from a measurement;
  3. hand the fix to the Queen as an issue that passes tools/queen/task_shape.py, with every expected output computed by a reference run before filing;
  4. watch it land (/queen/status, the board, a queen-<N> branch);
  5. report in three lines.

It points at the rules' homes (the plan, the reviewer's docstring) instead of copying them, and it lists what the agent never does: merge, approve, auto-merge, post as the bot, print a secret, build or test on the workstation.

Why

On 2026-10-04 every reviewer run ended on HTTP 504 for three hours and two watch ticks read it as transient. The watch had no written method outside one chat session. The tools it names are being written by the Queen's bees (#6086, #6087, #6056); until they land, the agent's round still runs with reviewer.py tick and /queen/status.

Checks

  • One file, 38 lines, ASCII only (LC_ALL=C grep -cP '[^\x00-\x7F]' prints 0).
  • Committed onto master 2e852cf82 without a checkout (the workstation is overloaded; nothing to build).

🤖 Generated with Claude Code

…oses #6088)

The review-speed work (#5776, epic #5906) was watched by one chat session's
cron ticks. This names the job for any session: where the plan, the reviewer,
the Queen's status and the task-shape checker are; the round (measure, hand
the fix to the Queen as a task that passes tools/queen/task_shape.py, watch it
land, report); and what is never done. It points at the rules' homes instead
of copying them.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@github-actions

github-actions Bot commented Oct 4, 2026

Copy link
Copy Markdown
Contributor

PR Dashboard

Generated at: 2026-10-04 13:56:15 UTC

Summary

Status Count
Total Open PRs 50
PRs with Failing Checks 41
PRs with All Checks Green 9
READY 8
FAILING 41
PENDING 0
NO CHECKS YET 0

These columns do not partition: 8 + 41 + 0 + 0 = 49, and there are 50 open PRs. A PR is being counted twice or not at all.

Seal Status

  • ⚠️ STALE -- sha256(compiler.rs)=a47525693295 != manifest seal=87e5cbd3ad94.
    The committed NMSE numbers were certified against an older compiler.rs.
    Run scripts/reseal-check.sh locally for the two-step reseal command (advisory; not a merge gate).

… and read her review events (Refs #6088)

task_shape.py said ready for three filed tasks whose criteria she could not
use: numbered lines she does not read, and a check her runner refuses
(an absolute path). Measured with tools/queen/criteria_backfill.py, the
twin pinned to her parser; her public activity showed two escalations.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@github-actions

github-actions Bot commented Oct 4, 2026

Copy link
Copy Markdown
Contributor

PR Dashboard

Generated at: 2026-10-04 15:03:41 UTC

Summary

Status Count
Total Open PRs 50
PRs with Failing Checks 40
PRs with All Checks Green 10
READY 9
FAILING 40
PENDING 0
NO CHECKS YET 0

These columns do not partition: 9 + 40 + 0 + 0 = 49, and there are 50 open PRs. A PR is being counted twice or not at all.

Seal Status

  • ⚠️ STALE -- sha256(compiler.rs)=32e37152eb23 != manifest seal=87e5cbd3ad94.
    The committed NMSE numbers were certified against an older compiler.rs.
    Run scripts/reseal-check.sh locally for the two-step reseal command (advisory; not a merge gate).

…im and ledger, installed tools, reports on #5776 (Refs #6088)

The scheduled task review-warden-round loads this file every 2 h. Tools that are not on master (reviewer.py, criteria_backfill.py, the plan) are named by their installed copies or their branch; tri cards still in the Queen's hands are skipped until they land; the round never uses the Queen's token.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@gHashTag

gHashTag commented Oct 4, 2026

Copy link
Copy Markdown
Owner Author

6764a9f27: the warden now runs without the chat that built it.

Same shape as t27c-steward (#6097) and queen-reel-keeper (999 #3590).

First delegated task through this path: #6099 was accepted by the Queen at 15:11:39Z. queen-6099 (5460c0eed) is the reference diff exactly: tools/duplicate_bodies_baseline.txt, +4/-0.

@github-actions

github-actions Bot commented Oct 4, 2026

Copy link
Copy Markdown
Contributor

PR Dashboard

Generated at: 2026-10-04 15:32:58 UTC

Summary

Status Count
Total Open PRs 50
PRs with Failing Checks 41
PRs with All Checks Green 9
READY 7
FAILING 41
PENDING 0
NO CHECKS YET 0

These columns do not partition: 7 + 41 + 0 + 0 = 48, and there are 50 open PRs. A PR is being counted twice or not at all.

Seal Status

  • ⚠️ STALE -- sha256(compiler.rs)=32e37152eb23 != manifest seal=87e5cbd3ad94.
    The committed NMSE numbers were certified against an older compiler.rs.
    Run scripts/reseal-check.sh locally for the two-step reseal command (advisory; not a merge gate).

…ces a report token (Refs #5776)

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@github-actions

github-actions Bot commented Oct 4, 2026

Copy link
Copy Markdown
Contributor

PR Dashboard

Generated at: 2026-10-04 15:58:01 UTC

Summary

Status Count
Total Open PRs 50
PRs with Failing Checks 43
PRs with All Checks Green 7
READY 6
FAILING 43
PENDING 0
NO CHECKS YET 0

These columns do not partition: 6 + 43 + 0 + 0 = 49, and there are 50 open PRs. A PR is being counted twice or not at all.

Seal Status

  • ⚠️ STALE -- sha256(compiler.rs)=32e37152eb23 != manifest seal=87e5cbd3ad94.
    The committed NMSE numbers were certified against an older compiler.rs.
    Run scripts/reseal-check.sh locally for the two-step reseal command (advisory; not a merge gate).

@gHashTag
gHashTag merged commit 6b06321 into master Oct 4, 2026
22 of 23 checks passed
@gHashTag
gHashTag deleted the feat/review-warden-agent branch October 4, 2026 17:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Review Warden: a profile agent for the review-speed watch (.claude/agents/review-warden.md)

1 participant