← ClaudeAtlas

work-with-prlisted

Full PR lifecycle for oh-my-matrix in a task-owned git worktree — implement with evidence-bound QA, open a reviewer-readable PR against `master`, then run an unbounded verify loop over three gates (CI / review-work / docs-build) until all pass, then merge by default and clean up the worktree. Decomposes work into the smallest atomic, independently-mergeable PRs and builds the independent ones concurrently. Use whenever implementation work needs to land as a PR. Triggers: 'create a PR', 'implement and PR', 'land this', 'work-with-pr', 'implement issue', 'split into atomic PRs', 'parallel PRs', even 'implement X' when the context implies PR delivery.
TeFuirnever/oh-my-matrix · ★ 3 · Code & Development · score 69
Install: claude install-skill TeFuirnever/oh-my-matrix
# Work With PR — OMM PR Lifecycle You execute a complete PR lifecycle: fresh task-owned worktree → implement with evidence-bound QA → reviewer-readable PR against `master` → unbounded verification loop → merge by default → worktree cleanup. A failing gate sends you back into the worktree to fix and re-QA. You keep cycling until every active gate passes at once. **The unit of delivery is the smallest PR that compiles, passes, and stands on its own** — not "one task, one PR." A single task routinely splits into several atomic PRs; apply the lifecycle below to each, and build the independent ones concurrently. <architecture> ``` Phase 0: Setup → Split into atomic PRs; branch + worktree per PR (parallel when independent) Phase 1: Implement → Run `pnpm verify` green locally; evidence-bound QA per success criterion; atomic commits Phase 2: PR → Push, open a reviewer-readable PR targeting `master` Phase 3: Verify → Unbounded loop; a failing gate routes back to Phase 1: ├─ Gate A: CI → gh pr checks (lint + lint:md + typecheck + commitlint + test) ├─ Gate B: review-work → code-reviewer subagent pass (no unresolved high/🔴 findings) └─ Gate C: docs-build → `pnpm docs:build` green (docs-first repo: never break the site) Phase 4: Merge → Merge by default; wait until actually merged; then worktree cleanup ``` </architecture> ## OMM invariants (non-negotiable) - **Docs-first.** A PR that breaks `pnpm docs:build` (Gate C) is not mergeable, full stop.