omp-fabric 1.25.12 → 1.25.14

package/CHANGELOG.md

@@ -1,5 +1,17 @@
 # Changelog
 
+## 1.25.14
+
+### Changed
+
+- Host pins move from `18.4.10` to `18.5.0`; the declared peer floor stays `>=18.4.4`. 18.5.0's two breaking changes (`task.completionProbeMs` and the `SessionStorage` session-claim API) touch nothing Fabric uses. Typecheck, the full suite, and `certify:context` (31 checks) pass against the new pin, and `docs/certification.md` records the re-certified host.
+
+## 1.25.13
+
+### Fixed
+
+- 1.25.12 told every full-code session to call OMP-loaded MCP tools as `extensions.mcp__<server>_<tool>(args)`. That holds where OMP registers MCP through the extension runner, as in `omp -p` and the Paperclip adapter, but an interactive session loads its MCP servers after startup straight into the host tool registry, Fabric never captures them, and the call fails with `Unknown Fabric action` while the tools stay on the model's direct tool list. The guidance now follows the catalog: it names the `extensions.mcp__` path only when Fabric holds captured MCP tools, and otherwise says those tools stay direct calls outside `fabric_exec`.
+
 ## 1.25.12
 
 ### Fixed

package/dist/core/system-guidance.d.ts

@@ -1,5 +1,5 @@
 export declare const fabricExecutionKernelGuidance: (fullCodeMode: boolean, deniedCoreTools?: readonly string[]) => string;
-export declare const defaultFabricExecutionGuidance: (fullCodeMode: boolean, deniedCoreTools?: readonly string[]) => string;
+export declare const defaultFabricExecutionGuidance: (fullCodeMode: boolean, deniedCoreTools?: readonly string[], hostMcpCaptured?: boolean) => string;
 export interface ExtensionRosterToolSource {
     name: string;
     sourceInfo?: {

package/dist/core/system-guidance.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"system-guidance.d.ts","sourceRoot":"","sources":["../../src/core/system-guidance.ts"],"names":[],"mappings":"AA6BA,eAAO,MAAM,6BAA6B,GACxC,cAAc,OAAO,EACrB,kBAAiB,SAAS,MAAM,EAAO,KACtC,MAqBU,CAAC;AAEd,eAAO,MAAM,8BAA8B,GACzC,cAAc,OAAO,EACrB,kBAAiB,SAAS,MAAM,EAAO,KACtC,MAKmW,CAAC;AAGvW,MAAM,WAAW,yBAAyB;IACxC,IAAI,EAAE,MAAM,CAAC;IACb,UAAU,CAAC,EAAE;QAAE,MAAM,CAAC,EAAE,MAAM,CAAC;QAAC,IAAI,CAAC,EAAE,MAAM,CAAA;KAAE,CAAC;IAChD,UAAU,CAAC,EAAE;QAAE,aAAa,CAAC,EAAE,MAAM,CAAA;KAAE,CAAC;CACzC;AAwCD,eAAO,MAAM,2BAA2B,GACtC,OAAO,aAAa,CAAC,yBAAyB,CAAC,EAC/C,eAAe,WAAW,CAAC,MAAM,CAAC,KACjC,MAAM,GAAG,SAwCX,CAAC;AAEF,eAAO,MAAM,oBAAoB,GAAI,MAAM,KAAK,GAAG,OAAO,GAAG,SAAS,KAAG,MAAM,GAAG,SAQjF,CAAC"}
+{"version":3,"file":"system-guidance.d.ts","sourceRoot":"","sources":["../../src/core/system-guidance.ts"],"names":[],"mappings":"AA6BA,eAAO,MAAM,6BAA6B,GACxC,cAAc,OAAO,EACrB,kBAAiB,SAAS,MAAM,EAAO,KACtC,MAqBU,CAAC;AAEd,eAAO,MAAM,8BAA8B,GACzC,cAAc,OAAO,EACrB,kBAAiB,SAAS,MAAM,EAAO,EACvC,yBAAsB,KACrB,MASmW,CAAC;AAGvW,MAAM,WAAW,yBAAyB;IACxC,IAAI,EAAE,MAAM,CAAC;IACb,UAAU,CAAC,EAAE;QAAE,MAAM,CAAC,EAAE,MAAM,CAAC;QAAC,IAAI,CAAC,EAAE,MAAM,CAAA;KAAE,CAAC;IAChD,UAAU,CAAC,EAAE;QAAE,aAAa,CAAC,EAAE,MAAM,CAAA;KAAE,CAAC;CACzC;AAwCD,eAAO,MAAM,2BAA2B,GACtC,OAAO,aAAa,CAAC,yBAAyB,CAAC,EAC/C,eAAe,WAAW,CAAC,MAAM,CAAC,KACjC,MAAM,GAAG,SAwCX,CAAC;AAEF,eAAO,MAAM,oBAAoB,GAAI,MAAM,KAAK,GAAG,OAAO,GAAG,SAAS,KAAG,MAAM,GAAG,SAQjF,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,CA+yBxE;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,CAmzBxE;AAED,cAAc,kBAAkB,CAAC;AACjC,cAAc,oBAAoB,CAAC;AACnC,cAAc,eAAe,CAAC"}

package/dist/index.js

@@ -2201,7 +2201,7 @@ var fabricExecutionKernelGuidance = (fullCodeMode, deniedCoreTools = []) => [
   ...fullCodeMode && !deniedCoreTools.includes("wait") ? ["A program that needs that result awaits `omp.wait()`, which takes no arguments and returns the next finished job; never sleep or poll for it."] : [],
   ...fullCodeMode && deniedCoreTools.length > 0 ? [`OMP has these guest 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, 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 tools from MCP servers Fabric itself pools (`mcp.$servers()`) as `mcp.<sanitized_server>.<sanitized_tool>(args)`; MCP servers OMP loaded are captured tools like any other, so call those and every other captured tool as `extensions.<tool>(args)`, e.g. `extensions.mcp__<server>_<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 = [], hostMcpCaptured = true) => 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 tools from MCP servers Fabric itself pools (`mcp.$servers()`) as `mcp.<sanitized_server>.<sanitized_tool>(args)`; " + (hostMcpCaptured ? "MCP servers OMP loaded are captured tools like any other, so call those and every other captured tool as `extensions.<tool>(args)`, e.g. `extensions.mcp__<server>_<tool>(args)`;" : "MCP tools OMP loaded in this session stay direct tool calls outside fabric_exec; call 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 isFilesystemSourcePath = (value) => path4.posix.isAbsolute(value) || path4.win32.isAbsolute(value);
 var packageNameFromManifest = (startPath) => {
@@ -6095,7 +6095,11 @@ async function ompFabric(omp) {
       target: process.env.OMP_FABRIC_PARENT_RUN ? "participant" : "main",
       defaults: [{
         slot: FABRIC_EXECUTION_GUIDANCE_SLOT,
-        content: defaultFabricExecutionGuidance(effectiveFullCodeMode, deniedCoreTools)
+        content: defaultFabricExecutionGuidance(
+          effectiveFullCodeMode,
+          deniedCoreTools,
+          capturedTools.list().some((entry) => entry.definition.mcpServerName !== void 0)
+        )
       }]
     });
     const overrideGuidance = effectiveFullCodeMode ? coreOverridePromptGuidance(capturedTools).trim() : void 0;