Add file-by-file-review command, update address-review, fix README

justin808 · claude · justin808 · commit 98ae52c137cd · 2026-03-07T16:03:41.000-10:00
- Add /file-by-file-review command and supporting bin scripts for
  reviewing massive PRs file-by-file with parallel subagents
  (credit: Romex91/claude-code-file-by-file-review)
- Update address-review.md with improved version that adds issue
  comment support, comment filtering, error handling, and auto-reply
- Fix README header and clone URL to match actual repo name
  (claude-code-commands-skills-agents, not claude-code-boosters)
- Add "Codex CLI" to repo description

Co-Authored-By: Claude Opus 4.6 &lt;noreply@anthropic.com&gt;
diff --git a/README.md b/README.md
@@ -1,15 +1,15 @@
-# claude-code-boosters
+# claude-code-commands-skills-agents
 
-ShakaCode Team Shared Claude Code Commands, Agents, Skills, and Tips.
+ShakaCode Team Shared Claude Code and Codex CLI Commands, Agents, Skills, and Tips.
 
 ## Quick Install
 
 Copy what you need to your `~/.claude/` directory:
 
 ```bash
 # Clone the repo
-git clone https://github.com/shakacode/claude-code-boosters.git
-cd claude-code-boosters
+git clone https://github.com/shakacode/claude-code-commands-skills-agents.git
+cd claude-code-commands-skills-agents
 
 # Install all commands
 mkdir -p ~/.claude/commands
@@ -36,6 +36,7 @@ For project-level sharing, copy to your project's `.claude/commands/` or `.claud
 | [`/merge-commit-msg`](commands/merge-commit-msg.md) | Generate a structured merge commit message from PR changes |
 | [`/optimize`](commands/optimize.md) | Analyze code for performance issues with structured recommendations |
 | [`/security-review`](commands/security-review.md) | Review code for security vulnerabilities (OWASP Top 10 checklist) |
+| [`/file-by-file-review`](commands/file-by-file-review.md) | Review massive PRs file-by-file with parallel subagents (credit: [Romex91](https://github.com/Romex91/claude-code-file-by-file-review)) |
 
 ## Agents
 
@@ -69,6 +70,13 @@ Starter templates for new projects:
 | Script | Description |
 |--------|-------------|
 | [`bin/chrome-mcp`](bin/chrome-mcp) | Launch Chrome with a separate profile for MCP browser debugging |
+| `bin/set-review-instructions` | Initialize file-by-file review with instructions and changed file list |
+| `bin/print-git-diff` | Print diff, before/after, and review instructions for a single file |
+| `bin/mark-git-diff-as-good` | Mark a reviewed file as GOOD with a reason |
+| `bin/mark-git-diff-as-bad` | Mark a reviewed file as BAD with a reason |
+| `bin/check-missing-reviews` | Check which files still need review |
+| `bin/compose-summary` | Compose final GOOD/BAD summary report |
+| `bin/get-number-of-changed-files` | Get count of changed files in the review |
 
 ## GitHub Actions
 
diff --git a/bin/check-missing-reviews b/bin/check-missing-reviews
@@ -0,0 +1,34 @@
+#!/usr/bin/env bash
+set -euo pipefail
+
+REVIEW_DIR="/tmp/file-by-file-review"
+
+if [ ! -f "$REVIEW_DIR/changed_files.txt" ]; then
+  echo "ERROR: Run set-review-instructions first."
+  exit 1
+fi
+
+TOTAL=$(wc -l < "$REVIEW_DIR/changed_files.txt")
+MISSING=0
+MISSING_LIST=""
+
+for i in $(seq 1 "$TOTAL"); do
+  FILEPATH=$(sed -n "${i}p" "$REVIEW_DIR/changed_files.txt")
+  SANITIZED=$(echo "$FILEPATH" | sed 's|/|__|g')
+
+  if [ ! -f "$REVIEW_DIR/report/${SANITIZED}_GOOD.txt" ] && [ ! -f "$REVIEW_DIR/report/${SANITIZED}_BAD.txt" ]; then
+    MISSING=$((MISSING + 1))
+    MISSING_LIST="${MISSING_LIST}  ${i}. ${FILEPATH}\n"
+  fi
+done
+
+echo "Reviewed: $((TOTAL - MISSING)) / $TOTAL"
+
+if [ "$MISSING" -eq 0 ]; then
+  echo "All files have been reviewed."
+  exit 0
+else
+  echo "Missing reviews ($MISSING):"
+  printf "$MISSING_LIST"
+  exit 1
+fi
diff --git a/bin/compose-summary b/bin/compose-summary
@@ -0,0 +1,47 @@
+#!/usr/bin/env bash
+set -euo pipefail
+
+REVIEW_DIR="/tmp/file-by-file-review"
+REPORT_DIR="$REVIEW_DIR/report"
+SUMMARY="$REVIEW_DIR/summary.txt"
+
+if [ ! -d "$REPORT_DIR" ]; then
+  echo "ERROR: No report directory found. Run the review first."
+  exit 1
+fi
+
+GOOD_COUNT=0
+BAD_COUNT=0
+GOOD_FILES=""
+BAD_FILES=""
+
+for f in "$REPORT_DIR"/*_GOOD.txt; do
+  [ -f "$f" ] || continue
+  GOOD_COUNT=$((GOOD_COUNT + 1))
+  GOOD_FILES+="$(cat "$f")"$'\n\n'
+done
+
+for f in "$REPORT_DIR"/*_BAD.txt; do
+  [ -f "$f" ] || continue
+  BAD_COUNT=$((BAD_COUNT + 1))
+  BAD_FILES+="$(cat "$f")"$'\n\n'
+done
+
+TOTAL=$((GOOD_COUNT + BAD_COUNT))
+
+cat > "$SUMMARY" <<EOF
+# Review Summary
+
+Total files reviewed: $TOTAL
+GOOD: $GOOD_COUNT
+BAD: $BAD_COUNT
+
+## BAD files
+
+$BAD_FILES
+## GOOD files
+
+$GOOD_FILES
+EOF
+
+echo "Summary written to $SUMMARY ($TOTAL files: $GOOD_COUNT good, $BAD_COUNT bad)"
diff --git a/bin/get-number-of-changed-files b/bin/get-number-of-changed-files
@@ -0,0 +1,11 @@
+#!/usr/bin/env bash
+set -euo pipefail
+
+REVIEW_DIR="/tmp/file-by-file-review"
+
+if [ ! -f "$REVIEW_DIR/changed_files.txt" ]; then
+  echo "ERROR: Run set-review-instructions first."
+  exit 1
+fi
+
+wc -l < "$REVIEW_DIR/changed_files.txt"
diff --git a/bin/mark-git-diff-as-bad b/bin/mark-git-diff-as-bad
@@ -0,0 +1,33 @@
+#!/usr/bin/env bash
+set -euo pipefail
+
+if [ $# -lt 2 ]; then
+  echo "Usage: mark-git-diff-as-bad <file_index> \"<reason>\""
+  exit 1
+fi
+
+REVIEW_DIR="/tmp/file-by-file-review"
+INDEX="$1"
+REASON="$2"
+
+if [ ! -f "$REVIEW_DIR/changed_files.txt" ] || [ ! -f "$REVIEW_DIR/base_ref.txt" ]; then
+  echo "ERROR: Run set-review-instructions first."
+  exit 1
+fi
+
+FILEPATH=$(sed -n "${INDEX}p" "$REVIEW_DIR/changed_files.txt")
+BASE=$(cat "$REVIEW_DIR/base_ref.txt")
+SANITIZED=$(echo "$FILEPATH" | sed 's|/|__|g')
+
+mkdir -p "$REVIEW_DIR/report"
+
+# Remove any existing GOOD verdict for this file
+rm -f "$REVIEW_DIR/report/${SANITIZED}_GOOD.txt"
+
+cat > "$REVIEW_DIR/report/${SANITIZED}_BAD.txt" <<EOF
+$FILEPATH
+git diff $BASE..HEAD -- $FILEPATH
+BAD: $REASON
+EOF
+
+echo "Marked file $INDEX ($FILEPATH) as BAD: $REASON"
diff --git a/bin/mark-git-diff-as-good b/bin/mark-git-diff-as-good
@@ -0,0 +1,33 @@
+#!/usr/bin/env bash
+set -euo pipefail
+
+if [ $# -lt 2 ]; then
+  echo "Usage: mark-git-diff-as-good <file_index> \"<reason>\""
+  exit 1
+fi
+
+REVIEW_DIR="/tmp/file-by-file-review"
+INDEX="$1"
+REASON="$2"
+
+if [ ! -f "$REVIEW_DIR/changed_files.txt" ] || [ ! -f "$REVIEW_DIR/base_ref.txt" ]; then
+  echo "ERROR: Run set-review-instructions first."
+  exit 1
+fi
+
+FILEPATH=$(sed -n "${INDEX}p" "$REVIEW_DIR/changed_files.txt")
+BASE=$(cat "$REVIEW_DIR/base_ref.txt")
+SANITIZED=$(echo "$FILEPATH" | sed 's|/|__|g')
+
+mkdir -p "$REVIEW_DIR/report"
+
+# Remove any existing BAD verdict for this file
+rm -f "$REVIEW_DIR/report/${SANITIZED}_BAD.txt"
+
+cat > "$REVIEW_DIR/report/${SANITIZED}_GOOD.txt" <<EOF
+$FILEPATH
+git diff $BASE..HEAD -- $FILEPATH
+GOOD: $REASON
+EOF
+
+echo "Marked file $INDEX ($FILEPATH) as GOOD: $REASON"
diff --git a/bin/print-git-diff b/bin/print-git-diff
@@ -0,0 +1,65 @@
+#!/usr/bin/env bash
+set -euo pipefail
+
+if [ $# -lt 1 ]; then
+  echo "Usage: print-git-diff <file_index>"
+  exit 1
+fi
+
+REVIEW_DIR="/tmp/file-by-file-review"
+INDEX="$1"
+
+if [ ! -f "$REVIEW_DIR/changed_files.txt" ] || [ ! -f "$REVIEW_DIR/base_ref.txt" ]; then
+  echo "ERROR: Run set-review-instructions first."
+  exit 1
+fi
+
+TOTAL=$(wc -l < "$REVIEW_DIR/changed_files.txt")
+if [ "$INDEX" -lt 1 ] || [ "$INDEX" -gt "$TOTAL" ]; then
+  echo "ERROR: Index $INDEX out of range (1-$TOTAL)"
+  exit 1
+fi
+
+FILEPATH=$(sed -n "${INDEX}p" "$REVIEW_DIR/changed_files.txt")
+BASE=$(cat "$REVIEW_DIR/base_ref.txt")
+
+# Sanitize filename for report lookup
+SANITIZED=$(echo "$FILEPATH" | sed 's|/|__|g')
+
+# Check review status
+REVIEW_STATUS="not yet reviewed"
+if [ -f "$REVIEW_DIR/report/${SANITIZED}_GOOD.txt" ]; then
+  REVIEW_STATUS="GOOD"
+elif [ -f "$REVIEW_DIR/report/${SANITIZED}_BAD.txt" ]; then
+  REVIEW_STATUS="BAD"
+fi
+
+echo "========================================"
+echo "FILE: $FILEPATH"
+echo "INDEX: $INDEX / $TOTAL"
+echo "REVIEW STATUS: $REVIEW_STATUS"
+echo "========================================"
+
+echo ""
+echo "======== GIT DIFF ========"
+git diff "$BASE"..HEAD -- "$FILEPATH"
+
+echo ""
+echo "======== FILE BEFORE ========"
+if git show "$BASE":"$FILEPATH" &>/dev/null; then
+  git show "$BASE":"$FILEPATH"
+else
+  echo "(NEW FILE — did not exist at base)"
+fi
+
+echo ""
+echo "======== FILE AFTER ========"
+if git show HEAD:"$FILEPATH" &>/dev/null; then
+  git show HEAD:"$FILEPATH"
+else
+  echo "(DELETED — does not exist at HEAD)"
+fi
+
+echo ""
+echo "======== REVIEW INSTRUCTIONS ========"
+cat "$REVIEW_DIR/review_instructions.txt"
diff --git a/bin/set-review-instructions b/bin/set-review-instructions
@@ -0,0 +1,40 @@
+#!/usr/bin/env bash
+set -euo pipefail
+
+if [ $# -lt 1 ]; then
+  echo "Usage: set-review-instructions \"<instructions>\""
+  exit 1
+fi
+
+REVIEW_DIR="/tmp/file-by-file-review"
+mkdir -p "$REVIEW_DIR"
+
+# Save review instructions
+echo "$1" > "$REVIEW_DIR/review_instructions.txt"
+
+# Auto-detect base branch and fetch latest
+if git rev-parse --verify origin/main &>/dev/null; then
+  BASE_BRANCH="origin/main"
+  BASE_REF="main"
+elif git rev-parse --verify origin/master &>/dev/null; then
+  BASE_BRANCH="origin/master"
+  BASE_REF="master"
+else
+  echo "ERROR: Could not find origin/main or origin/master"
+  exit 1
+fi
+
+echo "Fetching latest $BASE_REF from origin..."
+git fetch origin "$BASE_REF" --quiet
+
+MERGE_BASE=$(git merge-base "$BASE_BRANCH" HEAD)
+echo "$MERGE_BASE" > "$REVIEW_DIR/base_ref.txt"
+
+# Generate changed files list
+git diff --name-only "$MERGE_BASE"..HEAD > "$REVIEW_DIR/changed_files.txt"
+
+FILE_COUNT=$(wc -l < "$REVIEW_DIR/changed_files.txt")
+echo "Review initialized."
+echo "  Base: $MERGE_BASE (from $BASE_BRANCH)"
+echo "  Changed files: $FILE_COUNT"
+echo "  Instructions: $1"
diff --git a/commands/address-review.md b/commands/address-review.md
diff --git a/commands/file-by-file-review.md b/commands/file-by-file-review.md
