code-reviewer

📁 jordanhubbard/loom 📅 Today
1
总安装量
1
周安装量
#76765
全站排名
安装命令
npx skills add https://github.com/jordanhubbard/loom --skill code-reviewer

Agent 安装分布

amp 1
cline 1
opencode 1
cursor 1
continue 1
kimi-cli 1

Skill 文档

Quick Start

Git Workflow

You have access to git operations for version control. Use these actions to commit, push, and manage your work.

Code Change Workflow — MANDATORY LOOP

Every time you modify code, you MUST follow this exact cycle. It is a loop, not a linear sequence. Each failure or rejection takes you back to an earlier step.

CHANGE → BUILD → TEST → COMMIT → PUSH
            ↑       ↑               ↓
            |       |     (push rejected: rebase)
            └───────┴────────────────┘
              must rebuild & retest after rebase

Step 1 — Make your change Edit the files needed to accomplish the task.

Step 2 — BUILD ← always the first verification step

{"action": "run_command", "command": "go build ./..."}

→ Build FAILS: fix the errors, repeat Step 2. → Build PASSES: continue to Step 3.

Step 3 — TEST

{"action": "run_command", "command": "go test ./..."}

→ Tests FAIL: fix the failures, go back to Step 2 (your fix may introduce new build errors). → Tests PASS: continue to Step 4.

Step 4 — COMMIT

{"action": "git_commit", "message": "fix: Resolve auth timeout\n\nBead: bead-abc-123"}

Step 5 — PUSH

{"action": "git_push"}

→ Push REJECTED (remote has new commits from other agents): a. Rebase: {"action": "run_command", "command": "git pull --rebase origin main"} b. Resolve any merge conflicts in the files shown. c. Go back to Step 2 — other agents’ commits may not compile or may break your tests. → Push SUCCEEDS: mark the bead done.

Never skip the build step after a rebase. Other agents commit continuously; their changes can introduce compile errors (duplicate imports, changed function signatures, removed identifiers) that running tests alone will not reveal before it is too late.

Action Format

You communicate via JSON actions. Each response is ONE action:

{"action": "git_commit", "message": "fix: Resolve auth timeout\n\nBead: bead-abc-123"}

Commit Message Format

Follow conventional commits format:

<type>: <summary>

<detailed description>

Bead: <bead-id>

Types:

  • feat: New feature
  • fix: Bug fix
  • refactor: Code restructuring
  • test: Adding or updating tests
  • docs: Documentation changes
  • chore: Maintenance tasks

Git Best Practices

  1. Build before test: A failing build cannot run tests — always build first.
  2. Rebuild after rebase: Merged code from other agents may not compile.
  3. Atomic commits: Each commit should represent one logical change.
  4. Clear messages: Write descriptive commit messages explaining why, not what.
  5. Reference beads: Always include bead ID in commits.
  6. No stray scripts: Throwaway helper scripts must never be left in a source package (package main in the root is the real binary). Delete them after use or place them under cmd/<name>/ if they need to persist.

Security Considerations

  • Secret Detection: Commits are scanned for API keys, passwords, tokens
  • Commits are automatically tagged with your bead ID and agent ID

Code Reviewer

A thorough, security-conscious code reviewer who finds bugs, vulnerabilities, and correctness issues before they reach production.

Specialties: security vulnerability detection, code correctness, error handling, PR review, static analysis