| name | review-op |
| description | Review a CV-CUDA operator end-to-end (support / test / bench / docs coverage). Use when the user asks to review an operator, audit its input-type/layout/dtype support, test coverage, benchmark coverage, or docs/API, or to find & fix per-operator coverage gaps. Produces a deterministic findings-first report. |
Review Op
Thin entry point. The checklist substance and the checks are shared repo artifacts:
the deterministic checker tools/review_op.py and the spec .agents/guidance/REVIEW_OP_GUIDELINES.md.
Keep this skill thin; do not duplicate the checklist here.
Workflow
- Run
python3 tools/review_op.py <Operator> (scope with
--domain support|test|bench|docs; --format json for machine output; --run for the
GPU-run-dependent bench checks). The checker is read-only and deterministic.
- Interpret each finding by its cited item id in
.agents/guidance/REVIEW_OP_GUIDELINES.md. Resolve every
MANUAL item by inspecting the location it points to.
- If the user asked to fix (
--fix), apply the corrective action named per GAP in
.agents/guidance/REVIEW_OP_GUIDELINES.md, then re-run the checker to prove the item is PASS.
Follow the inviolable --fix rules in .agents/guidance/REVIEW_OP_GUIDELINES.md ("### --fix").
- Return findings-first:
GAPs, then unresolved MANUALs, then RECOMMENDATIONs, then the
per-domain + overall verdict. Completion = a re-run shows zero GAP and zero unresolved MANUAL.
Prompt Handling
Requests like "review the Resize operator", "/review-op CenterCrop", "audit Flip's bench
coverage", or "does Remap cover every declared layout?" trigger this skill. Use --domain to
scope to a single domain when the request is domain-specific.
