Skip to content

fix(queen): read the path roots from the tree, not from a typed list - #4741

Merged
gHashTag merged 1 commit into
masterfrom
feat/boundary-roots-from-tree
Sep 24, 2026
Merged

gHashTag merged 1 commit into
masterfrom
feat/boundary-roots-from-tree

Conversation

@gHashTag

Copy link
Copy Markdown
Owner

Closes #4740

ROOTS = "specs|tools|scripts|src|bootstrap|conformance|docs|bindings|apps"

Somebody typed that, and it was wrong in the direction that hides work. What this repository actually holds:

directory files in the list?
contrib 195 no
tests 162 no
research 160 no
fpga 150 no
.github 83 no
cli 69 no
gen 67 no

So "Add a regression test suite for gen/zig/measure.py" read as naming nothing at all, and sat in the pile that looks unreachable.

A list nobody re-reads drifts from the repository it describes. git ls-tree -d cannot.

Result

18 more boundaries the old rule could not see, including .claude/skills/ci-gates/SKILL.md — a path whose whole directory was invisible to the regex.

The typed list stays as the fallback for a run with no checkout, which is what --self-test uses, and both branches are pinned there.

The board today

2026-09-23 now
open issues without a boundary 565 296
with one 82 315
claimed 41 108
refusal "nothing to choose" None
active bees 1 of 20 12

The 296 that remain mostly name no file at all. There is nothing to draft from, and a guess there would be the wrong boundary this tool exists to avoid.

🤖 Generated with Claude Code

`ROOTS = "specs|tools|scripts|src|bootstrap|conformance|docs|bindings|apps"`
was typed by hand, and it was wrong in the direction that hides work. This
repository holds `fpga` (150 files), `tests` (162), `contrib` (195),
`research` (160), `.github` (83), `cli` (69), `gen` (67), `proofs` and
`experiments`, and none of them was in it.

So "Add a regression test suite for gen/zig/measure.py" read as naming nothing
at all, and sat in the pile that looks unreachable. A list nobody re-reads
drifts from the repository it describes; `git ls-tree -d` cannot.

Derived from the tree it finds 18 more boundaries the old rule could not see,
including `.claude/skills/ci-gates/SKILL.md` - a path whose whole directory
was invisible to the regex.

The typed list stays as the fallback for a run with no checkout, which is what
--self-test uses, and both branches are pinned there.

Closes #4740

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-24 03:43:22 UTC

Summary

Status Count
Total Open PRs 42
PRs with Failing Checks 30
PRs with All Checks Green 12
READY 3
FAILING 30
PENDING 0
NO CHECKS YET 0

These columns do not partition: 3 + 30 + 0 + 0 = 33, and there are 42 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).

@gHashTag
gHashTag merged commit bf18fb8 into master Sep 24, 2026
24 of 30 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.

The path rule was a typed list, and it never had fpga, tests, gen or cli in it

1 participant