diffprism 0.27.0 → 0.30.0

package/dist/bin.js

@@ -6,7 +6,7 @@ import {
   startGlobalServer,
   startReview,
   startWatch
-} from "./chunk-BNCXGE4V.js";
+} from "./chunk-OJ723D6Z.js";
 
 // cli/src/index.ts
 import { Command } from "commander";
@@ -140,10 +140,46 @@ Use this pattern:
 
 You can also check for feedback without blocking by calling \`get_review_result\` without \`wait\` at natural breakpoints (between tasks, before committing, etc.).
 
+## Self-Review: Headless Analysis Tools
+
+DiffPrism provides two headless tools that return analysis data as JSON without opening a browser. Use these to check your own work before requesting human review.
+
+### Available Headless Tools
+
+- **\`mcp__diffprism__get_diff\`** \u2014 Returns a structured \`DiffSet\` (files, hunks, additions/deletions) for a given diff ref. Use this to inspect exactly what changed.
+- **\`mcp__diffprism__analyze_diff\`** \u2014 Returns a \`ReviewBriefing\` with summary, file triage, impact detection, complexity scores, test coverage gaps, and pattern flags (security issues, TODOs, console.logs left in, etc.).
+
+Both accept a \`diff_ref\` parameter: \`"staged"\`, \`"unstaged"\`, \`"working-copy"\`, or a git range like \`"HEAD~3..HEAD"\`.
+
+### Self-Review Loop
+
+When you've finished writing code and before requesting human review, use this pattern:
+
+1. **Analyze your changes:** Call \`mcp__diffprism__analyze_diff\` with \`diff_ref: "working-copy"\`
+2. **Check the briefing for issues:**
+   - \`patterns\` \u2014 Look for console.logs, TODOs, security flags, disabled tests
+   - \`testCoverage\` \u2014 Check if changed source files have corresponding test changes
+   - \`complexity\` \u2014 Review high-complexity scores
+   - \`impact.newDependencies\` \u2014 Verify any new deps are intentional
+   - \`impact.breakingChanges\` \u2014 Confirm breaking changes are expected
+3. **Fix any issues found** \u2014 Remove debug statements, add missing tests, address security flags
+4. **Re-analyze** \u2014 Run \`analyze_diff\` again to confirm the issues are resolved
+5. **Open for human review** \u2014 Once clean, use \`/review\` or \`open_review\` for final human sign-off
+
+This loop catches common issues (leftover console.logs, missing tests, security anti-patterns) before the human reviewer sees them, making reviews faster and more focused.
+
+### When to Use Headless Tools
+
+- **After completing a coding task** \u2014 Self-check before requesting review
+- **During implementation** \u2014 Periodically check for patterns and issues as you work
+- **Before committing** \u2014 Quick sanity check on what's about to be committed
+- **Do NOT use these as a replacement for human review** \u2014 They complement, not replace, \`/review\`
+
 ## Behavior Rules
 
 - **IMPORTANT: Do NOT open reviews automatically.** Only open a review when the user explicitly invokes \`/review\` or directly asks for a review.
 - Do NOT open reviews before commits, after code changes, or at any other time unless the user requests it.
+- Headless tools (\`get_diff\`, \`analyze_diff\`) can be used proactively during development without explicit user request \u2014 they don't open a browser or interrupt the user.
 - Power users can create \`diffprism.config.json\` manually to customize defaults (diff scope, reasoning).
 `;
 
@@ -203,7 +239,9 @@ function setupClaudeSettings(baseDir, force) {
   const toolNames = [
     "mcp__diffprism__open_review",
     "mcp__diffprism__update_review_context",
-    "mcp__diffprism__get_review_result"
+    "mcp__diffprism__get_review_result",
+    "mcp__diffprism__get_diff",
+    "mcp__diffprism__analyze_diff"
   ];
   const allPresent = toolNames.every((t) => allow.includes(t));
   if (allPresent && !force) {
@@ -433,7 +471,9 @@ function isGlobalSetupDone() {
   const toolNames = [
     "mcp__diffprism__open_review",
     "mcp__diffprism__update_review_context",
-    "mcp__diffprism__get_review_result"
+    "mcp__diffprism__get_review_result",
+    "mcp__diffprism__get_diff",
+    "mcp__diffprism__analyze_diff"
   ];
   return toolNames.every((t) => allow.includes(t));
 }
@@ -476,7 +516,9 @@ function teardownClaudePermissions(baseDir) {
   const toolNames = [
     "mcp__diffprism__open_review",
     "mcp__diffprism__update_review_context",
-    "mcp__diffprism__get_review_result"
+    "mcp__diffprism__get_review_result",
+    "mcp__diffprism__get_diff",
+    "mcp__diffprism__analyze_diff"
   ];
   const filtered = allow.filter((t) => !toolNames.includes(t));
   if (filtered.length === allow.length) {
@@ -836,7 +878,7 @@ async function serverStop() {
 
 // cli/src/index.ts
 var program = new Command();
-program.name("diffprism").description("Local-first code review tool for agent-generated changes").version(true ? "0.27.0" : "0.0.0-dev");
+program.name("diffprism").description("Local-first code review tool for agent-generated changes").version(true ? "0.30.0" : "0.0.0-dev");
 program.command("review [ref]").description("Open a browser-based diff review").option("--staged", "Review staged changes").option("--unstaged", "Review unstaged changes").option("-t, --title <title>", "Review title").option("--dev", "Use Vite dev server with HMR instead of static files").action(review);
 program.command("start [ref]").description("Set up DiffPrism and start watching for changes").option("--staged", "Watch staged changes").option("--unstaged", "Watch unstaged changes").option("-t, --title <title>", "Review title").option("--interval <ms>", "Poll interval in milliseconds (default: 1000)").option("--dev", "Use Vite dev server with HMR instead of static files").option("--global", "Install skill globally (~/.claude/skills/)").option("--force", "Overwrite existing configuration files").action(start);
 program.command("watch [ref]").description("Start a persistent diff watcher with live-updating browser UI").option("--staged", "Watch staged changes").option("--unstaged", "Watch unstaged changes").option("-t, --title <title>", "Review title").option("--interval <ms>", "Poll interval in milliseconds (default: 1000)").option("--dev", "Use Vite dev server with HMR instead of static files").action(watch);