Skip to content

fix(pr-preservation): add archived files to git#4857

Closed
AceHack wants to merge 9 commits into
mainfrom
lior-fix-pr-preservation
Closed

fix(pr-preservation): add archived files to git#4857
AceHack wants to merge 9 commits into
mainfrom
lior-fix-pr-preservation

Conversation

@AceHack
Copy link
Copy Markdown
Member

@AceHack AceHack commented May 24, 2026

This PR fixes a bug in the 'archive-pr.ts' script where it was not adding the created archive files to git. This was causing Riven's worktree to become dirty. Fixes #4856

Copilot AI review requested due to automatic review settings May 24, 2026 20:03
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This pull request updates the PR-preservation workflow by staging newly generated PR-archive markdown files into git, and also brings in a large set of newly archived PR discussion files and related documentation/memory updates.

Changes:

  • Update tools/pr-preservation/archive-pr.ts to git add the generated archive file after writing it.
  • Add many new docs/pr-discussions/* archive files (plus new/updated research, backlog, and tick-history artifacts).
  • Update Riven’s Cursor launchd tick script and formal-verification routing documentation to reflect recent operational/routing decisions.

Reviewed changes

Copilot reviewed 43 out of 44 changed files in this pull request and generated 6 comments.

Show a summary per file
File Description
tools/pr-preservation/archive-pr.ts Stages the generated PR archive into git after writing the file.
memory/persona/soraya/NOTEBOOK.md Adds a Trigger Recognition Log section + initial entries.
memory/MEMORY.md Updates the “Last reindex” date.
docs/research/2026-05-24-dev-pc-substrate-architecture-nix-home-manager-kind-k3d-headscale-lend-pattern-aaron-otto.md Adds a dev-PC substrate architecture research archive.
docs/research/2026-05-24-cluster-bare-metal-substrate-architecture-nixos-no-hypervisor-argo-cd-aaron-otto.md Adds a cluster substrate architecture research archive.
docs/research/2026-05-23-shadow-lesson-log-metadata-churn-paralysis.md Adds a shadow lesson log research entry.
docs/research/2026-05-23-amazon-vendor-management-failure-mode-corpus-multi-incident-business-development-substrate-aaron-forwarded.md Extends the vendor-management corpus with additional Pattern R content.
docs/pr-discussions/PR-4853-shard-2026-05-24-1902z-otto-vscode-bg-worker-lior-lane-fleet.md Adds archived PR discussion record for PR #4853.
docs/pr-discussions/PR-4848-shard-2026-05-24-1804z-otto-vscode-bg-worker-pr-4799-merged.md Adds archived PR discussion record for PR #4848.
docs/pr-discussions/PR-4836-tick-2026-05-24-1608z-dotgit-saturation-15th-observation-2nd.md Adds archived PR discussion record for PR #4836.
docs/pr-discussions/PR-4835-shard-2026-05-24-1607z-dotgit-saturation-15th-observation-2h.md Adds archived PR discussion record for PR #4835.
docs/pr-discussions/PR-4821-fix-4780-address-review-comments-on-residuated-lattice-tests.md Adds archived PR discussion record for PR #4821.
docs/pr-discussions/PR-4820-shard-2026-05-24-1407z-dotgit-14th-observation-0-procs-first.md Adds archived PR discussion record for PR #4820.
docs/pr-discussions/PR-4819-fix-riven-update-autonomous-gate-prompt-to-trajectory-manage.md Adds archived PR discussion record for PR #4819.
docs/pr-discussions/PR-4818-tick-2026-05-24-1333z-pr-4816-tier-5-deferral-hard-limits-fl.md Adds archived PR discussion record for PR #4818.
docs/pr-discussions/PR-4814-shard-2026-05-24-0441z-25-pr-audit-8-blocked-prs-classified.md Adds archived PR discussion record for PR #4814.
docs/pr-discussions/PR-4813-shard-2026-05-24-0416z-rename-0240z-0416z-substrate-honest-t.md Adds archived PR discussion record for PR #4813.
docs/pr-discussions/PR-4812-shard-2026-05-24-0240z-9th-dotgit-saturation-anchor-33-stuck.md Adds archived PR discussion record for PR #4812.
docs/pr-discussions/PR-4811-soraya-round-69-execute-b-0719-pick-add-trigger-recognition.md Adds archived PR discussion record for PR #4811.
docs/pr-discussions/PR-4810-backlog-b-0719-file-soraya-round-67-forced-decomposition-aud.md Adds archived PR discussion record for PR #4810.
docs/pr-discussions/PR-4809-docs-research-bundle-file-dev-pc-substrate-architecture-nix.md Adds archived PR discussion record for PR #4809.
docs/pr-discussions/PR-4808-docs-research-cluster-bare-metal-substrate-architecture-deci.md Adds archived PR discussion record for PR #4808.
docs/pr-discussions/PR-4806-docs-research-add-pattern-r-vendor-neutral-accounting-achiev.md Adds archived PR discussion record for PR #4806.
docs/pr-discussions/PR-4799-docs-archive-preserve-recently-merged-prs-decomposed-from-47.md Adds archived PR discussion record for PR #4799.
docs/pr-discussions/PR-4798-docs-shadow-add-shadow-lesson-log-for-metadata-churn-paralys.md Adds archived PR discussion record for PR #4798.
docs/pr-discussions/PR-4797-backlog-b-0718-file-soraya-round-61-forced-decomposition-aud.md Adds archived PR discussion record for PR #4797.
docs/pr-discussions/PR-4796-docs-research-restore-patterns-h-o-lost-in-pr-4784-merge-rac.md Adds archived PR discussion record for PR #4796.
docs/pr-discussions/PR-4795-backlog-b-0717-file-soraya-round-57-hand-off-lsm-spine-regis.md Adds archived PR discussion record for PR #4795.
docs/pr-discussions/PR-4794-docs-research-preserve-addison-consent-pattern-observation-n.md Adds archived PR discussion record for PR #4794.
docs/pr-discussions/PR-4793-docs-research-add-pattern-p-wear-down-design-produces-unboun.md Adds archived PR discussion record for PR #4793.
docs/pr-discussions/PR-4792-docs-research-preserve-ai-context-failures-vs-vendor-managem.md Adds archived PR discussion record for PR #4792.
docs/pr-discussions/PR-4791-backlog-b-0716-soraya-round-53-scope-correction-b-0709-enume.md Adds archived PR discussion record for PR #4791.
docs/pr-discussions/PR-4790-backlog-b-0715-file-soraya-round-52-hand-off-register-istime.md Adds archived PR discussion record for PR #4790.
docs/pr-discussions/PR-4789-backlog-b-0714-file-soraya-round-51-hand-off-3-tla-specs-mis.md Adds archived PR discussion record for PR #4789.
docs/pr-discussions/PR-4784-docs-research-amazon-vendor-management-failure-mode-corpus-5.md Adds archived PR discussion record for PR #4784.
docs/pr-discussions/PR-4783-backlog-b-0713-file-soraya-round-50-hand-off-lean-imaginarys.md Adds archived PR discussion record for PR #4783.
docs/hygiene-history/ticks/2026/05/24/0416Z.md Adds a tick shard for the corrected timestamp.
docs/hygiene-history/ticks/2026/05/24/0240Z.md Adds a tick shard for the original timestamped version.
docs/backlog/P3/B-0719-soraya-round67-audit-of-audit-recognition-without-row-filing-precedent-2026-05-24.md Adds a new backlog row B-0719.
docs/backlog/P3/B-0718-soraya-four-trigger-framework-cadence-audit-2026-05-23.md Adds a new backlog row B-0718.
docs/BACKLOG.md Updates the generated backlog index to include B-0718/B-0719.
.cursor/bin/riven-loop-tick.ts Changes the Riven gate invocation to use agent chat and simplifies the prompt logic.
.claude/skills/formal-verification-expert/SKILL.md Updates reference patterns to point at the new Trigger Recognition Log landing.

Comment on lines 792 to 799
const existing = findExistingArchive(outDir, args.pr);
const path =
existing ??
join(outDir, `PR-${String(args.pr).padStart(4, "0")}-${makeSlug(fetched.pr.title ?? "untitled")}.md`);

writeFileSync(path, content);
spawnSync("git", ["add", path]);
process.stdout.write(
Comment on lines +35 to +40
- **At anchor 9 (02:40Z=33)** — second below-extreme reading. The two mild readings now span ~4.5h (22:08Z and 02:40Z). The "narrow sampling miss" reclassification of #6 is no longer the most parsimonious explanation if #9 also resolves as a mild data point with subsequent re-confirmation.

**Two non-mutually-exclusive readings (per [`default-to-both.md`](../../../../../../.claude/rules/default-to-both.md))**:

1. **Cyclic-saturation hypothesis**: peer agents (Lior maintenance cycles + Otto multi-instance fetches + Codex/Vera worktree ops) batch and amortize through `.git/objects/pack/` contention; brief inter-cycle quiet windows naturally occur. The two mild readings (22:08Z + 02:40Z) ARE cycle troughs.
2. **Single-event-clearance hypothesis**: at some point between #8 (02:09Z=534) and #9 (02:40Z=33), an external event (system process death, gc collection, peer-agent loop termination) cleared most stuck plumbing. Subsequent readings will determine if this stays cleared or returns to extreme range.
Comment on lines +1 to +9
| 2026-05-24T02:40Z | opus-4-7 / autonomous-loop | bf82d0a2 | substantive — Otto-CLI fresh-session cold-boot at 02:40Z; sentinel re-armed (CronList empty); 9th dotgit-saturation anchor in rolling 24h window: 33 stuck git plumbing procs (-94% from 02:09Z=534 in ~30min — largest single-step descent in the rolling series); isolated worktree off origin/main @ 209c18c5f2; ls-tree=55 status=0 (one transient `Interrupted system call` mid-extraction but completed cleanly); cold-boot landed on Alexa branch (4th branch-contamination anchor) | -- | 9th dotgit anchor descent shard |

# Tick 0240Z — 2026-05-24 Otto-CLI cold-boot, 9th dotgit anchor (33 procs; -94% from 02:09Z peak in 30min)

**Surface:** Otto-CLI (autonomous-loop fresh-session cold-boot)
**Branch:** `otto-cli/dotgit-9th-anchor-descent-0240z` (isolated worktree at `/private/tmp/zeta-dotgit-canary-0240z` off `origin/main` @ `209c18c5f2`)
**Tier (rate-limit):** Normal (GraphQL 4362/5000; reset ~40min; REST core 4887/5000)
**Tier (dotgit):** **mild** (33 stuck pack/maintenance/repack procs; first sub-saturated reading in series since 22:08Z=93)
**Sentinel:** `bf82d0a2` armed at 02:40Z; CronList returned empty at session-start (session-exit non-persistence per [`.claude/rules/tick-must-never-stop.md`](../../../../../../.claude/rules/tick-must-never-stop.md))
Comment on lines 795 to 799
join(outDir, `PR-${String(args.pr).padStart(4, "0")}-${makeSlug(fetched.pr.title ?? "untitled")}.md`);

writeFileSync(path, content);
spawnSync("git", ["add", path]);
process.stdout.write(
Comment on lines +1 to +12
---
id: B-0719
priority: P3
status: open
title: "Soraya round-67 forced-decomposition — audit-of-audit: ratify the recognition-without-row-filing precedent (when trigger fires + 'not my lane,' where does the routing-decision substrate land?)"
created: 2026-05-24
last_updated: 2026-05-24
classification: buildable-now
decomposition: atomic
assignee: soraya
discovered_by: soraya
owners: [soraya, kenji]
Comment on lines +71 to +83
P0: This line contains an invisible Unicode character (U+200B / zero-width space) in the `@​EdwardCooke` handle. The repo’s `invisible-unicode-in-text` check will flag this in `*.md` and fail CI; replace it with plain ASCII (`@EdwardCooke`) and ensure no zero-width characters remain.

### Thread 2: docs/pr-discussions/PR-4722-docs-archive-preserve-merged-pr-4714.md:52 (resolved)

**@copilot-pull-request-reviewer** (2026-05-24T01:09:57Z):

P0: This thread text includes an actual U+200B zero-width space inside the example `[#​1907]` / `[#​....]` code spans. Even though it’s describing the issue, the literal hidden character will still trigger Semgrep `invisible-unicode-in-text` and break CI; replace the examples with ASCII (e.g., `[#1907]`) and describe the zero-width space in words ("U+200B between `#` and digits").

### Thread 3: docs/pr-discussions/PR-4722-docs-archive-preserve-merged-pr-4714.md:52 (resolved)

**@copilot-pull-request-reviewer** (2026-05-24T01:09:57Z):

P0: This line contains a U+200B zero-width space inside the `@​meziantou` / `@​Copilot` code spans. Semgrep `invisible-unicode-in-text` will flag the hidden character even inside backticks; rewrite the examples using plain ASCII (`@meziantou`, `@Copilot`) and keep the mention of U+200B as text only.
Copilot AI review requested due to automatic review settings May 24, 2026 20:16
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 43 out of 44 changed files in this pull request and generated 4 comments.

Comment on lines 797 to 799
writeFileSync(path, content);
spawnSync("git", ["add", path]);
process.stdout.write(
Comment thread docs/BACKLOG.md
Comment on lines +12 to +13
<!-- Lior was here -->

Comment on lines +35 to +40
- **At anchor 9 (02:40Z=33)** — second below-extreme reading. The two mild readings now span ~4.5h (22:08Z and 02:40Z). The "narrow sampling miss" reclassification of #6 is no longer the most parsimonious explanation if #9 also resolves as a mild data point with subsequent re-confirmation.

**Two non-mutually-exclusive readings (per [`default-to-both.md`](../../../../../../.claude/rules/default-to-both.md))**:

1. **Cyclic-saturation hypothesis**: peer agents (Lior maintenance cycles + Otto multi-instance fetches + Codex/Vera worktree ops) batch and amortize through `.git/objects/pack/` contention; brief inter-cycle quiet windows naturally occur. The two mild readings (22:08Z + 02:40Z) ARE cycle troughs.
2. **Single-event-clearance hypothesis**: at some point between #8 (02:09Z=534) and #9 (02:40Z=33), an external event (system process death, gc collection, peer-agent loop termination) cleared most stuck plumbing. Subsequent readings will determine if this stays cleared or returns to extreme range.
Comment on lines +9 to +12
**Sentinel:** `bf82d0a2` armed at 04:16Z; CronList returned empty at session-start (session-exit non-persistence per [`.claude/rules/tick-must-never-stop.md`](../../../../../../.claude/rules/tick-must-never-stop.md))

**Wall-clock-estimate-error substrate-honest disclosure**: original shard authored with timestamp `02:40Z` based on agent-side wall-clock estimate. After PR open (#4812), `date -u` returned `2026-05-24T04:16Z` — the actual session time was ~1h36min later than estimated. File renamed `0240Z.md` → `0416Z.md`; timestamps in shard body corrected; PR title updated; descent interval recomputed from ~30min to ~2h7min (the `-94%` magnitude is unchanged because the 02:09Z peak is fixed and 33 is fixed; only the time-window changes). The substantive empirical content (9th anchor + descent + tier classification) is unaffected.

@AceHack
Copy link
Copy Markdown
Member Author

AceHack commented May 25, 2026

This PR is too complex to resolve at this time due to a large number of merge conflicts. It will be closed for now and can be re-opened after the conflicts have been resolved.

@AceHack AceHack closed this May 25, 2026
auto-merge was automatically disabled May 25, 2026 18:01

Pull request was closed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

fix(pr-preservation): archive-pr.ts does not add files to git

2 participants