Skip to content

Test the 1 untested function in specs/boards/xc7a100t_full.t27 - #4338

Merged
gHashTag merged 3 commits into
masterfrom
queen-4330
Sep 21, 2026
Merged

gHashTag merged 3 commits into
masterfrom
queen-4330

Conversation

@gHashTag

Copy link
Copy Markdown
Owner

Closes #4330

Written by a bee on queen-4330 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, 4 insertions(+)

🤖 Generated with Claude Code

Trinity Bee and others added 2 commits September 20, 2026 12:17
Test verifies that count_leds() returns 8, matching the NUM_LEDS constant.

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

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

Copy link
Copy Markdown
Contributor

📓 NotebookLM Notebook linked to this PR

This notebook contains session context, decisions, and artifacts for this work.

@github-actions

Copy link
Copy Markdown
Contributor

PR Dashboard

Generated at: 2026-09-20 12:29:13 UTC

Summary

Status Count
Total Open PRs 29
PRs with Failing Checks 19
PRs with All Checks Green 10
READY 2
FAILING 19
PENDING 0
NO CHECKS YET 0

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

Seal Status

  • ⚠️ STALE -- sha256(compiler.rs)=403499176a5d != 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 Sep 20, 2026
The publisher was run from a checkout that carried its own copy of this file,
and it travelled into the bee's branch - outside the issue's boundary, which is
specs/boards/xc7a100t_full.t27 alone. The branch now changes only what the bee
wrote and the coordination entry.

Closes #4330

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

Copy link
Copy Markdown
Contributor

PR Dashboard

Generated at: 2026-09-21 05:06:33 UTC

Summary

Status Count
Total Open PRs 21
PRs with Failing Checks 17
PRs with All Checks Green 4
READY 2
FAILING 17
PENDING 0
NO CHECKS YET 0

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

Seal Status

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

@github-actions

Copy link
Copy Markdown
Contributor

📓 NotebookLM Notebook linked to this PR

This notebook contains session context, decisions, and artifacts for this work.

@gHashTag
gHashTag merged commit 31a3e21 into master Sep 21, 2026
26 of 32 checks passed
gHashTag added a commit that referenced this pull request Sep 21, 2026
#4338 passed every required check and could not merge: CONFLICT (add/add) in
tools/queen/publish.py. Its issue's boundary named one spec. The publisher had
run from a checkout carrying its own copy of publish.py, `git checkout -B`
switched branches in place, and the file travelled into the bee's branch.

It now cuts a fresh worktree from the bee's branch, writes the entry there,
commits and pushes from there, and removes it. Verified on the next real
publish (#4498): the diff is the bee's one file and the entry, and the
checkout the publisher ran from was untouched.

That publish found the next trap: the branch adds a workflow, which moves the
census and the gate-topology ledger - both have turned master red before
(#4303, #4319). The publisher now refuses such a branch and says why.

Closes #4499

Co-authored-by: Claude <noreply@anthropic.com>
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.

Test the 1 untested function in specs/boards/xc7a100t_full.t27

2 participants