@open-code-review/cli 2.3.0 → 2.4.0

package/dist/dashboard/server.js

@@ -35769,6 +35769,7 @@ var import_express14 = __toESM(require_express2(), 1);
 var import_yaml = __toESM(require_dist(), 1);
 import { existsSync as existsSync12, readFileSync as readFileSync8 } from "node:fs";
 import { join as join16 } from "node:path";
+var MAX_INSTANCES_PER_PERSONA = 50;
 function loadTeamConfig(ocrDir) {
   const configPath = join16(ocrDir, "config.yaml");
   if (!existsSync12(configPath)) {
@@ -35830,6 +35831,11 @@ function parseEntry(persona, entry) {
         `default_team.${persona}: count must be a positive integer (got ${entry})`
       );
     }
+    if (entry > MAX_INSTANCES_PER_PERSONA) {
+      throw new Error(
+        `default_team.${persona}: count must be <= ${MAX_INSTANCES_PER_PERSONA} (got ${entry})`
+      );
+    }
     return Array.from({ length: entry }, () => ({}));
   }
   if (Array.isArray(entry)) {
@@ -35838,6 +35844,11 @@ function parseEntry(persona, entry) {
         `default_team.${persona}: list form must contain at least one instance`
       );
     }
+    if (entry.length > MAX_INSTANCES_PER_PERSONA) {
+      throw new Error(
+        `default_team.${persona}: list form must contain <= ${MAX_INSTANCES_PER_PERSONA} instances (got ${entry.length})`
+      );
+    }
     return entry.map((item, idx) => parseListItem(persona, idx, item));
   }
   if (entry && typeof entry === "object") {
@@ -35859,6 +35870,11 @@ function parseEntry(persona, entry) {
         `default_team.${persona}: count must be a positive integer (got ${String(count)})`
       );
     }
+    if (count > MAX_INSTANCES_PER_PERSONA) {
+      throw new Error(
+        `default_team.${persona}: count must be <= ${MAX_INSTANCES_PER_PERSONA} (got ${count})`
+      );
+    }
     const teamModel = readOptionalString(obj, "model", `default_team.${persona}.model`);
     return Array.from({ length: count }, () => ({ teamModel }));
   }

package/dist/index.js

@@ -30024,6 +30024,7 @@ function ensureGitignore(ocrDir) {
 var import_yaml = __toESM(require_dist(), 1);
 import { existsSync as existsSync2, readFileSync as readFileSync3 } from "node:fs";
 import { join as join2 } from "node:path";
+var MAX_INSTANCES_PER_PERSONA = 50;
 function loadTeamConfig(ocrDir) {
   const configPath = join2(ocrDir, "config.yaml");
   if (!existsSync2(configPath)) {
@@ -30085,6 +30086,11 @@ function parseEntry(persona, entry) {
         `default_team.${persona}: count must be a positive integer (got ${entry})`
       );
     }
+    if (entry > MAX_INSTANCES_PER_PERSONA) {
+      throw new Error(
+        `default_team.${persona}: count must be <= ${MAX_INSTANCES_PER_PERSONA} (got ${entry})`
+      );
+    }
     return Array.from({ length: entry }, () => ({}));
   }
   if (Array.isArray(entry)) {
@@ -30093,6 +30099,11 @@ function parseEntry(persona, entry) {
         `default_team.${persona}: list form must contain at least one instance`
       );
     }
+    if (entry.length > MAX_INSTANCES_PER_PERSONA) {
+      throw new Error(
+        `default_team.${persona}: list form must contain <= ${MAX_INSTANCES_PER_PERSONA} instances (got ${entry.length})`
+      );
+    }
     return entry.map((item, idx) => parseListItem(persona, idx, item));
   }
   if (entry && typeof entry === "object") {
@@ -30114,6 +30125,11 @@ function parseEntry(persona, entry) {
         `default_team.${persona}: count must be a positive integer (got ${String(count)})`
       );
     }
+    if (count > MAX_INSTANCES_PER_PERSONA) {
+      throw new Error(
+        `default_team.${persona}: count must be <= ${MAX_INSTANCES_PER_PERSONA} (got ${count})`
+      );
+    }
     const teamModel = readOptionalString(obj, "model", `default_team.${persona}.model`);
     return Array.from({ length: count }, () => ({ teamModel }));
   }
@@ -30206,6 +30222,71 @@ function resolveTeamComposition(team, override) {
   }
   return result;
 }
+var REVIEWER_ID_PATTERN = /^[a-z0-9]+(?:-[a-z0-9]+)*$/;
+function parseTeamSpec(spec, aliases = {}, defaultModel = null) {
+  const trimmed = spec.trim();
+  if (trimmed.length === 0) {
+    throw new Error(
+      "--team spec is empty; expected reviewer-id:count[,reviewer-id:count...]"
+    );
+  }
+  const model = resolveModel(null, null, aliases, defaultModel);
+  if (model !== null) assertSafeModelId(model, "--team");
+  const seen = /* @__PURE__ */ new Set();
+  const result = [];
+  for (const rawEntry of trimmed.split(",")) {
+    const entry = rawEntry.trim();
+    if (entry.length === 0) {
+      throw new Error(
+        `--team has an empty entry (stray or trailing comma) in "${spec}"`
+      );
+    }
+    const colon = entry.indexOf(":");
+    if (colon === -1) {
+      throw new Error(
+        `--team entry "${entry}" must be "reviewer-id:count" \u2014 the :count is required`
+      );
+    }
+    const persona = entry.slice(0, colon).trim();
+    const countRaw = entry.slice(colon + 1).trim();
+    if (!REVIEWER_ID_PATTERN.test(persona)) {
+      throw new Error(
+        `--team reviewer id "${persona}" is invalid; expected lowercase letters, digits, and single hyphens (e.g. principal, martin-fowler)`
+      );
+    }
+    if (seen.has(persona)) {
+      throw new Error(
+        `--team lists "${persona}" more than once; combine its instances into a single entry (e.g. ${persona}:2)`
+      );
+    }
+    if (!/^[0-9]+$/.test(countRaw)) {
+      throw new Error(
+        `--team count for "${persona}" must be a positive integer (got "${countRaw}")`
+      );
+    }
+    const count = Number(countRaw);
+    if (count < 1) {
+      throw new Error(
+        `--team count for "${persona}" must be a positive integer (got ${count})`
+      );
+    }
+    if (count > MAX_INSTANCES_PER_PERSONA) {
+      throw new Error(
+        `--team count for "${persona}" must be <= ${MAX_INSTANCES_PER_PERSONA} (got ${count})`
+      );
+    }
+    seen.add(persona);
+    for (let i = 0; i < count; i++) {
+      result.push({
+        persona,
+        instance_index: i + 1,
+        name: `${persona}-${i + 1}`,
+        model
+      });
+    }
+  }
+  return result;
+}
 
 // src/lib/installer.ts
 init_src();
@@ -30770,7 +30851,7 @@ ${hint}
 }
 
 // src/lib/version.ts
-var CLI_VERSION = true ? "2.3.0" : createRequire(import.meta.url)("../../package.json").version;
+var CLI_VERSION = true ? "2.4.0" : createRequire(import.meta.url)("../../package.json").version;
 
 // src/lib/deps.ts
 init_src();
@@ -36641,6 +36722,9 @@ function parseSessionOverride(raw) {
   return result;
 }
 var resolveSubcommand = new Command("resolve").description("Resolve and print the team composition for the active workspace").option(
+  "--team <spec>",
+  "Session team override replacing default_team. Format: reviewer-id:count[,reviewer-id:count...] (e.g. principal:2,architect:1)"
+).option(
   "--session-override <json>",
   "JSON array of ReviewerInstance overrides applied on top of disk config"
 ).option("--session-override-stdin", "Read --session-override JSON from stdin").option("--json", "Emit JSON for programmatic consumption (the AI workflow uses this)").action(
@@ -36649,7 +36733,8 @@ var resolveSubcommand = new Command("resolve").description("Resolve and print th
     requireOcrSetup(targetDir);
     const ocrDir = join21(targetDir, ".ocr");
     try {
-      const { team } = loadTeamConfig(ocrDir);
+      const { team, aliases, defaultModel } = loadTeamConfig(ocrDir);
+      const baseTeam = options.team ? parseTeamSpec(options.team, aliases, defaultModel) : team;
       let override;
       if (options.sessionOverride) {
         override = parseSessionOverride(options.sessionOverride);
@@ -36659,7 +36744,7 @@ var resolveSubcommand = new Command("resolve").description("Resolve and print th
           override = parseSessionOverride(raw);
         }
       }
-      const resolved = resolveTeamComposition(team, override);
+      const resolved = resolveTeamComposition(baseTeam, override);
       if (options.json) {
         console.log(JSON.stringify(resolved, null, 2));
         return;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@open-code-review/cli",
-  "version": "2.3.0",
+  "version": "2.4.0",
   "description": "CLI for Open Code Review - Multi-environment setup and progress tracking",
   "type": "module",
   "bin": {
@@ -45,14 +45,14 @@
     "ora": "^8.1.1",
     "socket.io": "^4.8",
     "yaml": "^2.8.3",
-    "@open-code-review/agents": "2.3.0"
+    "@open-code-review/agents": "2.4.0"
   },
   "publishConfig": {
     "access": "public"
   },
   "devDependencies": {
-    "@open-code-review/platform": "0.0.0",
+    "@open-code-review/config": "0.0.0",
     "@open-code-review/persistence": "0.0.0",
-    "@open-code-review/config": "0.0.0"
+    "@open-code-review/platform": "0.0.0"
   }
 }