@open-code-review/cli 1.1.1 → 1.3.0

package/README.md

@@ -57,16 +57,60 @@ ocr progress
 ocr progress --session 2026-01-26-feature-auth
 ```
 
-Shows: current phase, elapsed time, reviewer status, finding counts, completion percentage.
+Shows: current phase, elapsed time, reviewer status, finding counts, completion percentage, and **current round**.
+
+**Multi-round support**: The progress display shows which round is active and tracks completion across rounds. When a round completes, running `/ocr-review` again starts a new round (`round-2/`, `round-3/`, etc.).
 
 ### `ocr update`
 
-Update OCR skills and commands to the latest version.
+Update OCR skills and commands to the latest version after upgrading the package.
 
 ```bash
+# Update everything
 ocr update
+
+# Preview changes first
+ocr update --dry-run
+
+# Update specific components
+ocr update --commands    # Commands only
+ocr update --skills      # Skills and references only
+ocr update --inject      # AGENTS.md/CLAUDE.md only
+```
+
+**What it does:**
+
+1. Detects which AI tools you configured during `ocr init`
+2. Updates `.ocr/skills/` (SKILL.md, workflow, discourse rules)
+3. Updates tool-specific commands (`.windsurf/workflows/`, etc.)
+4. Updates managed blocks in `AGENTS.md` / `CLAUDE.md`
+
+**What is NOT modified:**
+
+- `.ocr/config.yaml` — Your team composition and context are preserved
+- `.ocr/skills/references/reviewers/` — All reviewers preserved (default and custom)
+- `.ocr/sessions/` — Review history remains untouched
+
+## Session Storage
+
+The CLI reads session state from `.ocr/sessions/{date}-{branch}/`:
+
+```
+.ocr/sessions/2026-01-26-feature-auth/
+├── state.json              # Workflow state (read by progress command)
+└── rounds/
+    ├── round-1/
+    │   ├── reviews/*.md    # Individual reviewer outputs
+    │   ├── discourse.md
+    │   └── final.md        # Completion indicator
+    └── round-2/            # Additional rounds if re-reviewed
 ```
 
+The CLI derives round information from the filesystem:
+- **Round count**: Enumerated from `rounds/round-*/` directories
+- **Round completion**: Determined by `final.md` presence
+- **Reviewer progress**: Listed from `rounds/round-{n}/reviews/*.md`
+
 ## Supported AI Tools
 
 | Tool | Config Directory |

package/dist/index.js

@@ -329,6 +329,16 @@ sessions/
 function detectInstalledTools(targetDir, tools) {
   return tools.filter((tool) => {
     const configPath = join(targetDir, tool.configDir);
+    if (tool.id === "github-copilot") {
+      const copilotInstructions = join(
+        targetDir,
+        ".github",
+        "copilot-instructions.md"
+      );
+      const copilotDir = join(targetDir, ".github", "copilot");
+      const copilotCommands = join(targetDir, ".github", "commands");
+      return existsSync(copilotInstructions) || existsSync(copilotDir) || existsSync(copilotCommands);
+    }
     return existsSync(configPath);
   });
 }
@@ -417,6 +427,50 @@ function printBanner() {
   console.log();
 }
 
+// packages/cli/src/lib/cli-config.ts
+import { existsSync as existsSync3, readFileSync as readFileSync3, writeFileSync as writeFileSync3 } from "node:fs";
+import { join as join3 } from "node:path";
+var CLI_CONFIG_FILE = "cli-config.json";
+function getCliConfigPath(targetDir) {
+  return join3(targetDir, ".ocr", CLI_CONFIG_FILE);
+}
+function loadCliConfig(targetDir) {
+  const configPath = getCliConfigPath(targetDir);
+  if (!existsSync3(configPath)) {
+    return null;
+  }
+  try {
+    const content = readFileSync3(configPath, "utf-8");
+    return JSON.parse(content);
+  } catch {
+    return null;
+  }
+}
+function saveCliConfig(targetDir, config) {
+  const configPath = getCliConfigPath(targetDir);
+  try {
+    const configWithMeta = {
+      ...config,
+      lastUpdated: (/* @__PURE__ */ new Date()).toISOString()
+    };
+    writeFileSync3(configPath, JSON.stringify(configWithMeta, null, 2) + "\n");
+    return true;
+  } catch {
+    return false;
+  }
+}
+function getConfiguredToolIds(targetDir) {
+  const config = loadCliConfig(targetDir);
+  return config?.configuredTools ?? [];
+}
+function setConfiguredToolIds(targetDir, toolIds) {
+  const existing = loadCliConfig(targetDir) ?? { configuredTools: [] };
+  return saveCliConfig(targetDir, {
+    ...existing,
+    configuredTools: toolIds
+  });
+}
+
 // packages/cli/src/commands/init.ts
 var initCommand = new Command("init").description("Set up OCR for AI coding environments").option("-t, --tools <tools>", 'Comma-separated tool IDs or "all"').option("--no-inject", "Skip injecting instructions into AGENTS.md/CLAUDE.md").action(async (options) => {
   printBanner();
@@ -481,6 +535,8 @@ var initCommand = new Command("init").description("Set up OCR for AI coding envi
     for (const result of successful) {
       console.log(`  ${chalk2.green("\u2713")} ${result.tool.name}`);
     }
+    const successfulToolIds = successful.map((r) => r.tool.id);
+    setConfiguredToolIds(targetDir, successfulToolIds);
   }
   if (failed.length > 0) {
     console.log();
@@ -528,21 +584,21 @@ var initCommand = new Command("init").description("Set up OCR for AI coding envi
 import { Command as Command2 } from "commander";
 import chalk4 from "chalk";
 import { watch } from "chokidar";
-import { existsSync as existsSync4, readdirSync as readdirSync2, readFileSync as readFileSync3, statSync } from "node:fs";
-import { join as join4, basename } from "node:path";
+import { existsSync as existsSync5, readdirSync as readdirSync2, readFileSync as readFileSync4, statSync } from "node:fs";
+import { join as join5, basename } from "node:path";
 import logUpdate from "log-update";
 
 // packages/cli/src/lib/guards.ts
-import { existsSync as existsSync3, mkdirSync as mkdirSync2 } from "node:fs";
-import { join as join3 } from "node:path";
+import { existsSync as existsSync4, mkdirSync as mkdirSync2 } from "node:fs";
+import { join as join4 } from "node:path";
 import chalk3 from "chalk";
 function checkOcrSetup(targetDir) {
-  const ocrDir = join3(targetDir, ".ocr");
-  const skillsDir = join3(ocrDir, "skills");
-  const sessionsDir = join3(ocrDir, "sessions");
-  const hasOcrDir = existsSync3(ocrDir);
-  const hasSkills = existsSync3(skillsDir);
-  const hasSessions = existsSync3(sessionsDir);
+  const ocrDir = join4(targetDir, ".ocr");
+  const skillsDir = join4(ocrDir, "skills");
+  const sessionsDir = join4(ocrDir, "sessions");
+  const hasOcrDir = existsSync4(ocrDir);
+  const hasSkills = existsSync4(skillsDir);
+  const hasSessions = existsSync4(sessionsDir);
   return {
     valid: hasOcrDir && hasSkills,
     ocrDir,
@@ -558,7 +614,7 @@ function requireOcrSetup(targetDir) {
     console.log();
     console.log(chalk3.red.bold("  \u2717 OCR is not set up in this directory"));
     console.log();
-    if (!existsSync3(status.ocrDir)) {
+    if (!existsSync4(status.ocrDir)) {
       console.log(chalk3.dim("  The .ocr directory was not found."));
     } else if (!status.hasSkills) {
       console.log(chalk3.dim("  The .ocr/skills directory is missing."));
@@ -578,38 +634,55 @@ function requireOcrSetup(targetDir) {
   return status;
 }
 function ensureSessionsDir(targetDir) {
-  const sessionsDir = join3(targetDir, ".ocr", "sessions");
-  if (!existsSync3(sessionsDir)) {
+  const sessionsDir = join4(targetDir, ".ocr", "sessions");
+  if (!existsSync4(sessionsDir)) {
     mkdirSync2(sessionsDir, { recursive: true });
   }
   return sessionsDir;
 }
 
 // packages/cli/src/commands/progress.ts
+function debounce(fn, delay) {
+  let timeoutId = null;
+  return (...args) => {
+    if (timeoutId) {
+      clearTimeout(timeoutId);
+    }
+    timeoutId = setTimeout(() => {
+      fn(...args);
+      timeoutId = null;
+    }, delay);
+  };
+}
+var lastRenderType = null;
+var lastLineCount = 0;
 var TOTAL_PHASES = 8;
 function isSessionActive(sessionPath) {
-  const statePath = join4(sessionPath, "state.json");
-  if (!existsSync4(statePath)) {
+  const statePath = join5(sessionPath, "state.json");
+  if (!existsSync5(statePath)) {
     return true;
   }
   try {
-    const stateContent = readFileSync3(statePath, "utf-8");
+    const stateContent = readFileSync4(statePath, "utf-8");
     const state = JSON.parse(stateContent);
-    return state.status !== "closed";
+    if (state.status === "closed" || state.current_phase === "complete") {
+      return false;
+    }
+    return true;
   } catch {
     return true;
   }
 }
 function findLatestActiveSession(sessionsDir) {
-  if (!existsSync4(sessionsDir)) {
+  if (!existsSync5(sessionsDir)) {
     return null;
   }
   const sessions = readdirSync2(sessionsDir).filter((name) => {
-    const sessionPath = join4(sessionsDir, name);
+    const sessionPath = join5(sessionsDir, name);
     return statSync(sessionPath).isDirectory();
   }).sort().reverse();
   for (const session of sessions) {
-    const sessionPath = join4(sessionsDir, session);
+    const sessionPath = join5(sessionsDir, session);
     if (isSessionActive(sessionPath)) {
       return session;
     }
@@ -617,10 +690,10 @@ function findLatestActiveSession(sessionsDir) {
   return null;
 }
 function countFindings(filePath) {
-  if (!existsSync4(filePath)) {
+  if (!existsSync5(filePath)) {
     return 0;
   }
-  const content = readFileSync3(filePath, "utf-8");
+  const content = readFileSync4(filePath, "utf-8");
   const findingMatches = content.match(/^##\s+(Finding|Issue|Suggestion)/gm);
   return findingMatches?.length ?? 0;
 }
@@ -635,12 +708,12 @@ function formatReviewerName(filename) {
 }
 function parseSessionState(sessionPath, preservedStartTime) {
   const session = basename(sessionPath);
-  const statePath = join4(sessionPath, "state.json");
-  if (!existsSync4(statePath)) {
+  const statePath = join5(sessionPath, "state.json");
+  if (!existsSync5(statePath)) {
     return null;
   }
   try {
-    const stateContent = readFileSync3(statePath, "utf-8");
+    const stateContent = readFileSync4(statePath, "utf-8");
     const state = JSON.parse(stateContent);
     return parseFromStateJson(session, state, sessionPath, preservedStartTime);
   } catch {
@@ -648,16 +721,21 @@ function parseSessionState(sessionPath, preservedStartTime) {
   }
 }
 function parseFromStateJson(session, state, sessionPath, preservedStartTime) {
-  const startTime = preservedStartTime ?? (state.started_at ? new Date(state.started_at).getTime() : Date.now());
-  const completed = new Set(state.completed_phases);
-  const reviewsDir = join4(sessionPath, "reviews");
+  const effectiveStartTime = state.round_started_at ?? state.started_at;
+  const startTime = preservedStartTime ?? (effectiveStartTime ? new Date(effectiveStartTime).getTime() : Date.now());
+  const roundsDir = join5(sessionPath, "rounds");
+  const rounds = deriveRoundsFromFilesystem(roundsDir);
+  const highestExistingRound = rounds.length > 0 ? Math.max(...rounds.map((r) => r.round)) : 1;
+  const stateRound = state.current_round ?? 1;
+  const currentRound = Math.min(stateRound, highestExistingRound);
+  const currentRoundDir = join5(roundsDir, `round-${currentRound}`);
+  const reviewsDir = join5(currentRoundDir, "reviews");
   const reviewers = [];
-  if (existsSync4(reviewsDir)) {
-    const reviewFiles = readdirSync2(reviewsDir).filter(
-      (f) => f.endsWith(".md")
-    );
+  if (existsSync5(reviewsDir)) {
+    const entries = readdirSync2(reviewsDir);
+    const reviewFiles = entries.filter((f) => f.endsWith(".md"));
     for (const file of reviewFiles) {
-      const reviewPath = join4(reviewsDir, file);
+      const reviewPath = join5(reviewsDir, file);
       const findings = countFindings(reviewPath);
       reviewers.push({
         name: file.replace(".md", ""),
@@ -667,38 +745,79 @@ function parseFromStateJson(session, state, sessionPath, preservedStartTime) {
       });
     }
   }
+  const contextComplete = existsSync5(
+    join5(sessionPath, "discovered-standards.md")
+  );
+  const changeContextComplete = existsSync5(join5(sessionPath, "context.md"));
+  const analysisComplete = changeContextComplete;
+  const reviewsComplete = state.phase_number > 4;
+  const discourseComplete = existsSync5(join5(currentRoundDir, "discourse.md"));
+  const synthesisComplete = existsSync5(join5(currentRoundDir, "final.md"));
   return {
     session,
     phase: state.current_phase,
     phaseNumber: state.phase_number,
     totalPhases: TOTAL_PHASES,
-    contextComplete: completed.has("context"),
-    requirementsComplete: completed.has("requirements"),
-    analysisComplete: completed.has("analysis"),
-    reviewsComplete: completed.has("reviews"),
-    aggregationComplete: completed.has("aggregation"),
-    discourseComplete: completed.has("discourse"),
-    synthesisComplete: completed.has("synthesis"),
+    contextComplete,
+    changeContextComplete,
+    analysisComplete,
+    reviewsComplete,
+    aggregationComplete: reviewsComplete,
+    // Aggregation is inline
+    discourseComplete,
+    synthesisComplete,
+    currentRound,
+    rounds,
     reviewers,
     startTime,
     complete: state.current_phase === "complete"
   };
 }
+function deriveRoundsFromFilesystem(roundsDir) {
+  if (!existsSync5(roundsDir)) {
+    return [];
+  }
+  const roundDirs = readdirSync2(roundsDir).filter((d) => d.match(/^round-\d+$/)).sort((a, b) => {
+    const numA = parseInt(a.replace("round-", ""));
+    const numB = parseInt(b.replace("round-", ""));
+    return numA - numB;
+  });
+  return roundDirs.map((dir) => {
+    const roundNum = parseInt(dir.replace("round-", ""));
+    const roundPath = join5(roundsDir, dir);
+    const reviewsPath = join5(roundPath, "reviews");
+    const finalPath = join5(roundPath, "final.md");
+    const reviewers = [];
+    if (existsSync5(reviewsPath)) {
+      const files = readdirSync2(reviewsPath).filter((f) => f.endsWith(".md"));
+      reviewers.push(...files.map((f) => f.replace(".md", "")));
+    }
+    return {
+      round: roundNum,
+      isComplete: existsSync5(finalPath),
+      reviewers
+    };
+  });
+}
 function formatDuration(ms) {
-  const totalSeconds = Math.floor(ms / 1e3);
+  const absMs = Math.abs(ms);
+  const totalSeconds = Math.floor(absMs / 1e3);
   const hours = Math.floor(totalSeconds / 3600);
   const minutes = Math.floor(totalSeconds % 3600 / 60);
   const seconds = totalSeconds % 60;
+  let duration;
   if (hours > 0) {
-    return `${hours}h ${minutes}m ${seconds}s`;
+    duration = `${hours}h ${minutes}m ${seconds}s`;
   } else if (minutes > 0) {
-    return `${minutes}m ${seconds}s`;
+    duration = `${minutes}m ${seconds}s`;
+  } else {
+    duration = `${seconds}s`;
   }
-  return `${seconds}s`;
+  return duration;
 }
 var PHASE_INFO = [
   { key: "context", label: "Context Discovery" },
-  { key: "requirements", label: "Requirements Gathering" },
+  { key: "change-context", label: "Change Context" },
   { key: "analysis", label: "Tech Lead Analysis" },
   { key: "reviews", label: "Parallel Reviews" },
   { key: "aggregation", label: "Aggregate Findings" },
@@ -727,8 +846,9 @@ function renderProgress(state) {
   log(chalk4.bold.white("  Open Code Review"));
   log();
   const elapsed = Date.now() - state.startTime;
+  const roundInfo = state.currentRound > 1 ? chalk4.cyan(` Round ${state.currentRound}`) + chalk4.dim("  \xB7  ") : "";
   log(
-    chalk4.dim("  ") + chalk4.white(state.session) + chalk4.dim("  \xB7  ") + chalk4.white(formatDuration(elapsed))
+    chalk4.dim("  ") + chalk4.white(state.session) + chalk4.dim("  \xB7  ") + roundInfo + chalk4.white(formatDuration(elapsed))
   );
   log();
   const progressPhases = state.complete ? 8 : state.phaseNumber;
@@ -739,7 +859,7 @@ function renderProgress(state) {
   const phaseCompletion = {
     waiting: false,
     context: state.contextComplete,
-    requirements: state.requirementsComplete,
+    "change-context": state.changeContextComplete,
     analysis: state.analysisComplete,
     reviews: state.reviewsComplete,
     aggregation: state.aggregationComplete,
@@ -761,6 +881,9 @@ function renderProgress(state) {
     }
     log(`  ${status} ${label}`);
     if (phase.key === "reviews" && state.reviewers.length > 0) {
+      if (state.currentRound > 1) {
+        log(chalk4.dim("    ") + chalk4.cyan(`Round ${state.currentRound}`));
+      }
       const reviewerLine = state.reviewers.map((r) => {
         const icon = r.status === "complete" ? chalk4.green("\u2713") : chalk4.dim("\u25CB");
         const name = chalk4.dim(r.displayName);
@@ -768,6 +891,17 @@ function renderProgress(state) {
         return `${icon} ${name}${count}`;
       }).join(chalk4.dim("  \u2502  "));
       log(chalk4.dim("    ") + reviewerLine);
+      if (state.rounds.length > 1) {
+        const prevRounds = state.rounds.slice(0, -1);
+        for (const round of prevRounds) {
+          const roundLabel = chalk4.dim(`    Round ${round.round}`);
+          const reviewerCount = round.reviewers.length;
+          const status2 = round.isComplete ? chalk4.green("\u2713") : chalk4.dim("\u25CB");
+          log(
+            `${roundLabel} ${status2} ${chalk4.dim(`${reviewerCount} reviewers`)}`
+          );
+        }
+      }
     }
   }
   log();
@@ -782,12 +916,22 @@ function renderProgress(state) {
       )
     );
     log(
-      chalk4.dim("    ") + chalk4.dim("\u2192 ") + chalk4.white(`.ocr/sessions/${state.session}/final.md`)
+      chalk4.dim("    ") + chalk4.dim("\u2192 ") + chalk4.white(
+        `.ocr/sessions/${state.session}/rounds/round-${state.currentRound}/final.md`
+      )
     );
   } else {
     log(chalk4.dim("  Ctrl+C to exit"));
   }
   log();
+  if (lastRenderType !== "progress") {
+    logUpdate.clear();
+  }
+  lastRenderType = "progress";
+  while (lines.length < lastLineCount) {
+    lines.push("");
+  }
+  lastLineCount = lines.length;
   logUpdate(lines.join("\n"));
 }
 function renderWaiting() {
@@ -807,16 +951,24 @@ function renderWaiting() {
   log();
   log(chalk4.dim("  Ctrl+C to exit"));
   log();
+  if (lastRenderType !== "waiting") {
+    logUpdate.clear();
+  }
+  lastRenderType = "waiting";
+  while (lines.length < lastLineCount) {
+    lines.push("");
+  }
+  lastLineCount = lines.length;
   logUpdate(lines.join("\n"));
 }
 var progressCommand = new Command2("progress").description("Watch real-time progress of a code review session").option("-s, --session <name>", "Specify session name").action(async (options) => {
   const targetDir = process.cwd();
   requireOcrSetup(targetDir);
   const sessionsDir = ensureSessionsDir(targetDir);
-  const ocrDir = join4(targetDir, ".ocr");
+  const ocrDir = join5(targetDir, ".ocr");
   if (options.session) {
-    const sessionPath = join4(sessionsDir, options.session);
-    if (!existsSync4(sessionPath)) {
+    const sessionPath = join5(sessionsDir, options.session);
+    if (!existsSync5(sessionPath)) {
       console.log(chalk4.red(`Session not found: ${options.session}`));
       process.exit(1);
     }
@@ -846,7 +998,7 @@ var progressCommand = new Command2("progress").description("Watch real-time prog
     const watcher = watch(sessionPath, {
       persistent: true,
       ignoreInitial: true,
-      depth: 2
+      depth: 3
     });
     watcher.on("all", () => {
       const newState = parseSessionState(sessionPath, preservedStartTime2);
@@ -864,11 +1016,24 @@ var progressCommand = new Command2("progress").description("Watch real-time prog
     return;
   }
   let currentSession = findLatestActiveSession(sessionsDir);
-  let currentSessionPath = currentSession ? join4(sessionsDir, currentSession) : null;
+  let currentSessionPath = currentSession ? join5(sessionsDir, currentSession) : null;
   let sessionWatcher = null;
   let preservedStartTime;
-  const updateDisplay = () => {
-    if (currentSessionPath && existsSync4(currentSessionPath)) {
+  const updateDisplayImpl = () => {
+    if (!currentSessionPath || !existsSync5(currentSessionPath) || !isSessionActive(currentSessionPath)) {
+      const latestActive = findLatestActiveSession(sessionsDir);
+      if (latestActive && latestActive !== currentSession) {
+        currentSession = latestActive;
+        currentSessionPath = join5(sessionsDir, latestActive);
+        preservedStartTime = void 0;
+        watchSession(currentSessionPath);
+      } else if (!latestActive) {
+        currentSession = null;
+        currentSessionPath = null;
+        preservedStartTime = void 0;
+      }
+    }
+    if (currentSessionPath && existsSync5(currentSessionPath)) {
       const state = parseSessionState(currentSessionPath, preservedStartTime);
       if (state) {
         if (!preservedStartTime) {
@@ -883,6 +1048,7 @@ var progressCommand = new Command2("progress").description("Watch real-time prog
       renderWaiting();
     }
   };
+  const updateDisplay = debounce(updateDisplayImpl, 50);
   const watchSession = (sessionPath) => {
     if (sessionWatcher) {
       sessionWatcher.close();
@@ -890,24 +1056,24 @@ var progressCommand = new Command2("progress").description("Watch real-time prog
     sessionWatcher = watch(sessionPath, {
       persistent: true,
       ignoreInitial: true,
-      depth: 2
+      depth: 3
     });
     sessionWatcher.on("all", updateDisplay);
   };
-  updateDisplay();
+  updateDisplayImpl();
   if (currentSessionPath) {
     watchSession(currentSessionPath);
   }
   const timerInterval = setInterval(updateDisplay, 1e3);
-  const watchDir = existsSync4(ocrDir) ? ocrDir : targetDir;
+  const watchDir = existsSync5(ocrDir) ? ocrDir : targetDir;
   const dirWatcher = watch(watchDir, {
     persistent: true,
     ignoreInitial: true,
     depth: 3
   });
   dirWatcher.on("addDir", (dirPath) => {
-    const parentDir = join4(dirPath, "..");
-    const isDirectChild = parentDir.endsWith("sessions") || parentDir.endsWith(join4(".ocr", "sessions"));
+    const parentDir = join5(dirPath, "..");
+    const isDirectChild = parentDir.endsWith("sessions") || parentDir.endsWith(join5(".ocr", "sessions"));
     if (isDirectChild && !dirPath.endsWith("sessions")) {
       const newSession = basename(dirPath);
       currentSession = newSession;
@@ -932,16 +1098,16 @@ var progressCommand = new Command2("progress").description("Watch real-time prog
 import { Command as Command3 } from "commander";
 import chalk5 from "chalk";
 import ora2 from "ora";
-import { existsSync as existsSync5 } from "node:fs";
-import { join as join5 } from "node:path";
+import { existsSync as existsSync6 } from "node:fs";
+import { join as join6 } from "node:path";
 function detectConfiguredTools(targetDir) {
   return AI_TOOLS.filter((tool) => {
     if (tool.commandStrategy === "subdirectory") {
-      const ocrDir = join5(targetDir, tool.commandsDir, "ocr");
-      return existsSync5(ocrDir);
+      const ocrDir = join6(targetDir, tool.commandsDir, "ocr");
+      return existsSync6(ocrDir);
     } else {
-      const reviewCmd = join5(targetDir, tool.commandsDir, "ocr-review.md");
-      return existsSync5(reviewCmd);
+      const reviewCmd = join6(targetDir, tool.commandsDir, "ocr-review.md");
+      return existsSync6(reviewCmd);
     }
   });
 }
@@ -954,11 +1120,17 @@ var updateCommand = new Command3("update").description("Update OCR assets after
   console.log();
   console.log(chalk5.bold.cyan("  Open Code Review - Update"));
   console.log();
+  const savedToolIds = getConfiguredToolIds(targetDir);
   const configuredTools = detectConfiguredTools(targetDir);
   const installedTools = detectInstalledTools(targetDir, AI_TOOLS);
-  const toolsToUpdate = AI_TOOLS.filter(
-    (tool) => configuredTools.some((t) => t.id === tool.id) || installedTools.some((t) => t.id === tool.id)
-  );
+  let toolsToUpdate;
+  if (savedToolIds.length > 0) {
+    toolsToUpdate = AI_TOOLS.filter((tool) => savedToolIds.includes(tool.id));
+  } else {
+    toolsToUpdate = AI_TOOLS.filter(
+      (tool) => configuredTools.some((t) => t.id === tool.id) || installedTools.some((t) => t.id === tool.id)
+    );
+  }
   if (toolsToUpdate.length === 0) {
     console.log(chalk5.yellow("  No configured AI tools found."));
     console.log(chalk5.dim("  Run `ocr init` to set up OCR first."));
@@ -1034,10 +1206,10 @@ var updateCommand = new Command3("update").description("Update OCR assets after
   if (updateInject) {
     if (options.dryRun) {
       console.log(chalk5.dim("  Would update:"));
-      if (existsSync5(join5(targetDir, "AGENTS.md"))) {
+      if (existsSync6(join6(targetDir, "AGENTS.md"))) {
         console.log(chalk5.dim("    \u2022 AGENTS.md (OCR managed block)"));
       }
-      if (existsSync5(join5(targetDir, "CLAUDE.md"))) {
+      if (existsSync6(join6(targetDir, "CLAUDE.md"))) {
         console.log(chalk5.dim("    \u2022 CLAUDE.md (OCR managed block)"));
       }
       console.log();

package/dist/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@open-code-review/cli",
-  "version": "1.0.3",
+  "version": "1.2.0",
   "description": "CLI for Open Code Review - Multi-environment setup and progress tracking",
   "type": "module",
   "bin": {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@open-code-review/cli",
-  "version": "1.1.1",
+  "version": "1.3.0",
   "description": "CLI for Open Code Review - Multi-environment setup and progress tracking",
   "type": "module",
   "bin": {
@@ -34,7 +34,7 @@
     "commander": "^13.0.0",
     "log-update": "^7.0.2",
     "ora": "^8.1.1",
-    "@open-code-review/agents": "1.1.1"
+    "@open-code-review/agents": "1.3.0"
   },
   "publishConfig": {
     "access": "public"