architecture-principle…
Architecture principles for manifest-dev: how its maintainer wants skills, wrappers, references, state and the workflow…
Read-only inspection of a single GitHub PR lifecycle — checks CI, review feedback (threads, review bodies, top-level comments), description sync, and mergeability, and returns PASS or FAIL with per-gate findings. The read-only lifecycle check that babysit-pr calls. Use when
Claude Code
$ npx -y skills add doodledood/manifest-dev --skill check-pr --agent claude-code
Codex
$ npx skills add doodledood/manifest-dev --skill check-pr -a codex -y
Cursor
$ npx skills add doodledood/manifest-dev --skill check-pr -a cursor -y
Gemini CLI
$ npx skills add doodledood/manifest-dev --skill check-pr -a gemini-cli -y
GitHub Copilot
$ npx skills add doodledood/manifest-dev --skill check-pr -a github-copilot -y
OpenCode
$ npx skills add doodledood/manifest-dev --skill check-pr -a opencode -y
Windsurf
$ npx skills add doodledood/manifest-dev --skill check-pr -a windsurf -y
name: check-pr description: 'Read-only inspection of a single GitHub PR lifecycle — checks CI, review feedback (threads, review bodies, top-level comments), description sync, and mergeability, and returns PASS or FAIL with per-gate findings. The read-only lifecycle check that babysit-pr calls. Use when verifying a PR is ready to merge, polling lifecycle progress, or checking mergeability.' user-invocable: true
This skill turns the activating session into a read-only PR lifecycle inspector. While it is active, you inspect exactly one GitHub PR, decide whether it is mergeable, and report findings — what's blocking the PR from being mergeable. Inspect and report only: do not mutate PR or repository state, perform a merge, or direct a caller to merge. The caller (typically a workflow orchestrator like `/do`) decides what to do with the findings.
Return PASS when the PR is mergeable; return FAIL with per-gate findings when it isn't. Report observable PR state and action targets only.
The PR is ready when each gate holds:
1. **PR exists** — exactly one open PR on the named branch. A merged PR is a terminal state of its own: report who merged it and when, and that nothing remains to tend, as the merged-PR example below shows. 2. **CI green** — required checks pass on the current head commit. 3. **Review feedback addressed** — inbound review feedback has been inspected through its latest reply. That inbox is: open review threads; non-empty review bodies of each author's latest submitted `CHANGES_REQUESTED` or `COMMENTED` review; inbound human top-level PR comments; and comments stamped `<!-- manifest-dev:review-pr -->` (the marker is an inbox signal, not a skip). Skip empty, pending, dismissed, and superseded reviews; skip `APPROVED` bodies; skip unthreaded GitHub-App IssueComments (bot nits that arrive as threads still count). A thread is addressed when it is resolved, has a reply clearly addressing its concern, or is addressed by a follow-up commit on the current head. A review body or top-level comment is addressed when someone other than its author — the PR author, or the identity this inspection runs as, the one tending the PR — has later replied on the Conversation with a response clearly tied to that item, the review is dismissed, or a newer submitted review from the same reviewer supersedes it — a follow-up commit or unrelated later update does not address a body or comment. Identify the addressing signal for each item without judging the substantive validity of the answer. Do not pass from summary counts or stale state; new pushback or replies keep this gate failing. 4. **Description in sync** — PR description reflects the current diff's intent. 5. **Mergeable** — GitHub's current merge state and applicable repository requirements permit the PR to merge. Interpret the returned state under the API's documented semantics; an unknown state remains a wait finding, never a guessed PASS.
User-defined gates from steering evaluate additively — the PR is ready only when both baseline and user gates hold.
Always emit exactly one of two shapes.
**PASS** — a one-line confirmation plus a do-not-merge reminder:
## check-pr: PASS PR #N is mergeable. <one-line summary> Do not merge unless the operator explicitly authorized it — PASS is a mergeability report, not authorization to press the merge button. Merging is a separate operator decision.
Include the reminder line on every PASS.
**FAIL** — a per-gate breakdown with a **finding** per failing gate. A finding's `Suggested:` field carries either a workflow-neutral GitHub-action directive (action intent, drawn from the fixed vocabulary below — the caller maps it to available capabilities) or a free-form prose description (solvable-but-novel observation — the caller reads with judgment).
The workflow-neutral directive vocabulary (each names a concrete GitHub-state action, not a workflow step):
`reply` and `reply-and-resolve` name a review-thread GraphQL ID. A review body or top-level comment has none: FAIL as a prose finding with action-ready context. When any outstanding item has no thread id, `Suggested:` is prose that names every outstanding item; thread ids stay in `Context:`. Do not invent a thread id. `re-request-review` answers review *state*, not unanswered content.
For situations that don't fit any of these (an unexpected CI fingerprint that looks suspicious, a steering ambiguity, a terminal state like PR-closed-externa
Hand your agent a task. Get back work you can trust. manifest-dev is a set of free, open-source skills for coding agents.
Architecture principles for manifest-dev: how its maintainer wants skills, wrappers, references, state and the workflow…
Ship a change to a file setup-project copies or emits into projects (ADR_FORMAT.md, NORTH_STAR_FORMAT.md, the verification…
Design principles for manifest-dev: how its maintainer wants the visual and interaction design guidance the plugin ships, and the…
Harden a manifest-dev task guidance file for one-shot quality — either /define''s quality-gate/Default set or figure-out''s probe…
Analyze Claude Code sessions to learn what went right/wrong and suggest high-confidence improvements to skills. Use when asked to…
The red-flag lists mirrored between figure-out''s and define''s task files, so a change to one side lands on the other. Use when…
You MUST use this before any creative work - creating features, building components, adding functionality, modifying behavior, or…
Ask which skill or flow fits your situation. A router over the skills in this repo.