@hyav/pi-provider 0.1.5 → 0.1.6

package/CHANGELOG.md

@@ -2,6 +2,10 @@
 
 This file is the authoritative user-facing release history for `@hyav/pi-provider`.
 
+## 0.1.6 - 2026-08-24
+
+- Prevent Charm Hyper model-refresh warnings when no credentials are configured by registering OAuth-capable Providers without unresolved optional environment API keys and clearing stale API-key configuration after `/reload`; configured environment and stored API keys remain supported.
+
 ## 0.1.5 - 2026-08-24
 
 - Skip network model-catalog refreshes for Providers whose environment-backed API key is not configured and which have no resolved stored credential, retaining cached models without Pi refresh warnings.

package/core/adapter-extensions.ts

@@ -61,9 +61,10 @@ function getStartupBridge(pi: ExtensionAPI): StartupBridge {
 	);
 }
 
-function emitRegistration(pi: ExtensionAPI, envelope: AdapterRegistrationEnvelope): void {
+function emitRegistration(pi: ExtensionAPI, envelope: AdapterRegistrationEnvelope, onSessionStart?: () => void): void {
 	pi.events.emit(PI_PROVIDER_ADAPTER_EVENT, envelope);
 	pi.on("session_start", () => {
+		onSessionStart?.();
 		pi.events.emit(PI_PROVIDER_ADAPTER_EVENT, envelope);
 	});
 }
@@ -92,17 +93,26 @@ function createAdapterExtension<TAdapter extends ProviderAdapter | StatusAdapter
 			// Host runs in a different Pi module context and cannot use this
 			// extension's module-local state. The factory is retained so a Host
 			// loaded later can recreate the adapter with its configured runtime.
-			registerProviderAdapter(pi, providerAdapter, bridge.dependencies, {}, modelDrafts);
-			emitRegistration(pi, {
-				version: PI_PROVIDER_ADAPTER_PROTOCOL_VERSION,
-				kind: "provider",
-				id,
-				token,
-				adapter: providerAdapter,
-				factory: createAdapter,
-				startupDependencies: bridge.dependencies,
-				modelDrafts,
-			});
+			const registeredProvider = registerProviderAdapter(pi, providerAdapter, bridge.dependencies, {}, modelDrafts);
+			emitRegistration(
+				pi,
+				{
+					version: PI_PROVIDER_ADAPTER_PROTOCOL_VERSION,
+					kind: "provider",
+					id,
+					token,
+					adapter: providerAdapter,
+					factory: createAdapter,
+					startupDependencies: bridge.dependencies,
+					modelDrafts,
+				},
+				// During extension loading Pi can only remove queued registrations;
+				// repeat OAuth-only replacement after binding so /reload clears a
+				// raw environment key retained by Pi's merge semantics.
+				registeredProvider.apiKey === undefined
+					? () => registerProviderAdapter(pi, providerAdapter, bridge.dependencies, {}, modelDrafts)
+					: undefined,
+			);
 			return;
 		}
 		if (kind === "status") {

package/core/provider-registration.ts

@@ -1,4 +1,4 @@
-import type { ExtensionAPI, ProviderConfig } from "@earendil-works/pi-coding-agent";
+import type { ProviderConfig } from "@earendil-works/pi-coding-agent";
 import { validateProviderModelDrafts } from "./adapter-validation.ts";
 import { applyOfficialModelCosts, findOfficialMeta, type OfficialModelMeta } from "./official-pricing.ts";
 import { resolvePricingDetails } from "./pricing-adjustments.ts";
@@ -17,6 +17,11 @@ import type {
 const DEFAULT_CONTEXT_WINDOW = 128_000;
 const DEFAULT_MAX_TOKENS = 16_384;
 
+type ProviderRegistrationApi = {
+	registerProvider(name: string, config: ProviderConfig): void;
+	unregisterProvider?(name: string): void;
+};
+
 function finiteNonNegative(value: unknown): number {
 	return typeof value === "number" && Number.isFinite(value) && value >= 0 ? value : 0;
 }
@@ -233,12 +238,23 @@ function replaceModels(target: ProviderModel[], source: ProviderModel[]): Provid
 	return target;
 }
 
-function lacksCatalogRefreshCredential(apiKey: string, context: ProviderRefreshContext): boolean {
-	if (context.allowNetwork !== true || context.credential !== undefined) return false;
+function hasMissingEnvironmentReference(apiKey: string): boolean {
 	const environmentNames = getEnvironmentReferences(apiKey);
 	return environmentNames.length > 0 && environmentNames.some((name) => !process.env[name]);
 }
 
+function lacksCatalogRefreshCredential(apiKey: string, context: ProviderRefreshContext): boolean {
+	return context.allowNetwork === true && context.credential === undefined && hasMissingEnvironmentReference(apiKey);
+}
+
+function shouldOmitOptionalApiKey(adapter: ProviderAdapter, runtime: PiProviderDependencies): boolean {
+	return (
+		adapter.provider.oauth !== undefined &&
+		hasMissingEnvironmentReference(adapter.provider.apiKey) &&
+		runtime.readStoredCredential(adapter.id)?.type !== "api_key"
+	);
+}
+
 /**
  * Register a normalized Provider before the Host has assembled its final
  * registry. The original drafts remain attached to the adapter so a Host in a
@@ -276,6 +292,10 @@ export function prepareProviderRegistration(
 
 	const { models: _draftModels, refreshModels: originalRefresh, ...providerMetadata } = adapter.provider;
 	const registeredProvider: ProviderConfig = { ...providerMetadata, models };
+	// Pi 0.84.2 resolves a declared environment API key before it can skip an
+	// unauthenticated catalog refresh. Register OAuth-only until the optional
+	// key exists, while preserving API-key credentials already stored by Pi.
+	if (shouldOmitOptionalApiKey(adapter, runtime)) delete registeredProvider.apiKey;
 	if (originalRefresh) {
 		registeredProvider.refreshModels = async (options: ProviderRefreshContext) => {
 			// Pi may ask every dynamic Provider to refresh. Keep the current catalog
@@ -329,7 +349,7 @@ export function cancelDeferredProviderRegistrations(providers: readonly Provider
 }
 
 export function refreshProviderRegistrations(
-	pi: Pick<ExtensionAPI, "registerProvider">,
+	pi: ProviderRegistrationApi,
 	providers: readonly ProviderAdapter[],
 	runtime: PiProviderDependencies,
 	officialPricing: Record<string, OfficialModelMeta>,
@@ -348,13 +368,16 @@ export function refreshProviderRegistrations(
 }
 
 export function registerProviderAdapter(
-	pi: Pick<ExtensionAPI, "registerProvider">,
+	pi: ProviderRegistrationApi,
 	adapter: ProviderAdapter,
 	runtime: PiProviderDependencies,
 	officialPricing: Record<string, OfficialModelMeta> = {},
 	modelDrafts?: ProviderModelDraft[],
 ): ProviderConfig {
 	const registeredProvider = prepareProviderRegistration(adapter, runtime, officialPricing, modelDrafts);
+	// Pi merges re-registrations, so omission alone cannot clear a raw API key
+	// left by the previous extension instance during /reload.
+	if (registeredProvider.apiKey === undefined) pi.unregisterProvider?.(adapter.id);
 	pi.registerProvider(adapter.id, registeredProvider);
 	return registeredProvider;
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@hyav/pi-provider",
-  "version": "0.1.5",
+  "version": "0.1.6",
   "description": "Provider extension toolkit for Pi to integrate and manage custom LLM providers with dynamic models, request tuners, and account status.",
   "author": "hyav",
   "license": "MIT",