omp-fabric 1.25.1 → 1.25.2

package/dist/host-compatibility.d.ts

@@ -1,5 +1,11 @@
 export declare const MINIMUM_OMP_HOST_VERSION = "18.3.0";
 export declare const compareVersions: (left: string, right: string) => number | undefined;
 export declare const detectOmpHostVersion: (cliPath?: string | undefined) => string | undefined;
+export interface HostVersionWitnesses {
+    argvPath?: string | undefined;
+    execPath?: string | undefined;
+    hostVersion?: () => Promise<string | undefined>;
+}
+export declare const resolveHostVersion: (witnesses?: HostVersionWitnesses) => Promise<string | undefined>;
 export declare const ompHostCompatibilityWarning: (version?: string | undefined) => string | undefined;
 //# sourceMappingURL=host-compatibility.d.ts.map

package/dist/host-compatibility.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"host-compatibility.d.ts","sourceRoot":"","sources":["../src/host-compatibility.ts"],"names":[],"mappings":"AAGA,eAAO,MAAM,wBAAwB,WAAW,CAAC;AAoBjD,eAAO,MAAM,eAAe,GAAI,MAAM,MAAM,EAAE,OAAO,MAAM,KAAG,MAAM,GAAG,SAYtE,CAAC;AAEF,eAAO,MAAM,oBAAoB,GAC/B,UAAS,MAAM,GAAG,SAA2B,KAC5C,MAAM,GAAG,SA8BX,CAAC;AAEF,eAAO,MAAM,2BAA2B,GACtC,UAAS,MAAM,GAAG,SAAkC,KACnD,MAAM,GAAG,SAKX,CAAC"}
+{"version":3,"file":"host-compatibility.d.ts","sourceRoot":"","sources":["../src/host-compatibility.ts"],"names":[],"mappings":"AAGA,eAAO,MAAM,wBAAwB,WAAW,CAAC;AAoBjD,eAAO,MAAM,eAAe,GAAI,MAAM,MAAM,EAAE,OAAO,MAAM,KAAG,MAAM,GAAG,SAYtE,CAAC;AAsCF,eAAO,MAAM,oBAAoB,GAC/B,UAAS,MAAM,GAAG,SAA2B,KAC5C,MAAM,GAAG,SAAgE,CAAC;AAE7E,MAAM,WAAW,oBAAoB;IACnC,QAAQ,CAAC,EAAE,MAAM,GAAG,SAAS,CAAC;IAC9B,QAAQ,CAAC,EAAE,MAAM,GAAG,SAAS,CAAC;IAC9B,WAAW,CAAC,EAAE,MAAM,OAAO,CAAC,MAAM,GAAG,SAAS,CAAC,CAAC;CACjD;AAQD,eAAO,MAAM,kBAAkB,GAC7B,YAAW,oBAAyB,KACnC,OAAO,CAAC,MAAM,GAAG,SAAS,CAM5B,CAAC;AAEF,eAAO,MAAM,2BAA2B,GACtC,UAAS,MAAM,GAAG,SAAkC,KACnD,MAAM,GAAG,SAKX,CAAC"}

package/dist/index.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"index.d.ts","sourceRoot":"","sources":["../src/index.ts"],"names":[],"mappings":"AACA,OAAO,KAAK,EACV,YAAY,EAEb,MAAM,2BAA2B,CAAC;AAsKnC,wBAA8B,SAAS,CAAC,GAAG,EAAE,YAAY,GAAG,OAAO,CAAC,IAAI,CAAC,CA8yBxE;AAED,cAAc,kBAAkB,CAAC;AACjC,cAAc,oBAAoB,CAAC;AACnC,cAAc,eAAe,CAAC"}
+{"version":3,"file":"index.d.ts","sourceRoot":"","sources":["../src/index.ts"],"names":[],"mappings":"AACA,OAAO,KAAK,EACV,YAAY,EAEb,MAAM,2BAA2B,CAAC;AAsKnC,wBAA8B,SAAS,CAAC,GAAG,EAAE,YAAY,GAAG,OAAO,CAAC,IAAI,CAAC,CA+yBxE;AAED,cAAc,kBAAkB,CAAC;AACjC,cAAc,oBAAoB,CAAC;AACnC,cAAc,eAAe,CAAC"}

package/dist/index.js

@@ -2199,7 +2199,7 @@ var fabricExecutionKernelGuidance = (fullCodeMode, deniedCoreTools = []) => [
   ...fullCodeMode && !deniedCoreTools.includes("edit") && !deniedCoreTools.includes("write") ? ["Prefer `omp.edit`/`omp.write` for source edits over hand-assembled shell or Node string literals, where nested escaping breaks; running a formatter or codemod through `omp.bash` is a different case."] : [],
   ...fullCodeMode && deniedCoreTools.length > 0 ? [`OMP has these core tools turned off, so ${deniedCoreTools.map((name) => `\`omp.${name}\``).join(", ")} ${deniedCoreTools.length === 1 ? "is" : "are"} unavailable and calling ${deniedCoreTools.length === 1 ? "it" : "them"} fails.`] : []
 ].join(" ");
-var defaultFabricExecutionGuidance = (fullCodeMode) => fullCodeMode ? "Examples and returns: `omp.read('/x')`, `omp.grep('TODO','src')` / `omp.grep({pattern:'TODO', path:'src', ignoreCase:true, context:2})`, `omp.find({pattern:'*.ts', path:'src', limit:20})`, and `omp.ls('src')` return strings; `omp.bash({cmd:'ls'})`, `omp.edit({path:'/x', old:'a', new:'b'})`, and `omp.write({path:'/y', text:'z'})` return `{ok, output, details}` (read `.output`); failed core calls reject, including shell tools on an ordinary nonzero exit; pass `settle: true` to `omp.bash` to get `{ ok: false, exitCode, output, error }` instead. Timeout, cancellation, approval, and security failures still reject.\n`tools` is discovery + generic calls only (`providers`/`catalog`/`list`/`search`/`describe`/`call`/`models`). Call known MCP tools as `mcp.<sanitized_server>.<sanitized_tool>(args)`, captured tools as `extensions.<tool>(args)`, and stable providers as `memory.*`, `state.*`, `schema.*`, `compact.*`, or `judgment.*` (calibrated choice/bool/score answers about one state). Use `tools.call({ref,args})` for computed refs. Use `judge.filter(items, instructions)` or `judge.bool(state, instructions)` for fast semantic triage in one request without subagents (e.g. `await judge.filter(logs, 'fatal error')`). `omp` is the core tools; `omp.<key>` reads a named `payloads` value (not a tool)." : "Call known actions through `mcp.<sanitized_server>.<sanitized_tool>(args)`, `memory.*`, `state.*`, `schema.*`, `components.*`, `compact.*`, `codemap.*`, `judgment.*`, `agents.*`, or `mesh.*`; use `tools.catalog`/`search`/`describe`/`list` for discovery and `tools.call({ref,args})` for computed refs. Other surfaces are opt-in via user-loaded skills.";
+var defaultFabricExecutionGuidance = (fullCodeMode, deniedCoreTools = []) => fullCodeMode ? "Examples and returns: `omp.read('/x')`, `omp.grep('TODO','src')` / `omp.grep({pattern:'TODO', path:'src', ignoreCase:true, context:2})`, `omp.find({pattern:'*.ts', path:'src', limit:20})`" + (deniedCoreTools.includes("ls") ? "" : ", and `omp.ls('src')`") + " return strings; `omp.bash({cmd:'ls'})`, `omp.edit({path:'/x', old:'a', new:'b'})`, and `omp.write({path:'/y', text:'z'})` return `{ok, output, details}` (read `.output`); failed core calls reject, including shell tools on an ordinary nonzero exit; pass `settle: true` to `omp.bash` to get `{ ok: false, exitCode, output, error }` instead. Timeout, cancellation, approval, and security failures still reject.\n`tools` is discovery + generic calls only (`providers`/`catalog`/`list`/`search`/`describe`/`call`/`models`). Call known MCP tools as `mcp.<sanitized_server>.<sanitized_tool>(args)`, captured tools as `extensions.<tool>(args)`, and stable providers as `memory.*`, `state.*`, `schema.*`, `compact.*`, or `judgment.*` (calibrated choice/bool/score answers about one state). Use `tools.call({ref,args})` for computed refs. Use `judge.filter(items, instructions)` or `judge.bool(state, instructions)` for fast semantic triage in one request without subagents (e.g. `await judge.filter(logs, 'fatal error')`). `omp` is the core tools; `omp.<key>` reads a named `payloads` value (not a tool)." : "Call known actions through `mcp.<sanitized_server>.<sanitized_tool>(args)`, `memory.*`, `state.*`, `schema.*`, `components.*`, `compact.*`, `codemap.*`, `judgment.*`, `agents.*`, or `mesh.*`; use `tools.catalog`/`search`/`describe`/`list` for discovery and `tools.call({ref,args})` for computed refs. Other surfaces are opt-in via user-loaded skills.";
 var manifestNameCache = /* @__PURE__ */ new Map();
 var packageNameFromManifest = (startPath) => {
   if (!startPath) return void 0;
@@ -2594,7 +2594,7 @@ Relevant to the current request: ${suggestion.name}. Ignore this if it does not
 
 // src/fabric-exec-tool.ts
 import { Container as Container2, Text as Text2 } from "@oh-my-pi/pi-tui";
-import { Type } from "@oh-my-pi/omptype/typebox";
+import { Type } from "@oh-my-pi/pi-coding-agent/extensibility/legacy-typebox";
 
 // src/speculation/summary.ts
 var fabricSpeculationSummary = (stats) => {
@@ -4274,30 +4274,43 @@ var compareVersions = (left, right) => {
   if (a.prerelease === b.prerelease) return 0;
   return (a.prerelease ?? "").localeCompare(b.prerelease ?? "");
 };
-var detectOmpHostVersion = (cliPath = process.argv[1]) => {
-  if (!cliPath) return void 0;
+var manifestHostVersion = (directory) => {
+  const manifestPath = path6.join(directory, "package.json");
+  if (!existsSync3(manifestPath)) return void 0;
+  try {
+    const manifest = JSON.parse(readFileSync4(manifestPath, "utf8"));
+    if (typeof manifest.name === "string" && OMP_HOST_PACKAGE_NAMES[manifest.name] === true && typeof manifest.version === "string") {
+      return manifest.version;
+    }
+  } catch {
+  }
+  return void 0;
+};
+var walkForHostVersion = (startPath) => {
   let directory;
   try {
-    directory = path6.dirname(realpathSync2(cliPath));
+    directory = path6.dirname(realpathSync2(startPath));
   } catch {
     return void 0;
   }
   while (true) {
-    const manifestPath = path6.join(directory, "package.json");
-    if (existsSync3(manifestPath)) {
-      try {
-        const manifest = JSON.parse(readFileSync4(manifestPath, "utf8"));
-        if (typeof manifest.name === "string" && OMP_HOST_PACKAGE_NAMES[manifest.name] === true && typeof manifest.version === "string") {
-          return manifest.version;
-        }
-      } catch {
-      }
-    }
+    const version = manifestHostVersion(directory);
+    if (version) return version;
     const parent = path6.dirname(directory);
     if (parent === directory) return void 0;
     directory = parent;
   }
 };
+var detectOmpHostVersion = (cliPath = process.argv[1]) => cliPath ? walkForHostVersion(cliPath) : void 0;
+var hostModuleVersion = async () => await import("@oh-my-pi/pi-coding-agent").then(
+  (host) => typeof host.VERSION === "string" ? host.VERSION : void 0,
+  () => void 0
+);
+var resolveHostVersion = async (witnesses = {}) => {
+  const argvPath = "argvPath" in witnesses ? witnesses.argvPath : process.argv[1];
+  const execPath = "execPath" in witnesses ? witnesses.execPath : process.execPath;
+  return (argvPath ? walkForHostVersion(argvPath) : void 0) ?? (execPath ? walkForHostVersion(execPath) : void 0) ?? await (witnesses.hostVersion ?? hostModuleVersion)();
+};
 var ompHostCompatibilityWarning = (version = detectOmpHostVersion()) => {
   if (!version) return void 0;
   const comparison = compareVersions(version, MINIMUM_OMP_HOST_VERSION);
@@ -5812,7 +5825,7 @@ async function ompFabric(omp) {
     refreshProxyLedger(context);
     if (!compatibilityWarningShown) {
       compatibilityWarningShown = true;
-      const warning = ompHostCompatibilityWarning();
+      const warning = ompHostCompatibilityWarning(await resolveHostVersion());
       if (warning) {
         console.warn(`[omp-fabric] ${warning}`);
         if (context.hasUI) context.ui.notify(warning, "warning");
@@ -5896,8 +5909,9 @@ async function ompFabric(omp) {
   omp.on("agent_end", async (event, context) => {
     if (event.willContinue === true) return;
     await lcmRuntime?.syncAndSchedule();
-    if (!state.initialized) {
-      await compactAtConfiguredThreshold(context, state.config);
+    const config = state.bootstrapped ? state.config : void 0;
+    if (!state.initialized || !config) {
+      if (config) await compactAtConfiguredThreshold(context, config);
       return;
     }
     const sessionId = context.sessionManager.getSessionId();
@@ -6064,21 +6078,19 @@ async function ompFabric(omp) {
       { signal: AbortSignal.timeout(SKILL_GATE_BUDGET_MS) }
     ) : void 0;
     const currentModel = context.model ? `${context.model.provider}/${context.model.id}` : void 0;
+    const deniedCoreTools = effectiveFullCodeMode ? deniedOmpCoreTools(toolOwnership.hostActiveTools()) : [];
     const resolvedGuidance = resolveFabricModelGuidance(state.modelGuidance(), {
       ...currentModel ? { model: currentModel } : {},
       target: process.env.OMP_FABRIC_PARENT_RUN ? "participant" : "main",
       defaults: [{
         slot: FABRIC_EXECUTION_GUIDANCE_SLOT,
-        content: defaultFabricExecutionGuidance(effectiveFullCodeMode)
+        content: defaultFabricExecutionGuidance(effectiveFullCodeMode, deniedCoreTools)
       }]
     });
     const overrideGuidance = effectiveFullCodeMode ? coreOverridePromptGuidance(capturedTools).trim() : void 0;
     const extensionRoster = effectiveFullCodeMode ? extensionToolRosterGuidance(capturedTools.list(), new Set(OMP_CORE_TOOL_NAMES)) : void 0;
     const guidance = [
-      fabricExecutionKernelGuidance(
-        effectiveFullCodeMode,
-        effectiveFullCodeMode ? deniedOmpCoreTools(toolOwnership.hostActiveTools()) : []
-      ),
+      fabricExecutionKernelGuidance(effectiveFullCodeMode, deniedCoreTools),
       resolvedGuidance.slotText,
       fabricSchemaGuidance(schemaMode),
       overrideGuidance,