omp-fabric 1.25.25 → 1.25.27

package/CHANGELOG.md

@@ -1,5 +1,27 @@
 # Changelog
 
+## 1.25.27
+
+### Fixed
+
+- Correction to the 1.25.26 entry: it says its host pins moved from 18.10.1, but the working tree was already on 18.10.4 (shipped in 1.25.25). The range was 18.10.4 to 18.11.0.
+- **Session start no longer blocks on the LCM ledger's size.** `LcmLedger` construction ran `migrateSchema()` and `PRAGMA integrity_check` — both O(file size) — inline, so every `session_start` handler paid them: 5929ms, 9210ms, 17393ms and 22245ms across measured runs against this project's 611MB ledger (the ledger directory holds several 1.5-2.4GB files), inside the host's 30000ms per-handler budget. The host log has the receipts: `Extension handler timed out` for `event: "session_start"`, 17 times on 2026-10-15 alone, including one at 11:12:57 with 1.25.26 already installed. The FTS table and trigger DDL stays synchronous, because it is cheap and `searchRaw` needs those objects to exist; migrations, their backfill and the integrity check moved to `LcmLedger.runStartupChecks()`, which the detached session warmup runs after a macrotask yield. Compaction is unaffected: the compaction hook's `getEngine()` reports `omp` until the runtime is `ready`, so a cold session hands compaction to the host core instead of summarising through an unverified ledger.
+- Same phase marks before and after: `session_start` returned after 5929-22245ms before and 672ms after (compat 155ms, bootstrap 3ms, LCM prepare 513ms, activation 0ms). The deferred migrations and integrity check still cost about 2s on the 611MB ledger, now on a background macrotask outside the handler.
+- `tests/storage/lcm-ledger.test.ts` now drives `runStartupChecks()` explicitly for the five migration cases (orphan quarantine, FTS backfill, fts5-less fallback, index re-sync, purge counts). The contracts are unchanged; only their timing moved.
+
+## 1.25.26
+
+### Changed
+
+- Host pins move from 18.10.1 to 18.11.0 (dependencies pi-agent-core, pi-ai, pi-utils, omptype; dev pi-coding-agent, pi-natives, pi-tui); the declared peer floor stays >=18.4.4. Nothing fabric imports changed: `formatApprovalDetails` lost its optional options parameter and `truncateApprovalDetail`/`ApprovalDetailsOptions` are gone, but fabric calls none of them, the grep per-file caps stay 20/200, and every wrapped tool's truncation contract is unchanged. bun run check green at 254 files / 3256 tests, certify:context 28/28 checks against the new host.
+- Extension startup stops waiting on avoidable work. The Shiki language catalog (measured at 1263ms to import) now loads on demand instead of at module load, and registered-tool capture installs detached from the extension factory, whose body measures 5-10ms either way. Host readyMs did not move measurably at N=8 in either run order (old 6400ms vs new 6900ms; single runs 4.4-12.7s), so this is a tail-risk reduction rather than a speedup: the one sample above the host's 30000ms handler budget (34.3s) happened on the old bundle.
+- `session_start` awaits the LCM runtime before activation and leaves ledger reconcile, sync and sweep detached behind the session epoch. The order is load-bearing: the memory provider samples the LCM context once at activation, so the runtime must exist first.
+
+### Fixed
+
+- Compaction is no longer cancelled before the LCM runtime is up: `getEngine()` reported `lcm` with no runtime behind it, and the hook cancels a compaction it cannot summarize. It now reports `omp` (the host's own engine) until the runtime loads. Covered by tests/compaction-cold-start.test.ts.
+- The memory provider is installed with its LCM context present. Covered by tests/lcm-memory-context.test.ts, which fails if the runtime is prepared after activation.
+
 ## 1.25.25
 
 ### Changed

package/dist/chunks/{chunk-7NWMFYD3.js → chunk-5U6HKMJV.js}

@@ -18,9 +18,21 @@ import {
 // src/ui/highlight.ts
 import { readFileSync, statSync } from "node:fs";
 import { basename, extname } from "node:path";
-import { bundledLanguages } from "shiki/langs";
 import { bundledThemesInfo } from "shiki/themes";
 
+// src/ui/shiki-languages.ts
+var bundledLanguageIds;
+var loadingLanguageIds;
+var languageIdsLoaded = () => bundledLanguageIds;
+var loadLanguageIds = () => {
+  if (bundledLanguageIds) return Promise.resolve(bundledLanguageIds);
+  loadingLanguageIds ??= import("shiki/langs").then((module) => {
+    bundledLanguageIds = new Set(Object.keys(module.bundledLanguages));
+    return bundledLanguageIds;
+  });
+  return loadingLanguageIds;
+};
+
 // src/ui/shiki-theme.ts
 var THEME_IMPORTS = {
   "andromeeda": () => import("@shikijs/themes/andromeeda"),
@@ -341,18 +353,20 @@ var effectiveShikiThemeIsLight = () => THEME_TYPE.get(currentTheme) === "light";
 var observedThemeVariant = () => observedVariant;
 function languageFromPath(filePath) {
   if (!filePath) return void 0;
+  const catalog = languageIdsLoaded();
+  const supported = (id) => catalog === void 0 || catalog.has(id);
   const name = basename(filePath).toLowerCase();
   if (name.startsWith(".env")) {
     const candidate = "dotenv";
-    return candidate in bundledLanguages ? candidate : void 0;
+    return supported(candidate) ? candidate : void 0;
   }
   if (name === "dockerfile" || name.startsWith("dockerfile.")) {
-    return "dockerfile" in bundledLanguages ? "dockerfile" : void 0;
+    return supported("dockerfile") ? "dockerfile" : void 0;
   }
   const exact = EXACT_BASENAMES.get(name);
-  if (exact && exact in bundledLanguages) return exact;
+  if (exact && supported(exact)) return exact;
   const byExt = EXTENSION_ALIASES.get(extname(name));
-  return byExt && byExt in bundledLanguages ? byExt : void 0;
+  return byExt && supported(byExt) ? byExt : void 0;
 }
 function configureHighlighting(themePreferenceValue, syntaxEnabled = true) {
   const preference = themePreferenceValue.trim() || "auto";
@@ -376,6 +390,7 @@ function configureHighlighting(themePreferenceValue, syntaxEnabled = true) {
     resetFileHighlighting();
     return;
   }
+  void loadLanguageIds();
   const themeChanged = syncEffectiveTheme(preference, observedVariant);
   if (!themeChanged && !wasEnabled && !highlighter && !initializingTheme) {
     void initHighlighting(currentTheme, syntaxEnabled);
@@ -388,6 +403,7 @@ async function initHighlighting(theme, syntaxEnabled = true) {
   const version = ++initVersion;
   initializingTheme = theme;
   try {
+    await loadLanguageIds();
     const { createHighlighter } = await import("shiki");
     const themeObject = await resolveShikiThemeObject(theme);
     if (!themeObject) {
@@ -527,7 +543,12 @@ function highlightCode(text, lang, invalidate) {
     return null;
   }
   const shikiLang = normalizeLanguage(lang);
-  if (!(shikiLang in bundledLanguages)) return null;
+  const catalog = languageIdsLoaded();
+  if (!catalog) {
+    requestInit(invalidate);
+    return null;
+  }
+  if (!catalog.has(shikiLang)) return null;
   const cacheKey = `${currentTheme}\0${shikiLang}\0${text.length}\0${hashString(text)}`;
   const cached = renderCache.get(cacheKey);
   if (cached) {
@@ -667,7 +688,12 @@ function highlightFileLines(filePath, lang, from, to, invalidate) {
     return null;
   }
   const shikiLang = normalizeLanguage(lang);
-  if (!(shikiLang in bundledLanguages)) return null;
+  const catalog = languageIdsLoaded();
+  if (!catalog) {
+    requestInit(invalidate);
+    return null;
+  }
+  if (!catalog.has(shikiLang)) return null;
   if (!loadedLanguages.has(shikiLang)) {
     requestLanguageLoad(shikiLang, invalidate);
     return null;
@@ -727,7 +753,12 @@ function highlightSourceLines(cacheKey, sourceLines, lang, from, to, invalidate)
     return null;
   }
   const shikiLang = normalizeLanguage(lang);
-  if (!(shikiLang in bundledLanguages)) return null;
+  const catalog = languageIdsLoaded();
+  if (!catalog) {
+    requestInit(invalidate);
+    return null;
+  }
+  if (!catalog.has(shikiLang)) return null;
   if (!loadedLanguages.has(shikiLang)) {
     requestLanguageLoad(shikiLang, invalidate);
     return null;
@@ -941,7 +972,6 @@ var formatToolCallDuration = (startedAt, endedAt) => {
 
 // src/ui/core-tool-render.ts
 import { diffLines } from "diff";
-import { bundledLanguages as bundledLanguages2 } from "shiki/langs";
 
 // src/ui/diff-background.ts
 import { truncateToWidth, visibleWidth } from "@oh-my-pi/pi-tui";
@@ -2921,7 +2951,8 @@ var resolveLanguage = (filePath, content) => {
       php: "php"
     };
     const language = executable ? shebang[executable] : void 0;
-    if (language && language in bundledLanguages2) return language;
+    const catalog = languageIdsLoaded();
+    if (language && (catalog === void 0 || catalog.has(language))) return language;
   }
   if (!content || content.length > CONTENT_LANGUAGE_DETECTION_CHARS) return void 0;
   const trimmed = content.trim();
@@ -5083,4 +5114,4 @@ export {
   formatJsonAsYaml,
   formatFabricValue
 };
-//# sourceMappingURL=chunk-7NWMFYD3.js.map
+//# sourceMappingURL=chunk-5U6HKMJV.js.map