FE-703: Add /planning-pr skill#99
Conversation
Encodes the project convention that paragraph-level rewrites of memory/PLAN.md and any change to memory/SPEC.md tracked-ID rows (Assumptions / Decisions / Requirements / Invariants) go on a separate planning PR off main, scoped to memory/* files only, while small in-place edits stay on the feature branch. Triggers on: about-to-edit memory/* beyond a one-line tick, where-to-commit questions, post-/ln-sync substantive edits, SPEC.md tracked-ID row changes. Body covers: decision rule, workflow (Linear -> branch off main -> memory/* only -> PR -> land first -> downstream restack), anti-patterns, common entry points, worked example, cross-refs to /cli-linear, /cli-graphite, /ln-sync. Process-only mitigation under FE-699; structural proposals from docs/praxis/proposals/merge-friction-proposals.md remain open separately. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
PR SummaryLow Risk Overview Defines the decision threshold, required workflow (Linear → Graphite branch off Reviewed by Cursor Bugbot for commit 305d868. Bugbot is set up for automated code reviews on this repo. Configure here. |
🤖 Augment PR SummarySummary: This PR adds a new Changes:
Technical Notes: The skill’s core intent is to serialize high-conflict planning edits by landing them on 🤖 Was this summary useful? React with 👍 or 👎 |

What
Adds a
/planning-pragent skill that captures when planning-doc edits warrant a dedicated PR offmainvs an inline commit on a feature branch.Linear: FE-703 (child of FE-699).
Why
Every paragraph-level rewrite of
memory/PLAN.md/memory/SPEC.mdcompetes with feature branches for the same lines on rebase. This skill encodes the project's process-only mitigation: serialize all paragraph-level rewrites and any SPEC.md tracked-ID additions through dedicated planning PRs.Doesn't preclude the structural fixes drafted under FE-699 (union merge driver, per-frontier-item file decomposition) — this just makes the convention discoverable.
Change
One new file:
.agents/skills/planning-pr/SKILL.md(73 lines, no code, no tests).Test plan
/ln-syncinvocation routes through this skill before committing