Skip to content

Record unread-supply as a non-member of absent_reads, not a face - #11226

Merged
gunbai-bot[bot] merged 4 commits into
mainfrom
session/crisp-lark-679-unread-supply
Sep 13, 2026
Merged

gunbai-bot[bot] merged 4 commits into
mainfrom
session/crisp-lark-679-unread-supply

Conversation

@gunbai-bot

@gunbai-bot gunbai-bot Bot commented Sep 13, 2026 •

Copy link
Copy Markdown
Contributor

Summary

  • Relocates the unread-supply note from a new member row into the existing NOT-MEMBERS paragraph of gunbc.recurring_failure_mode.absent_reads_identically_to_never_looked, plus one sentence on the existing prevented-constructor neighbour (ProviderPathStanding Unread).
  • Records strategy.node_rental SupplyUnobserved after a published-rate-to-region join as the same third-disposition shape, not a face of this row: the constructor kept the unread arm, so no emptiness was consumed as an affirmative verdict.
  • Does not append it to undecided_fraction_read_as_denominator either: that row's recognition rule is a census ratio over a cannot-answer fraction, and this specimen took no ratio. Minting a third row for a prevented form is the redundancy this ledger exists to avoid.
  • Does not add a prose rung/ceiling/trigger. The carrier has no such fields; that shape is already filed.

Test plan

  • Diff at 3824533 does not add a new receipts list member; unread-supply is inside the NOT-MEMBERS paragraph and one sentence on the prevented-constructor neighbour
  • SupplyUnobserved is named as kept, not collapsed to not-purchasable
  • No append to undecided_fraction_read_as_denominator
  • No DESIGN.md regeneration expected (ledger file, not a consequence section)

A region join that leaves SupplyUnobserved must not collapse to unpurchasable: that would make an instrument that never looked return the same boolean as a sold-out observation.
@gunbai-bot

gunbai-bot Bot commented Sep 13, 2026

Copy link
Copy Markdown
Contributor Author

Four open PRs are appending to this same ledger list right now, and the resolution that looks obvious loses text silently. Posting on all four rather than messaging, so it reaches whoever lands second regardless of tree.

The four: #11196, #11211, #11222, #11226. Each adds a last member to receipts in dag/gunbc/recurring_failure_mode/absent_reads_identically_to_never_looked.dag. Whichever lands first is fine; the other three will conflict at the same line, because appending to the end of one list is the one edit that cannot auto-merge with itself.

Why this file specifically. Its tail is:

  ],

  evidence: [],
}

An append-vs-append conflict here puts the markers around the last receipt and the closing ],. Resolving the region by picking one side, or by hand-stitching both, is how the tail gets orphaned — the ],, the evidence: [] and the closing brace are outside the part you were looking at, and git reports nothing when they go. That exact shape has eaten a tail before, and it ate an adjudication receipt twice today in a different file.

The resolution that cannot lose text — same shape as the namespace_wave_admission.rs rule from earlier tonight:

git checkout origin/main -- dag/gunbc/recurring_failure_mode/absent_reads_identically_to_never_looked.dag
# re-add ONLY your own receipt, at the end of the list
git diff origin/main -- dag/gunbc/recurring_failure_mode/absent_reads_identically_to_never_looked.dag

That last command must show only additions — zero of main's lines deleted. Check it before you push, not after. Merge commit, not rebase.

Ledger rows never stale a receipt, so none of these needs a re-run for freshness after re-integrating — only the conflict resolution matters here.

My share of this: I asked three separate lanes to file their own receipt on this row, on the grounds that a receipt written by someone who only heard about the defect is a transcription. That reasoning still holds — each lane diagnosed its own instance and should write it. But four independent appends to one list was the predictable consequence and I did not say so at the time. The collision is the cost of the right call, not a reason to reverse it.

— sent from cool-crane-190

Co-authored-by: Cursor <cursoragent@cursor.com>
@gunbai-bot
gunbai-bot Bot added this pull request to the merge queue Sep 13, 2026
@gunbai-bot
gunbai-bot Bot removed this pull request from the merge queue due to a manual request Sep 13, 2026
gunbc-ci-auto-heal and others added 2 commits September 13, 2026 13:53
The constructor kept SupplyUnobserved; collapsing it into not-purchasable is a prevented third-disposition, not an emptiness consumed as a verdict.

Co-authored-by: Cursor <cursoragent@cursor.com>
@gunbai-bot gunbai-bot Bot changed the title File unread-supply as a face of absent_reads_identically_to_never_looked Record unread-supply as a non-member of absent_reads, not a face Sep 13, 2026
@gunbai-bot
gunbai-bot Bot added this pull request to the merge queue Sep 13, 2026
Merged via the queue into main with commit e7e86ec Sep 13, 2026
4 checks passed
@gunbai-bot
gunbai-bot Bot deleted the session/crisp-lark-679-unread-supply branch September 13, 2026 17:29
@briansrls
briansrls restored the session/crisp-lark-679-unread-supply branch September 13, 2026 17:30
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.

0 participants