Skip to content

tri review-log: a health card from a window of the reviewer bee's own log (scripts/tri_loop/review_log.py) - #6745

Merged
gHashTag merged 4 commits into
masterfrom
queen-6086
Oct 7, 2026
Merged

gHashTag merged 4 commits into
masterfrom
queen-6086

Conversation

@gHashTag

@gHashTag gHashTag commented Oct 6, 2026

Copy link
Copy Markdown
Owner

Closes #6086

Written by a bee on queen-6086 and published by tools/queen/publish.py. The branch itself is the bee's; the second commit is the coordination entry every pull request must add, which a bee has no way to know about.

1 file changed, 265 insertions(+)

🤖 Generated with Claude Code

Trinity Bee and others added 4 commits October 6, 2026 07:06
The turn ended with these files edited and never committed. Uncommitted
work is invisible to the review - it reads the branch - so the attempt
would have been released as empty and the next bee would have started
beside this work rather than from it.

This commit is not a claim that the work is correct. It is the bee's
work, committed on its behalf, and it is judged exactly like any other:
the adversarial reviewer reads it, the compiler runs on it, and the
issue's own criteria are measured against it.

Issue: #6086
Turn: d0665635-e202-4a17-a453-63d009411003
Ending: finished (the turn closed)
Committed: 1 path(s)
Left uncommitted: 0 path(s) outside the declared boundary
The turn ended with these files edited and never committed. Uncommitted
work is invisible to the review - it reads the branch - so the attempt
would have been released as empty and the next bee would have started
beside this work rather than from it.

This commit is not a claim that the work is correct. It is the bee's
work, committed on its behalf, and it is judged exactly like any other:
the adversarial reviewer reads it, the compiler runs on it, and the
issue's own criteria are measured against it.

Issue: #6086
Turn: deb4ebed-8060-411e-8a3d-7f7a87bdf201
Ending: finished (the turn closed)
Committed: 1 path(s)
Left uncommitted: 0 path(s) outside the declared boundary
- Add classify_review_line function to categorize log line types
- Add review_health_exit function to determine health status based on listing errors
- Add review_window_exit function to handle window validation
- Include comprehensive tests covering all classification scenarios
- Implement exact classification logic from Python specification
- Use custom string functions to avoid compiler compatibility issues

Closes #6086
A pull request must add exactly one docs/now entry and a bee has no way
to know that: its brief names a boundary file and acceptance criteria,
and docs/now/ is neither. The publisher adds it rather than failing the
gate.

Closes #6086

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

github-actions Bot commented Oct 6, 2026

Copy link
Copy Markdown
Contributor

PR Dashboard

Generated at: 2026-10-06 11:41:55 UTC

Summary

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

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

Seal Status

  • ⚠️ STALE -- sha256(compiler.rs)=3c78f3c7ffb7 != 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).

This was referenced Oct 6, 2026
This was referenced Oct 6, 2026
@gHashTag
gHashTag enabled auto-merge (squash) October 7, 2026 07:27
@gHashTag
gHashTag merged commit 0bb1ea8 into master Oct 7, 2026
23 of 26 checks passed
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.

tri review-log: a health card from a window of the reviewer bee's own log (scripts/tri_loop/review_log.py)

2 participants