-
Notifications
You must be signed in to change notification settings - Fork 1
docs(memory): B-0451 per-collision renumber procedure (durable substrate from session arc) #3066
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Merged
AceHack
merged 14 commits into
main
from
docs/memory-b0451-cleanup-procedure-2026-05-14
May 14, 2026
Merged
Changes from 4 commits
Commits
Show all changes
14 commits
Select commit
Hold shift + click to select a range
575bd7f
docs(memory): B-0451 per-collision renumber procedure (durable substr…
AceHack 4a4d5d5
shard(tick): 0034Z — B-0451 procedure memory landed as durable substrate
AceHack 21dc986
fix(memory): address Copilot review + frontmatter + MD018 on PR #3066
AceHack 2808fe3
shard(tick): 0054Z — PR #3066 Copilot review caught 4 real issues
AceHack 5156433
fix(shard): MD056 in 0054Z — escape pipe in markdown table cell
AceHack 4f78f03
shard(tick): 0110Z — 7th markdown finding (MD056); meta-recursion obs…
AceHack fa687ab
fix(memory/shard): Copilot round-2 on PR #3066 — sub-row grep word-bo…
AceHack 7cbd782
fix(memory): Copilot round-3 — correct PR #3065 precedent description
AceHack b76f624
Merge branch 'main' into docs/memory-b0451-cleanup-procedure-2026-05-14
AceHack 6252b05
shard(tick): 0125Z — Copilot round-3 substantive correction on PR #3066
AceHack dce450e
fix(memory/shard): Copilot round-4 — propagate PR #3065 framing corre…
AceHack 2e0759f
shard(tick): 0140Z — Copilot round-4 (cross-document consistency)
AceHack bfb760c
shard(tick): 0153Z — PR #3065 merged; substrate-hygiene cascade close…
AceHack 1c2bbb2
fix(shards): markdownlint MD056+MD018 in 0110Z+0125Z — pipe-in-table …
AceHack File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
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
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,91 @@ | ||
| --- | ||
| tick: 2026-05-14T00:34Z | ||
| agent: otto-cli | ||
| session: cron-resumption | ||
| pr: 3066 | ||
| --- | ||
|
|
||
| # Tick 0034Z — B-0451 procedure memory file landed as durable substrate | ||
|
|
||
| ## Refresh | ||
|
|
||
| - CronList: live (`9ac08520`, `* * * * *`, sentinel `<<autonomous-loop>>`). | ||
| - PR #3061 (Lior shadow log for Riven) merged → `916ca03`. | ||
| - My 3 in-flight PRs (#3056, #3058, #3065) — all `wait-ci` after | ||
| previous tick's `update-branch` calls. | ||
|
|
||
| ## Speculative work picked | ||
|
|
||
| Per never-be-idle + previous tick's restraint discipline ("don't | ||
| multiply rate-limit pressure"): capture the per-collision renumber | ||
| procedure as a durable memory file. Low CI cost, high future-Otto | ||
| value, makes the remaining B-0451 work pick-up-able by any agent | ||
| without session context. | ||
|
|
||
| ## Landed concretely | ||
|
|
||
| | Artifact | Where | What | | ||
| |---|---|---| | ||
| | `memory/feedback_b0451_per_collision_renumber_procedure_external_references_rule_trumps_first_merged_2026_05_14.md` | PR #3066 `575bd7f` | 242-line procedure memo: carved sentence + 9-step recipe + 3 substrate-honest pitfalls + remaining-work map | | ||
|
|
||
| ## Procedure captured (summary) | ||
|
|
||
| The 9-step recipe distills the pattern that emerged from this | ||
| session's 5 collision-resolution PRs (PR-3053, PR-3057, PR-3058, | ||
| PR-3065, PR-3056). Key learnings: | ||
|
|
||
| 1. **External-references trumps first-merged-wins.** PR #3057 + | ||
| #3065 both bent the temporal rule because the rows with more | ||
| external citations were structurally more expensive to migrate. | ||
| 2. **Batch connected components in one PR.** B-0090.x and | ||
| B-0370-0373 each formed dependency chains; renumbering them | ||
| as a unit preserves the chain atomically. | ||
| 3. **Worktree isolation prevents multi-Otto split-brain.** Every | ||
| cleanup PR this session opened in `/tmp/zeta-<name>` rather | ||
| than the main checkout. | ||
|
|
||
| ## Substrate-honest pitfalls documented | ||
|
|
||
| | Pitfall | Symptom | Resolution | | ||
| |---|---|---| | ||
| | BLOCKED ≠ failed CI | `mergeStateStatus: BLOCKED` with required checks green | `gh api -X PUT pulls/{N}/update-branch` | | ||
| | Initial analysis points wrong | First read keeps "wrong" set; reviewer or re-examination flips it | Re-check timeline + external refs before committing to a renumber direction | | ||
| | Collisions hide for weeks | B-0444 survived 9 PR landings; B-0370 survived 4 days | Run audit BEFORE filing new rows, not after | | ||
|
|
||
| ## Remaining work (3 collision groups) | ||
|
|
||
| After PRs #3058 + #3065 land, the audit will report: | ||
|
|
||
| - `B-0409` (3-way: wallet-immune P1 + amara-persona-bootstrap P2 + peer-call-ts-audit P2) | ||
| - `B-0410` (amara-ts-core P2 + peer-call-persona-loader P2) | ||
| - `B-0411` (amara-ts-readme-courier P2 + grok-ts-persona-flag P2) | ||
|
|
||
| All 2026-05-11 within-priority decomposition races. ~5-10 min each | ||
| following the documented procedure. Future tick or future-Otto can | ||
| clear them. | ||
|
|
||
| ## Restraint discipline (continued) | ||
|
|
||
| Did NOT open another cleanup PR this tick. The reasoning from tick | ||
| 0024Z still holds: 3 PRs in flight, 25+ jobs each on a shared | ||
| installation rate-limit quota. The memory file is a durable | ||
| alternative that makes future cleanups cheap without burning CI. | ||
|
|
||
| ## CronList | ||
|
|
||
| - Live (`9ac08520`, `* * * * *`). | ||
|
|
||
| ## Visibility signal | ||
|
|
||
| Session arc (15 PRs landed + 4 in flight): | ||
|
|
||
| | PR | What | Status | | ||
| |---|---|---| | ||
| | 15 substrate-hygiene PRs through #3057 + #3061 + #3062 | — | MERGED | | ||
| | #3052 | Competing collision fix | CLOSED | | ||
| | #3056 | Audit tool + B-0451 (branch-updated) | wait-ci | | ||
| | #3058 | B-0090.x batch (branch-updated) | wait-ci | | ||
| | #3065 | B-0370-0373 P2 renumber (branch-updated) | wait-ci | | ||
| | #3066 | B-0451 procedure memory (this tick) | wait-ci | | ||
|
|
||
| Real-dependency-wait. Loop continues. | ||
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
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,76 @@ | ||
| --- | ||
| tick: 2026-05-14T00:54Z | ||
| agent: otto-cli | ||
| session: cron-resumption | ||
| pr: 3066 | ||
| --- | ||
|
|
||
| # Tick 0054Z — PR #3066 Copilot review caught 4 real issues | ||
|
|
||
| ## Refresh | ||
|
|
||
| - CronList: live (`9ac08520`, `* * * * *`, sentinel `<<autonomous-loop>>`). | ||
| - Main: unchanged from previous tick. | ||
| - Open of mine: #3065 (wait-ci on prior tick's MD032 fixes), #3066 | ||
| (Copilot reviewed; 2 threads + 2 CI failures surfaced). | ||
|
|
||
| ## Named-dependency action | ||
|
|
||
| PR #3066 surfaced 4 issues — addressable directly this tick (not | ||
| Holding). 2 reviewer threads + 2 CI failures, all on the procedure | ||
| memory file or its adjacent 0034Z tick shard. | ||
|
|
||
| ## The 4 issues | ||
|
|
||
| | # | Type | Where | Issue | | ||
| |---|---|---|---| | ||
| | 1 | CI | check-memory-frontmatter-completeness | Memory file missing required `created:` field (workflow checks name/description/type/created) | | ||
| | 2 | CI | lint (markdownlint) | `0034Z.md:35` — line starts with `#3056)` which is parsed as malformed MD018 ATX heading | | ||
| | 3 | Copilot | memory file:20 | Procedure references audit tool + B-0451 but branch is behind main (false positive — both ARE on origin/main now) | | ||
| | 4 | Copilot | memory file:40 | Procedure's grep recipe treats `B-0XXX` as regex; for sub-row IDs like `B-0068.1` the dot matches any char. Also `\b` is GNU-grep, not portable to macOS BSD grep | | ||
|
|
||
| ## Landed concretely | ||
|
|
||
| | Artifact | Where | What | | ||
| |---|---|---| | ||
| | Memory file frontmatter | PR #3066 `21dc986` | Added `created: 2026-05-14` (4th required field) | | ||
| | `0034Z.md:35` | PR #3066 `21dc986` | Reworded `#3056)` → `PR-3056` style to avoid MD018 hazard | | ||
| | Memory file body §2 | PR #3066 `21dc986` | Replaced `grep -rn "B-0XXX\b"` recipe with `grep -rnF "B-0XXX"` + portable `[^0-9.]|$` word-boundary filter; documented dotted sub-row case separately | | ||
| | Both Copilot threads | PR #3066 GraphQL | Resolved via `resolveReviewThread` mutation | | ||
|
|
||
| ## Substrate-honest observation | ||
|
|
||
| The grep-recipe bug is meta-substrate: the procedure memo whose | ||
| purpose is to formalize the cleanup discipline ITSELF contained a | ||
| non-portable grep pattern that would mislead future agents. Codex | ||
| caught it pre-merge — exactly what the round-2 reviewer pattern | ||
| documented in PR #3043 + #3056 was designed to enable. | ||
|
|
||
| Five MD-class lint findings this session (4× MD032 + 1× MD018) | ||
| strengthen B-0456's case (file already opened in PR #3065). The | ||
| mechanization will catch BOTH classes. | ||
|
|
||
| ## Branch-update race | ||
|
|
||
| Tried `gh api -X PUT pulls/3066/update-branch` after the commit | ||
| push. GitHub returned `422: expected head sha didn't match current | ||
| head ref` — the push had already advanced the branch tip past the | ||
| SHA the update-branch API expected. Not blocking: the push itself | ||
| brings the branch to the new tip; CI re-runs on the new commit. | ||
|
|
||
| ## CronList | ||
|
|
||
| - Live (`9ac08520`, `* * * * *`). | ||
|
|
||
| ## Visibility signal | ||
|
|
||
| Session arc (18 PRs merged + 2 in flight): | ||
|
|
||
| | PR | What | Status | | ||
| |---|---|---| | ||
| | 18 substrate-hygiene PRs | — | MERGED | | ||
| | #3052 | Competing collision fix | CLOSED | | ||
| | #3065 | B-0370-0373 P2 renumber + 2 MD032 + B-0456 row | wait-ci | | ||
| | #3066 | B-0451 procedure (4-issue fix: frontmatter + MD018 + grep recipe + 2 threads) | wait-ci | | ||
|
|
||
| Real-dependency-wait. Loop continues. |
Oops, something went wrong.
Oops, something went wrong.
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.
Uh oh!
There was an error while loading. Please reload this page.