Repository navigation
Rules: worktrees, away defaults, estimates, score tables, scope check - #331
Merged
Merged
Conversation
… estimates, away defaults, ruling scope (@architect) All approved by Ola; prose rule files only. - CLAUDE.md §3: new worktrees are made with tools/new_worktree.py (2026-10-21-merge-326.md proposal 2). REQUIRED-READING drops its `git worktree add` hint, which said otherwise. - briefs/common.md: no polling loop; stop every background command you started before handing back (merge-326 proposal 1). - perf.md §2: check outside data's dates and extent against the run's period and domain; py-spy needs root on macOS for native frames, otherwise cProfile and `sample` (merge-326 proposal 3, its wording: the brief said "uses py-spy", which the proposal Ola approved qualifies; py-spy is not installed here either). - perf.md §3: score tables show KGE with r, alpha, beta, NSE, half-water date. Public file only; the private perf.md is outside this write limit. - tester.md §1: a red step greps the tests for lists a new command or module must join (merge-326 proposal 4). - architect.md §5: one line, cost estimates probe unknown drivers; a timing gate's factor sums each new term's estimate plus noise (2026-10-22-merge-lsm39.md proposals 1 and 2). - REQUIRED-READING unattended mode: defaults taken while away go on disk as ASK OLA lines ending "default taken: <what>" and are all listed in the first report after the window; a build-at-all question waits; a refused push may be redone once Ola's later grant names the branch (2026-10-22-night.md proposals 2-4, review-applied wording at 4061f23). "Morning report" is undefined in the rules, so the text says "the first report to Ola after the window". - reviewer.md check 6: a design's files and pieces are checked against Ola's verbatim ruling; anything beyond its words blocks until Ola rules. Offsets: CLAUDE.md "Prohibited dependencies" bullet (restates §2, gated; night retro §6); tester.md's sentence restating the LOC exemption of tests. rule_sizes.py total 11,951 before edits on this base? No: 11,714 at base, 11,913 after (+199). Co-Authored-By: Claude Opus 5.5 <[email protected]>
…ter.md sentence on the size ceiling restored (cut not approved) Co-Authored-By: Claude Opus 5.5 <[email protected]>
… CLAUDE.md §2 is its only statement) Co-Authored-By: Claude Opus 5.5 <[email protected]>
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.
Rule round approved by Ola: new worktrees via tools/new_worktree.py; a refused push redone once Ola's later grant names the branch; defaults taken while away written down and all reported, build-or-not questions wait; agents stop their own background commands; architect counts unknown cost drivers and sums timing-gate terms; perf checks outside data's dates and extent, profiling fallback, score tables with KGE r/α/β, NSE and half-water date; reviewer checks a design's files against Ola's verbatim ruling; tester greps for lists a new command joins. Cuts: CLAUDE.md's prohibited-dependencies bullet and tester.md's restatement of the size ceiling. Review: APPROVED WITH WORDINGS on 1d77855 (@Reviewer); applied.
🤖 Generated with Claude Code