@open-code-review/cli 1.6.0 → 1.7.0

package/dist/index.js

@@ -20361,6 +20361,153 @@ function installCommandsForTool(tool, commandsSource, targetDir) {
     return false;
   }
 }
+var BUILTIN_ICON_MAP = {
+  architect: "blocks",
+  fullstack: "layers",
+  reliability: "activity",
+  "staff-engineer": "compass",
+  principal: "crown",
+  frontend: "layout",
+  backend: "server",
+  infrastructure: "cloud",
+  performance: "gauge",
+  accessibility: "accessibility",
+  data: "database",
+  devops: "rocket",
+  dx: "terminal",
+  mobile: "smartphone",
+  security: "shield-alert",
+  quality: "sparkles",
+  testing: "test-tubes",
+  ai: "bot",
+  "docs-writer": "file-text"
+};
+var HOLISTIC_IDS = /* @__PURE__ */ new Set(["architect", "fullstack", "reliability", "staff-engineer", "principal"]);
+var SPECIALIST_IDS = /* @__PURE__ */ new Set([
+  "frontend",
+  "backend",
+  "infrastructure",
+  "performance",
+  "accessibility",
+  "data",
+  "devops",
+  "dx",
+  "mobile",
+  "security",
+  "quality",
+  "testing",
+  "ai",
+  "docs-writer"
+]);
+var PERSONA_IDS = /* @__PURE__ */ new Set([
+  "martin-fowler",
+  "kent-beck",
+  "john-ousterhout",
+  "anders-hejlsberg",
+  "vladimir-khorikov",
+  "kent-dodds",
+  "tanner-linsley",
+  "kamil-mysliwiec",
+  "sandi-metz",
+  "rich-hickey"
+]);
+function classifyTier(id) {
+  if (PERSONA_IDS.has(id)) return "persona";
+  if (HOLISTIC_IDS.has(id)) return "holistic";
+  if (SPECIALIST_IDS.has(id)) return "specialist";
+  return "custom";
+}
+function extractReviewerName(content) {
+  const match = content.match(/^#\s+(.+?)(?:\s+—\s+Reviewer|\s+Reviewer)\s*$/m);
+  if (match?.[1]) return match[1];
+  const titleMatch = content.match(/^#\s+(.+)$/m);
+  return titleMatch?.[1]?.replace(/\s*Reviewer\s*$/, "").trim() ?? "Unknown";
+}
+function extractReviewerDescription(content) {
+  const lines = content.split("\n");
+  for (const line of lines) {
+    const trimmed = line.trim();
+    if (!trimmed || trimmed.startsWith("#") || trimmed.startsWith(">")) continue;
+    if (trimmed.startsWith("You are a **") || trimmed.startsWith("You are reviewing")) {
+      return trimmed.replace(/\*\*/g, "").replace(/^You are a /, "").replace(/^You are reviewing code through the lens of .*?\.\s*/, "").trim();
+    }
+  }
+  return "";
+}
+function extractFocusAreas(content) {
+  const areas = [];
+  const focusMatch = content.match(/## Your Focus Areas\n([\s\S]*?)(?=\n##|\n---|\z)/);
+  if (focusMatch?.[1]) {
+    const bullets = focusMatch[1].match(/- \*\*(.+?)\*\*/g);
+    if (bullets) {
+      for (const b of bullets) {
+        const m = b.match(/- \*\*(.+?)\*\*/);
+        if (m?.[1]) areas.push(m[1]);
+      }
+    }
+  }
+  return areas;
+}
+function extractPersonaFields(content) {
+  const knownMatch = content.match(/>\s*\*\*Known for\*\*:\s*(.+)/);
+  const philMatch = content.match(/>\s*\*\*Philosophy\*\*:\s*([\s\S]*?)(?=\n(?!>)|\n\n)/);
+  const result = {};
+  if (knownMatch?.[1]) result.known_for = knownMatch[1].trim();
+  if (philMatch?.[1]) {
+    result.philosophy = philMatch[1].split("\n").map((l) => l.replace(/^>\s*/, "").trim()).join(" ").trim();
+  }
+  return result;
+}
+function generateReviewersMeta(reviewersDir, configPath) {
+  if (!existsSync2(reviewersDir)) return null;
+  const files = readdirSync(reviewersDir).filter((f) => f.endsWith(".md"));
+  if (files.length === 0) return null;
+  const defaultTeamIds = /* @__PURE__ */ new Set();
+  if (existsSync2(configPath)) {
+    try {
+      const configContent = readFileSync3(configPath, "utf-8");
+      const teamMatch = configContent.match(/default_team:\s*\n((?:\s+\w[\w-]*:\s*\d+\s*(?:#[^\n]*)?\n?)*)/);
+      if (teamMatch?.[1]) {
+        const entries = teamMatch[1].matchAll(/\s+([\w-]+):\s*\d+/g);
+        for (const entry of entries) {
+          if (entry[1]) defaultTeamIds.add(entry[1]);
+        }
+      }
+    } catch {
+    }
+  }
+  const reviewers = [];
+  for (const file of files) {
+    const id = file.replace(/\.md$/, "");
+    try {
+      const content = readFileSync3(join2(reviewersDir, file), "utf-8");
+      const tier = classifyTier(id);
+      const isBuiltin = HOLISTIC_IDS.has(id) || SPECIALIST_IDS.has(id) || PERSONA_IDS.has(id);
+      const reviewer = {
+        id,
+        name: extractReviewerName(content),
+        tier,
+        icon: BUILTIN_ICON_MAP[id] ?? (tier === "persona" ? "brain" : "user"),
+        description: extractReviewerDescription(content),
+        focus_areas: extractFocusAreas(content),
+        is_default: defaultTeamIds.has(id),
+        is_builtin: isBuiltin
+      };
+      if (tier === "persona") {
+        const persona = extractPersonaFields(content);
+        if (persona.known_for) reviewer.known_for = persona.known_for;
+        if (persona.philosophy) reviewer.philosophy = persona.philosophy;
+      }
+      reviewers.push(reviewer);
+    } catch {
+    }
+  }
+  return {
+    schema_version: 1,
+    generated_at: (/* @__PURE__ */ new Date()).toISOString(),
+    reviewers
+  };
+}
 function installForTool(tool, targetDir) {
   const agentsPath = getAgentsPackagePath();
   const ocrSkillsSource = join2(agentsPath, "skills", "ocr");
@@ -20435,6 +20582,14 @@ function installForTool(tool, targetDir) {
       }
     }
   }
+  const metaPath = join2(ocrDir, "reviewers-meta.json");
+  try {
+    const meta = generateReviewersMeta(reviewersDir, configPath);
+    if (meta) {
+      writeFileSync3(metaPath, JSON.stringify(meta, null, 2) + "\n");
+    }
+  } catch {
+  }
   const commandsOk = installCommandsForTool(tool, commandsSource, targetDir);
   if (!commandsOk) {
     return {
@@ -25616,14 +25771,145 @@ var doctorCommand = new Command("doctor").description("Check OCR installation an
   console.log();
 });
 
+// src/commands/reviewers.ts
+import { writeFileSync as writeFileSync8, renameSync as renameSync2 } from "node:fs";
+import { join as join18 } from "node:path";
+async function readStdin2() {
+  const chunks = [];
+  for await (const chunk of process.stdin) {
+    chunks.push(Buffer.isBuffer(chunk) ? chunk : Buffer.from(chunk));
+  }
+  const data = Buffer.concat(chunks).toString("utf-8").trim();
+  if (data.length === 0) {
+    throw new Error("No data received on stdin");
+  }
+  return data;
+}
+var VALID_TIERS = /* @__PURE__ */ new Set(["holistic", "specialist", "persona", "custom"]);
+var SLUG_RE = /^[a-z][a-z0-9-]*$/;
+function validateReviewersMeta(data) {
+  if (typeof data !== "object" || data === null || Array.isArray(data)) {
+    throw new Error("Payload must be a JSON object");
+  }
+  const obj = data;
+  if (obj.schema_version !== 1) {
+    throw new Error(`schema_version must be 1, got ${JSON.stringify(obj.schema_version)}`);
+  }
+  if (typeof obj.generated_at !== "string" || obj.generated_at.length === 0) {
+    throw new Error("generated_at must be a non-empty ISO 8601 string");
+  }
+  if (!Array.isArray(obj.reviewers)) {
+    throw new Error("reviewers must be an array");
+  }
+  const seenIds = /* @__PURE__ */ new Set();
+  for (let i = 0; i < obj.reviewers.length; i++) {
+    const r = obj.reviewers[i];
+    const prefix = `reviewers[${i}]`;
+    if (typeof r.id !== "string" || !SLUG_RE.test(r.id)) {
+      throw new Error(`${prefix}.id must be a lowercase slug (got ${JSON.stringify(r.id)})`);
+    }
+    if (seenIds.has(r.id)) {
+      throw new Error(`Duplicate reviewer id: "${r.id}"`);
+    }
+    seenIds.add(r.id);
+    if (typeof r.name !== "string" || r.name.length === 0) {
+      throw new Error(`${prefix}.name must be a non-empty string`);
+    }
+    if (!VALID_TIERS.has(r.tier)) {
+      throw new Error(`${prefix}.tier must be one of: ${[...VALID_TIERS].join(", ")} (got ${JSON.stringify(r.tier)})`);
+    }
+    if (typeof r.description !== "string" || r.description.length === 0) {
+      throw new Error(`${prefix}.description must be a non-empty string`);
+    }
+    if (!Array.isArray(r.focus_areas)) {
+      throw new Error(`${prefix}.focus_areas must be an array`);
+    }
+    if (r.known_for !== void 0 && typeof r.known_for !== "string") {
+      throw new Error(`${prefix}.known_for must be a string if provided`);
+    }
+    if (r.philosophy !== void 0 && typeof r.philosophy !== "string") {
+      throw new Error(`${prefix}.philosophy must be a string if provided`);
+    }
+  }
+  return data;
+}
+var syncSubcommand2 = new Command("sync").description("Sync reviewers-meta.json from reviewer markdown files or structured JSON").option("--stdin", "Read reviewers JSON from stdin (for AI-invoked sync)").action(async (options) => {
+  const targetDir = process.cwd();
+  requireOcrSetup(targetDir);
+  const ocrDir = join18(targetDir, ".ocr");
+  if (!options.stdin) {
+    try {
+      const reviewersDir = join18(ocrDir, "skills", "references", "reviewers");
+      const configPath = join18(ocrDir, "config.yaml");
+      const meta = generateReviewersMeta(reviewersDir, configPath);
+      if (!meta || meta.reviewers.length === 0) {
+        console.error(source_default.yellow("No reviewer files found in .ocr/skills/references/reviewers/"));
+        process.exit(1);
+      }
+      const metaPath = join18(ocrDir, "reviewers-meta.json");
+      const tmpPath = metaPath + ".tmp";
+      writeFileSync8(tmpPath, JSON.stringify(meta, null, 2) + "\n");
+      renameSync2(tmpPath, metaPath);
+      const tierCounts = meta.reviewers.reduce(
+        (acc, r) => {
+          acc[r.tier] = (acc[r.tier] ?? 0) + 1;
+          return acc;
+        },
+        {}
+      );
+      const breakdown = Object.entries(tierCounts).map(([tier, count]) => `${count} ${tier}`).join(", ");
+      console.log(source_default.green(`Synced ${meta.reviewers.length} reviewer(s) (${breakdown}).`));
+    } catch (error) {
+      console.error(
+        source_default.red(`Error: ${error instanceof Error ? error.message : "Failed to sync reviewers"}`)
+      );
+      process.exit(1);
+    }
+    return;
+  }
+  try {
+    const raw = await readStdin2();
+    let parsed;
+    try {
+      parsed = JSON.parse(raw);
+    } catch {
+      throw new Error("Invalid JSON on stdin");
+    }
+    const meta = validateReviewersMeta(parsed);
+    const metaPath = join18(ocrDir, "reviewers-meta.json");
+    const tmpPath = metaPath + ".tmp";
+    writeFileSync8(tmpPath, JSON.stringify(meta, null, 2) + "\n");
+    renameSync2(tmpPath, metaPath);
+    const tierCounts = meta.reviewers.reduce(
+      (acc, r) => {
+        acc[r.tier] = (acc[r.tier] ?? 0) + 1;
+        return acc;
+      },
+      {}
+    );
+    const breakdown = Object.entries(tierCounts).map(([tier, count]) => `${count} ${tier}`).join(", ");
+    console.log(
+      source_default.green(`Synced ${meta.reviewers.length} reviewer(s) (${breakdown}).`)
+    );
+  } catch (error) {
+    console.error(
+      source_default.red(
+        `Error: ${error instanceof Error ? error.message : "Failed to sync reviewers"}`
+      )
+    );
+    process.exit(1);
+  }
+});
+var reviewersCommand = new Command("reviewers").description("Manage OCR reviewer metadata").addCommand(syncSubcommand2);
+
 // src/lib/update-check.ts
 import { homedir } from "node:os";
-import { join as join18 } from "node:path";
-import { readFileSync as readFileSync10, writeFileSync as writeFileSync8, mkdirSync as mkdirSync6 } from "node:fs";
+import { join as join19 } from "node:path";
+import { readFileSync as readFileSync10, writeFileSync as writeFileSync9, mkdirSync as mkdirSync6 } from "node:fs";
 var PACKAGE_NAME = "@open-code-review/cli";
 var REGISTRY_URL = `https://registry.npmjs.org/${PACKAGE_NAME}/latest`;
-var CACHE_DIR = join18(homedir(), ".ocr");
-var CACHE_FILE = join18(CACHE_DIR, "update-check.json");
+var CACHE_DIR = join19(homedir(), ".ocr");
+var CACHE_FILE = join19(CACHE_DIR, "update-check.json");
 var CHECK_INTERVAL_MS = 4 * 60 * 60 * 1e3;
 var FETCH_TIMEOUT_MS = 3e3;
 function readCache(cacheFile) {
@@ -25635,8 +25921,8 @@ function readCache(cacheFile) {
 }
 function writeCache(cacheFile, cache) {
   try {
-    mkdirSync6(join18(cacheFile, ".."), { recursive: true });
-    writeFileSync8(cacheFile, JSON.stringify(cache));
+    mkdirSync6(join19(cacheFile, ".."), { recursive: true });
+    writeFileSync9(cacheFile, JSON.stringify(cache));
   } catch {
   }
 }
@@ -25656,7 +25942,7 @@ async function checkForUpdate(currentVersion, options) {
   if (process.env.CI || process.env.OCR_NO_UPDATE_CHECK) {
     return null;
   }
-  const cacheFile = join18(options?.cacheDir ?? CACHE_DIR, "update-check.json");
+  const cacheFile = join19(options?.cacheDir ?? CACHE_DIR, "update-check.json");
   try {
     const cache = readCache(cacheFile);
     if (cache && Date.now() - cache.lastCheck < CHECK_INTERVAL_MS) {
@@ -25700,7 +25986,7 @@ ${line2}
 }
 
 // src/index.ts
-var cliVersion = true ? "1.6.0" : createRequire(import.meta.url)("../package.json").version;
+var cliVersion = true ? "1.7.0" : createRequire(import.meta.url)("../package.json").version;
 var HUMAN_COMMANDS = /* @__PURE__ */ new Set(["init", "update", "doctor", "dashboard", "progress"]);
 var subcommand = process.argv[2];
 var updateCheck = subcommand && HUMAN_COMMANDS.has(subcommand) ? checkForUpdate(cliVersion) : null;
@@ -25712,6 +25998,7 @@ program2.addCommand(stateCommand);
 program2.addCommand(updateCommand);
 program2.addCommand(dashboardCommand);
 program2.addCommand(doctorCommand);
+program2.addCommand(reviewersCommand);
 await program2.parseAsync();
 if (updateCheck) {
   const updateResult = await Promise.race([

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@open-code-review/cli",
-  "version": "1.6.0",
+  "version": "1.7.0",
   "description": "CLI for Open Code Review - Multi-environment setup and progress tracking",
   "type": "module",
   "bin": {
@@ -50,7 +50,7 @@
     "ora": "^8.1.1",
     "socket.io": "^4.8",
     "sql.js": "^1.14.1",
-    "@open-code-review/agents": "1.6.0"
+    "@open-code-review/agents": "1.7.0"
   },
   "publishConfig": {
     "access": "public"

package/dist/dashboard/client/assets/channel-yW2sWou_.js

@@ -1 +0,0 @@
-import{ap as o,aq as n}from"./mermaid-renderer-DGUmIWXY.js";const t=(a,r)=>o.lang.round(n.parse(a)[r]);export{t as c};

package/dist/dashboard/client/assets/classDiagram-2ON5EDUG-1pMX5UXO.js

@@ -1 +0,0 @@
-import{s as a,c as s,a as e,C as t}from"./chunk-B4BG7PRW-BXzTPbH1.js";import{_ as i}from"./mermaid-renderer-DGUmIWXY.js";import"./chunk-FMBD7UC4-B4g9S67N.js";import"./chunk-55IACEB6-C4SjgsZO.js";import"./chunk-QN33PNHL-Dyk7Hc0J.js";import"./index-BzQ3i_QR.js";var u={parser:e,get db(){return new t},renderer:s,styles:a,init:i(r=>{r.class||(r.class={}),r.class.arrowMarkerAbsolute=r.arrowMarkerAbsolute},"init")};export{u as diagram};

package/dist/dashboard/client/assets/classDiagram-v2-WZHVMYZB-1pMX5UXO.js

@@ -1 +0,0 @@
-import{s as a,c as s,a as e,C as t}from"./chunk-B4BG7PRW-BXzTPbH1.js";import{_ as i}from"./mermaid-renderer-DGUmIWXY.js";import"./chunk-FMBD7UC4-B4g9S67N.js";import"./chunk-55IACEB6-C4SjgsZO.js";import"./chunk-QN33PNHL-Dyk7Hc0J.js";import"./index-BzQ3i_QR.js";var u={parser:e,get db(){return new t},renderer:s,styles:a,init:i(r=>{r.class||(r.class={}),r.class.arrowMarkerAbsolute=r.arrowMarkerAbsolute},"init")};export{u as diagram};

package/dist/dashboard/client/assets/clone-DQwdw3YR.js

@@ -1 +0,0 @@
-import{b as r}from"./_baseUniq-Bx8loabg.js";var e=4;function a(o){return r(o,e)}export{a as c};