diff --git a/docs/pr-discussions/PR-2762-launch-draft-zeta-twitter-launch-post-multi-agent-review-req.md b/docs/pr-discussions/PR-2762-launch-draft-zeta-twitter-launch-post-multi-agent-review-req.md index 57391f46c4..8bf9410f32 100644 --- a/docs/pr-discussions/PR-2762-launch-draft-zeta-twitter-launch-post-multi-agent-review-req.md +++ b/docs/pr-discussions/PR-2762-launch-draft-zeta-twitter-launch-post-multi-agent-review-req.md @@ -8,7 +8,7 @@ merged_at: "2026-05-11T23:40:26Z" closed_at: "2026-05-11T23:40:26Z" head_ref: "launch/zeta-twitter-post-multi-agent-review-2026-05-11" base_ref: "main" -archived_at: "2026-05-14T07:40:07Z" +archived_at: "2026-05-14T08:25:41Z" archive_tool: "tools/pr-preservation/archive-pr.ts" --- diff --git a/docs/pr-discussions/PR-3074-feat-pm2-add-mira-to-expert-registry-forward-radar-scaffold.md b/docs/pr-discussions/PR-3074-feat-pm2-add-mira-to-expert-registry-forward-radar-scaffold.md index a158ff41dd..ceb81b90b5 100644 --- a/docs/pr-discussions/PR-3074-feat-pm2-add-mira-to-expert-registry-forward-radar-scaffold.md +++ b/docs/pr-discussions/PR-3074-feat-pm2-add-mira-to-expert-registry-forward-radar-scaffold.md @@ -8,7 +8,7 @@ merged_at: "2026-05-14T01:02:38Z" closed_at: "2026-05-14T01:02:38Z" head_ref: "feat/pm2-forward-radar-b0145-2026-05-13" base_ref: "main" -archived_at: "2026-05-14T07:40:05Z" +archived_at: "2026-05-14T08:25:42Z" archive_tool: "tools/pr-preservation/archive-pr.ts" --- diff --git a/docs/pr-discussions/PR-3075-feat-hygiene-check-md032-mechanize-md032-blanks-around-lists.md b/docs/pr-discussions/PR-3075-feat-hygiene-check-md032-mechanize-md032-blanks-around-lists.md index 5b6a3b27fb..472ab3e703 100644 --- a/docs/pr-discussions/PR-3075-feat-hygiene-check-md032-mechanize-md032-blanks-around-lists.md +++ b/docs/pr-discussions/PR-3075-feat-hygiene-check-md032-mechanize-md032-blanks-around-lists.md @@ -8,7 +8,7 @@ merged_at: "2026-05-14T02:52:39Z" closed_at: "2026-05-14T02:52:39Z" head_ref: "feat/b0456-md032-blanks-around-lists-helper-2026-05-14" base_ref: "main" -archived_at: "2026-05-14T07:40:06Z" +archived_at: "2026-05-14T08:25:44Z" archive_tool: "tools/pr-preservation/archive-pr.ts" --- diff --git a/docs/pr-discussions/PR-3114-backlog-b-0424-mark-in-progress-slices-1-8-merged-apply-next.md b/docs/pr-discussions/PR-3114-backlog-b-0424-mark-in-progress-slices-1-8-merged-apply-next.md new file mode 100644 index 0000000000..f4b548d895 --- /dev/null +++ b/docs/pr-discussions/PR-3114-backlog-b-0424-mark-in-progress-slices-1-8-merged-apply-next.md @@ -0,0 +1,99 @@ +--- +pr_number: 3114 +title: "backlog(b-0424): mark in-progress \u2014 slices 1-8 merged, --apply next-step documented" +author: "AceHack" +state: "MERGED" +created_at: "2026-05-14T08:09:22Z" +merged_at: "2026-05-14T08:16:17Z" +closed_at: "2026-05-14T08:16:17Z" +head_ref: "feat/b-0424-status-update-2026-05-14" +base_ref: "main" +archived_at: "2026-05-14T08:25:44Z" +archive_tool: "tools/pr-preservation/archive-pr.ts" +--- + +# PR #3114: backlog(b-0424): mark in-progress — slices 1-8 merged, --apply next-step documented + +## PR description + +## Summary + +- Slices B-0424.1–B-0424.8 (PRs #2994, #2996, #3003, #3019, #3025, #3026, #3027, #3028) are all merged +- Tests: **30/30 pass** (`bun test tools/scaffold/create-repo.ts`) +- Dry-run verified 2026-05-14: 12 operations planned for both `forge` and `ace` — no errors +- B-0424 status updated `open` → `in-progress`; completed-slices table and `--apply` execution instructions added + +## What's left (for full B-0424 closure) + +The only remaining step is running `--apply` to actually create the GitHub repos — an irreversible external action that requires Aaron's explicit review. The dry-run shows exactly what will happen before he decides. + +```bash +bun tools/scaffold/create-repo.ts --repo forge --apply +bun tools/scaffold/create-repo.ts --repo ace --apply +``` + +After `--apply`, six manual follow-ups remain (documented in the updated backlog row): SVG preview upload, merge-queue toggle, Semgrep CI wiring, required-status-check population, and budget-cap verification. + +## Test plan + +- [x] `bun test tools/scaffold/create-repo.test.ts` — 30 pass, 0 fail +- [x] `bun tools/scaffold/create-repo.ts --repo forge --dry-run` — 12 operations planned +- [x] `bun tools/scaffold/create-repo.ts --repo ace --dry-run` — 12 operations planned +- [x] No changes to source code — documentation-only update to backlog row + +🤖 Generated with [Claude Code](https://claude.com/claude-code) + +## Reviews + +### COMMENTED — @copilot-pull-request-reviewer (2026-05-14T08:12:20Z) + +## Pull request overview + +Documentation-only backlog update for B-0424, marking Stage 1 scaffold work as partially complete and documenting the remaining repo-creation execution step. + +**Changes:** +- Updates B-0424 frontmatter metadata and status. +- Adds a completed-slices table for B-0424.1–B-0424.8. +- Documents dry-run verification, `--apply` commands, and post-apply manual follow-ups. + +### COMMENTED — @copilot-pull-request-reviewer (2026-05-14T08:17:27Z) + +## Pull request overview + +Copilot reviewed 1 out of 1 changed files in this pull request and generated 1 comment. + +## Review threads + +### Thread 1: docs/backlog/P1/B-0424-three-repo-split-stage1-create-forge-ace-with-scaffolding-aaron-2026-05-13.md:4 (resolved) + +**@copilot-pull-request-reviewer** (2026-05-14T08:12:20Z): + +P1: `in-progress` is not a valid backlog frontmatter status per `tools/backlog/README.md:63`, which only permits `open`, `closed`, `superseded-by-B-NNNN`, or `deferred`. This row will no longer conform to the documented schema; keep it `open` or add/update schema/tooling support for this new state in the same PR. + +### Thread 2: docs/backlog/P1/B-0424-three-repo-split-stage1-create-forge-ace-with-scaffolding-aaron-2026-05-13.md:108 (resolved) + +**@copilot-pull-request-reviewer** (2026-05-14T08:12:20Z): + +P1: This tells the operator to run the irreversible `--apply` commands locally, bypassing the `scaffold-stage1-create-repos` workflow that was added as the safe apply entrypoint with actor allowlisting, a `CONFIRM` gate, PAT-secret handling, and concurrency protection. Since the row itself calls B-0424.2 the safe apply entrypoint, the next-step instructions should route through that workflow (or explicitly justify why these controls are being bypassed). + +### Thread 3: docs/backlog/P1/B-0424-three-repo-split-stage1-create-forge-ace-with-scaffolding-aaron-2026-05-13.md:79 (unresolved) + +**@copilot-pull-request-reviewer** (2026-05-14T08:17:27Z): + +P1: The PR title/summary says this backlog row is being marked `in-progress` (`open` → `in-progress`), but the row still uses `status: open` in frontmatter, and the backlog schema only allows `open`, `closed`, `superseded-by-B-NNNN`, or `deferred` (tools/backlog/README.md:63). Please align the PR text with the valid status model or update the schema/tooling before documenting a new status. + +## General comments + +### @chatgpt-codex-connector (2026-05-14T08:14:41Z) + +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-14T08:14:46Z) + +Fixed both Copilot P1 findings: + +1. **MD032 markdownlint**: added blank line before numbered list (commit 6d16f693) +2. **status enum**: `in-progress` → `open` — not a valid schema value per `tools/backlog/README.md` +3. **--apply bypass**: Next-step instructions now route through the `scaffold-stage1-create-repos` GitHub Actions workflow (actor allowlist + CONFIRM gate + PAT secret + concurrency protection) instead of raw local bash commands; dry-run-only bash snippet retained for local preview + +Threads resolved. CI rerunning. diff --git a/docs/pr-discussions/PR-3115-docs-preserve-pr-discussions-for-3112-and-3113-close-b-0451.md b/docs/pr-discussions/PR-3115-docs-preserve-pr-discussions-for-3112-and-3113-close-b-0451.md new file mode 100644 index 0000000000..8502c80d38 --- /dev/null +++ b/docs/pr-discussions/PR-3115-docs-preserve-pr-discussions-for-3112-and-3113-close-b-0451.md @@ -0,0 +1,74 @@ +--- +pr_number: 3115 +title: "docs: preserve PR discussions for 3112 and 3113; close B-0451" +author: "AceHack" +state: "MERGED" +created_at: "2026-05-14T08:13:07Z" +merged_at: "2026-05-14T08:21:26Z" +closed_at: "2026-05-14T08:21:27Z" +head_ref: "lior/preserve-prs-3112-3113" +base_ref: "main" +archived_at: "2026-05-14T08:25:45Z" +archive_tool: "tools/pr-preservation/archive-pr.ts" +--- + +# PR #3115: docs: preserve PR discussions for 3112 and 3113; close B-0451 + +## PR description + +Preserving recently merged PRs per step 4b PR archive discipline. + +Also closes B-0451 (duplicate row-ID substrate cleanup) — all 12 collision +groups resolved, audit tool exits 0. CI-wiring follow-up tracked in +B-0451 Future work section (separate slice, does not block closure). + +Closes #B-0451 + +## Reviews + +### COMMENTED — @copilot-pull-request-reviewer (2026-05-14T08:16:38Z) + +## Pull request overview + +This PR primarily preserves GitHub PR discussion archives for merged PRs #3112 and #3113 under `docs/pr-discussions/`, but it also includes a hygiene tick shard and closes backlog item B-0451. + +**Changes:** +- Adds generated PR discussion archives for PR #3112 and PR #3113. +- Adds a hygiene-history tick shard for 2026-05-14T05:21Z. +- Marks B-0451 closed and updates `docs/BACKLOG.md` accordingly. + +### Reviewed changes + +Copilot reviewed 5 out of 5 changed files in this pull request and generated 2 comments. + +
+Show a summary per file + +| File | Description | +| ---- | ----------- | +| `docs/pr-discussions/PR-3113-docs-lior-antigravity-check-pr-preservation-and-blob-drift-l.md` | Adds preserved PR #3113 discussion metadata and review content. | +| `docs/pr-discussions/PR-3112-docs-b-0289-green-lantern-hardware-spec-local-inference-requ.md` | Adds preserved PR #3112 discussion metadata, reviews, and review threads. | +| `docs/hygiene-history/ticks/2026/05/14/0521Z.md` | Adds tick-session notes covering PR gate status and B-0451 closure work. | +| `docs/backlog/P1/B-0451-duplicate-row-id-substrate-cleanup-2026-05-13.md` | Marks B-0451 closed and updates acceptance criteria status. | +| `docs/BACKLOG.md` | Updates the generated backlog index to show B-0451 as closed. | +
+ +### COMMENTED — @copilot-pull-request-reviewer (2026-05-14T08:22:18Z) + +## Pull request overview + +Copilot reviewed 3 out of 3 changed files in this pull request and generated no new comments. + +## Review threads + +### Thread 1: docs/backlog/P1/B-0451-duplicate-row-id-substrate-cleanup-2026-05-13.md:93 (resolved) + +**@copilot-pull-request-reviewer** (2026-05-14T08:16:37Z): + +This closes B-0451 while leaving one acceptance criterion unchecked. If CI wiring is no longer required for this row, move it out of the acceptance-criteria checklist and link/create the follow-up row; otherwise keep the backlog item open until the criterion is complete. + +### Thread 2: docs/backlog/P1/B-0451-duplicate-row-id-substrate-cleanup-2026-05-13.md:5 (resolved) + +**@copilot-pull-request-reviewer** (2026-05-14T08:16:37Z): + +The PR title and description only describe preserving PR discussions for #3112/#3113, but this change also closes backlog item B-0451. Please either update the PR metadata to include the backlog closure or split this state change into a separate PR so reviewers can evaluate the scope accurately. diff --git a/docs/pr-discussions/PR-3116-docs-lior-antigravity-check-shadow-log-for-vera-and-riven-dr.md b/docs/pr-discussions/PR-3116-docs-lior-antigravity-check-shadow-log-for-vera-and-riven-dr.md new file mode 100644 index 0000000000..a6650fb943 --- /dev/null +++ b/docs/pr-discussions/PR-3116-docs-lior-antigravity-check-shadow-log-for-vera-and-riven-dr.md @@ -0,0 +1,69 @@ +--- +pr_number: 3116 +title: "docs(lior): antigravity check - shadow log for Vera and Riven drift" +author: "AceHack" +state: "MERGED" +created_at: "2026-05-14T08:13:17Z" +merged_at: "2026-05-14T08:17:19Z" +closed_at: "2026-05-14T08:17:19Z" +head_ref: "lior/shadow-log-vera-riven-drift" +base_ref: "main" +archived_at: "2026-05-14T08:25:46Z" +archive_tool: "tools/pr-preservation/archive-pr.ts" +--- + +# PR #3116: docs(lior): antigravity check - shadow log for Vera and Riven drift + +## PR description + +Addressing drift in Vera and Riven where narration-over-action and missing auto-healing were observed. + +## Reviews + +### COMMENTED — @copilot-pull-request-reviewer (2026-05-14T08:18:11Z) + +## Pull request overview + +This docs-only PR records an observed drift pattern (narration-over-action / missing auto-healing) and updates backlog hygiene artifacts to mark B-0451 as closed, including corresponding index and tick-history updates. + +**Changes:** +- Added a new research shadow-log capturing the Vera/Riven drift observation + correction framing. +- Added a new tick-history shard documenting the session state and B-0451 closure actions. +- Closed backlog row **B-0451** (frontmatter) and updated the generated `docs/BACKLOG.md` checkbox entry. + +### Reviewed changes + +Copilot reviewed 4 out of 4 changed files in this pull request and generated 3 comments. + +| File | Description | +| ---- | ----------- | +| docs/research/2026-05-14-shadow-lesson-log-vera-riven-drift.md | New research note documenting the observed drift and correction. | +| docs/hygiene-history/ticks/2026/05/14/0521Z.md | New tick shard capturing the session checklist, PR gate status, and B-0451 closure actions. | +| docs/backlog/P1/B-0451-duplicate-row-id-substrate-cleanup-2026-05-13.md | Marks B-0451 as closed and updates acceptance criteria status text. | +| docs/BACKLOG.md | Updates the generated index entry to reflect B-0451 as closed. | + +## Review threads + +### Thread 1: docs/hygiene-history/ticks/2026/05/14/0521Z.md:6 (unresolved) + +**@copilot-pull-request-reviewer** (2026-05-14T08:18:09Z): + +P1: This new tick shard doesn’t include the YAML frontmatter metadata (e.g., `tick`, `agent`, `session`, `pr`) that’s consistently present in other tick shards in this directory (e.g., `.../0152Z.md`). Please add the frontmatter (and ideally align the H1/header format) so downstream tooling and readers can reliably locate the tick timestamp/PR context. + +### Thread 2: docs/backlog/P1/B-0451-duplicate-row-id-substrate-cleanup-2026-05-13.md:93 (unresolved) + +**@copilot-pull-request-reviewer** (2026-05-14T08:18:10Z): + +P1: This row is being marked `status: closed`, but the acceptance criteria still contains an unchecked item that explicitly calls for a separate follow-up row (“Wire audit-duplicate-row-ids.ts into a CI workflow…”). Please either (a) file/link that follow-up backlog row here, or (b) move this item out of acceptance criteria (or keep this row open/deferred) so the closed status reflects completed ACs. + +### Thread 3: docs/BACKLOG.md:257 (unresolved) + +**@copilot-pull-request-reviewer** (2026-05-14T08:18:10Z): + +P2: The PR title/description focus on the Vera/Riven drift shadow-log, but this PR also closes backlog item B-0451 (row frontmatter + regenerated `docs/BACKLOG.md`) and adds a tick shard documenting that closure. Please update the PR description to reflect the full scope (or split into separate PRs) so reviewers/readers don’t miss the backlog-status change. + +## General comments + +### @chatgpt-codex-connector (2026-05-14T08:13:22Z) + +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).