docs(ux): enforce strict scoping and prevent opportunistic refactors

This commit is contained in:
Keith Guerin
2026-03-31 13:05:17 -07:00
parent 7b842aa844
commit 82c675d58e
2 changed files with 5 additions and 0 deletions
@@ -65,6 +65,10 @@ underlying "rigor."
pushing a PR.
9. **Copyright Headers**: NEVER modify or update the copyright header comments
(e.g., year) in existing files.
10. **Strict Scoping**: Keep work tightly scoped to the user's explicit request
to ensure PRs remain tight and clean. Do not make sweeping, loosely tied
changes or opportunistic refactors. If a scope change or expansion is
recommended, you MUST ask the user for input and receive approval FIRST.
## Mandatory Workflow Triggers
@@ -105,6 +105,7 @@ If operating in a sibling worktree (e.g., `feature-xyz/`):
- **Diff Verification**: After reverting, run `git diff origin/main...HEAD` on the specific reverted files to ensure their diff is completely empty.
#### 7. Diff Minimization & Refactor Isolation (Jacob's Protocol)
- **Strict Scoping**: Keep work tightly scoped to the user's explicit request to ensure PRs remain tight and clean. Do not make sweeping, loosely tied changes or opportunistic refactors. If a scope change/expansion is recommended, ask the user for input and receive approval FIRST.
- **Mandatory Two-Step Process**: Never move code between files AND make logic changes in the same commit.
- **Refactor Commit**: If your task requires moving code or reorganization, create a "zero-modification" commit first. Verify that `npm run typecheck` passes but no functional logic has changed.
- **Logic Commit**: Apply logic changes or new features in a separate, follow-up commit.