Skip to content

t27b checked: test blocks for 7 specs in specs/tools/tri (2 of 8) - #6583

Merged
gHashTag merged 2 commits into
masterfrom
queen-6463
Oct 5, 2026
Merged

gHashTag merged 2 commits into
masterfrom
queen-6463

Conversation

@gHashTag

@gHashTag gHashTag commented Oct 5, 2026

Copy link
Copy Markdown
Owner

Closes #6463

Written by a bee on queen-6463 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.

7 files changed, 49 insertions(+)

🤖 Generated with Claude Code

Trinity Bee and others added 2 commits October 5, 2026 15:39
Each spec now has one test block with 4 assertions that verify
constant values. This moves these specs from pass_vacuous to
checked passes in t27b coverage testing.

Closes #6463
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 #6463

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

github-actions Bot commented Oct 5, 2026

Copy link
Copy Markdown
Contributor

PR Dashboard

Generated at: 2026-10-05 18:05:18 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)=9f2c8a4829f6 != 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).

@t27-bees t27-bees Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Reviewer bee verdict for head fd6ad1e750fcf2f888faf8ddc6ecf0d71a182b5b (tools/bees/reviewer.py, zai glm-4.7-flash, 12 turns, 613 s).

BEE-VERDICT: REQUEST_CHANGES
summary: FR-006 not met: PR body is empty and must contain "t27b-blog: +7 " line per issue #6463
criterion: FR-006 (PR body blog line) -- unmet -- evidence: PR body empty, no t27b-blog: +7 line
discounted-check: duplicate-bodies -- same failure on master 7e49725
discounted-check: spec-guards -- same failure on master 7e49725

Evidence

I'll continue the review by analyzing the final checks:

Criterion 1: FR-006 (PR body line) - UNMET

The PR body is completely empty. FR-006 requires adding a blog entry line t27b-blog: +7 <spec paths> to the PR body to document which bee work moved the t27b passes count.

Evidence: PR body shows no text (empty), diff shows no PR body content, issue #6463 explicitly states "write t27b-blog: +N <spec paths> in the PR body".

Red Check Analysis

duplicate-bodies -- DISCOUNTED

  • Why: Same failure on master (commit 7e49725). The log shows "duplicate bodies: 606 of 5228" and the PR did not introduce new duplicates.
  • Evidence: brief.md line 109: "on master: master 7e49725: failure" and line 153: "duplicate bodies: 606 of 5228 in 177 group(s) across 1283 spec(s)"

spec-guards -- DISCOUNTED

  • Why: Same failure on master (commit 7e49725). The log shows DRIFTED issues that existed before this PR.
  • Evidence: brief.md line 161: "on master: master 7e49725: failure" and the DRIFTED items (ring-096-rust, ring-097-rust) were already present

Summary

All 14 success criteria from issue #6463 are met:

  • Each of the 7 specs has exactly one test block with 4 assertions ✓
  • All assertions pass (t27c test-report prints FAIL 0 and pass count 2) ✓
  • All files are ASCII-only and contain only .t27 files ✓
  • Each spec has constants properly tested ✓

However, FR-006 is not met: the PR body lacks the required t27b-blog: +7 <spec paths> line. This is a clear formatting requirement that the PR body should contain.

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.

t27b checked: test blocks for 7 specs in specs/tools/tri (2 of 8)

2 participants