pi-supernova 0.0.3 → 0.0.4

package/CHANGELOG.md

@@ -6,6 +6,12 @@
 
 - Native adapters are now included in `nova.search` / `nova.describe` with schemas matching the callable adapter, even when the host catalog omits parameters or the adapter name.
 
+## [0.0.4] - 2026-09-03
+
+### Fixed
+
+- OMP no longer hangs on "Loading plugins…" — removed top-level import of `@oh-my-pi/pi-coding-agent/tui` from the extension (portable framed chrome only).
+
 ## [0.0.3] - 2026-09-03
 
 ### Changed

package/index.js

@@ -1,16 +1,4 @@
-
-let Type;
-try {
-  Type = (await import("typebox")).Type;
-} catch {
-  Type = {
-    Object: (props, opts) => ({ type: "object", properties: props || {}, additionalProperties: false, ...opts }),
-    String: (opts) => ({ type: "string", ...opts }),
-    Integer: (opts) => ({ type: "integer", ...opts }),
-    Optional: (s) => ({ ...s }),
-  };
-}
-
+import { createRequire } from "node:module";
 import { isString, isFunction } from "./decode.js";
 import { buildCatalog, searchCatalog, describeTool, mergeNativeToolDefinitions } from "./catalog.js";
 import { loadConfig } from "./config.js";
@@ -25,6 +13,20 @@ import {
 
 export { extractOperationsFromCode, renderSupernovaCall, renderSupernovaResult, SafeText };
 
+// Sync only — never top-level await. Dynamic import of host/deps hung OMP plugin load.
+const require = createRequire(import.meta.url);
+let Type;
+try {
+  Type = require("typebox").Type;
+} catch {
+  Type = {
+    Object: (props, opts) => ({ type: "object", properties: props || {}, additionalProperties: false, ...opts }),
+    String: (opts) => ({ type: "string", ...opts }),
+    Integer: (opts) => ({ type: "integer", ...opts }),
+    Optional: (s) => ({ ...s }),
+  };
+}
+
 function result(text, details) {
   return { content: [{ type: "text", text }], details };
 }

package/omp-frame.js

@@ -1,10 +1,8 @@
 /**
- * OMP-native tool chrome for supernova.
+ * OMP-style rounded tool chrome for supernova.
  *
- * Prefer host `framedBlock` + `renderStatusLine` from
- * `@oh-my-pi/pi-coding-agent/tui` (same path read/write/edit use). Fall back to a
- * portable rounded frame that matches that look when the import is unavailable
- * (Pi path-install / unit tests).
+ * Intentionally self-contained — never dynamic-imports `@oh-my-pi/pi-coding-agent`
+ * (that hung OMP plugin load). Geometry matches native write/edit framedBlock.
  */
 
 import { clampLine, measureWidth, wrapPlainToWidth } from "./render-measure.js";
@@ -20,32 +18,12 @@ const DEFAULT_BOX = {
 	teeRight: "├",
 };
 
-let hostFramedBlock = null;
-let hostRenderStatusLine = null;
-let hostMarkFramed = null;
-let hostResolved = false;
-
-export async function ensureOmpChrome() {
-	if (hostResolved) return;
-	hostResolved = true;
-	for (const spec of ["@oh-my-pi/pi-coding-agent/tui", "@oh-my-pi/pi-coding-agent/tui/index.js"]) {
-		try {
-			const tui = await import(spec);
-			if (typeof tui.framedBlock === "function") hostFramedBlock = tui.framedBlock;
-			if (typeof tui.renderStatusLine === "function") hostRenderStatusLine = tui.renderStatusLine;
-			if (typeof tui.markFramedBlockComponent === "function") hostMarkFramed = tui.markFramedBlockComponent;
-			if (hostFramedBlock && hostRenderStatusLine) return;
-		} catch {
-			// Pi / tests / plain node — use portable frame.
-		}
-	}
+export function hasHostFramedBlock() {
+	return false;
 }
 
-/** Kick off resolution at module load (extension hosts support top-level await). */
-await ensureOmpChrome();
-
-export function hasHostFramedBlock() {
-	return typeof hostFramedBlock === "function" && typeof hostRenderStatusLine === "function";
+export async function ensureOmpChrome() {
+	// No-op: portable frame only. Kept so call sites stay stable.
 }
 
 function boxOf(theme) {
@@ -69,8 +47,6 @@ function borderPaint(theme, state, borderColor) {
 }
 
 function statusHeader(theme, { title, description, state, spinnerFrame, icon, iconOverride }) {
-	// Match native Write/Edit: framed call heads omit the pending hourglass; only
-	// pass an icon when the caller sets one (error / running / success glyph).
 	const resolvedIcon =
 		iconOverride !== undefined
 			? undefined
@@ -79,21 +55,12 @@ function statusHeader(theme, { title, description, state, spinnerFrame, icon, ic
 				: state === "error"
 					? "error"
 					: undefined;
-	if (hostRenderStatusLine) {
-		return hostRenderStatusLine(
-			{
-				icon: resolvedIcon,
-				iconOverride,
-				spinnerFrame,
-				title,
-				description,
-			},
-			theme,
-		);
-	}
 	const titleText = theme?.fg ? theme.fg("accent", title) : title;
 	const descText = description ? (theme?.fg ? theme.fg("muted", description) : description) : "";
-	const prefix = state === "error" ? (theme?.fg ? theme.fg("error", "✗ ") : "✗ ") : "";
+	let prefix = "";
+	if (iconOverride) prefix = `${iconOverride} `;
+	else if (resolvedIcon === "error") prefix = theme?.fg ? theme.fg("error", "✗ ") : "✗ ";
+	else if (resolvedIcon === "running" || spinnerFrame) prefix = theme?.fg ? theme.fg("dim", "… ") : "… ";
 	return descText ? `${prefix}${titleText}: ${descText}` : `${prefix}${titleText}`;
 }
 
@@ -135,9 +102,6 @@ function bgFnForState(theme, state) {
 	return undefined;
 }
 
-/**
- * Portable rounded frame matching OMP output-block geometry.
- */
 export function renderPortableFrame(theme, { header, sections = [], state = "pending", borderColor, width }) {
 	const w = Math.max(8, width | 0);
 	const box = boxOf(theme);
@@ -161,7 +125,7 @@ export function renderPortableFrame(theme, { header, sections = [], state = "pen
 		return padLine(`${border(left)}${trimmed}${border(h.repeat(fill))}${border(right)}`, w, bgFn);
 	};
 
-	const contentWidth = Math.max(1, w - 2 - 2); // borders + 1-col pad each side
+	const contentWidth = Math.max(1, w - 2 - 2);
 	const lines = [];
 	lines.push(paintBar(box.topLeft, box.topRight, header));
 
@@ -188,7 +152,7 @@ export function createPortableFramedComponent(theme, build) {
 	let cacheWidth;
 	let cacheKey;
 	let cacheLines;
-	const comp = {
+	return {
 		render(width) {
 			const opts = build(width);
 			const key = `${opts.state}|${opts.borderColor}|${opts.header}|${(opts.sections || [])
@@ -206,17 +170,9 @@ export function createPortableFramedComponent(theme, build) {
 			cacheWidth = undefined;
 		},
 	};
-	if (hostMarkFramed) return hostMarkFramed(comp);
-	return comp;
 }
 
-/**
- * Build a framed nova card. Uses host framedBlock when available (OMP).
- */
 export function novaFramedBlock(theme, build) {
-	if (hostFramedBlock) {
-		return hostFramedBlock(theme, build);
-	}
 	return createPortableFramedComponent(theme, build);
 }
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "pi-supernova",
-  "version": "0.0.3",
+  "version": "0.0.4",
   "description": "Dual-host CodeMode for Pi/OMP: progressive tool discovery, result bottleneck, and Amdahl Auto parallel.",
   "type": "module",
   "author": "AdityaVG13",

package/render.js

@@ -19,7 +19,7 @@ import {
 	wrapPlainToWidth,
 	fitPath,
 } from "./render-measure.js";
-import { ensureOmpChrome, hasHostFramedBlock, novaFramedBlock, novaStatusLine } from "./omp-frame.js";
+import { novaFramedBlock, novaStatusLine } from "./omp-frame.js";
 
 export { measureWidth, hardTruncate, clampLine, wrapPlainToWidth, fitPath };
 
@@ -348,7 +348,6 @@ function detectResultHost(options, ctxOrArgs) {
 	) {
 		return "omp";
 	}
-	if (hasHostFramedBlock()) return "omp";
 	return "pi";
 }
 
@@ -445,7 +444,7 @@ function formatOpBodyLine(theme, op) {
 }
 
 function shouldUseOmpFrame(host) {
-	return host === "omp" || hasHostFramedBlock();
+	return host === "omp";
 }
 
 /**
@@ -529,7 +528,6 @@ export function renderSupernovaCall(a, b, c) {
 	const timeStr = formatElapsed(context?.state);
 
 	if (shouldUseOmpFrame(host)) {
-		void ensureOmpChrome();
 		return renderOmpCallCard(theme, {
 			ops,
 			timeStr,
@@ -642,7 +640,6 @@ export function renderSupernovaResult(resultArg, optionsArg, themeArg, contextAr
 	const useOmp = shouldUseOmpFrame(host);
 
 	if (useOmp) {
-		void ensureOmpChrome();
 		if (isPartial && !isErr) {
 			// Streaming partials stay quiet until body content exists — same as Pi.
 			const partialBody = buildResultBody(theme, { payload, context, expanded });