rule-observance: report which clauses have a program behind them (Refs #2622) - #2625
Merged
Conversation
Read LOOP-RULES.md properly instead of the three clauses a regex could reach, and checked every one that can be checked at all [measured]: R1 cost output format the tool itself observed R5 tool committed loop-tools-tracked.sh observed R6 seal = identity only tri loop-rules observed R10 absent != passing check_pr_branch_filters observed R15 six categories diffbin, at runtime observed --- R0 tick ledger nothing cron_tracking/ never existed R11 branch prefix w699- nothing 0 of 30 R11 provenance tag nothing 1 of 25 The split is TOTAL. Every clause with a program behind it is followed; every clause without one is at zero. R15 makes the point about its own enforcement -- "the set must be ASSERTED to partition the corpus at runtime; claiming it in a docstring is not a check" -- and diffbin.py:458 sums the six categories against the file total and errors on mismatch. The rule anticipated that a rule about categories, kept only in prose, would drift. The three clauses at zero are the ones that took their own advice least. So this is not carelessness. A rule kept in a document is a wish; a rule kept in a program is a rule, and this file holds both kinds under one heading, indistinguishable to a reader and equally sealed by a checksum that certifies neither. Suggested to the owner in #2622 as a fourth option beside the three already there: mark each clause with what enforces it, so a clause with an empty column is visibly a wish rather than silently one. Still reports, still does not gate. Refs #2622
Contributor
|
📓 NotebookLM Notebook linked to this PR
This notebook contains session context, decisions, and artifacts for this work. |
Contributor
PR DashboardGenerated at: 2026-08-23 22:53:48 UTC
Summary
Seal Status
|
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
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.
Read
LOOP-RULES.mdproperly instead of the three clauses a regex could reach, and checked every one that can be checked at all[measured]:costoutput formatloop-tools-tracked.shgit add)tri loop-rulescheck_pr_branch_filters.pyunchanged≠ "could not compare"diffbin, at runtimecron_tracking/w699-<topic>The split is total. Not "mostly".
R15 makes the point about its own enforcement:
diffbin.py:458sums the six categories against the file total and errors on mismatch. The rule anticipated that a rule about categories, kept only in prose, would drift — and the three clauses at zero are the ones that took their own advice least.So this is not carelessness by anyone. A rule kept in a document is a wish; a rule kept in a program is a rule, and this file holds both kinds under one heading, indistinguishable to a reader and equally sealed by a checksum that certifies neither.
Suggested to the owner in #2622 as a fourth option: mark each clause with what enforces it, so a clause with an empty column is visibly a wish rather than silently one.
Still reports, still does not gate. §105.
Refs #2622