Repository navigation
Bless the eight copies the bees could not avoid - #4497
Merged
Merged
Conversation
Duplicate Body Ratchet went red on master after yesterday's bee pull requests landed. Eight groups each grew by one: magmul 21 -> 22, magsub 30 -> 31, relu 9 -> 10, relu_prime 5 -> 6, sadd 29 -> 30, scale_q 4 -> 5, smul 19 -> 20, and one more - 576 copies became 584. Blessed rather than fixed because the fix does not exist yet: `use m::f;` compiles to a comment and an unqualified call and the Zig fails (#4298), so a bee that needs sadd cannot import it and must either copy it or leave its criteria unmet. The ratchet still refuses any NEW group and any further growth. Closes #4496 Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
gHashTag
enabled auto-merge (squash)
September 21, 2026 05:04
Contributor
Contributor
|
📓 NotebookLM Notebook linked to this PR
This notebook contains session context, decisions, and artifacts for this work. |
This was referenced Oct 2, 2026
Closed
dmitrii-f-t27
added a commit
that referenced
this pull request
Oct 10, 2026
Eleven groups were in no ledger on master 756bcff. Scanner. tools/dupe_scan.py now reads the lexical forms t27c has: a declaration ending in ';' no longer borrows the next function's body (digest_for x8 and githubCiDeps_checkRuns_impl x5 were that), comments and string literals are masked, ';' at column 1 and '#' are line comments, and a single quote ends at the newline as the lexer ends it. The prototype, comment and literal repair comes from #5473. The quote and ';' rules are new: an apostrophe in "; the Queen's ..." opened a literal that hid three bodies in specs/queen/views.t27 and specs/tools/catalog.t27. Eleven self-test shapes pass; the #5473 lexer fails the two new ones. forwardPass x3: one empty-result literal in specs/hslm/forward_pass.t27 is shared (from #5473). Both seal files naming the spec are resealed with the t27c built from this tree; all four gen hashes equal the #5473 seals. rng_int_range x2: the gen_fuzz port had two 'return undefined;' stubs. They are now a xorshift64 generator (shifts and xors only: t27c emits u64 '+' and '*' without wrapping in Zig and Rust), and the port has five tests that can fail. Zig: 8/8 pass; five mutants are each caught. Seven copies across two files are ledgered, as #4497 did, because 'use a::b;' still generates no import (#4610): bytes_eq, session_status_str, led_config, validate_led_config, startup_config, update_oscillator_chain, shift_ir. Three rows whose groups no longer exist are dropped: generate_all, next, is_orphan. Refs #5497
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Closes #4496
Duplicate Body Ratchetis red on master: eight groups each grew by one copy after yesterday's bee pull requests landed (576 → 584). A check that is red on master is noise the next real failure hides in.Blessed, not fixed, because the fix does not exist yet — cross-module reuse does not generate (#4298), so a bee that needs
saddhas to copy it or leave its criteria unmet. The ratchet still refuses any new group and any further growth.🤖 Generated with Claude Code