omp-fabric 1.24.2 → 1.25.1

package/CHANGELOG.md

@@ -1,5 +1,22 @@
 # Changelog
 
+## 1.25.1
+
+### Fixed
+
+- `MINIMUM_OMP_HOST_VERSION` raised to `18.3.0`. The startup host check still read `18.1.10`, so it stayed quiet on a host that cannot load fabric at all. The README and the certification host pin carried the same stale number.
+
+## 1.25.0
+
+### Fixed
+
+- Fabric runs again on OMP 18.3.x. The host replaced `Settings.instance.get("<path>")` with typed registry handles, so every fabric read through the old accessor threw `Settings.instance.get is not a function` — `fabric_exec` reported it on the first shell call, where `async.enabled` decides whether a guest command may background. Reads now go through `cfgAsyncEnabled`, `cfgSkillful`, `cfgHideThinkingBlock` and `cfgTerminalShowImages`.
+- `resolveJudge` no longer takes a `backend` key; fabric stopped sending one. The host removed the local-model judge selector, so the value was ignored while the call site still claimed it chose the chain.
+
+### Changed
+
+- Requires OMP 18.3.0 or newer (`@oh-my-pi/*` peer ranges and `MINIMUM_OMP_HOST_VERSION` raised from `18.1.10`). The registry settings modules do not exist on older hosts, so there is no version fabric can satisfy at once.
+
 ## 1.24.2
 
 ### Fixed

package/README.md

@@ -62,7 +62,7 @@ Independent calls run in parallel, and the returned object enters the model cont
 
 ## Install
 
-Requires Node.js 24+ and OMP 18.1.10+. Fabric also checks a detectable OMP host version at startup and warns when an older host may ignore continuation APIs such as actor `triggerTurn`.
+Requires Node.js 24+ and OMP 18.3.0+. Fabric also checks a detectable OMP host version at startup and warns when an older host may ignore continuation APIs such as actor `triggerTurn`.
 
 ```bash
 omp install npm:omp-fabric

package/dist/chunks/{chunk-NCPJZXLM.js → chunk-NOZBB266.js}

@@ -8,14 +8,18 @@ import { homedir } from "node:os";
 import path from "node:path";
 import { Ellipsis, truncateToWidth, visibleWidth } from "@oh-my-pi/pi-tui";
 async function readConversationAppearance(cwd, agentDir, projectTrusted) {
-  const { Settings } = await import("@oh-my-pi/pi-coding-agent/config/settings");
+  const [{ Settings }, { cfgHideThinkingBlock }, { cfgTerminalShowImages }] = await Promise.all([
+    import("@oh-my-pi/pi-coding-agent/config/settings"),
+    import("@oh-my-pi/pi-coding-agent/session/settings"),
+    import("@oh-my-pi/pi-coding-agent/modes/settings")
+  ]);
   const settings = projectTrusted ? await Settings.loadReadOnly({ cwd, agentDir }) : Settings.isolated();
   return {
     editorPaddingX: 0,
     outputPad: 0,
     codeBlockIndent: "  ",
-    hideThinkingBlock: settings.get("hideThinkingBlock") === true,
-    showImages: settings.get("terminal.showImages") === true
+    hideThinkingBlock: cfgHideThinkingBlock.get(settings) === true,
+    showImages: cfgTerminalShowImages.get(settings) === true
   };
 }
 var footerPath = (cwd) => {
@@ -54,4 +58,4 @@ export {
   readConversationAppearance,
   conversationFooter
 };
-//# sourceMappingURL=chunk-NCPJZXLM.js.map
+//# sourceMappingURL=chunk-NOZBB266.js.map

package/dist/chunks/chunk-NOZBB266.js.map

@@ -0,0 +1,7 @@
+{
+  "version": 3,
+  "sources": ["../../src/ui/conversation-chrome.ts"],
+  "sourcesContent": ["import { homedir } from \"node:os\";\nimport path from \"node:path\";\nimport type { Theme } from \"@oh-my-pi/pi-coding-agent\";\nimport { Ellipsis, truncateToWidth, visibleWidth } from \"@oh-my-pi/pi-tui\";\nimport { formatTokens, safeText } from \"./format.js\";\nimport type { FabricConversationTarget } from \"./conversation.js\";\n\nexport interface FabricConversationAppearance {\n  editorPaddingX: number;\n  outputPad: 0 | 1;\n  codeBlockIndent?: string;\n  hideThinkingBlock?: boolean;\n  showImages?: boolean;\n  imageWidthCells?: number;\n}\n\n/** Read OMP appearance settings; cwd and agentDir apply only to trusted project loading. */\nexport async function readConversationAppearance(cwd: string, agentDir: string, projectTrusted: boolean): Promise<FabricConversationAppearance> {\n  const [{ Settings }, { cfgHideThinkingBlock }, { cfgTerminalShowImages }] = await Promise.all([\n    import(\"@oh-my-pi/pi-coding-agent/config/settings\"),\n    import(\"@oh-my-pi/pi-coding-agent/session/settings\"),\n    import(\"@oh-my-pi/pi-coding-agent/modes/settings\"),\n  ]);\n  const settings = projectTrusted\n    ? await Settings.loadReadOnly({ cwd, agentDir })\n    : Settings.isolated();\n  return {\n    editorPaddingX: 0,\n    outputPad: 0,\n    codeBlockIndent: \"  \",\n    hideThinkingBlock: cfgHideThinkingBlock.get(settings) === true,\n    showImages: cfgTerminalShowImages.get(settings) === true,\n  };\n}\n\nconst footerPath = (cwd: string): string => {\n  const home = homedir();\n  if (cwd === home) return \"~\";\n  return cwd.startsWith(`${home}${path.sep}`) ? `~${cwd.slice(home.length)}` : cwd;\n};\n\n/** Native-shaped footer backed by this participant, never by Main's session stats. */\nexport function conversationFooter(target: FabricConversationTarget | undefined, theme: Theme, width: number): string[] {\n  if (!target || width <= 0) return [];\n  const location = target.cwd ? footerPath(target.cwd) : \"cwd unavailable\";\n  const branch = target.branch ? ` (${target.branch})` : \"\";\n  const pwd = safeText(`${location}${branch} \u2022 ${target.name}`);\n  const stats = [safeText(target.status)];\n  if (target.usage) {\n    const usage = target.usage;\n    // Agent usage is run-scoped; actor values describe the current/last activation.\n    stats.push(\"run\", `\u2191${formatTokens(usage.input)}`, `\u2193${formatTokens(usage.output)}`);\n    if (usage.cacheRead > 0) stats.push(`R${formatTokens(usage.cacheRead)}`);\n    if (usage.cacheWrite > 0) stats.push(`W${formatTokens(usage.cacheWrite)}`);\n    if (Number.isFinite(usage.cost)) stats.push(`$${usage.cost.toFixed(3)}`);\n  } else {\n    stats.push(\"usage unavailable\");\n  }\n  // Cumulative run tokens are not current context occupancy. Do not invent a percentage.\n  stats.push(target.contextWindow ? `?/${formatTokens(target.contextWindow)} ctx` : \"ctx ?\");\n  const model = safeText(target.model ?? `${target.runner ?? \"\"} model unavailable`).trim();\n  const thinking = target.thinking ? ` \u2022 ${safeText(target.thinking === \"off\" ? \"thinking off\" : target.thinking)}` : \"\";\n  const left = truncateToWidth(stats.join(\" \"), width, Ellipsis.Unicode);\n  const right = truncateToWidth(`${model}${thinking}`, Math.max(0, width - visibleWidth(left) - 2), \"\");\n  const gap = \" \".repeat(Math.max(0, width - visibleWidth(left) - visibleWidth(right)));\n  return [\n    truncateToWidth(theme.fg(\"dim\", pwd), width, Ellipsis.Unicode),\n    theme.fg(\"dim\", `${left}${gap}${right}`),\n  ];\n}\n"],
+  "mappings": ";;;;;;AAAA,SAAS,eAAe;AACxB,OAAO,UAAU;AAEjB,SAAS,UAAU,iBAAiB,oBAAoB;AAcxD,eAAsB,2BAA2B,KAAa,UAAkB,gBAAgE;AAC9I,QAAM,CAAC,EAAE,SAAS,GAAG,EAAE,qBAAqB,GAAG,EAAE,sBAAsB,CAAC,IAAI,MAAM,QAAQ,IAAI;AAAA,IAC5F,OAAO,2CAA2C;AAAA,IAClD,OAAO,4CAA4C;AAAA,IACnD,OAAO,0CAA0C;AAAA,EACnD,CAAC;AACD,QAAM,WAAW,iBACb,MAAM,SAAS,aAAa,EAAE,KAAK,SAAS,CAAC,IAC7C,SAAS,SAAS;AACtB,SAAO;AAAA,IACL,gBAAgB;AAAA,IAChB,WAAW;AAAA,IACX,iBAAiB;AAAA,IACjB,mBAAmB,qBAAqB,IAAI,QAAQ,MAAM;AAAA,IAC1D,YAAY,sBAAsB,IAAI,QAAQ,MAAM;AAAA,EACtD;AACF;AAEA,IAAM,aAAa,CAAC,QAAwB;AAC1C,QAAM,OAAO,QAAQ;AACrB,MAAI,QAAQ,KAAM,QAAO;AACzB,SAAO,IAAI,WAAW,GAAG,IAAI,GAAG,KAAK,GAAG,EAAE,IAAI,IAAI,IAAI,MAAM,KAAK,MAAM,CAAC,KAAK;AAC/E;AAGO,SAAS,mBAAmB,QAA8C,OAAc,OAAyB;AACtH,MAAI,CAAC,UAAU,SAAS,EAAG,QAAO,CAAC;AACnC,QAAM,WAAW,OAAO,MAAM,WAAW,OAAO,GAAG,IAAI;AACvD,QAAM,SAAS,OAAO,SAAS,KAAK,OAAO,MAAM,MAAM;AACvD,QAAM,MAAM,SAAS,GAAG,QAAQ,GAAG,MAAM,WAAM,OAAO,IAAI,EAAE;AAC5D,QAAM,QAAQ,CAAC,SAAS,OAAO,MAAM,CAAC;AACtC,MAAI,OAAO,OAAO;AAChB,UAAM,QAAQ,OAAO;AAErB,UAAM,KAAK,OAAO,SAAI,aAAa,MAAM,KAAK,CAAC,IAAI,SAAI,aAAa,MAAM,MAAM,CAAC,EAAE;AACnF,QAAI,MAAM,YAAY,EAAG,OAAM,KAAK,IAAI,aAAa,MAAM,SAAS,CAAC,EAAE;AACvE,QAAI,MAAM,aAAa,EAAG,OAAM,KAAK,IAAI,aAAa,MAAM,UAAU,CAAC,EAAE;AACzE,QAAI,OAAO,SAAS,MAAM,IAAI,EAAG,OAAM,KAAK,IAAI,MAAM,KAAK,QAAQ,CAAC,CAAC,EAAE;AAAA,EACzE,OAAO;AACL,UAAM,KAAK,mBAAmB;AAAA,EAChC;AAEA,QAAM,KAAK,OAAO,gBAAgB,KAAK,aAAa,OAAO,aAAa,CAAC,SAAS,OAAO;AACzF,QAAM,QAAQ,SAAS,OAAO,SAAS,GAAG,OAAO,UAAU,EAAE,oBAAoB,EAAE,KAAK;AACxF,QAAM,WAAW,OAAO,WAAW,WAAM,SAAS,OAAO,aAAa,QAAQ,iBAAiB,OAAO,QAAQ,CAAC,KAAK;AACpH,QAAM,OAAO,gBAAgB,MAAM,KAAK,GAAG,GAAG,OAAO,SAAS,OAAO;AACrE,QAAM,QAAQ,gBAAgB,GAAG,KAAK,GAAG,QAAQ,IAAI,KAAK,IAAI,GAAG,QAAQ,aAAa,IAAI,IAAI,CAAC,GAAG,EAAE;AACpG,QAAM,MAAM,IAAI,OAAO,KAAK,IAAI,GAAG,QAAQ,aAAa,IAAI,IAAI,aAAa,KAAK,CAAC,CAAC;AACpF,SAAO;AAAA,IACL,gBAAgB,MAAM,GAAG,OAAO,GAAG,GAAG,OAAO,SAAS,OAAO;AAAA,IAC7D,MAAM,GAAG,OAAO,GAAG,IAAI,GAAG,GAAG,GAAG,KAAK,EAAE;AAAA,EACzC;AACF;",
+  "names": []
+}

package/dist/core/skill-prompt.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"skill-prompt.d.ts","sourceRoot":"","sources":["../../src/core/skill-prompt.ts"],"names":[],"mappings":"AACA,OAAO,EAA6B,KAAK,KAAK,EAAE,MAAM,2BAA2B,CAAC;AAalF,eAAO,MAAM,YAAY,QAAO,SAAS,KAAK,EAAuB,CAAC;AAEtE,eAAO,MAAM,cAAc,GAAI,QAAQ,SAAS,KAAK,EAAE,KAAG,SAAS,KAAK,EAOvE,CAAC;AAEF,eAAO,MAAM,8BAA8B,GACzC,cAAc,MAAM,EACpB,QAAQ,SAAS,KAAK,EAAE,KACvB,MAYF,CAAC"}
+{"version":3,"file":"skill-prompt.d.ts","sourceRoot":"","sources":["../../src/core/skill-prompt.ts"],"names":[],"mappings":"AACA,OAAO,EAA6B,KAAK,KAAK,EAAE,MAAM,2BAA2B,CAAC;AAclF,eAAO,MAAM,YAAY,QAAO,SAAS,KAAK,EAAuB,CAAC;AAEtE,eAAO,MAAM,cAAc,GAAI,QAAQ,SAAS,KAAK,EAAE,KAAG,SAAS,KAAK,EAOvE,CAAC;AAEF,eAAO,MAAM,8BAA8B,GACzC,cAAc,MAAM,EACpB,QAAQ,SAAS,KAAK,EAAE,KACvB,MAYF,CAAC"}

package/dist/fabric-runtime-state.js

@@ -8857,7 +8857,6 @@ var FabricJudgmentLane = class {
 };
 
 // src/judgment/host-judge.ts
-var ONLINE_JUDGMENT_BACKEND = "online";
 var resolveHostJudge = async (context) => {
   if (context.modelRegistry === void 0) return void 0;
   const [judgment, settingsModule] = await Promise.all([
@@ -8870,7 +8869,6 @@ var resolveHostJudge = async (context) => {
   return resolveJudge({
     settings,
     registry: context.modelRegistry,
-    backend: ONLINE_JUDGMENT_BACKEND,
     ...context.model !== void 0 ? { sessionModel: context.model } : {},
     ...context.sessionId !== void 0 ? { sessionId: context.sessionId } : {}
   });
@@ -10659,6 +10657,7 @@ var createPreviewWriteToolDefinition = (cwd, session) => {
 // src/providers/omp-session-scope.ts
 import { AgentRegistry } from "@oh-my-pi/pi-coding-agent";
 import { isSettingsInitialized, Settings } from "@oh-my-pi/pi-coding-agent/config/settings";
+import { cfgAsyncEnabled } from "@oh-my-pi/pi-coding-agent/tools/settings";
 var liveOwner = (sessionId) => {
   if (!sessionId) return void 0;
   for (const ref of AgentRegistry.global().list()) {
@@ -10677,7 +10676,7 @@ var ompJobScope = (getSessionId) => {
     agentId: () => liveOwner(getSessionId())?.agentId ?? null
   };
 };
-var hostAsyncEnabled = () => isSettingsInitialized() ? Settings.instance.get("async.enabled") : true;
+var hostAsyncEnabled = () => isSettingsInitialized() ? cfgAsyncEnabled.get(Settings.instance) : true;
 var shellBackgroundSettings = (jobs, asyncEnabled = hostAsyncEnabled) => ({
   "async.enabled": jobs ? asyncEnabled() : false,
   "bash.autoBackground.enabled": false