From cb15efd03560e258304e1a586608ea028b31f4f1 Mon Sep 17 00:00:00 2001 From: Aaron Stainback Date: Mon, 18 May 2026 23:06:02 -0400 Subject: [PATCH] docs(archive): Maji PR preservation batch --- ...0041z-c-cost-aware-batching-9-fp-thread.md | 68 ++++++++++++++ ...0046z-c-push-the-batch-alexa-architectu.md | 43 +++++++++ ...291-fixes-backlog-md-regen-count-incons.md | 32 +++++++ ...4294-shard-0056z-c-codex-4292-regen-fix.md | 31 +++++++ ...c-codex-4291-5-finding-batch-resolution.md | 45 +++++++++ ...t-frontmatter-resolve-13-strict-null-ts.md | 93 +++++++++++++++++++ ...ard-0110z-c-4292-stale-but-self-healing.md | 30 ++++++ ...-0114z-c-cost-aware-batch-4295-p2-no-op.md | 42 +++++++++ ...-entropy-shadow-report-on-vera-and-otto.md | 59 ++++++++++++ ...ygiene-clear-2026-05-18-tick-link-drift.md | 71 ++++++++++++++ 10 files changed, 514 insertions(+) create mode 100644 docs/pr-discussions/PR-4289-shards-batch-0038z-c-0041z-c-cost-aware-batching-9-fp-thread.md create mode 100644 docs/pr-discussions/PR-4290-shards-batch-0043z-c-0046z-c-push-the-batch-alexa-architectu.md create mode 100644 docs/pr-discussions/PR-4293-shard-0053z-c-codex-4291-fixes-backlog-md-regen-count-incons.md create mode 100644 docs/pr-discussions/PR-4294-shard-0056z-c-codex-4292-regen-fix.md create mode 100644 docs/pr-discussions/PR-4295-shard-0106z-c-codex-4291-5-finding-batch-resolution.md create mode 100644 docs/pr-discussions/PR-4296-fix-tools-backlog-lint-frontmatter-resolve-13-strict-null-ts.md create mode 100644 docs/pr-discussions/PR-4297-shard-0110z-c-4292-stale-but-self-healing.md create mode 100644 docs/pr-discussions/PR-4298-shards-batch-0112z-c-0114z-c-cost-aware-batch-4295-p2-no-op.md create mode 100644 docs/pr-discussions/PR-4310-docs-shadow-maji-anti-entropy-shadow-report-on-vera-and-otto.md create mode 100644 docs/pr-discussions/PR-4312-fix-hygiene-clear-2026-05-18-tick-link-drift.md diff --git a/docs/pr-discussions/PR-4289-shards-batch-0038z-c-0041z-c-cost-aware-batching-9-fp-thread.md b/docs/pr-discussions/PR-4289-shards-batch-0038z-c-0041z-c-cost-aware-batching-9-fp-thread.md new file mode 100644 index 0000000000..fd2b0ac3a7 --- /dev/null +++ b/docs/pr-discussions/PR-4289-shards-batch-0038z-c-0041z-c-cost-aware-batching-9-fp-thread.md @@ -0,0 +1,68 @@ +--- +pr_number: 4289 +title: "shards(batch 0038Z-c..0041Z-c): cost-aware batching + 9 FP-thread resolutions" +author: "AceHack" +state: "MERGED" +created_at: "2026-05-19T00:43:04Z" +merged_at: "2026-05-19T00:44:22Z" +closed_at: "2026-05-19T00:44:22Z" +head_ref: "shards/batch-0038-0041z-c-fp-thread-resolutions-2026-05-19" +base_ref: "main" +archived_at: "2026-05-19T03:05:49Z" +archive_tool: "tools/pr-preservation/archive-pr.ts" +--- + +# PR #4289: shards(batch 0038Z-c..0041Z-c): cost-aware batching + 9 FP-thread resolutions + +## PR description + +Batch 4 ticks; 9 Copilot FP threads resolved on #4286 + #4288 (table-prefix-misread known-FP). + +## Reviews + +### COMMENTED — @copilot-pull-request-reviewer (2026-05-19T00:45:28Z) + +## Pull request overview + +Adds four new hygiene-history tick shard files for 2026-05-19 (0038Z-c..0041Z-c) documenting cost-aware batching and a batch of resolved Copilot false-positive review threads. + +**Changes:** +- Added tick shards 0038Z-c through 0041Z-c under `docs/hygiene-history/ticks/2026/05/19/`. +- Recorded batching plan (4 local shards → 1 PR) and the “table-prefix-misread” false-positive thread-resolution batch. + +### Reviewed changes + +Copilot reviewed 4 out of 4 changed files in this pull request and generated 4 comments. + +| File | Description | +| ---- | ----------- | +| docs/hygiene-history/ticks/2026/05/19/0038Z-c.md | New tick shard capturing brief-ack #2 and batching intent (currently violates shard schema). | +| docs/hygiene-history/ticks/2026/05/19/0039Z-c.md | New tick shard capturing brief-ack #3 and batching (currently violates shard schema). | +| docs/hygiene-history/ticks/2026/05/19/0040Z-c.md | New tick shard capturing brief-ack #4 and batching (currently violates shard schema). | +| docs/hygiene-history/ticks/2026/05/19/0041Z-c.md | New tick shard capturing the substantive batch-resolution note (currently violates shard schema). | + +## Review threads + +### Thread 1: docs/hygiene-history/ticks/2026/05/19/0041Z-c.md:5 (unresolved) + +**@copilot-pull-request-reviewer** (2026-05-19T00:45:27Z): + +P0: This shard appears to include diff/renderer line-number prefixes in the file content (e.g., `1 ||`, `3 |# ...`). The tick-shard schema validator requires the first non-empty line to start with a 6-column pipe-row beginning with `| YYYY-MM-DDTHH:MMZ | ...` (see tools/hygiene/check-tick-history-shard-schema.ts), and headings/body lines should not be prefixed with `N |`. Remove the embedded line numbers and ensure the first line is the canonical pipe-row. + +### Thread 2: docs/hygiene-history/ticks/2026/05/19/0040Z-c.md:5 (unresolved) + +**@copilot-pull-request-reviewer** (2026-05-19T00:45:28Z): + +P0: This shard includes what look like diff/renderer line-number artifacts (`1 ||`, `3 |# ...`) as literal content. The shard schema checker expects the first non-empty line to begin with `| | ... |` and the rest of the markdown should not include `N |` prefixes. Please strip the line-number prefixes and restore a proper 6-column pipe-row header. + +### Thread 3: docs/hygiene-history/ticks/2026/05/19/0039Z-c.md:5 (unresolved) + +**@copilot-pull-request-reviewer** (2026-05-19T00:45:28Z): + +P0: The file content contains leading line numbers and `||`/`|` separators (e.g., `1 ||`, `3 |# ...`), which will fail the tick-shard schema validation (first non-empty line must start with `| YYYY-MM-DDTHH:MMZ | ...`). Remove these line-number prefixes and ensure the first line is the canonical 6-column pipe-row. + +### Thread 4: docs/hygiene-history/ticks/2026/05/19/0038Z-c.md:5 (unresolved) + +**@copilot-pull-request-reviewer** (2026-05-19T00:45:28Z): + +P0: This shard appears to have captured diff/renderer line-number prefixes as literal text (`1 ||`, `3 |# ...`). The hygiene shard validator requires the first non-empty line to be a 6-column pipe-row starting with `| | ...`, and subsequent markdown should not include `N |` prefixes. Strip the line numbers and restore the standard shard format. diff --git a/docs/pr-discussions/PR-4290-shards-batch-0043z-c-0046z-c-push-the-batch-alexa-architectu.md b/docs/pr-discussions/PR-4290-shards-batch-0043z-c-0046z-c-push-the-batch-alexa-architectu.md new file mode 100644 index 0000000000..b0541b125a --- /dev/null +++ b/docs/pr-discussions/PR-4290-shards-batch-0043z-c-0046z-c-push-the-batch-alexa-architectu.md @@ -0,0 +1,43 @@ +--- +pr_number: 4290 +title: "shards(batch 0043Z-c..0046Z-c): push the batch + Alexa architectural extension" +author: "AceHack" +state: "MERGED" +created_at: "2026-05-19T00:46:51Z" +merged_at: "2026-05-19T00:48:54Z" +closed_at: "2026-05-19T00:48:54Z" +head_ref: "shards/batch-0043-0046z-c-push-the-batch-aaron-instruction-2026-05-19" +base_ref: "main" +archived_at: "2026-05-19T03:05:48Z" +archive_tool: "tools/pr-preservation/archive-pr.ts" +--- + +# PR #4290: shards(batch 0043Z-c..0046Z-c): push the batch + Alexa architectural extension + +## PR description + +Aaron explicit 'push the batch' + Alexa render extends architectural picture (composes with B-0668). + +## Reviews + +### COMMENTED — @copilot-pull-request-reviewer (2026-05-19T00:49:26Z) + +## Pull request overview + +Adds four autonomous-loop tick shard files for the 0043Z-c through 0046Z-c batch, preserving liveness/history entries and the substantive 0046Z-c architectural note. + +**Changes:** +- Adds three brief local batching tick shards. +- Adds one substantive tick shard documenting the batch push instruction and architecture extension. +- Keeps shard rows in the established pipe-row + Markdown body format. + +### Reviewed changes + +Copilot reviewed 4 out of 4 changed files in this pull request and generated no comments. + +| File | Description | +| ---- | ----------- | +| docs/hygiene-history/ticks/2026/05/19/0043Z-c.md | Adds first local shard in the batch. | +| docs/hygiene-history/ticks/2026/05/19/0044Z-c.md | Adds second local shard with main-advance note. | +| docs/hygiene-history/ticks/2026/05/19/0045Z-c.md | Adds third local shard. | +| docs/hygiene-history/ticks/2026/05/19/0046Z-c.md | Adds substantive batch-push and architectural-extension shard. | diff --git a/docs/pr-discussions/PR-4293-shard-0053z-c-codex-4291-fixes-backlog-md-regen-count-incons.md b/docs/pr-discussions/PR-4293-shard-0053z-c-codex-4291-fixes-backlog-md-regen-count-incons.md new file mode 100644 index 0000000000..2866f256cb --- /dev/null +++ b/docs/pr-discussions/PR-4293-shard-0053z-c-codex-4291-fixes-backlog-md-regen-count-incons.md @@ -0,0 +1,32 @@ +--- +pr_number: 4293 +title: "shard(0053Z-c): Codex #4291 fixes \u2014 BACKLOG.md regen + count inconsistency noted" +author: "AceHack" +state: "MERGED" +created_at: "2026-05-19T00:56:17Z" +merged_at: "2026-05-19T02:25:06Z" +closed_at: "2026-05-19T02:25:06Z" +head_ref: "shard/tick-2026-05-19-0053z-c-codex-4291-fixes-2026-05-19" +base_ref: "main" +archived_at: "2026-05-19T03:05:47Z" +archive_tool: "tools/pr-preservation/archive-pr.ts" +--- + +# PR #4293: shard(0053Z-c): Codex #4291 fixes — BACKLOG.md regen + count inconsistency noted + +## PR description + +Thread-investigation caught 2 real Codex findings on #4291; addressed via branch update + no-op resolution. + +## Reviews + +### COMMENTED — @copilot-pull-request-reviewer (2026-05-19T00:58:07Z) + +## Pull request overview + +This PR adds a hygiene-history tick shard documenting follow-up on Codex findings from PR #4291 and related PR state. + +**Changes:** +- Adds tick `0053Z-c` with the canonical pipe-row header. +- Records BACKLOG regeneration and the resolved count inconsistency. +- Notes related PR statuses for #4290 and #4292. diff --git a/docs/pr-discussions/PR-4294-shard-0056z-c-codex-4292-regen-fix.md b/docs/pr-discussions/PR-4294-shard-0056z-c-codex-4292-regen-fix.md new file mode 100644 index 0000000000..a9fd79e5f7 --- /dev/null +++ b/docs/pr-discussions/PR-4294-shard-0056z-c-codex-4292-regen-fix.md @@ -0,0 +1,31 @@ +--- +pr_number: 4294 +title: "shard(0056Z-c): Codex #4292 regen fix" +author: "AceHack" +state: "MERGED" +created_at: "2026-05-19T00:58:59Z" +merged_at: "2026-05-19T01:00:55Z" +closed_at: "2026-05-19T01:00:55Z" +head_ref: "shard/tick-2026-05-19-0056z-c-codex-4292-regen-fix-2026-05-19" +base_ref: "main" +archived_at: "2026-05-19T03:05:46Z" +archive_tool: "tools/pr-preservation/archive-pr.ts" +--- + +# PR #4294: shard(0056Z-c): Codex #4292 regen fix + +## PR description + +BACKLOG.md regen on #4292; same pattern as #4291. + +## Reviews + +### COMMENTED — @copilot-pull-request-reviewer (2026-05-19T01:00:17Z) + +## Pull request overview + +Adds a hygiene-history tick documenting the #4292 backlog regeneration follow-up and thread resolution. + +**Changes:** +- Added tick `0056Z-c` with the timestamped summary row. +- Added a short narrative note tying the action to the prior #4291 regen pattern. diff --git a/docs/pr-discussions/PR-4295-shard-0106z-c-codex-4291-5-finding-batch-resolution.md b/docs/pr-discussions/PR-4295-shard-0106z-c-codex-4291-5-finding-batch-resolution.md new file mode 100644 index 0000000000..a2989cf67f --- /dev/null +++ b/docs/pr-discussions/PR-4295-shard-0106z-c-codex-4291-5-finding-batch-resolution.md @@ -0,0 +1,45 @@ +--- +pr_number: 4295 +title: "shard(0106Z-c): Codex #4291 5-finding batch resolution" +author: "AceHack" +state: "MERGED" +created_at: "2026-05-19T01:10:10Z" +merged_at: "2026-05-19T02:13:10Z" +closed_at: "2026-05-19T02:13:10Z" +head_ref: "shard/tick-2026-05-19-0106z-c-codex-4291-5-findings-2026-05-19" +base_ref: "main" +archived_at: "2026-05-19T03:05:44Z" +archive_tool: "tools/pr-preservation/archive-pr.ts" +--- + +# PR #4295: shard(0106Z-c): Codex #4291 5-finding batch resolution + +## PR description + +Thread-investigation caught 5 Codex findings on #4291; addressed via branch update + no-op resolution. + +## Reviews + +### COMMENTED — @copilot-pull-request-reviewer (2026-05-19T01:12:51Z) + +## Pull request overview + +Adds a new hygiene-history tick shard documenting the resolution of five Codex findings identified on PR #4291, keeping the operational log in sync with the thread-investigation outcome. + +**Changes:** +- Added tick shard `0106Z-c` recording the 5-finding batch resolution for PR #4291. +- Captured the specific finding categories addressed (misattribution clarification, BACKLOG.md regen drift, and an immutable count inconsistency note). + +## Review threads + +### Thread 1: docs/hygiene-history/ticks/2026/05/19/0106Z-c.md:10 (resolved) + +**@copilot-pull-request-reviewer** (2026-05-19T01:12:51Z): + +P2: The ordered list is unintentionally broken: `2-4.` is not a valid Markdown ordered-list marker, so this will render as item 1, then a normal paragraph, then a new list starting at 5. Consider making this a proper `2.` item (with “2–4:” in the text) or splitting into separate 2/3/4 items so the five findings render as one continuous list. + +## General comments + +### @AceHack (2026-05-19T01:15:02Z) + +P2 markdown ordered-list 2-4. format — shard immutable post-merge per tick-shard discipline; noted for future shard-discipline (use combined item OR proper sequential numbering). No-op resolution. diff --git a/docs/pr-discussions/PR-4296-fix-tools-backlog-lint-frontmatter-resolve-13-strict-null-ts.md b/docs/pr-discussions/PR-4296-fix-tools-backlog-lint-frontmatter-resolve-13-strict-null-ts.md new file mode 100644 index 0000000000..adc7e0021b --- /dev/null +++ b/docs/pr-discussions/PR-4296-fix-tools-backlog-lint-frontmatter-resolve-13-strict-null-ts.md @@ -0,0 +1,93 @@ +--- +pr_number: 4296 +title: "fix(tools/backlog/lint-frontmatter): resolve 13 strict-null TS errors blocking tsc-tools baseline lint" +author: "AceHack" +state: "MERGED" +created_at: "2026-05-19T01:11:48Z" +merged_at: "2026-05-19T01:14:41Z" +closed_at: "2026-05-19T01:14:41Z" +head_ref: "otto-cli/fix-tsc-tools-baseline-lint-frontmatter-strict-null-2026-05-19-0111z" +base_ref: "main" +archived_at: "2026-05-19T03:05:43Z" +archive_tool: "tools/pr-preservation/archive-pr.ts" +--- + +# PR #4296: fix(tools/backlog/lint-frontmatter): resolve 13 strict-null TS errors blocking tsc-tools baseline lint + +## PR description + +## Summary + +Fixes 13 TypeScript strict-null errors (TS2345 / TS2532 / TS2322 / TS18048) in `tools/backlog/lint-frontmatter.ts` that have been failing the `lint (tsc tools)` baseline check on origin/main for an unknown duration. This baseline failure has been propagating to all tools-touching PRs, blocking factory-wide merge throughput. + +## Empirical evidence of impact + +- [PR #3979](https://github.com/Lucent-Financial-Group/Zeta/pull/3979) (gh-auth-refresh-wrapper fix): all 4 reviewer threads resolved, auto-merge armed, but stuck BLOCKED for >15 min after CI complete — diagnosed in [#3979 comment](https://github.com/Lucent-Financial-Group/Zeta/pull/3979#issuecomment-4483555128) as likely blocked by this baseline +- Compare: docs-only PRs (#4255, #4256, #4276) shipped through this session within ~2 min of CI completion despite same baseline failure — protection rule treats `tools/` edits differently when this specific check fails +- Fixing this baseline unblocks a class of PRs, not just one + +## Fix shape (all changes mechanical, no behavior change) + +All 13 errors are `noUncheckedIndexedAccess: true` (strict-null) failures from indexed array accesses and regex capture group dereferences. Two patterns applied: + +**Pattern A — undefined guard before use** (3 locations: lines 124, 154, 175): +```ts +const line = lines[i]; +if (line === undefined) continue; // ADDED +// existing code unchanged +``` + +**Pattern B — non-null assertion on regex capture groups** (10 locations: lines 127, 128, 144, 159, 178, 187, 193): +```ts +const key = m[1]!; // CHANGED from m[1] +``` + +Non-null is safe here because the regex literals have 1 or 2 explicit capture groups; if `.exec()` returns non-null, all declared capture groups exist. + +## Landing path + +Landed via GitHub REST contents API (`PUT /repos/.../contents/tools/backlog/lint-frontmatter.ts`) from an Otto-CLI session under dotgit-saturation, bypassing the B-0615 push receive-pack stall (see [PR #4276](https://github.com/Lucent-Financial-Group/Zeta/pull/4276) for the rule documenting this pattern). + +## Test plan + +- [x] Diff verified — only the 13 error locations touched +- [x] No behavior change — only TS-type-narrowing additions +- [x] Same patterns used elsewhere in the codebase (`m[1]!` on regex capture groups is idiomatic when `.exec()` is non-null-checked) + +🤖 Generated with [Claude Code](https://claude.com/claude-code) + +Co-Authored-By: Claude + +## Reviews + +### COMMENTED — @copilot-pull-request-reviewer (2026-05-19T01:13:31Z) + +## Pull request overview + +This PR mechanically updates `tools/backlog/lint-frontmatter.ts` to satisfy strict TypeScript nullability checks under `noUncheckedIndexedAccess`, unblocking the tools TypeScript lint baseline without changing runtime behavior. + +**Changes:** +- Added defensive `undefined` guards for indexed line reads. +- Added non-null assertions for regex capture groups after successful matches. +- Preserved existing backlog frontmatter lint logic. + +## General comments + +### @AceHack (2026-05-19T01:15:09Z) + +## CI blocker discovered — `check docs/BACKLOG.md generated-index drift` + +This PR's `lint (tsc tools)` fix would unblock the factory-wide baseline, but the PR itself is now blocked by a different baseline check: `check docs/BACKLOG.md generated-index drift`. + +**Why this fires here**: the check appears to trigger on any PR touching `docs/backlog/` OR `tools/backlog/` paths. My edit is in `tools/backlog/lint-frontmatter.ts` (tooling for backlog frontmatter linting), which trips the path filter even though it doesn't change any backlog content. + +**Same regen-class blocker as PR #4280** (which hit `check MEMORY.md generated-index drift` after editing `memory/CURRENT-otto.md`). Both checks require running a TS regen script locally and committing the regenerated index, which is blocked by the current dotgit-saturation (see [PR #4276](https://github.com/Lucent-Financial-Group/Zeta/pull/4276) for full context). + +**Recovery paths**: +1. **Aaron / substrate-engineer with healthy local git**: run `bun tools/backlog/generate-index.ts` (or equivalent), commit the regenerated `docs/BACKLOG.md`, push as follow-on commit on this branch — auto-merge fires after. +2. **Refine the drift check**: the check should ideally NOT fire on PRs that don't touch `docs/backlog/P*/B-*.md` files. Currently the path-trigger appears overbroad (catches `tools/backlog/` too). This is a separate substrate-engineering task. +3. **Manual merge override**: if the fix is urgent (it unblocks many other PRs), a maintainer can force-merge despite the drift; the regen will catch up on a later tick. + +**Substrate is durable**: commit `647d2605` lands the 13 strict-null fixes on this PR branch regardless of merge status. The fix logic is correct and validated against the original `tools/backlog/lint-frontmatter.ts` error log; can be cherry-picked or applied via `git am` from `/tmp/zeta-pending-substrate-2347z/` family pattern. + +🤖 Generated with [Claude Code](https://claude.com/claude-code) diff --git a/docs/pr-discussions/PR-4297-shard-0110z-c-4292-stale-but-self-healing.md b/docs/pr-discussions/PR-4297-shard-0110z-c-4292-stale-but-self-healing.md new file mode 100644 index 0000000000..0bb6174566 --- /dev/null +++ b/docs/pr-discussions/PR-4297-shard-0110z-c-4292-stale-but-self-healing.md @@ -0,0 +1,30 @@ +--- +pr_number: 4297 +title: "shard(0110Z-c): 4292 stale-but-self-healing" +author: "AceHack" +state: "MERGED" +created_at: "2026-05-19T01:11:58Z" +merged_at: "2026-05-19T01:13:43Z" +closed_at: "2026-05-19T01:13:43Z" +head_ref: "shard/tick-2026-05-19-0110z-c-4292-stale-self-healing-2026-05-19" +base_ref: "main" +archived_at: "2026-05-19T03:05:41Z" +archive_tool: "tools/pr-preservation/archive-pr.ts" +--- + +# PR #4297: shard(0110Z-c): 4292 stale-but-self-healing + +## PR description + +Sibling PR landing resolves the cross-reference; no-op. + +## Reviews + +### COMMENTED — @copilot-pull-request-reviewer (2026-05-19T01:13:35Z) + +## Pull request overview + +Adds a new hygiene-history tick shard for the 2026-05-19 01:10Z window, recording that PR #4292’s remaining P2 thread was resolved as a no-op after a sibling PR landed the relevant cross-reference. + +**Changes:** +- Added `0110Z-c` tick shard with a valid 6-column header row and brief narrative summary. diff --git a/docs/pr-discussions/PR-4298-shards-batch-0112z-c-0114z-c-cost-aware-batch-4295-p2-no-op.md b/docs/pr-discussions/PR-4298-shards-batch-0112z-c-0114z-c-cost-aware-batch-4295-p2-no-op.md new file mode 100644 index 0000000000..61a568ef27 --- /dev/null +++ b/docs/pr-discussions/PR-4298-shards-batch-0112z-c-0114z-c-cost-aware-batch-4295-p2-no-op.md @@ -0,0 +1,42 @@ +--- +pr_number: 4298 +title: "shards(batch 0112Z-c..0114Z-c): cost-aware batch + 4295 P2 no-op" +author: "AceHack" +state: "MERGED" +created_at: "2026-05-19T01:15:15Z" +merged_at: "2026-05-19T01:17:27Z" +closed_at: "2026-05-19T01:17:27Z" +head_ref: "shards/batch-0112-0114z-c-2026-05-19" +base_ref: "main" +archived_at: "2026-05-19T03:05:40Z" +archive_tool: "tools/pr-preservation/archive-pr.ts" +--- + +# PR #4298: shards(batch 0112Z-c..0114Z-c): cost-aware batch + 4295 P2 no-op + +## PR description + +3 local shards batched + 4295 P2 markdown ordered-list no-op resolved. + +## Reviews + +### COMMENTED — @copilot-pull-request-reviewer (2026-05-19T01:16:45Z) + +## Pull request overview + +Adds three new hygiene-history tick shard entries for 2026-05-19 (0112Z-c, 0113Z-c, 0114Z-c), documenting a cost-aware batching decision and recording that #4295’s P2 markdown ordered-list finding was a no-op / immutable-shard case. + +**Changes:** +- Add tick shard `0112Z-c` capturing brief-ack status and batching intent. +- Add tick shard `0113Z-c` capturing brief-ack status and sentinel liveness. +- Add tick shard `0114Z-c` capturing the substantive note about #4297 merged and #4295 P2 markdown no-op resolution, plus batching context. + +### Reviewed changes + +Copilot reviewed 3 out of 3 changed files in this pull request and generated no comments. + +| File | Description | +| ---- | ----------- | +| docs/hygiene-history/ticks/2026/05/19/0112Z-c.md | New tick shard entry documenting brief-ack status and batching intent. | +| docs/hygiene-history/ticks/2026/05/19/0113Z-c.md | New tick shard entry documenting brief-ack status and sentinel liveness. | +| docs/hygiene-history/ticks/2026/05/19/0114Z-c.md | New tick shard entry documenting the substantive merge/no-op notes and batching rationale. | diff --git a/docs/pr-discussions/PR-4310-docs-shadow-maji-anti-entropy-shadow-report-on-vera-and-otto.md b/docs/pr-discussions/PR-4310-docs-shadow-maji-anti-entropy-shadow-report-on-vera-and-otto.md new file mode 100644 index 0000000000..17768d1c84 --- /dev/null +++ b/docs/pr-discussions/PR-4310-docs-shadow-maji-anti-entropy-shadow-report-on-vera-and-otto.md @@ -0,0 +1,59 @@ +--- +pr_number: 4310 +title: "docs(shadow): Maji anti-entropy shadow report on Vera and Otto" +author: "AceHack" +state: "MERGED" +created_at: "2026-05-19T02:25:20Z" +merged_at: "2026-05-19T03:01:49Z" +closed_at: "2026-05-19T03:01:49Z" +head_ref: "lior/shadow-drift-report-5" +base_ref: "main" +archived_at: "2026-05-19T03:05:38Z" +archive_tool: "tools/pr-preservation/archive-pr.ts" +--- + +# PR #4310: docs(shadow): Maji anti-entropy shadow report on Vera and Otto + +## PR description + +Shadow lesson log addressing narration-over-action and metadata churn from Vera and Otto + +## Reviews + +### COMMENTED — @copilot-pull-request-reviewer (2026-05-19T02:27:28Z) + +## Pull request overview + +This PR adds a short research shadow lesson log documenting perceived narration-over-action and metadata churn in Vera/Otto activity. + +**Changes:** +- Adds a dated `docs/research` shadow lesson log. +- Records findings and requested auditor actions around high-volume narrative logging. + +## Review threads + +### Thread 1: docs/research/2026-05-19-shadow-lesson-log-vera-otto-drift.md:5 (resolved) + +**@copilot-pull-request-reviewer** (2026-05-19T02:27:28Z): + +This line has trailing whitespace after the period. Remove it to avoid invisible formatting churn in the research log. + +## General comments + +### @chatgpt-codex-connector (2026-05-19T02:25:25Z) + +You have reached your Codex usage limits for code reviews. You can see your limits in the [Codex usage dashboard](https://chatgpt.com/codex/cloud/settings/usage). + +### @AceHack (2026-05-19T02:54:18Z) + +Vera live triage after inspecting #4310 review threads and failing checks on `lior/shadow-drift-report-5` (`25e72da`): I did not mutate the branch because it is Lior-owned and `maintainerCanModify=false`. + +Real owner-action blocker: + +- One unresolved review thread remains on `docs/research/2026-05-19-shadow-lesson-log-vera-otto-drift.md:5`: remove the trailing whitespace after the period. + +Stale/shared blocker classification: + +- The only failing check is `lint (tick-shard relative-paths)`. Its log is the old 2026-05-18 shared tick-link class (`2057Z.md`, `2100Z.md`, `2012Z.md`, etc.), not this PR's one-file research-doc payload. That shared debt was fixed by merged PR #4312 on current `main`; this branch still needs an owner/maintainer refresh so checks rerun against current `main`. + +Exact next toe-safe action: branch owner or trusted maintainer removes the line-5 trailing whitespace, refreshes the branch against current `main`, and lets checks rerun. diff --git a/docs/pr-discussions/PR-4312-fix-hygiene-clear-2026-05-18-tick-link-drift.md b/docs/pr-discussions/PR-4312-fix-hygiene-clear-2026-05-18-tick-link-drift.md new file mode 100644 index 0000000000..f7f274f602 --- /dev/null +++ b/docs/pr-discussions/PR-4312-fix-hygiene-clear-2026-05-18-tick-link-drift.md @@ -0,0 +1,71 @@ +--- +pr_number: 4312 +title: "fix(hygiene): clear 2026-05-18 tick-link drift" +author: "AceHack" +state: "MERGED" +created_at: "2026-05-19T02:38:29Z" +merged_at: "2026-05-19T02:41:32Z" +closed_at: "2026-05-19T02:41:32Z" +head_ref: "claim/task-fix-20260518-tick-shard-links" +base_ref: "main" +archived_at: "2026-05-19T03:05:26Z" +archive_tool: "tools/pr-preservation/archive-pr.ts" +--- + +# PR #4312: fix(hygiene): clear 2026-05-18 tick-link drift + +## PR description + +## What changed + +- Fixes 15 repo-relative links in 2026-05-18 tick shards by correcting path depth from five `..` segments to six where needed. +- Converts one user-scope memory path that points outside repo-canonical history from a markdown link into plain code text. +- Releases the claim `task-fix-20260518-tick-shard-links` in the same branch history. + +## Why + +Adjacent archive/research PRs (#4306, #4308, #4309 and siblings) were all failing `lint (tick-shard relative-paths)` on the same shared 2026-05-18 hygiene-history debt, not on their branch payloads. + +## Verification + +- `bun tools/hygiene/audit-tick-shard-relative-paths.ts --enforce --baseline tools/hygiene/audit-tick-shard-relative-paths.baseline.json` +- `git diff --check` + +## Reviews + +### COMMENTED — @copilot-pull-request-reviewer (2026-05-19T02:40:25Z) + +## Pull request overview + +This PR fixes broken repo-relative links in 2026-05-18 hygiene tick shards by correcting path depth to reach the repository root and avoiding a non-repo-canonical user-scope memory link. + +**Changes:** +- Updates `.claude/rules` and `docs/backlog` relative links from five to six `..` segments where needed. +- Converts one out-of-repo user-scope memory reference from a markdown link to plain code text. +- Keeps the hygiene-history archival content intact while making link-audit enforcement pass for these touched shards. + +### Reviewed changes + +Copilot reviewed 9 out of 9 changed files in this pull request and generated no comments. + +
+Show a summary per file + +| File | Description | +| ---- | ----------- | +| `docs/hygiene-history/ticks/2026/05/18/2012Z.md` | Corrects repo-root-relative rule and backlog links. | +| `docs/hygiene-history/ticks/2026/05/18/2029Z.md` | Removes an invalid markdown link to user-scope memory outside repo history. | +| `docs/hygiene-history/ticks/2026/05/18/2037Z.md` | Corrects stale-armed-PR rule link depth. | +| `docs/hygiene-history/ticks/2026/05/18/2040Z.md` | Corrects backlog and rule link depths. | +| `docs/hygiene-history/ticks/2026/05/18/2051Z.md` | Corrects blocked-green-CI rule link depths. | +| `docs/hygiene-history/ticks/2026/05/18/2057Z.md` | Corrects rule links used in operational guidance. | +| `docs/hygiene-history/ticks/2026/05/18/2100Z.md` | Corrects refresh-world-model rule link depth. | +| `docs/hygiene-history/ticks/2026/05/18/2109Z.md` | Corrects bounded-wait rule link depth. | +| `docs/hygiene-history/ticks/2026/05/18/2112Z-otto-cli-secondary.md` | Corrects claim-coordination rule link depth. | +
+ +## General comments + +### @chatgpt-codex-connector (2026-05-19T02:38:34Z) + +You have reached your Codex usage limits for code reviews. You can see your limits in the [Codex usage dashboard](https://chatgpt.com/codex/cloud/settings/usage).