pi-hypercharm-provider 1.3.35 → 1.3.36

package/CHANGELOG.md

@@ -1,5 +1,12 @@
 # Changelog
 
+## 1.3.36
+
+- Pin Pi SDK development dependencies to 1.0.0 while retaining wildcard host peers.
+- Verify real manifest loading, provider catalogs, startup/shutdown and native/bundled Pi hosts offline.
+- Exercise real transport adapters with Unicode text, tool calls, empty responses, usage, request hooks and cancellation; no live provider calls.
+- Make deprecated-catalog integration fixtures time-independent and emit shutdown before disposing test sessions.
+
 ## 1.3.35
 
 - Validate against Pi 0.99.0, including an offline real-host package-loading probe.

package/README.md

@@ -11,6 +11,11 @@ _Hyperoptimized coding models — DeepSeek, GLM, Kimi, Qwen, MiniMax, Gemma, GPT
 
 </div>
 
+## Pi 1.0 compatibility
+
+Tested with Pi **1.0.0**. Host-provided packages remain wildcard peers; development uses exact SDK pins.
+Run `bun run test:pi` for offline manifest, catalog, lifecycle and streaming checks. To test an installed host, set `PI1_HOST_PACKAGE` to its package directory; add `PI1_HOST_ENTRY=bundle` for its bundled CLI runtime. The probe stubs all network requests and never uses a live provider endpoint.
+
 ---
 
 ## Features

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "pi-hypercharm-provider",
-  "version": "1.3.35",
+  "version": "1.3.36",
   "description": "HyperCharm provider extension for pi - Access DeepSeek, GLM, Kimi, Qwen, MiniMax, Gemma, and GPT-OSS models through the Charm Hyper API",
   "type": "module",
   "main": "index.ts",
@@ -36,9 +36,9 @@
     "@earendil-works/pi-coding-agent": "*"
   },
   "devDependencies": {
-    "@earendil-works/pi-ai": "0.99.0",
-    "@earendil-works/pi-coding-agent": "0.99.0",
-    "@earendil-works/pi-tui": "0.99.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": "^24.10.1",
     "jiti": "2.7.0",
     "typescript": "6.0.3"

package/scripts/test-pi-compat.mjs

@@ -1,23 +1,30 @@
+// Offline Pi 1.0 manifest, lifecycle, catalog and real transport regression.
 import assert from 'node:assert/strict';
 import { mkdtemp, readFile, rm } from 'node:fs/promises';
 import { tmpdir } from 'node:os';
 import { dirname, join, resolve } from 'node:path';
-import { fileURLToPath } from 'node:url';
+import { fileURLToPath, pathToFileURL } from 'node:url';
 
 const root = resolve(dirname(fileURLToPath(import.meta.url)), '..');
-const home = await mkdtemp(join(tmpdir(), 'pi-extension-compat-'));
+const home = await mkdtemp(join(tmpdir(), 'pi-provider-compat-'));
 const previousHome = process.env.PI_CODING_AGENT_DIR;
 const previousFetch = globalThis.fetch;
 process.env.PI_CODING_AGENT_DIR = home;
+// Never allow a background catalog/account request to reach a real endpoint.
 globalThis.fetch = async () => new Response('', { status: 503 });
 let session;
 try {
-  const { createAgentSession, DefaultResourceLoader, ModelRuntime, SessionManager, SettingsManager, VERSION } = await import('@earendil-works/pi-coding-agent');
-  assert.equal(VERSION, '0.99.0', 'test the actual pinned Pi host, not a stale override');
+  const host = process.env.PI1_HOST_PACKAGE;
+  const hostEntry = process.env.PI1_HOST_ENTRY === 'bundle' ? 'dist/bundle/index.js' : 'dist/index.js';
+  const sdk = await import(host ? pathToFileURL(join(host, hostEntry)).href : '@earendil-works/pi-coding-agent');
+  const { createAgentSession, DefaultResourceLoader, ModelRuntime, SessionManager, SettingsManager, VERSION } = sdk;
+  assert.equal(VERSION, '1.0.0', 'executing host version');
+  assert.equal((await import('@earendil-works/pi-coding-agent')).VERSION, '1.0.0', 'development host version');
   const manifest = JSON.parse(await readFile(join(root, 'package.json'), 'utf8'));
   for (const name of ['@earendil-works/pi-ai', '@earendil-works/pi-agent-core', '@earendil-works/pi-coding-agent', '@earendil-works/pi-tui', 'typebox']) {
-    assert.equal(manifest.dependencies?.[name], undefined, `${name}: host packages must not be runtime dependencies`);
+    assert.equal(manifest.dependencies?.[name], undefined, `${name}: do not bundle host packages`);
     if (manifest.peerDependencies?.[name] !== undefined) assert.equal(manifest.peerDependencies[name], '*');
+    if (name !== 'typebox' && manifest.devDependencies?.[name]) assert.equal(manifest.devDependencies[name], '1.0.0');
   }
   const settingsManager = SettingsManager.inMemory({ packages: [root], compaction: { enabled: false }, retry: { enabled: false } });
   const resourceLoader = new DefaultResourceLoader({ cwd: home, agentDir: home, settingsManager,
@@ -26,25 +33,83 @@ try {
   const loaded = resourceLoader.getExtensions();
   assert.deepEqual(loaded.errors, []);
   assert.deepEqual(loaded.warnings ?? [], []);
-  assert.ok(loaded.extensions.length > 0, 'manifest entrypoints must load');
+  assert.equal(loaded.extensions.length, manifest.pi.extensions.length, 'all manifest entrypoints load');
+  // Endpoint/header-only re-registration (e.g. MCR) retains the prior catalog.
+  const registrations = [...loaded.runtime.pendingProviderRegistrations.reduce((byName, { name, config }) =>
+    byName.set(name, { name, config: { ...byName.get(name)?.config, ...config } }), new Map()).values()];
+  assert(registrations.length > 0, 'providers register before session startup');
   const modelRuntime = await ModelRuntime.create({ authPath: join(home, 'auth.json'), modelsPath: null, modelsStorePath: join(home, 'models-cache'), allowModelNetwork: false });
   ({ session } = await createAgentSession({ cwd: home, agentDir: home, resourceLoader, modelRuntime, settingsManager, sessionManager: SessionManager.inMemory(home) }));
-  const names = new Set();
-  for (const extension of loaded.extensions) {
-    for (const [name, { definition }] of extension.tools) {
-      assert.equal(definition.name, name);
-      assert.equal(typeof definition.execute, 'function');
-      assert.equal(typeof definition.parameters, 'object');
-      assert.ok(!names.has(name), `duplicate tool ${name}`);
-      names.add(name);
-      assert.ok(session.getAllTools().some(tool => tool.name === name), `${name}: tool must be installed in the real session`);
+  const errors = [];
+  await session.bindExtensions({ mode: 'print', onError: error => errors.push(error) });
+  await new Promise(done => setImmediate(done));
+  const tools = new Set();
+  for (const extension of loaded.extensions) for (const [name, { definition }] of extension.tools) {
+    assert.equal(definition.name, name); assert.equal(typeof definition.execute, 'function');
+    assert.equal(typeof definition.parameters, 'object'); assert(!tools.has(name)); tools.add(name);
+    assert(session.getAllTools().some(tool => tool.name === name));
+  }
+  let modelCount = 0, streamChecks = 0;
+  for (const { name, config } of registrations) {
+    const models = modelRuntime.getAllModels(name);
+    assert.equal(models.length, config.models.length, `${name}: catalog survives registration`);
+    assert(models.length > 0);
+    assert.equal(new Set(models.map(m => `${m.type}/${m.id}`)).size, models.length);
+    for (const model of models) {
+      assert.equal(model.provider, name); assert(model.api && model.baseUrl && model.id && model.name);
+      if (model.type === 'image') assert(model.output.includes('image'));
+      else assert(model.contextWindow > 0);
+      if ((model.type ?? 'chat') === 'chat') assert(model.maxTokens > 0);
+      assert(model.input.includes('text'));
+      for (const cost of Object.values(model.cost)) assert(Number.isFinite(cost) && cost >= 0);
+    }
+    const chatModels = modelRuntime.getModels(name);
+    const model = chatModels.find(m => m.api === 'openai-completions') ?? chatModels[0];
+    assert(model, 'a chat model is available for offline transport checks');
+    const user = { role: 'user', content: 'hello', timestamp: 1 };
+    async function probe(tool = false, empty = false) {
+      let payloadCalls = 0, responseCalls = 0, observed = 0, wire;
+      const fetch = async (_url, init) => {
+        wire = JSON.parse(init.body);
+        const delta = tool ? { tool_calls: [{ index: 0, id: 'call_probe', type: 'function', function: { name: 'probe', arguments: '{"value":7}' } }] } : { content: empty ? '' : 'Hello 界' };
+        const chunks = [{ id: 'offline', choices: [{ index: 0, delta: { role: 'assistant', ...delta }, finish_reason: null }] }, { id: 'offline', choices: [{ index: 0, delta: {}, finish_reason: tool ? 'tool_calls' : 'stop' }], usage: { prompt_tokens: 10, completion_tokens: 2, total_tokens: 12 } }];
+        return new Response(chunks.map(c => `data: ${JSON.stringify(c)}\n\n`).join('') + 'data: [DONE]\n\n', { headers: { 'content-type': 'text/event-stream' } });
+      };
+      const stream = modelRuntime.streamSimple(model, { messages: [{ role: 'system', content: 'Offline probe', toolsAdded: [{ name: 'probe', description: 'Probe', parameters: { type: 'object', properties: { value: { type: 'integer' } }, required: ['value'] } }], timestamp: 0 }, user] }, {
+        apiKey: 'offline-placeholder', maxTokens: 32, maxRetries: 0, fetch,
+        onPayload: p => { payloadCalls++; return { ...p, temperature: 0.123 }; },
+        onResponse: () => { responseCalls++; }, onProviderStreamEvent: () => { observed++; },
+      });
+      const events = []; for await (const event of stream) events.push(event.type);
+      const result = await stream.result();
+      assert.equal(result.stopReason, tool ? 'toolUse' : 'stop', result.errorMessage);
+      assert.equal(events.filter(e => e === 'start').length, 1);
+      assert.equal(events.filter(e => e === 'done' || e === 'error').length, 1);
+      assert.equal(events.at(-1), 'done');
+      assert.equal(payloadCalls, 1); assert.equal(responseCalls, 1); assert(observed >= 2);
+      assert.equal(wire.temperature, 0.123); assert.equal(wire.tools[0].function.name, 'probe');
+      assert.equal(result.usage.totalTokens, 12);
+      if (tool) assert.deepEqual(result.content.find(c => c.type === 'toolCall').arguments, { value: 7 });
+      else if (!empty) assert.equal(result.content.find(c => c.type === 'text').text, 'Hello 界');
+      streamChecks++;
     }
+    await probe(); await probe(true); await probe(false, true);
+    // Abort after preflight: the SDK's lazy preflight reports an already-aborted
+    // auth operation as an error; this checks provider transport cancellation.
+    const controller = new AbortController();
+    const aborted = await modelRuntime.streamSimple(model, { messages: [user] }, {
+      apiKey: 'offline-placeholder', signal: controller.signal, maxRetries: 0,
+      onPayload: () => { controller.abort(); },
+      fetch: async () => { throw new DOMException('Aborted', 'AbortError'); },
+    }).result();
+    assert.equal(aborted.stopReason, 'aborted', aborted.errorMessage); streamChecks++;
+    modelCount += models.length;
   }
-  console.log(`${manifest.name}: Pi ${VERSION} warning-free manifest load; ${loaded.extensions.length} extensions, ${names.size} tools registered`);
+  await session.extensionRunner.emit({ type: 'session_shutdown', reason: 'quit' });
+  assert.deepEqual(errors, []);
+  console.log(JSON.stringify({ repo: manifest.name, pi: VERSION, hostEntry, extensions: loaded.extensions.length, tools: tools.size, models: modelCount, streamChecks, lifecycle: 'passed' }));
 } finally {
-  session?.dispose();
-  globalThis.fetch = previousFetch;
-  if (previousHome === undefined) delete process.env.PI_CODING_AGENT_DIR;
-  else process.env.PI_CODING_AGENT_DIR = previousHome;
+  session?.dispose(); globalThis.fetch = previousFetch;
+  if (previousHome === undefined) delete process.env.PI_CODING_AGENT_DIR; else process.env.PI_CODING_AGENT_DIR = previousHome;
   await rm(home, { recursive: true, force: true });
 }

package/tests/provider.integration.test.ts

@@ -144,7 +144,12 @@ async function load(options = {}) {
 		noThemes: true,
 		noContextFiles: true,
 	});
-	await loader.reload();
+	// Pin only catalog loading, not timeout clocks or subsequent provider work.
+	const now = Date.now;
+	try {
+		if (options.catalogNow !== undefined) Date.now = () => options.catalogNow;
+		await loader.reload();
+	} finally { Date.now = now; }
 	assert.deepEqual(loader.getExtensions().errors, [], "extension must load without errors");
 
 	const sessionManager = options.sessionFile
@@ -163,7 +168,10 @@ async function load(options = {}) {
 	// the extension session path is part of what this suite verifies.
 	const runner = session.extensionRunner;
 	await runner.emit({ type: "session_start", reason: "startup" });
-	return { credentials, runtime, loader, session, sessionManager, runner };
+	return { credentials, runtime, loader, session, sessionManager, runner, async shutdown() {
+		try { await runner.emit({ type: "session_shutdown", reason: "quit" }); }
+		finally { session.dispose(); }
+	} };
 }
 
 function assistantMessage(overrides = {}) {
@@ -191,7 +199,9 @@ test("loads offline: embedded catalog, deprecated grace, namespaced registration
 	const stderr = captureStderr();
 	let harness;
 	try {
-		harness = await load();
+		const graveyard = Object.values(JSON.parse(readFileSync(deprecatedModelsPath, "utf8")));
+		const graceNow = Math.max(...graveyard.map(entry => Date.parse(entry.deprecatedAt))) + 24 * 60 * 60 * 1000;
+		harness = await load({ catalogNow: graceNow });
 
 		const embedded = harness.runtime.getModel("hypercharm", "deepseek-v4-flash");
 		assert.ok(embedded, "embedded catalog must register without any network");
@@ -210,8 +220,8 @@ test("loads offline: embedded catalog, deprecated grace, namespaced registration
 			).map((model) => model.id),
 		);
 		const deprecatedOnly = Object.values(deprecated).filter((entry) => !embeddedIds.has(entry.id));
-		const fresh = deprecatedOnly.filter((entry) => Date.now() - Date.parse(entry.deprecatedAt) <= DEPRECATED_TTL_MS);
-		const stale = deprecatedOnly.filter((entry) => Date.now() - Date.parse(entry.deprecatedAt) > DEPRECATED_TTL_MS);
+		const fresh = deprecatedOnly.filter((entry) => graceNow - Date.parse(entry.deprecatedAt) <= DEPRECATED_TTL_MS);
+		const stale = deprecatedOnly.filter((entry) => graceNow - Date.parse(entry.deprecatedAt) > DEPRECATED_TTL_MS);
 		assert.ok(fresh.length > 0, "expected at least one model inside the deprecated grace window");
 		for (const entry of fresh) {
 			assert.ok(harness.runtime.getModel("hypercharm", entry.id), entry.id + " must be served during its grace period");
@@ -235,7 +245,7 @@ test("loads offline: embedded catalog, deprecated grace, namespaced registration
 		assert.match(warning, /network disabled/);
 	} finally {
 		stderr.restore();
-		harness?.session.dispose();
+		await harness?.shutdown();
 	}
 });
 
@@ -268,7 +278,7 @@ test("refreshes from /v1/provider, caches the catalog, and retains it when refre
 		assert.ok(Array.isArray(cached), "catalog cache written as an array");
 		assert.ok(cached.some((entry) => entry.id === "fixture-model"), "cache holds the refreshed catalog");
 	} finally {
-		first.session.dispose();
+		await first.shutdown();
 	}
 
 	// A later session keeps serving the cached catalog even though Hyper is down.
@@ -280,7 +290,7 @@ test("refreshes from /v1/provider, caches the catalog, and retains it when refre
 		assert.match(warning, /model catalog/);
 	} finally {
 		stderr.restore();
-		second.session.dispose();
+		await second.shutdown();
 	}
 });
 
@@ -368,7 +378,7 @@ test("records prism routing as durable session entries", async () => {
 		// renderer registration both survive.
 		const sessionFile = sessionManager.getSessionFile();
 		assert.ok(sessionFile);
-		harness.session.dispose();
+		await harness.shutdown();
 		disposed = true;
 
 		const restored = await load({ sessionFile });
@@ -389,10 +399,10 @@ test("records prism routing as durable session entries", async () => {
 			assert.ok(restoredComponent);
 			assert.ok(restoredComponent.render(80).join("\n").includes("Prism \u2192 GLM 5.3 Flash"));
 		} finally {
-			restored.session.dispose();
+			await restored.shutdown();
 		}
 	} finally {
-		if (!disposed) harness.session.dispose();
+		if (!disposed) await harness.shutdown();
 	}
 });
 
@@ -428,7 +438,7 @@ test("co-installs with the official identifier surface without interference", as
 		}
 		assert.equal(ui.statusKeys.includes("hyper") && ownKeys.includes("hyper"), false);
 	} finally {
-		harness.session.dispose();
+		await harness.shutdown();
 	}
 });
 
@@ -480,6 +490,6 @@ test("shows the footer widget on model selection and clears it when the provider
 			"no reinstall while another provider's model is active",
 		);
 	} finally {
-		harness.session.dispose();
+		await harness.shutdown();
 	}
 });