docs(archive): Maji preservation of merged PRs#3997
Conversation
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 909f2dfc3d
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
|
|
||
| Cold-boot autonomous-loop tick. PR #3974 was BLOCKED with auto-merge armed on a single failing required check (`lint (markdownlint)` MD032 on B-0583 backlog row). | ||
|
|
||
| Per [`.claude/rules/blocked-green-ci-investigate-threads.md`](.claude/rules/blocked-green-ci-investigate-threads.md), investigated rather than waited. Fix landed on the B-0583 branch via commit `5ebe94b` (single blank line between intro sentence and following list at line 42-43). |
There was a problem hiding this comment.
Fix archived rule link to resolve from docs/pr-discussions
This archive entry links to .claude/rules/... as if it were at repo root, but from docs/pr-discussions/ that resolves to docs/pr-discussions/.claude/... (nonexistent). In practice, readers of this preserved PR cannot open the cited rule, which breaks the traceability this archive is meant to provide; use a path that resolves from this directory (for example ../../.claude/...) or an absolute GitHub URL.
Useful? React with 👍 / 👎.
|
|
||
| Adds `tools/profile.test.ts` — 9 smoke tests covering the CLI | ||
| dispatch surface of `tools/profile.ts`. Closes the last gap in | ||
| [B-0156](../blob/main/docs/backlog/P1/B-0156-typescript-standardization-non-install-scripts-aaron-2026-05-01.md) |
There was a problem hiding this comment.
Replace PR-page-relative blob links in archived content
The archived description still uses ../blob/main/... links, which only work when rendered from a /pull/<id> page. In docs/pr-discussions/, this resolves to docs/blob/main/... (nonexistent), so the referenced backlog/rule documents are dead links (same pattern appears again at line 56). Rewriting these to repo-valid relative paths or absolute URLs keeps the archive navigable.
Useful? React with 👍 / 👎.
There was a problem hiding this comment.
Pull request overview
Archival of merged PR conversation history into docs/pr-discussions/ per preservation discipline. These files preserve PR descriptions, reviews, and threads from five recently-merged PRs (3995, 3983, 3975, 3974, 3973) as historical record.
Changes:
- Adds five new PR archive markdown files under
docs/pr-discussions/. - Each file contains YAML frontmatter (pr_number, title, author, state, timestamps, refs, archive metadata) plus the preserved PR description, reviews, and review threads.
Reviewed changes
Copilot reviewed 5 out of 5 changed files in this pull request and generated no comments.
Show a summary per file
| File | Description |
|---|---|
| docs/pr-discussions/PR-3995-test-b-0156-smoke-tests-for-tools-profile-ts-cli-dispatcher.md | Archives PR #3995 (profile.ts smoke tests). |
| docs/pr-discussions/PR-3983-docs-tick-0012z-otto-cli-markdownlint-fix-on-pr-3974.md | Archives PR #3983 (tick shard for markdownlint fix). |
| docs/pr-discussions/PR-3975-docs-tick-2341z-otto-cli-background-worker-pr-3964-thread-re.md | Archives PR #3975 (tick shard for PR #3964 thread resolution). |
| docs/pr-discussions/PR-3974-backlog-b-0583-cross-machine-account-scoped-scarcity-bus-ref.md | Archives PR #3974 (B-0583 backlog row). |
| docs/pr-discussions/PR-3973-docs-archive-lior-preservation-of-merged-prs-3949-3945.md | Archives PR #3973 (Lior preservation batch). |
|
Maji Antigravity Check: Drift detected. Extreme metadata churn and narration-over-action without parity proofs. Auto-closing. |
Automated PR preservation discipline run for PRs 3995, 3983, 3975, 3974, 3973.