pi-vision-handoff 0.10.6 → 0.10.8

package/README.md

@@ -16,6 +16,14 @@ _Describe images with a vision model you pick, then feed the text to models that
 
 ---
 
+## Pi 1.0 compatibility (0.10.8)
+
+Tested with Pi **1.0.0**. Host-provided Pi packages and TypeBox are peers (`*`), not bundled runtime dependencies; development uses exact Pi 1.0.0 pins and host-compatible TypeBox where needed.
+
+Vision calls use the native ModelRegistry streaming runtime (including native providers/auth/hooks). Fetch energy capture remains necessary for raw SSE comments, but cleanup now preserves later wrappers and retained predecessors cannot recurse after reinstall. Native nested read results are covered.
+
+Run `bun run test:host` for the offline real-host load, native codemode/nested-call, module-identity and reload checks. Set `PI1_HOST_PACKAGE` to an installed Pi package directory to test that host explicitly; add `PI1_HOST_ENTRY=bundle` to check the bundled CLI runtime's constructors.
+
 ## The Problem
 
 Some of the best coding models are blind. You paste a screenshot, a UI mock, a stack trace, or a diagram into pi — and a text-only model either silently ignores the image or rejects the request outright. Up to now your only options were to describe the image yourself, or switch to a (often weaker-for-coding) vision model just to read it.

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "pi-vision-handoff",
-  "version": "0.10.6",
+  "version": "0.10.8",
   "description": "Give text-only pi models vision — describe images with a vision model you pick via an interactive picker, then hand off the text description to non-vision models",
   "type": "module",
   "author": "Tom X Nguyen",
@@ -31,6 +31,7 @@
     "README.md"
   ],
   "scripts": {
+    "test:host": "node scripts/pi1-host.mjs",
     "test": "vitest run",
     "test:watch": "vitest",
     "test:coverage": "vitest run --coverage",
@@ -38,9 +39,9 @@
     "lint:dead": "knip --no-gitignore"
   },
   "devDependencies": {
-    "@earendil-works/pi-ai": "0.87.0",
-    "@earendil-works/pi-coding-agent": "0.87.0",
-    "@earendil-works/pi-tui": "0.87.0",
+    "@earendil-works/pi-ai": "1.0.0",
+    "@earendil-works/pi-coding-agent": "1.0.0",
+    "@earendil-works/pi-tui": "1.0.0",
     "@types/node": "25.9.1",
     "@vitest/coverage-v8": "4.1.7",
     "knip": "6.14.1",

package/src/describer.ts

@@ -178,6 +178,11 @@ export async function completeVisionModel(
   // here, then hand the same transcript to both the registered-provider stream and
   // the compat `completeSimple` fallback (which also accepts a TranscriptContext).
   const transcript: TranscriptContext = normalizeContext(context);
+  // Pi 1.0's runtime owns native providers, auth, environment and request hooks.
+  // Calling a legacy provider config directly bypasses those contracts.
+  if (typeof modelRegistry.streamSimple === "function") {
+    return modelRegistry.streamSimple(model, transcript, isolatedOptions).result();
+  }
   const provider = modelRegistry.getRegisteredProviderConfig?.(model.provider);
   if (provider?.streamSimple && provider.api === model.api) {
     return provider.streamSimple(model, transcript, isolatedOptions).result();

package/src/usage.ts

@@ -218,44 +218,40 @@ export interface DescribeContext {
 export const describeAls = new AsyncLocalStorage<DescribeContext>();
 
 let fetchInterceptRefCount = 0;
-let savedRealFetch: typeof globalThis.fetch | null = null;
+let installation: { previous: typeof globalThis.fetch; wrapper: typeof globalThis.fetch; active: boolean } | undefined;
 
-/** Install the globalThis.fetch interceptor. Refcounted: the first caller
- *  patches fetch; later callers just bump the count. Idempotent per install. */
+/** Refcounted, ownership-safe interception. Each wrapper closes over its own
+ * predecessor: another extension may retain it after our session shuts down. */
 export function installFetchInterceptor(): void {
   if (fetchInterceptRefCount === 0) {
-    savedRealFetch = globalThis.fetch;
-    globalThis.fetch = interceptedFetch;
+    const current = { previous: globalThis.fetch, wrapper: globalThis.fetch, active: true };
+    current.wrapper = (async (input: RequestInfo | URL, init?: RequestInit): Promise<Response> => {
+      const response = await current.previous(input, init);
+      const store = describeAls.getStore();
+      if (!current.active || !store || !response.body) return response;
+      const [bodyForSdk, bodyForEnergy] = response.body.tee();
+      store.energyReader = readEnergyFromTee(bodyForEnergy);
+      return new Response(bodyForSdk, {
+        headers: response.headers, status: response.status, statusText: response.statusText,
+      });
+    }) as typeof globalThis.fetch;
+    installation = current;
+    globalThis.fetch = current.wrapper;
   }
   fetchInterceptRefCount++;
 }
 
-/** Remove the interceptor. Refcounted: only the last caller restores fetch. */
 export function uninstallFetchInterceptor(): void {
   if (fetchInterceptRefCount > 0) fetchInterceptRefCount--;
-  if (fetchInterceptRefCount === 0 && savedRealFetch) {
-    globalThis.fetch = savedRealFetch;
-    savedRealFetch = null;
+  if (fetchInterceptRefCount === 0 && installation) {
+    installation.active = false;
+    // Never overwrite a wrapper installed by someone else after ours.
+    if (globalThis.fetch === installation.wrapper) globalThis.fetch = installation.previous;
+    installation = undefined;
   }
 }
 
-/** Current refcount — 0 means the interceptor is not installed. Test hook. */
+/** Current refcount — 0 means capture is inactive. Test hook. */
 export function fetchInterceptorRefcount(): number {
   return fetchInterceptRefCount;
 }
-
-async function interceptedFetch(input: RequestInfo | URL, init?: RequestInit): Promise<Response> {
-  const real = savedRealFetch ?? globalThis.fetch;
-  const response = await real(input, init);
-  const store = describeAls.getStore();
-  // Outside a describer call (no ALS store) or for bodiless responses: pass
-  // through untouched.
-  if (!store || !response.body) return response;
-  const [bodyForSdk, bodyForEnergy] = response.body.tee();
-  store.energyReader = readEnergyFromTee(bodyForEnergy);
-  return new Response(bodyForSdk, {
-    headers: response.headers,
-    status: response.status,
-    statusText: response.statusText,
-  });
-}