Skills

aid-review

Frontmatter

name — aid-review
description — Review/assess an existing artifact — code, a change/diff, a design, a PR, a ticket, a document, a UI, whatever the request names — against criteria, and return findings + recommendations NOW, in one pass. Single-shot and (except the findings ledger + optional approved publish) read-only: it never plans-and-halts. Grounded in the Knowledge Base (.aid/knowledge/) and the project source — every finding cites a KB doc or a file:line. The review is produced by the aid-reviewer agent in a clean context and independently verified before you see it; you approve before anything is published to an external target (PR/ticket/doc). Allocates a work-NNN folder for isolation; does not fix anything (findings hand off to /aid-fix).
allowed-tools — Read, Glob, Grep, Bash, Write, Edit, Agent
argument-hint — [target] — what to review (a file/dir, PR link, ticket id, work-NNN, ‘my changes’, or a described target)

Definition: canonical/skills/aid-review/SKILL.md

Flow

flowchart TB
  classDef aidNode color:#fff
  classDef aidEntry fill:#166534,stroke:#14532d,color:#fff
  classDef aidExit fill:#991b1b,stroke:#7f1d1d,color:#fff
  classDef aidDecision fill:#92400e,stroke:#78350f,color:#fff
  classDef aidLoopBack fill:#1e3a8a,stroke:#1e3a8a,color:#fff
  classDef aidStep fill:#1a2035,stroke:#d4a853,color:#f1f5f9
  n1(["INTAKE"])
  n2["REVIEW"]
  n3["VERIFY<br/>who reviews the reviewer"]
  n4{"PRESENT-FINDINGS"}
  n5["PUBLISH<br/>only on approval"]
  n6(["DONE"])
  n1 --> n2
  n2 --> n3
  n3 -.-> n2
  n3 --> n4
  n4 -->|"on approval"| n5
  n4 -->|"otherwise"| n6
  n5 --> n6
  class n1 aidEntry
  class n2 aidStep
  class n3 aidLoopBack
  class n4 aidDecision
  class n5 aidStep
  class n6 aidExit
  class n1 aidNode
  class n2 aidNode
  class n3 aidNode
  class n4 aidNode
  class n5 aidNode
  class n6 aidNode

Source fragments

Every node in the chart above, in chart order, with the exact canonical/ text it was derived from.

1 · INTAKE · entry

## State: INTAKE

Purpose: resolve the target + criteria, pick the path, allocate the work folder.

Source: canonical/skills/aid-review/SKILL.md#L38-L40 · full step: canonical/skills/aid-review/SKILL.md#L38-L118

2 · REVIEW · step

## State: REVIEW

Purpose: gather evidence and produce the grounded findings ledger.

Source: canonical/skills/aid-review/SKILL.md#L122-L124 · full step: canonical/skills/aid-review/SKILL.md#L122-L143

3 · VERIFY — who reviews the reviewer · loop-back

## State: VERIFY  (who reviews the reviewer)

Purpose: ensure the review is grounded, correct, and complete before the human sees it.

Source: canonical/skills/aid-review/SKILL.md#L147-L149 · full step: canonical/skills/aid-review/SKILL.md#L147-L167

4 · PRESENT-FINDINGS — always a hard stop — human final say · decision

## State: PRESENT-FINDINGS  (always a hard stop -- human final say)

Set STATE `lifecycle: Paused-Awaiting-Input`. Present:

Source: canonical/skills/aid-review/SKILL.md#L171-L173 · full step: canonical/skills/aid-review/SKILL.md#L171-L184

5 · PUBLISH — only on approval · step

## State: PUBLISH  (only on approval)

Deliver by the method appropriate to the target, chosen by judgment (not a hardcoded
enum): a PR comment via `gh`; a ticket comment via `/aid-update-ticket comment
[<connector>:]<ticket-id> <text>` (still user-authorized -- the approval just given at
PRESENT-FINDINGS is what authorizes this call, and the skill previews the exact payload again
at its own CONFIRM before posting); a findings report in the work folder (+ optional
inline-comment suggestions) for code; inline notes for a document. **Graceful fallback:** no PR /
no catalogued connector / unknown target -> present the exact text for the human to paste.
Publishing is optional and never blocks DONE.

Source: canonical/skills/aid-review/SKILL.md#L188-L197 · full step: canonical/skills/aid-review/SKILL.md#L188-L199

6 · DONE · exit · UNSPECIFIED

## State: DONE

Set STATE `lifecycle: Completed`, `updated` now, append a `## Lifecycle History` row.
Leave the findings ledger on disk (`.aid/.temp/review-pending/<work>-review.md`) so a
follow-up `/aid-fix` can consume it. Keep the work folder as the audit record.

Source: canonical/skills/aid-review/SKILL.md#L203-L207 · full step: canonical/skills/aid-review/SKILL.md#L203-L207

Report an issue with this page →