@brainervirus/workit-pi 8.32.0 → 8.34.0

package/dist/workit.js

@@ -7410,10 +7410,11 @@ 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)}` }]
-    });
+  if (!actions.includes(action)) {
+    const common = actions.find((name) => READ_ONLY_ACTIONS.has(name)) ?? actions[0];
+    const example = JSON.stringify(exampleOperation(family, common));
+    return failure2("invalid_input", `${family}: action must be one of ${oneOf(actions)}. Example: ${example}`, { fields: [{ path: "action", reason: `got ${JSON.stringify(request.action ?? null)}` }] });
+  }
   const own = actionKeys(family, action);
   const fields = [];
   const sentence = [];
@@ -16205,7 +16206,14 @@ var sessionMarkerFile = (input, session, store) => {
   return path19.join(dir, "hooks", `skills-${name}.json`);
 };
 var list2 = (value) => Array.isArray(value) ? value.filter((item) => typeof item === "string") : [];
-var EMPTY = { nudged: [], loaded: [], raw: [], stops: [], judges: [] };
+var EMPTY = {
+  nudged: [],
+  loaded: [],
+  raw: [],
+  stops: [],
+  judges: [],
+  model: null
+};
 var readRaw = (file) => {
   try {
     return fs15.readFileSync(file, "utf8");
@@ -16221,7 +16229,8 @@ var parseMarker = (text) => {
       loaded: list2(value.loaded),
       raw: list2(value.raw),
       stops: list2(value.stops),
-      judges: list2(value.judges)
+      judges: list2(value.judges),
+      model: typeof value.model === "string" ? value.model : null
     };
   } catch {
     return { ...EMPTY };
@@ -16266,6 +16275,18 @@ var trackJudge = (input, agentId, running) => {
     return { ...marker, judges: running ? [...rest, agentId] : rest };
   });
 };
+var modelSwitchedOnDisk = (input) => {
+  const session = input.session.id;
+  const model = input.model;
+  const file = session && model ? sessionMarkerFile(input, session) : null;
+  if (!file || !model)
+    return false;
+  const previous = readSessionMarker(file).model;
+  if (previous === model)
+    return false;
+  updateSessionMarker(file, (marker) => ({ ...marker, model }));
+  return previous !== null;
+};
 
 // packages/workit-core/src/hooks/shell-words.ts
 class ShellAnalysisError extends Error {
@@ -24561,7 +24582,9 @@ function decide(input, deps) {
       if (!usable2(descriptor.context.perTurn))
         return NONE6;
       const current = turnContextText(input, descriptor);
-      const text = resendsOnChange(descriptor) ? changedTurnContext(input, current) : current;
+      const turn = resendsOnChange(descriptor) ? changedTurnContext(input, current) : current;
+      const text = input.model && modelSwitchedOnDisk(input) ? [invariantBootstrap(), turn].filter(Boolean).join(`
+`) : turn;
       return withContextLine(text ? { kind: "context", text } : NONE6, promptNudge(input, event.prompt));
     }
     case "shell.pre": {
@@ -24767,17 +24790,6 @@ var PI_DESCRIPTOR = {
   ]
 };
 // packages/workit-core/src/hooks/model-switch.ts
-var MAX_SESSIONS = 256;
-var modelSwitched = (models, sessionId, model) => {
-  if (!model)
-    return false;
-  const previous = models.get(sessionId);
-  models.delete(sessionId);
-  models.set(sessionId, model);
-  if (models.size > MAX_SESSIONS)
-    models.delete(models.keys().next().value);
-  return previous !== undefined && previous !== model;
-};
 var modelKey = (model) => {
   if (model === null || typeof model !== "object")
     return null;
@@ -24813,13 +24825,15 @@ var taskTurnContext = (ctx) => {
     return null;
   }
 };
-var sessionModels = new Map;
-var turnContext = (ctx) => {
-  const task = taskTurnContext(ctx);
-  const switched = modelSwitched(sessionModels, ctx.sessionManager.getSessionId(), modelKey(ctx.model));
-  return switched && ctx.isProjectTrusted() ? `${invariantBootstrap()}${task ? `
-${task}` : ""}` : task;
+var switchedSessions = new Set;
+var noteModelSelect = (event, ctx) => {
+  const next = modelKey(event.model);
+  const previous = modelKey(event.previousModel);
+  if (next && previous && next !== previous)
+    switchedSessions.add(ctx.sessionManager.getSessionId());
 };
+var modelSwitchContract = (ctx) => switchedSessions.delete(ctx.sessionManager.getSessionId()) && ctx.isProjectTrusted() ? invariantBootstrap() : null;
+var turnContext = (ctx) => taskTurnContext(ctx);
 var turnInput = (ctx) => ({
   host: "pi",
   cwd: ctx.cwd,
@@ -24866,7 +24880,7 @@ var unfinishedTaskOffer2 = (ctx) => {
 // packages/workit-core/src/hooks/repeat-failure.ts
 import { createHash as createHash18 } from "node:crypto";
 var SAME_CALL_FAILED = "Same call failed twice:";
-var MAX_SESSIONS2 = 256;
+var MAX_SESSIONS = 256;
 var stable = (value) => JSON.stringify(value, (_key, entry) => entry && typeof entry === "object" && !Array.isArray(entry) ? Object.fromEntries(Object.entries(entry).toSorted(([a], [b]) => a.localeCompare(b))) : entry) ?? "null";
 var hashOf = (call) => createHash18("sha256").update(call).digest("hex").slice(0, 32);
 var record2 = (memo, sessionId, call, failed, next) => {
@@ -24876,7 +24890,7 @@ var record2 = (memo, sessionId, call, failed, next) => {
     return null;
   const hash = hashOf(call);
   memo.set(sessionId, hash);
-  if (memo.size > MAX_SESSIONS2)
+  if (memo.size > MAX_SESSIONS)
     memo.delete(memo.keys().next().value);
   return previous === hash && next ? `${SAME_CALL_FAILED} ${next}` : null;
 };
@@ -24894,6 +24908,26 @@ var escalateToolFailure = (memo, sessionId, tool, input, result) => {
   return line && failed ? { ...result, error: `${result.error}
 ${line}` } : result;
 };
+var WORKIT_COMMAND = /^\s*(?:[A-Za-z_][A-Za-z0-9_]*=\S*\s+)*workit\s/;
+var unblockOf = (output) => {
+  const human = /^\s*unblock: (.+)$/m.exec(output)?.[1];
+  if (human)
+    return human.trim();
+  const json = /"unblock":("(?:[^"\\]|\\.)*")/.exec(output)?.[1];
+  if (!json)
+    return null;
+  try {
+    return JSON.parse(json);
+  } catch {
+    return null;
+  }
+};
+var repeatedCliFailure = (memo, sessionId, command, output) => {
+  if (!WORKIT_COMMAND.test(command))
+    return null;
+  const failed = /^workit: /m.test(output) || /"ok":\s*false/.test(output);
+  return record2(memo, sessionId, `cli\x00${command.trim()}`, failed, failed ? unblockOf(output) : null);
+};
 
 // packages/workit-pi/src/tools.ts
 var readOnlyActions = new Set(["list", "inspect", "preview", "explain", "export"]);
@@ -25013,11 +25047,17 @@ var enforceToolPolicy = (event, ctx) => {
 };
 var observeToolResult = (event, ctx) => {
   try {
-    if (event.toolName !== "bash" || event.isError)
+    if (event.toolName !== "bash")
       return;
     const command = event.input?.command;
     if (typeof command !== "string")
       return;
+    const shown = event.content.map((part) => part.type === "text" ? part.text : "").join(`
+`);
+    const session = piContext(ctx).caller.actor;
+    const repeated = repeatedCliFailure(failures, session, command, shown);
+    if (event.isError)
+      return repeated ? { content: [...event.content, { type: "text", text: repeated }] } : undefined;
     const post = hookInput(ctx, {
       kind: "shell.post",
       command,
@@ -25029,7 +25069,10 @@ var observeToolResult = (event, ctx) => {
     const named = process.env.WORKIT_PI_WORKER_ROLE;
     const role = named === "implementer" ? "implementer" : named ? "judge" : null;
     const nudge = process.env.WORKIT_PI_WORKER_SESSION ? rawGitPre(post, command, { nudge: roleSteering(role, false).nudges }) : shellNudge(post, command);
-    return nudge.kind === "context" ? { content: [...event.content, { type: "text", text: nudge.text }] } : undefined;
+    const extra = [nudge.kind === "context" ? nudge.text : null, repeated].filter((text) => text !== null);
+    return extra.length > 0 ? { content: [...event.content, { type: "text", text: extra.join(`
+
+`) }] } : undefined;
   } catch {
     return;
   }
@@ -25953,6 +25996,7 @@ function extension(pi) {
     sessions.delete(ctx.sessionManager.getSessionId());
     reconcileLostWorkers(ctx);
   });
+  pi.on("model_select", (event, ctx) => noteModelSelect(event, ctx));
   pi.on("before_agent_start", (event, ctx) => {
     if (event.messageType !== "user")
       return;
@@ -25961,7 +26005,8 @@ function extension(pi) {
     const trusted = ctx.isProjectTrusted();
     const task = turnContext(ctx);
     let content;
-    if (sessions.has(id)) {
+    const firstTurn = !sessions.has(id);
+    if (!firstTurn) {
       content = trusted ? changedTurnContext(turnInput(ctx), task) : null;
     } else {
       sessions.add(id);
@@ -25973,6 +26018,10 @@ function extension(pi) {
 
 ${offer}` : ""}`;
     }
+    const contract = modelSwitchContract(ctx);
+    if (contract && !firstTurn)
+      content = content ? `${contract}
+${content}` : contract;
     if (content)
       return { message: contextMessage(ctx, nudge ? `${content}
 

package/package.json

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