@brainervirus/workit-core 8.32.0 → 8.33.0

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@brainervirus/workit-core",
-  "version": "8.32.0",
+  "version": "8.33.0",
   "private": false,
   "description": "Workit shared core — task, policy, evidence, review, decision, worker, and writer state for agentic coding workflows",
   "keywords": [

package/src/core/operation-input.ts

@@ -567,10 +567,16 @@ function invalidOperation(
   const request = isObject(input) ? input : {};
   const action = typeof request.action === "string" ? request.action : "";
   const actions = OPERATION_ACTIONS[family];
-  if (!actions.includes(action))
-    return failure("invalid_input", `${family}: action must be one of ${oneOf(actions)}`, {
-      fields: [{ path: "action", reason: `got ${JSON.stringify(request.action ?? null)}` }],
-    });
+  if (!actions.includes(action)) {
+    // The family's read (list, else the first action) is the safe corrected call.
+    const common = actions.find((name) => READ_ONLY_ACTIONS.has(name)) ?? actions[0];
+    const example = JSON.stringify(exampleOperation(family, common));
+    return failure(
+      "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: { path: string; reason: string }[] = [];
   const sentence: string[] = [];

package/src/hooks/handle.ts

@@ -30,7 +30,8 @@ import {
 } from "./skill-nudge";
 import { spawnDecision } from "./spawn-gate";
 import { seedStanding, standingDelta } from "./standing";
-import { trackJudge } from "./session-marker";
+import { modelSwitchedOnDisk, trackJudge } from "./session-marker";
+import { invariantBootstrap } from "../core/methods";
 import { stopDecision } from "./stop";
 import { liveOwner } from "../core/live-session";
 import { liveTargets, shellGate, writeGate } from "./write-gate";
@@ -166,7 +167,12 @@ function decide(input: HookInput, deps: HookDeps): HookDecision {
     case "context.turn": {
       if (!usable(descriptor.context.perTurn)) return NONE;
       const current = turnContextText(input, descriptor);
-      const text = resendsOnChange(descriptor) ? changedTurnContext(input, current) : current;
+      const turn = resendsOnChange(descriptor) ? changedTurnContext(input, current) : current;
+      // A model switch (#435): the compact contract again, once.
+      const text =
+        input.model && modelSwitchedOnDisk(input)
+          ? [invariantBootstrap(), turn].filter(Boolean).join("\n")
+          : turn;
       return withContextLine(
         text ? { kind: "context", text } : NONE,
         promptNudge(input, event.prompt),

package/src/hooks/hosts/codex.ts

@@ -361,6 +361,7 @@ export const codexAdapter: HostAdapter = {
         },
         permissionMode: input.permission_mode,
         transcriptPath: input.transcript_path,
+        model: input.model,
         event: protocolEvent(input),
       },
     };

package/src/hooks/protocol.ts

@@ -90,6 +90,8 @@ export type HookInput = {
   session: Session;
   permissionMode: string | null;
   transcriptPath: string | null;
+  /** The host's current model, where every hook input carries it (Codex). */
+  model?: string | null;
   event: HookEvent;
 };
 

package/src/hooks/session-marker.ts

@@ -17,6 +17,8 @@ export type SessionMarker = {
   stops: string[];
   /** Verifier and reviewer subagents of this lead that started and have not stopped. */
   judges: string[];
+  /** The last model the session reported (Codex #435): a change re-sends the contract. */
+  model: string | null;
 };
 
 /**
@@ -42,7 +44,14 @@ export const sessionMarkerFile = (
 const list = (value: unknown): string[] =>
   Array.isArray(value) ? value.filter((item): item is string => typeof item === "string") : [];
 
-const EMPTY: SessionMarker = { nudged: [], loaded: [], raw: [], stops: [], judges: [] };
+const EMPTY: SessionMarker = {
+  nudged: [],
+  loaded: [],
+  raw: [],
+  stops: [],
+  judges: [],
+  model: null,
+};
 
 const readRaw = (file: string): string => {
   try {
@@ -61,6 +70,7 @@ const parseMarker = (text: string): SessionMarker => {
       raw: list(value.raw),
       stops: list(value.stops),
       judges: list(value.judges),
+      model: typeof value.model === "string" ? value.model : null,
     };
   } catch {
     return { ...EMPTY };
@@ -123,3 +133,19 @@ export const trackJudge = (input: HookInput, agentId: string | null, running: bo
     return { ...marker, judges: running ? [...rest, agentId] : rest };
   });
 };
+
+/**
+ * Record the session's model and say whether it differs from the last one
+ * recorded (the hook is a fresh process each time). The first model is a
+ * start, not a switch. Fails open: no store or no model reads as no switch.
+ */
+export const modelSwitchedOnDisk = (input: HookInput): boolean => {
+  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;
+};