@brainervirus/workit-pi 8.27.0 → 8.28.1

package/dist/worker.js

@@ -6668,6 +6668,7 @@ var ENVELOPE = new Set([
   "expectedWorkspaceRevision",
   "judgment"
 ]);
+var NOT_FLAT = new Set(["schemaVersion", "action", "workspaceId"]);
 // packages/workit-core/src/core/config.ts
 import {
   copyFileSync,

package/dist/workit.js

@@ -7105,8 +7105,9 @@ function flatOperationJsonSchema(family, options = {}) {
   };
   const properties = { action: { type: "string", enum: actions } };
   const readField = (name) => name === "taskId" || (family === "task" ? READ_TASK_FIELDS.has(name) : family === "policy");
+  const taken = (name) => family === "policy" || actions.some((action) => actionKeys(family, action)?.accepted.has(name));
   for (const [name, field] of Object.entries({ ...COMMON, ...spec.fields }))
-    if (options.readOnly ? readField(name) : !(family === "state" && name === "expectedRevision"))
+    if (options.readOnly ? readField(name) : taken(name))
       properties[name] = property(field);
   return {
     $schema: "https://json-schema.org/draft/2020-12/schema",
@@ -7348,6 +7349,98 @@ function normalizeOperationInput(family, input, host) {
       request[key] = strings(request[key]);
   return request;
 }
+var defOf = (node) => node._zod.def;
+var unwrap = (node) => {
+  let inner = node;
+  while (["optional", "nullable"].includes(defOf(inner).type) && defOf(inner).innerType)
+    inner = defOf(inner).innerType;
+  return inner;
+};
+var FLAT_NAME = {
+  authorityRefs: "refs",
+  contentRefs: "refs",
+  statedChoice: "choice",
+  reviewContext: "reviewSession"
+};
+var NOT_FLAT = new Set(["schemaVersion", "action", "workspaceId"]);
+function actionKeys(family, action) {
+  const options = advertisedOperationSchemas[family].options;
+  const option = options.find((item) => {
+    const literal = defOf(item).shape?.action;
+    return literal._zod.def.values.includes(action);
+  });
+  if (!option)
+    return null;
+  const out = { accepted: new Set, required: [] };
+  const walk = (shape, nested) => {
+    for (const [key, node] of Object.entries(shape)) {
+      if (NOT_FLAT.has(key) || nested && key === "taskId")
+        continue;
+      const inner = unwrap(node);
+      const type = defOf(inner).type;
+      const needed = !["optional", "nullable"].includes(defOf(node).type);
+      if (key === "scope")
+        for (const name of ["scope", "paths", "exclusions"])
+          out.accepted.add(name);
+      else if (type === "object" && key !== "bundle" && !(key in FLAT_NAME)) {
+        out.accepted.add(key);
+        walk(defOf(inner).shape, true);
+      } else {
+        const name = FLAT_NAME[key] ?? key;
+        out.accepted.add(name);
+        if (needed && type !== "array" && type !== "record" && !out.required.includes(name))
+          out.required.push(name);
+      }
+    }
+  };
+  walk(defOf(option).shape, false);
+  return out;
+}
+function exampleOperation(family, action) {
+  const keys = actionKeys(family, action);
+  const example = { action };
+  for (const name of keys?.required ?? [])
+    example[name] = FLAT[family].fields[name]?.enum?.[0] ?? `<${name}>`;
+  if (family === "policy" && !keys?.required.length && action !== "explain")
+    example.riskTier = "normal";
+  return example;
+}
+var oneOf = (values) => values.join(" | ");
+function invalidOperation(family, input, error) {
+  const request = isObject2(input) ? input : {};
+  const action = typeof request.action === "string" ? request.action : "";
+  const actions = OPERATION_ACTIONS[family];
+  if (!actions.includes(action))
+    return failure2("invalid_input", `${family}: action must be one of ${oneOf(actions)}`, {
+      fields: [{ path: "action", reason: `got ${JSON.stringify(request.action ?? null)}` }]
+    });
+  const own = actionKeys(family, action);
+  const fields = [];
+  const sentence = [];
+  for (const issue of error.issues) {
+    if (issue.code === "unrecognized_keys") {
+      for (const key of issue.keys) {
+        const home = actions.find((other) => other !== action && actionKeys(family, other).accepted.has(key));
+        const reason = home ? `${key} belongs to action: ${home}` : `${key} is not a field of ${family} ${action}; it takes ${[...own.accepted].join(", ")}`;
+        fields.push({ path: key, reason });
+        sentence.push(reason);
+      }
+      continue;
+    }
+    const last = String(issue.path.at(-1) ?? "");
+    const name = Object.entries(FLAT_NAME).find(([contract]) => contract === last)?.[1] ?? last;
+    const missing = issue.code === "invalid_type" && /received undefined/.test(issue.message);
+    const absent = own.required.filter((key) => !(key in request));
+    const needs = missing && issue.path.length === 1 && absent.length ? absent.join(", ") : name;
+    const reason = missing ? `${family} ${action} requires ${needs}` : `${name}: ${issue.message}`;
+    fields.push({ path: issue.path.join("."), reason });
+    sentence.push(reason);
+  }
+  const example = JSON.stringify(exampleOperation(family, action));
+  return failure2("invalid_input", `${[...new Set(sentence)].join("; ")}. Example: ${example}`, {
+    fields
+  });
+}
 function parseAdvertisedOperation(family, input, host) {
   const request = normalizeOperationInput(family, input, host);
   if (isObject2(request) && request.schemaVersion !== SCHEMA_VERSION)
@@ -7355,12 +7448,7 @@ function parseAdvertisedOperation(family, input, host) {
   const parsed = advertisedOperationSchemas[family].safeParse(request);
   if (parsed.success)
     return success(null, null, request);
-  return failure2("invalid_input", "operation input is invalid", {
-    fields: parsed.error.issues.map((issue) => ({
-      path: issue.code === "unrecognized_keys" ? issue.keys.join(".") : issue.path.join("."),
-      reason: issue.message
-    }))
-  });
+  return invalidOperation(family, request, parsed.error);
 }
 // packages/workit-core/src/core/task-store.ts
 import { createHash as createHash9, randomUUID as randomUUID3 } from "node:crypto";
@@ -10013,11 +10101,16 @@ var branchPolicySnapshotFor = (workspaceRoot) => {
     fingerprint
   };
 };
+var rethrowRuntimeBug = (error) => {
+  if (error instanceof TypeError || error instanceof ReferenceError)
+    throw error;
+};
 var validateBranchNameFor = (workspaceRoot, name) => {
   try {
     const resolved = branchPolicySnapshotFor(workspaceRoot);
     return validateBranchNamePolicy(name, resolved.namePolicy, resolved.source);
   } catch (error) {
+    rethrowRuntimeBug(error);
     const correction = "repair or select a valid workspace policy before creating a branch";
     return {
       ok: false,
@@ -23839,6 +23932,138 @@ var standingDelta = (input) => {
   }
 };
 var sinceStart = new Map;
+// packages/workit-core/src/core/role-models.ts
+var MODEL_ROLES = [
+  "decider",
+  "implementer",
+  "verifier",
+  "reviewer",
+  "explore",
+  "babysit"
+];
+var MODEL_TIERS = ["mundane", "standard", "hard"];
+var HOST_KEYS = new Set([
+  "claude_code",
+  "opencode",
+  "codex",
+  "codex_cli",
+  "codex_desktop",
+  "cursor",
+  "pi",
+  "default"
+]);
+var BUILT_IN_CLAUDE_CODE = {
+  decider: { model: "fable", effort: "high" },
+  "implementer.hard": { model: "opus" },
+  "implementer.standard": { model: "sonnet" },
+  "implementer.mundane": { model: "haiku" },
+  verifier: { model: "sonnet" },
+  reviewer: { model: "opus" },
+  explore: { model: "haiku" },
+  babysit: { model: "haiku" }
+};
+var isObject7 = (value) => !!value && typeof value === "object" && !Array.isArray(value);
+var name = (value) => typeof value === "string" && value.trim() ? value.trim() : null;
+var leaf = (value) => {
+  const text = name(value);
+  if (text)
+    return { model: text, effort: null };
+  if (!isObject7(value))
+    return null;
+  const model = name(value.model);
+  return model ? { model, effort: name(value.effort) } : null;
+};
+var isFlat = (value) => typeof value === "string" || isObject7(value) && Object.hasOwn(value, "model");
+var forHost = (value, host) => {
+  if (isFlat(value))
+    return leaf(value);
+  if (!isObject7(value))
+    return null;
+  const family = host ? hostFamily(host) : null;
+  for (const key of [family, host, "default"])
+    if (key && Object.hasOwn(value, key)) {
+      const found = leaf(value[key]);
+      if (found)
+        return found;
+    }
+  return null;
+};
+var rolesOf = (models) => isObject7(models) && isObject7(models.roles) ? models.roles : null;
+var resolveRoleModel = (cwd, selector, host, dir) => {
+  const { role, tier } = selector;
+  for (const source of modelSources(cwd, dir)) {
+    const where = source.global ? "the global default" : `workspace ${source.name}`;
+    const roles = rolesOf(source.models);
+    const configured = roles && Object.hasOwn(roles, role) ? roles[role] : undefined;
+    const fromRoles = role === "implementer" ? tier && isObject7(configured) && Object.hasOwn(configured, tier) ? forHost(configured[tier], host) : null : forHost(configured, host);
+    if (fromRoles)
+      return { ...fromRoles, source: `models.roles in ${where}` };
+    if (role === "implementer" && tier && isObject7(source.tierModels)) {
+      const fromTiers = Object.hasOwn(source.tierModels, tier) ? forHost(source.tierModels[tier], host) : null;
+      if (fromTiers)
+        return { model: fromTiers.model, effort: null, source: `fanout.tierModels in ${where}` };
+    }
+  }
+  if (host && hostFamily(host) === "claude_code") {
+    const builtIn = BUILT_IN_CLAUDE_CODE[tier ? `${role}.${tier}` : role];
+    if (builtIn)
+      return { model: builtIn.model, effort: builtIn.effort ?? null, source: "built-in" };
+  }
+  return null;
+};
+var uniformImplementerModel = (cwd, host, dir) => {
+  const tiers = MODEL_TIERS.map((tier) => resolveRoleModel(cwd, { role: "implementer", tier }, host, dir));
+  const [first] = tiers;
+  return first && tiers.every((t) => t?.model === first.model && t.effort === first.effort) ? first : null;
+};
+var ROLE_SELECTORS = MODEL_ROLES.flatMap((role) => role === "implementer" ? MODEL_TIERS.map((tier) => ({ role, tier })) : [{ role }]);
+var roleForAgentType = (agentType, prompt) => {
+  switch (agentType) {
+    case "workit:implementer": {
+      const tier = typeof prompt === "string" ? /^[ \t]*TIER:[ \t]*(mundane|standard|hard)\b/im.exec(prompt)?.[1] : undefined;
+      return { role: "implementer", tier: tier?.toLowerCase() ?? "standard" };
+    }
+    case "workit:verifier":
+      return { role: "verifier" };
+    case "workit:reviewer":
+      return { role: "reviewer" };
+    case "Explore":
+      return { role: "explore" };
+    default:
+      return null;
+  }
+};
+
+// packages/workit-core/src/hooks/agent-model.ts
+var MODELS = new Set(["sonnet", "opus", "haiku", "fable"]);
+var EFFORTS = new Set(["low", "medium", "high", "xhigh", "max"]);
+var given = (value) => typeof value === "string" && value.trim() !== "";
+var agentModelDecision = (cwd, tool, env = process.env) => {
+  if (given(tool.model) || env.CLAUDE_CODE_SUBAGENT_MODEL_FORCE)
+    return { kind: "none" };
+  const selector = roleForAgentType(tool.subagent_type, tool.prompt);
+  if (!selector)
+    return { kind: "none" };
+  const resolved = resolveRoleModel(cwd, selector, "claude_code");
+  if (!resolved)
+    return { kind: "none" };
+  const model = resolved.model.toLowerCase();
+  const effort = resolved.effort?.toLowerCase() ?? null;
+  const input = { ...tool };
+  if (MODELS.has(model))
+    input.model = model;
+  if (effort && EFFORTS.has(effort) && !given(tool.effort))
+    input.effort = effort;
+  if (input.model === undefined && input.effort === tool.effort)
+    return { kind: "none" };
+  const label = selector.tier ? `${selector.role} ${selector.tier}` : selector.role;
+  return {
+    kind: "update-input",
+    input,
+    text: `Workit set ${String(input.model ?? "no model")}${input.effort !== tool.effort ? ` (effort ${String(input.effort)})` : ""} on this ${String(tool.subagent_type)} spawn: role ${label}, ${resolved.source}. Name a model to choose another.`
+  };
+};
+
 // packages/workit-core/src/hooks/cwd-normalize.ts
 var DIR = String.raw`[A-Za-z0-9_./@%+:,=-]+|'[^'$${"`"}\\\n]*'|"[^"$${"`"}\\!\n]*"`;
 var COMPOUND = new RegExp(String.raw`^\s*cd\s+(${DIR})\s*&&\s*(workit(?:\s[\s\S]*)?)$`, "u");
@@ -24202,7 +24427,7 @@ function handleHook(input, deps) {
   const delta = standingDelta(input);
   if (!delta)
     return decision;
-  return decision.kind === "rewrite" ? { ...decision, text: [decision.text, delta].filter(Boolean).join(`
+  return decision.kind === "rewrite" || decision.kind === "update-input" ? { ...decision, text: [decision.text, delta].filter(Boolean).join(`
 `) } : withContextLine(decision, delta);
 }
 function decide(input, deps) {
@@ -24315,6 +24540,8 @@ function decide(input, deps) {
     case "tool.pre":
       if (event.skill)
         recordSkillLoad(input, event.skill, "tool");
+      if (event.agentInput && input.host === "claude_code")
+        return agentModelDecision(input.cwd, event.agentInput);
       return NONE6;
     case "stop":
       return usable2(descriptor.events.stop.support) ? stopDecision(input, descriptor) : NONE6;
@@ -24697,92 +24924,6 @@ var cancelHint = (workerId) => `cancel it first: workit_worker {"action":"cancel
 import { spawn as nodeSpawn } from "node:child_process";
 import { randomUUID as randomUUID4 } from "node:crypto";
 
-// packages/workit-core/src/core/role-models.ts
-var MODEL_ROLES = [
-  "decider",
-  "implementer",
-  "verifier",
-  "reviewer",
-  "explore",
-  "babysit"
-];
-var MODEL_TIERS = ["mundane", "standard", "hard"];
-var HOST_KEYS = new Set([
-  "claude_code",
-  "opencode",
-  "codex",
-  "codex_cli",
-  "codex_desktop",
-  "cursor",
-  "pi",
-  "default"
-]);
-var BUILT_IN_CLAUDE_CODE = {
-  decider: { model: "fable", effort: "high" },
-  "implementer.hard": { model: "opus" },
-  "implementer.standard": { model: "sonnet" },
-  "implementer.mundane": { model: "haiku" },
-  verifier: { model: "sonnet" },
-  reviewer: { model: "opus" },
-  explore: { model: "haiku" },
-  babysit: { model: "haiku" }
-};
-var isObject7 = (value) => !!value && typeof value === "object" && !Array.isArray(value);
-var name = (value) => typeof value === "string" && value.trim() ? value.trim() : null;
-var leaf = (value) => {
-  const text = name(value);
-  if (text)
-    return { model: text, effort: null };
-  if (!isObject7(value))
-    return null;
-  const model = name(value.model);
-  return model ? { model, effort: name(value.effort) } : null;
-};
-var isFlat = (value) => typeof value === "string" || isObject7(value) && Object.hasOwn(value, "model");
-var forHost = (value, host) => {
-  if (isFlat(value))
-    return leaf(value);
-  if (!isObject7(value))
-    return null;
-  const family = host ? hostFamily(host) : null;
-  for (const key of [family, host, "default"])
-    if (key && Object.hasOwn(value, key)) {
-      const found = leaf(value[key]);
-      if (found)
-        return found;
-    }
-  return null;
-};
-var rolesOf = (models) => isObject7(models) && isObject7(models.roles) ? models.roles : null;
-var resolveRoleModel = (cwd, selector, host, dir) => {
-  const { role, tier } = selector;
-  for (const source of modelSources(cwd, dir)) {
-    const where = source.global ? "the global default" : `workspace ${source.name}`;
-    const roles = rolesOf(source.models);
-    const configured = roles && Object.hasOwn(roles, role) ? roles[role] : undefined;
-    const fromRoles = role === "implementer" ? tier && isObject7(configured) && Object.hasOwn(configured, tier) ? forHost(configured[tier], host) : null : forHost(configured, host);
-    if (fromRoles)
-      return { ...fromRoles, source: `models.roles in ${where}` };
-    if (role === "implementer" && tier && isObject7(source.tierModels)) {
-      const fromTiers = Object.hasOwn(source.tierModels, tier) ? forHost(source.tierModels[tier], host) : null;
-      if (fromTiers)
-        return { model: fromTiers.model, effort: null, source: `fanout.tierModels in ${where}` };
-    }
-  }
-  if (host && hostFamily(host) === "claude_code") {
-    const builtIn = BUILT_IN_CLAUDE_CODE[tier ? `${role}.${tier}` : role];
-    if (builtIn)
-      return { model: builtIn.model, effort: builtIn.effort ?? null, source: "built-in" };
-  }
-  return null;
-};
-var uniformImplementerModel = (cwd, host, dir) => {
-  const tiers = MODEL_TIERS.map((tier) => resolveRoleModel(cwd, { role: "implementer", tier }, host, dir));
-  const [first] = tiers;
-  return first && tiers.every((t) => t?.model === first.model && t.effort === first.effort) ? first : null;
-};
-var ROLE_SELECTORS = MODEL_ROLES.flatMap((role) => role === "implementer" ? MODEL_TIERS.map((tier) => ({ role, tier })) : [{ role }]);
-
 // packages/workit-pi/src/worker-protocol.ts
 var MAX_WORKER_LINE_BYTES = 64 * 1024;
 var MAX_WORKER_STDERR_BYTES = 64 * 1024;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@brainervirus/workit-pi",
-  "version": "8.27.0",
+  "version": "8.28.1",
   "description": "Workit native extension for Pi — tools, method skills, and session continuity",
   "keywords": [
     "agentic-coding",