omp-fabric 1.25.4 → 1.25.6

package/CHANGELOG.md

@@ -1,5 +1,19 @@
 # Changelog
 
+## 1.25.6
+
+### Fixed
+
+- A finished `fabric_exec` card rendered its call and its result as two transcript blocks. The host emergency layout then kept every finished card active and collapsed the session to one line per turn, which is the flat `Fabric` / `Task` / `Wait` stream. The card now sets `mergeCallAndResult`, the result keeps `display.name`, and the border shell no longer substitutes an empty container for a finished result.
+
+
+## 1.25.5
+
+### Fixed
+
+- `docs/providers.md` now names the symptom of the one surviving `omp.wait()` failure mode. A session that gains an owner after its provider was built can start an async job it can never collect: a `bash` call naming a `cwd` runs `async: true`, the job is running, and the result still reaches the agent as a follow-up, while `omp.wait()` reports that nothing is running. Nothing errors, and the guidance a program author had was only the general instruction to start a new session. The behaviour is unchanged; the documentation now says how to recognise it.
+- `skills/fabric-exec/SKILL.md` states its job-scope precondition once instead of twice in consecutive sentences.
+
 ## 1.25.4
 
 ### Fixed

package/dist/fabric-exec-tool.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"fabric-exec-tool.d.ts","sourceRoot":"","sources":["../src/fabric-exec-tool.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAS,cAAc,EAA2B,MAAM,2BAA2B,CAAC;AAWhG,OAAO,KAAK,EAAE,mBAAmB,EAAE,MAAM,sBAAsB,CAAC;AAChE,OAAO,EACL,KAAK,wBAAwB,EAG9B,MAAM,4BAA4B,CAAC;AASpC,OAAO,KAAK,EAAE,WAAW,EAAE,MAAM,mBAAmB,CAAC;AASrD,OAAO,KAAK,EAAE,oBAAoB,EAAE,MAAM,sBAAsB,CAAC;AA8CjE,OAAO,KAAK,EAAE,2BAA2B,EAAE,MAAM,sBAAsB,CAAC;AA6HxE,eAAO,MAAM,oBAAoB,GAC/B,OAAO,WAAW,EAClB,qBAAqB,mBAAmB,EACxC,iBAAiB,GAAG,CAAC,MAAM,EAAE,oBAAoB,CAAC,EAClD,gBAAe,wBAA+C,EAC9D,cAAc,2BAA2B,KACxC,cAAc,CAAC,GAAG,EAAE,GAAG,CAs0BzB,CAAC"}
+{"version":3,"file":"fabric-exec-tool.d.ts","sourceRoot":"","sources":["../src/fabric-exec-tool.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAS,cAAc,EAA2B,MAAM,2BAA2B,CAAC;AAYhG,OAAO,KAAK,EAAE,mBAAmB,EAAE,MAAM,sBAAsB,CAAC;AAChE,OAAO,EACL,KAAK,wBAAwB,EAG9B,MAAM,4BAA4B,CAAC;AASpC,OAAO,KAAK,EAAE,WAAW,EAAE,MAAM,mBAAmB,CAAC;AASrD,OAAO,KAAK,EAAE,oBAAoB,EAAE,MAAM,sBAAsB,CAAC;AA8CjE,OAAO,KAAK,EAAE,2BAA2B,EAAE,MAAM,sBAAsB,CAAC;AA6HxE,eAAO,MAAM,oBAAoB,GAC/B,OAAO,WAAW,EAClB,qBAAqB,mBAAmB,EACxC,iBAAiB,GAAG,CAAC,MAAM,EAAE,oBAAoB,CAAC,EAClD,gBAAe,wBAA+C,EAC9D,cAAc,2BAA2B,KACxC,cAAc,CAAC,GAAG,EAAE,GAAG,CA20BzB,CAAC"}

package/dist/index.js

@@ -484,9 +484,9 @@ var withCodePreviewShell = (tool, options = {}) => {
     const renderOptions = { ...options2, renderContext: context };
     return originalRenderCall ? originalRenderCall.call(tool, args, renderOptions, theme) : new Text(theme.fg("toolTitle", theme.bold(tool.label || tool.name)), 0, 0);
   };
-  const renderResult = (result, resultOptions, theme, context) => {
+  const renderResult = (result, resultOptions, theme, context, args) => {
     const renderOptions = { ...resultOptions, renderContext: context };
-    return originalRenderResult ? originalRenderResult.call(tool, result, renderOptions, theme) : new Container();
+    return originalRenderResult ? originalRenderResult.call(tool, result, renderOptions, theme, args) : new Container();
   };
   return {
     ...tool,
@@ -537,7 +537,7 @@ var withCodePreviewShell = (tool, options = {}) => {
       state.codePreviewBorderTheme = theme;
       return shell;
     },
-    renderResult(result, resultOptions, theme) {
+    renderResult(result, resultOptions, theme, args) {
       const renderOptions = resultOptions ?? {};
       const supplied = renderOptions.renderContext && typeof renderOptions.renderContext === "object" ? renderOptions.renderContext : renderOptions;
       const context = buildRenderContext(
@@ -552,7 +552,7 @@ var withCodePreviewShell = (tool, options = {}) => {
         return renderTimedResult(
           context,
           theme,
-          (next) => renderResult(result, resultOptions, theme, next),
+          (next) => renderResult(result, resultOptions, theme, next, args),
           label
         );
       }
@@ -561,7 +561,7 @@ var withCodePreviewShell = (tool, options = {}) => {
       const component = timingOnly && state.codePreviewBorderResultComponent ? state.codePreviewBorderResultComponent : renderResult(result, resultOptions, theme, {
         ...context,
         lastComponent: state.codePreviewBorderResultComponent
-      });
+      }, args);
       state.codePreviewBorderResultComponent = component;
       const shell = state.codePreviewBorderShell instanceof BorderedToolCall && state.codePreviewBorderTheme === theme ? state.codePreviewBorderShell : new BorderedToolCall(theme, state);
       shell.setCall(state.codePreviewBorderCallComponent);
@@ -570,7 +570,7 @@ var withCodePreviewShell = (tool, options = {}) => {
       shell.setTimingLabel(label);
       state.codePreviewBorderShell = shell;
       state.codePreviewBorderTheme = theme;
-      return shouldRenderResultSeparately(state, optionsRecord.isPartial) ? component : new Container();
+      return shouldRenderResultSeparately(state, optionsRecord.isPartial) ? component : state.codePreviewBorderShell ?? component;
     }
   };
 };
@@ -3002,6 +3002,7 @@ var createFabricExecTool = (state, codePreviewSettings, pendingHandoffs, decorat
     name: "fabric_exec",
     label: "Fabric",
     readsSkillUris: true,
+    mergeCallAndResult: true,
     description: "Execute type-checked TypeScript through Fabric's configured executor for OMP tools, MCP, Fabric providers, discovery, and extensions. QuickJS is isolated by default; the optional Node/Bun process is an unsafe trusted-code escape hatch. In full code mode, and always in Schema enforce mode, this is the exclusive model tool path.",
     promptGuidelines: [
       "Batch independent operations in one `fabric_exec` program (`Promise.all` for parallel, sequential `await` for ordered), not one call per tool; keep dependent/conditional steps sequential. Coalesce non-dependent replacements from one file snapshot into one `omp.edit({path, edits:[...]})`; use `all:true` only for intentional repeated exact anchors. Return only the compact final value; intermediate results stay in the sandbox.",
@@ -3377,10 +3378,10 @@ ${preview}` : ""}${hiddenHint}`,
         const lines = safeTerminalText(output).split(nl);
         const limit = expanded ? Math.min(lines.length, 200) : 12;
         const shown = lines.slice(0, limit);
+        const displayName = normalizeRunDisplay(params?.display)?.name;
+        const title = displayName ? theme.fg("toolTitle", theme.bold(safeTerminalText(displayName))) : compact ? theme.fg(failed ? "error" : "success", failed ? "\u2717 Failed" : "\u2713 Evaluated") : "";
         let text2 = styleOutputLines(shown).join(nl);
-        if (compact) {
-          text2 = theme.fg(failed ? "error" : "success", failed ? "\u2717 Failed" : "\u2713 Evaluated") + nl + text2;
-        }
+        if (title) text2 = title + nl + text2;
         if (lines.length > shown.length) {
           text2 += nl + theme.fg("dim", `\u2026 ${countLabel(lines.length - shown.length, "line")}`);
           if (!expanded) text2 += theme.fg("dim", " \xB7 ") + expandHint(theme);