SuperAPI AI Directory
Partner with us中文AI newsFavorites
Skillspeer-qa-review

peer-qa-review

General & other

View repository

Updated 10-16 01:35

About this skill

Peer QA Review (Round 1)

A teammate marked work ready for QA. Verify before it reaches customer acceptance, QA2, or close: re-run verification; check formal correctness, inventory, docs; post a structured comment; transition. Not rubber-stamping; detail in references/.

When it applies

Triggers: see description. Skip if: still In Progress (transition first); already QA2 (different scope); already closed (post-mortem only); or you are the implementer (no self-review; per-PR, §E).

A ticket-system skill is required (Jira: jira-communication) for Stage 0 discovery. Consult maintenance skills for overrides.

Lifecycle

  • -1 Claim — your FIRST tool call, before Stage 0: assign to yourself; someone else's, stop; never self-review (lifecycle.md §-1).
  • 0 Discover: ${CLAUDE_SKILL_DIR}/scripts/qa-gather.sh <KEY> (--json to parse).
  • 1 Formal (description, linkage, console-output, worklog); 2 Functional, Inventory, Guardrails (re-run; update inventory; check adjacent components, shared-layer downstream, default path); 3 Docs, Rollback, Communication.
  • 4 Verdict (routing below); 5 Comment, Transition, Worklog: internal QA comment; on QA2 also a customer handover.

Severity icons

Reuse the Atlassian set. (/) passed; (x) MUST/blocking (bounce); (!) SHOULD (document, follow-up if structural); (i) hint; (?) open question (block); (off) n/a (never (-) or *n/a*).

Output

One internal QA comment: header h3. IT Internal QA, h4 per pillar, severity icons, a verdict line.

On a QA2 verdict, post a second, separate comment: the customer handover. The internal QA comment is addressed to IT and never states the acceptance check, leaving the approver lost. The handover is plain (no jargon or icons): what was delivered, the one acceptance check, the next step.

Verdict routing

  • Pass, resolve: all (x) clear, IT-internal; QA to the project's terminal reviewed status — usually Resolved — then clear your assignment. Take the status from the transition list, not from this line: closing is frequently someone else's step, so a reviewer who walks a ticket further than the pass requires has changed something that was not theirs to change.
  • Pass, QA2: all (x) clear, customer-affecting; post the QA comment and a customer handover; QA to QA2, assign the product owner.
  • Bounce: any (x); QA to In Progress, comment the blocker, reassign.
  • Won't-do: blocking external prerequisite; document, file a follow-up, resolve Won't-do with a reopen condition.

When uncertain, default to QA2.

Anti-patterns

The cardinal one: no re-execution by the reviewer — copy-pasting the implementer's output is not QA. Others (giant final comments, Jira Markdown leakage, end-of-run inventory, tags without a green pipeline) in references/anti-patterns.md.

References

references/lifecycle.md, references/checklist.md (checks by pillar); references/severity.md; references/comment-template.md (template, examples, customer handover); references/edge-cases.md (QA2 routing, bounce, won't-do, self-review); references/anti-patterns.md; references/batch-review.md — open it before starting a multi-ticket review, or any review held alongside other in-window work; the trigger is reviewer wall-clock, not a ticket count.