pi-free 2.2.1 → 2.2.2

package/CHANGELOG.md

@@ -7,6 +7,22 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ## [Unreleased]
 
+## [2.2.2] - 2026-06-19
+
+### Refactored
+
+- `config.ts`: Introduced `PROVIDER_META` table that pairs each
+  provider's ID, env-var prefix, and typed config key. `getProviderShowPaid`
+  now delegates to a generic `resolveShowPaidForProvider` resolver
+  instead of a 17-case switch. New `PROVIDER_OPENROUTER`, `PROVIDER_OPENCODE`,
+  and `PROVIDER_FASTROUTER` constants added to `constants.ts` and used
+  in the table (DRYKISS DRY/Architecture findings).
+- `index.ts`: Wrap dynamic built-in provider import in `try/catch`
+  with full error+stack logging to both `~/.pi/free.log` and stderr
+  (DRYKISS Resilience finding).
+- `providers/bai/bai.ts`: Improve startup-failure message to point users
+  at `~/.pi/free.log` and their API key (DRYKISS Resilience finding).
+
 ## [2.2.1] - 2026-06-19
 
 ### Security

package/config.ts

@@ -11,11 +11,33 @@
 
 import { chmodSync, existsSync, readFileSync, writeFileSync } from "node:fs";
 import { join } from "node:path";
+import {
+	PROVIDER_BAI,
+	PROVIDER_CLINE,
+	PROVIDER_FASTROUTER,
+	PROVIDER_KILO,
+	PROVIDER_OLLAMA,
+	PROVIDER_OPENCODE,
+	PROVIDER_OPENROUTER,
+	PROVIDER_ROUTEWAY,
+	PROVIDER_TOKENROUTER,
+	PROVIDER_ZENMUX,
+	PROVIDER_CROFAI,
+	PROVIDER_CODESTRAL,
+	PROVIDER_LLM7,
+	PROVIDER_DEEPINFRA,
+	PROVIDER_SAMBANOVA,
+	PROVIDER_TOGETHER,
+	PROVIDER_NOVITA,
+} from "./constants.ts";
 export {
 	PROVIDER_BAI,
 	PROVIDER_CLINE,
+	PROVIDER_FASTROUTER,
 	PROVIDER_KILO,
 	PROVIDER_MODAL,
+	PROVIDER_OPENCODE,
+	PROVIDER_OPENROUTER,
 	PROVIDER_QWEN,
 	PROVIDER_ROUTEWAY,
 	PROVIDER_TOKENROUTER,
@@ -219,6 +241,56 @@ function resolveBool(envKey: string, fileVal?: boolean): boolean {
 	return fileVal === true;
 }
 
+// =============================================================================
+// Per-provider metadata table
+// Adding a new provider only requires a single entry here plus the
+// corresponding field in the PiFreeConfig interface and CONFIG_TEMPLATE.
+// Each entry pairs the provider ID with its env-var prefix (used for both
+// the API key and show_paid flag) and the typed key on PiFreeConfig.
+// =============================================================================
+
+interface ProviderMeta {
+	id: string;
+	/** Env var prefix, e.g. "KILO" => KILO_SHOW_PAID and KILO_API_KEY */
+	prefix: string;
+	/** Typed accessor returning the show_paid value from PiFreeConfig */
+	showPaidKey: keyof PiFreeConfig;
+}
+
+const PROVIDER_META: readonly ProviderMeta[] = [
+	{ id: PROVIDER_KILO, prefix: "KILO", showPaidKey: "kilo_show_paid" },
+	{ id: PROVIDER_CLINE, prefix: "CLINE", showPaidKey: "cline_show_paid" },
+	{ id: PROVIDER_ZENMUX, prefix: "ZENMUX", showPaidKey: "zenmux_show_paid" },
+	{ id: PROVIDER_CROFAI, prefix: "CROFAI", showPaidKey: "crofai_show_paid" },
+	{ id: PROVIDER_CODESTRAL, prefix: "CODESTRAL", showPaidKey: "codestral_show_paid" },
+	{ id: PROVIDER_LLM7, prefix: "LLM7", showPaidKey: "llm7_show_paid" },
+	{ id: PROVIDER_DEEPINFRA, prefix: "DEEPINFRA", showPaidKey: "deepinfra_show_paid" },
+	{ id: PROVIDER_SAMBANOVA, prefix: "SAMBANOVA", showPaidKey: "sambanova_show_paid" },
+	{ id: PROVIDER_TOGETHER, prefix: "TOGETHER", showPaidKey: "together_show_paid" },
+	{ id: PROVIDER_NOVITA, prefix: "NOVITA", showPaidKey: "novita_show_paid" },
+	{ id: PROVIDER_ROUTEWAY, prefix: "ROUTEWAY", showPaidKey: "routeway_show_paid" },
+	{ id: PROVIDER_TOKENROUTER, prefix: "TOKENROUTER", showPaidKey: "tokenrouter_show_paid" },
+	{ id: PROVIDER_BAI, prefix: "BAI", showPaidKey: "bai_show_paid" },
+	{ id: PROVIDER_FASTROUTER, prefix: "FASTROUTER", showPaidKey: "fastrouter_show_paid" },
+	{ id: PROVIDER_OLLAMA, prefix: "OLLAMA", showPaidKey: "ollama_show_paid" },
+	{ id: PROVIDER_OPENROUTER, prefix: "OPENROUTER", showPaidKey: "openrouter_show_paid" },
+	{ id: PROVIDER_OPENCODE, prefix: "OPENCODE", showPaidKey: "opencode_show_paid" },
+];
+
+const PROVIDER_META_BY_ID = new Map(PROVIDER_META.map((m) => [m.id, m]));
+
+/**
+ * Generic show_paid resolver backed by PROVIDER_META. Returns false
+ * for unknown provider IDs (matches the previous switch default).
+ */
+function resolveShowPaidForProvider(providerId: string): boolean {
+	const meta = PROVIDER_META_BY_ID.get(providerId);
+	if (!meta) return false;
+	const cfg = loadConfigFile();
+	const fileVal = cfg[meta.showPaidKey];
+	return resolveBool(`${meta.prefix}_SHOW_PAID`, fileVal as boolean | undefined);
+}
+
 // =============================================================================
 // Per-provider paid-model flags (getters so toggles reflect immediately)
 // =============================================================================
@@ -310,44 +382,7 @@ export function getOpencodeShowPaid(): boolean {
 }
 
 export function getProviderShowPaid(providerId: string): boolean {
-	switch (providerId) {
-		case "kilo":
-			return getKiloShowPaid();
-		case "cline":
-			return getClineShowPaid();
-		case "zenmux":
-			return getZenmuxShowPaid();
-		case "crofai":
-			return getCrofaiShowPaid();
-		case "codestral":
-			return getCodestralShowPaid();
-		case "llm7":
-			return getLlm7ShowPaid();
-		case "deepinfra":
-			return getDeepinfraShowPaid();
-		case "sambanova":
-			return getSambanovaShowPaid();
-		case "together":
-			return getTogetherShowPaid();
-		case "novita":
-			return getNovitaShowPaid();
-		case "routeway":
-			return getRoutewayShowPaid();
-		case "tokenrouter":
-			return getTokenrouterShowPaid();
-		case "bai":
-			return getBaiShowPaid();
-		case "fastrouter":
-			return getFastrouterShowPaid();
-		case "ollama-cloud":
-			return getOllamaShowPaid();
-		case "openrouter":
-			return getOpenrouterShowPaid();
-		case "opencode":
-			return getOpencodeShowPaid();
-		default:
-			return false;
-	}
+	return resolveShowPaidForProvider(providerId);
 }
 
 // =============================================================================

package/constants.ts

@@ -26,6 +26,11 @@ export const PROVIDER_ROUTEWAY = "routeway";
 export const PROVIDER_TOKENROUTER = "tokenrouter";
 export const PROVIDER_BAI = "bai";
 
+// Built-in pi providers that pi-free wraps with toggles
+export const PROVIDER_OPENROUTER = "openrouter";
+export const PROVIDER_OPENCODE = "opencode";
+export const PROVIDER_FASTROUTER = "fastrouter";
+
 export const ALL_UNIQUE_PROVIDERS = [
 	PROVIDER_KILO,
 	PROVIDER_CLINE,

package/index.ts

@@ -379,10 +379,22 @@ export default async function piFreeEntry(pi: ExtensionAPI) {
 
 	// Setup dynamic built-in providers (Mistral, Groq, Cerebras, xAI, Hugging Face,
 	// OpenRouter/OpenCode from Pi auth, and FastRouter public model discovery)
-	const { setupDynamicBuiltInProviders } = await import(
-		"./providers/dynamic-built-in/index.ts"
-	);
-	await setupDynamicBuiltInProviders(pi);
+	try {
+		const { setupDynamicBuiltInProviders } = await import(
+			"./providers/dynamic-built-in/index.ts"
+		);
+		await setupDynamicBuiltInProviders(pi);
+	} catch (err) {
+		// Dynamic providers are a best-effort enhancement — if the import
+		// or init fails (e.g. upstream API change), continue with the
+		// already-registered static providers rather than failing the whole
+		// extension load. Log full error (message + stack) to the structured
+		// log so the user can investigate, but never block startup.
+		_logger.error("[pi-free] Dynamic built-in providers failed to load", {
+			error: err instanceof Error ? err.message : String(err),
+			stack: err instanceof Error ? err.stack : undefined,
+		});
+	}
 
 	// Setup toggles for pi's built-in providers (e.g., OpenCode)
 	setupBuiltInProviderToggles(pi);

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "pi-free",
-	"version": "2.2.1",
+	"version": "2.2.2",
 	"type": "module",
 	"description": "AI model providers for Pi with free model filtering and dynamic model fetching",
 	"keywords": [

package/providers/bai/bai.ts

@@ -184,7 +184,12 @@ export default async function baiProvider(pi: ExtensionAPI) {
 	const allModels = await fetchBaiModels(apiKey);
 
 	if (allModels.length === 0) {
-		_logger.warn("[bai] No text chat models available");
+		// Either the API failed (already logged inside fetchBaiModels) or
+		// the API returned zero text chat models. We can't tell the user
+		// which, but we can give a hint to check the log / their key.
+		_logger.warn(
+			"[bai] No text chat models available — verify BAI_API_KEY is valid and see ~/.pi/free.log for details",
+		);
 		return;
 	}