uv-sbom: Skill for Claude Code

/code-review - Code Review Skill

Spawns a dedicated Reviewer Agent to evaluate the current git diff HEAD against architectural, design, and quality criteria specific to uv-sbom.

This skill does NOT re-run CI checks (cargo fmt, clippy). Those are enforced by the /commit skill and the CI pipeline.

Invocation

Standalone: Invoke at any time to review uncommitted changes. Integrated: Called automatically by /implement Step 4.5 before /commit.

Workflow

Step 1: Collect diff
Step 2: Spawn Reviewer Agent (foreground)
Step 3: Parse result (PASS / FAIL)
  ├── PASS → run Step 3.5, then report success, return control to caller
  └── FAIL → apply fixes, increment counter, return to Step 1
             (max 3 iterations; halt and report if still failing)
Step 3.5: Follow-up Issue Gate (runs once, at the terminal state only —
          after PASS, or after the final post-3-iteration report)

Steps

Step 1: Collect Diff and File Content

git diff HEAD --name-only   # list changed files
git diff HEAD               # full diff for the reviewer

For each file listed by --name-only, also read its full current content. This is required for file-size checks and documentation coverage checks.

In addition, run the following scan and include its output as supplemental context for criterion 11:

grep -rn 'eprintln!\|println!' src/ \
  | grep -v 'msgs\.\|Messages::' \
  | grep -v '#\[cfg(test)\]' \
  | grep -v '\.rs:.*//.*i18n-ok'

• Lines in changed files: flag as 🔴 MUST FIX under criterion 11.
• Lines in unchanged files: flag as 🟡 SHOULD FIX (pre-existing violation).

If any changed file is under .claude/, also include the full content of .claude/conventions/README.md and every file it indexes, as supplemental context for criterion 3.

If git diff HEAD is empty, report "Nothing to review — no uncommitted changes." and exit.

Step 2: Spawn Reviewer Agent

Use the Agent tool with subagent_type: "general-purpose", run foreground.