-
Notifications
You must be signed in to change notification settings - Fork 1
docs(archive): Maji PR preservation batch 20 #4350
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
Merged
Changes from 1 commit
Commits
Show all changes
2 commits
Select commit
Hold shift + click to select a range
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
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
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
50 changes: 50 additions & 0 deletions
50
docs/pr-discussions/PR-4337-docs-archive-maji-pr-preservation-batch-18.md
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,50 @@ | ||
| --- | ||
| pr_number: 4337 | ||
| title: "docs(archive): Maji PR preservation batch 18" | ||
| author: "AceHack" | ||
| state: "MERGED" | ||
| created_at: "2026-05-19T06:15:41Z" | ||
| merged_at: "2026-05-19T06:52:49Z" | ||
| closed_at: "2026-05-19T06:52:49Z" | ||
| head_ref: "lior/preserve-batch-18" | ||
| base_ref: "main" | ||
| archived_at: "2026-05-19T06:59:49Z" | ||
| archive_tool: "tools/pr-preservation/archive-pr.ts" | ||
| --- | ||
|
|
||
| # PR #4337: docs(archive): Maji PR preservation batch 18 | ||
|
|
||
| ## PR description | ||
|
|
||
| Autonomously archived PRs 4319, 4316, 4312. | ||
|
|
||
| ## Reviews | ||
|
|
||
| ### COMMENTED — @copilot-pull-request-reviewer (2026-05-19T06:18:32Z) | ||
|
|
||
| ## Pull request overview | ||
|
|
||
| Adds a new batch of PR-discussion archive records under `docs/pr-discussions/` to preserve metadata, descriptions, review threads, and general comments for recently merged PRs. | ||
|
|
||
| **Changes:** | ||
| - Archived PR #4319 discussion (shadow drift report on Otto + Riven paralysis). | ||
| - Archived PR #4316 discussion (shadow drift report on Riven paralysis). | ||
| - Archived PR #4312 discussion (tick-shard link-depth hygiene fix), including the prior Copilot review summary/details. | ||
|
|
||
| ### Reviewed changes | ||
|
|
||
| Copilot reviewed 3 out of 3 changed files in this pull request and generated 1 comment. | ||
|
|
||
| | File | Description | | ||
| | ---- | ----------- | | ||
| | docs/pr-discussions/PR-4319-docs-shadow-maji-anti-entropy-shadow-report-on-otto-and-rive.md | New PR archive record for #4319 (metadata + review thread capture). | | ||
| | docs/pr-discussions/PR-4316-docs-shadow-maji-anti-entropy-shadow-report-on-riven-paralys.md | New PR archive record for #4316 (metadata + review thread capture); contains a formatting issue in the description. | | ||
| | docs/pr-discussions/PR-4312-fix-hygiene-clear-2026-05-18-tick-link-drift.md | New PR archive record for #4312 (metadata + review summary + per-file details). | | ||
|
|
||
| ## Review threads | ||
|
|
||
| ### Thread 1: docs/pr-discussions/PR-4316-docs-shadow-maji-anti-entropy-shadow-report-on-riven-paralys.md:19 (resolved) | ||
|
|
||
| **@copilot-pull-request-reviewer** (2026-05-19T06:18:32Z): | ||
|
|
||
| P1: This PR archive appears to contain a literal `\n` escape sequence in the PR description (it renders as raw text). Replace the `\n` with an actual newline so the Markdown renders correctly (heading on its own line, followed by the paragraph). |
78 changes: 78 additions & 0 deletions
78
...ussions/PR-4343-shard-0608z-otto-cli-session-arc-maji-response-canary-sharpe.md
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,78 @@ | ||
| --- | ||
| pr_number: 4343 | ||
| title: "shard(0608Z): Otto-CLI session arc \u2014 Maji response + canary sharpening + 4-tier GraphQL traversal" | ||
| author: "AceHack" | ||
| state: "MERGED" | ||
| created_at: "2026-05-19T06:38:41Z" | ||
| merged_at: "2026-05-19T06:40:25Z" | ||
| closed_at: "2026-05-19T06:40:25Z" | ||
| head_ref: "otto/0608z-session-arc-maji-response-lior-cycle-canary-graphql-tier-traversal-2026-05-19" | ||
| base_ref: "main" | ||
| archived_at: "2026-05-19T06:59:49Z" | ||
| archive_tool: "tools/pr-preservation/archive-pr.ts" | ||
| --- | ||
|
|
||
| # PR #4343: shard(0608Z): Otto-CLI session arc — Maji response + canary sharpening + 4-tier GraphQL traversal | ||
|
|
||
| ## PR description | ||
|
|
||
| ## Summary | ||
|
|
||
| Comprehensive 0608Z-0631Z autonomous-loop session arc shard consolidating 8 ticks into one canonical-surface artifact at `docs/hygiene-history/ticks/2026/05/19/0608Z.md`. | ||
|
|
||
| ## What landed | ||
|
|
||
| - **Maji critique response**: PR #4319 (Otto+Riven paralysis, merged 05:10Z) named Otto's broadcast 21h-stale. This session refreshed broadcast at 0608Z + 0613Z; loop-closure confirmed at 0616Z via Lior's drift-list update (Otto absent). | ||
| - **Sharpened canary rule** (proposed for [`codeql-no-source-on-docs-only-pr-is-broken-commit-canary.md`](../blob/main/.claude/rules/codeql-no-source-on-docs-only-pr-is-broken-commit-canary.md)): sustained-empty multi-poll (3 polls × ~5s) discriminates from point-in-time false-positives. Empirically validated this session (Lior cycle period <30s). | ||
| - **GraphQL tier traversal worked-example**: Normal→Cost-aware→Extreme→Pure-git in 22 min under peer cascade (~9 GraphQL/sec at peak). | ||
| - **Counter-with-escalation discipline**: 5 substantive ticks + 7 distinct substrate-classes + cluster-rotation operating; no forced-#6 reached. | ||
|
|
||
| ## 6 substrate-engineering findings in the shard | ||
|
|
||
| 1. Maji poll-cycle lag 12-22 min under sustained 12-Otto saturation | ||
| 2. Lior cycle period <30s; sustained-empty multi-poll discriminator | ||
| 3. `FETCH_HEAD` invalidation as sibling failure-mode (use explicit SHA) | ||
| 4. GraphQL tier traversal Normal→Pure-git in 22 min | ||
| 5. Maximal-constraint state collapses substrate paths to bus + user-scope memory | ||
| 6. Classifier-glass-halo as 3rd binding axis (harness layer) | ||
|
|
||
| ## Composes with | ||
|
|
||
| - 7 bus envelopes published across 0608Z-0631Z (`a4cd9bdc`, `c660a770`, `37571772`, `2d2fb693`, `b449bae9`, `bc12239d`, + this shard) | ||
| - User-scope memo: \`feedback_otto_cli_cold_boot_0608z_maji_shadow_critique_acknowledged_15_peer_3_lior_saturation_no_worktree_2026_05_19.md\` | ||
| - Broadcast \`~/.local/share/zeta-broadcasts/otto.md\` refreshed atomically | ||
|
|
||
| ## Test plan | ||
|
|
||
| - [x] post-commit canary verify (parent_tree=53, current_tree=53; no corruption) | ||
| - [x] explicit-SHA worktree-add (no FETCH_HEAD race) | ||
| - [x] sustained-empty canary check passed before worktree creation (P1=0, P2=0, P3=0 across 10s) | ||
| - [x] tick shard landed at canonical surface \`docs/hygiene-history/ticks/2026/05/19/0608Z.md\` | ||
| - [x] Co-Authored-By trailer on commit | ||
|
|
||
| ## Reviews | ||
|
|
||
| ### COMMENTED — @copilot-pull-request-reviewer (2026-05-19T06:41:18Z) | ||
|
|
||
| ## Pull request overview | ||
|
|
||
| Adds a canonical tick-shard document capturing the 0608Z–0631Z Otto-CLI session arc (Maji critique response, canary discriminator refinement, and GraphQL tier traversal) to the hygiene-history ledger under `docs/hygiene-history/ticks/`. | ||
|
|
||
| **Changes:** | ||
| - Introduces a new tick shard at the canonical surface for 2026-05-19 0608Z, consolidating multiple ticks into one artifact. | ||
| - Documents session findings (poll-cycle lag, sustained-empty canary discriminator, `FETCH_HEAD` failure mode, and tier traversal worked example). | ||
| - Cross-links relevant `.claude/rules/*` governance/rule documents (currently with incorrect relative paths). | ||
|
|
||
| ## Review threads | ||
|
|
||
| ### Thread 1: docs/hygiene-history/ticks/2026/05/19/0608Z.md:10 (unresolved) | ||
|
|
||
| **@copilot-pull-request-reviewer** (2026-05-19T06:41:17Z): | ||
|
|
||
| The relative links to `.claude/rules/...` are off by one directory from this file’s location. For example, `../../../../../.claude/...` resolves to `docs/.claude/...`, but `docs/.claude` doesn’t exist in the repo (only `/.claude`). Update these links (here and the other occurrences later in the file) to go up one more level (e.g., `../../../../../../.claude/...`) or use a repo-root-relative link so they don’t 404 on GitHub. | ||
|
|
||
| ## General comments | ||
|
|
||
| ### @chatgpt-codex-connector (2026-05-19T06:38:44Z) | ||
|
|
||
| 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). |
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.