docs(archive): preserve PRs 3235, 3234, 3233, 3232, 3230#3242
Merged
Conversation
There was a problem hiding this comment.
Pull request overview
Adds archived PR discussion records under docs/pr-discussions/ for PRs #3235, #3234, #3233, #3232, and #3230 as part of the automated PR-preservation discipline.
Changes:
- Add five new
docs/pr-discussions/PR-<N>-<slug>.mdarchive files capturing PR metadata, descriptions, and review-thread snapshots. - Preserve cross-references and test-plan notes from the original PR descriptions/reviews in the archive format.
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-3235-feat-b-0488-ksk-kinetic-safeguard-kernel-persona-map.md | Adds the archived discussion record for PR #3235 (KSK persona-map work). |
| docs/pr-discussions/PR-3234-chore-b-0518-add-sharpenings-6-7-from-aaron-s-failure-mode-c.md | Adds the archived discussion record for PR #3234 (B-0518 sharpenings update). |
| docs/pr-discussions/PR-3233-shard-tick-2034z-promote-primary-defenses-to-claude-rules.md | Adds the archived discussion record for PR #3233 (tick shard record). |
| docs/pr-discussions/PR-3232-chore-rule-extend-zeta-expected-branch-with-primary-defenses.md | Adds the archived discussion record for PR #3232 (rule update promotion). |
| docs/pr-discussions/PR-3230-feat-workflow-add-stale-worktrees-self-test-job-to-factory-h.md | Adds the archived discussion record for PR #3230 (workflow job addition). |
Comments suppressed due to low confidence (2)
docs/pr-discussions/PR-3235-feat-b-0488-ksk-kinetic-safeguard-kernel-persona-map.md:45
- P1: This link to
docs/research/...is currently written asdocs/research/...from withindocs/pr-discussions/, so it will resolve to a non-existentdocs/pr-discussions/docs/research/.... Switch to../research/...(or an absolute repo URL) so the archived record stays navigable.
## Acceptance criteria
- [x] Template from [B-0485](docs/research/2026-05-14-persona-mapping-framework-b0485.md) applied
- [x] Primary personas fully documented (3, exceeds the "grey-hat / ethical researcher" minimum — folded that framing into `ksk-security-engineer`)
docs/pr-discussions/PR-3232-chore-rule-extend-zeta-expected-branch-with-primary-defenses.md:27
- P1: These links point to
.claude/rules/...but, fromdocs/pr-discussions/, they resolve to a non-existent path. Update them to../../.claude/rules/...(or repo-root URLs) so readers can follow the referenced taxonomy/inventory docs.
Per [claude-code-loading-taxonomy.md](.claude/rules/claude-code-loading-taxonomy.md): "I keep forgetting to do X" is the goldfish-ontology failure mode that needs direct-load surface (rule files), not router-loaded skills or grep-discoverable backlog rows. The defenses fire on every commit/PR call — every cold-boot Otto needs them in context from session start, not via grep.
Per [skill-router-as-substrate-inventory.md](.claude/rules/skill-router-as-substrate-inventory.md): extend the existing rule rather than mint a new one — same failure mode (wrong-branch commits under multi-Otto contention).
…scussion docs Links in archived PR discussions were written relative to the repo root but the files live under docs/pr-discussions/. Fix: - docs/backlog/ → ../backlog/ (PR-3235 lines 21, 44) - docs/research/ → ../research/ (PR-3235 line 44) - .claude/rules/ → ../../.claude/rules/ (PR-3232 lines 21, 25, 27) Resolves Copilot threads on PR #3242. Co-Authored-By: Claude <noreply@anthropic.com>
Member
Author
|
Fixed both broken-link threads in this push (commit a10004f):
All relative links now resolve correctly from |
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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
Automated PR preservation discipline run.