@juvio15/pi-cliproxyapi-provider 0.1.1 → 0.1.2

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@juvio15/pi-cliproxyapi-provider",
-  "version": "0.1.1",
+  "version": "0.1.2",
   "description": "pi provider for CLIProxyAPI: per-family APIs, model catalog, costs, web search, usage footer",
   "keywords": [
     "cliproxyapi",

package/src/core/catalog.ts

@@ -368,6 +368,13 @@ export const fetchCatalog = async (
   return "models" in payload ? payload.models : payload.data;
 };
 
+/**
+ * Drops models the latest catalog no longer lists. mergeCatalog keeps them so
+ * a session that uses one still resolves, but nobody should pick one anew.
+ */
+export const withoutStale = <T extends object>(models: readonly T[]): T[] =>
+  models.filter((model) => !("stale" in model && model.stale === true));
+
 export const mergeCatalog = (
   previous: readonly CliproxyModel[],
   fresh: readonly CliproxyModel[],

package/src/core/index.ts

@@ -13,6 +13,7 @@
  * matchModelCost(modelId: string, catalog: CostCatalog, fast?: boolean): ModelCost
  * mapCatalog(entries: readonly CatalogModel[], options: MappingOptions): MappedCatalog
  * mergeCatalog(previous: readonly CliproxyModel[], fresh: readonly CliproxyModel[], options?: MergeOptions): CliproxyModel[]
+ * withoutStale<T extends object>(models: readonly T[]): T[]
  * wrapCliproxyStreams(streams: ProviderStreams, settings: PayloadSettings): ProviderStreams
  * createCliproxyProvider(options: CliproxyProviderOptions): Provider<CliproxyApi>
  * quotaResetFromError(message: string, now?: number): number | undefined
@@ -32,7 +33,12 @@ import {
 import { createProvider } from "@earendil-works/pi-ai/models";
 import type { Provider } from "@earendil-works/pi-ai/models";
 
-import { fetchCatalog, mapCatalog, mergeCatalog } from "./catalog.ts";
+import {
+  fetchCatalog,
+  mapCatalog,
+  mergeCatalog,
+  withoutStale,
+} from "./catalog.ts";
 import type { CliproxyApi, CliproxyModel, MappingOptions } from "./catalog.ts";
 import { resolveEndpoints } from "./connection.ts";
 import { fetchCosts } from "./costs.ts";
@@ -46,6 +52,7 @@ export {
   mapCatalog,
   mergeCatalog,
   selectModelApi,
+  withoutStale,
   type CatalogFetchOptions,
   type CatalogModel,
   type CliproxyApi,
@@ -180,6 +187,7 @@ export const createCliproxyProvider = (
         staleTtlMs: options.staleTtlMs,
       });
     },
+    filterModels: (models) => withoutStale(models),
     id,
     models: options.models ?? [],
     name: options.providerName ?? "CLIProxyAPI",

package/src/extension/index.ts

@@ -4,7 +4,11 @@ import type {
   ExtensionContext,
 } from "@earendil-works/pi-coding-agent";
 
-import { resolveConnectionSources, resolveEndpoints } from "../core/index.ts";
+import {
+  resolveConnectionSources,
+  resolveEndpoints,
+  withoutStale,
+} from "../core/index.ts";
 import type { CliproxyModel } from "../core/index.ts";
 import { completeFrom } from "./completions.ts";
 import {
@@ -140,7 +144,7 @@ const registerCatalogCommands = (
         const models = controller.getModels();
         const stale = models.filter((model) => model.stale).length;
         ctx.ui.notify(
-          `Refreshed ${models.length} CLIProxyAPI models${stale ? ` (${stale} retained from cache)` : ""}.`,
+          `Refreshed ${models.length - stale} CLIProxyAPI models${stale ? `. Hid ${stale} that CLIProxyAPI no longer lists` : ""}.`,
           "info"
         );
       } catch (error) {
@@ -229,7 +233,7 @@ export default async function cliproxyapiProvider(
     configured: connection !== null,
     connection: initialConnection,
     fast,
-    filterAvailable: (models) => tracker.filter(models),
+    filterAvailable: (models) => tracker.filter(withoutStale(models)),
     onModels: (models) => {
       latestModels = models;
 
@@ -370,8 +374,9 @@ export default async function cliproxyapiProvider(
   });
   registerWebSearch(pi, {
     agentDir,
-    getAvailableModels: () => tracker.filter(controller.getModels()),
-    getModels: controller.getModels,
+    getAvailableModels: () =>
+      tracker.filter(withoutStale(controller.getModels())),
+    getModels: () => withoutStale(controller.getModels()),
     getStandardCost: controller.getStandardCost,
     providerId,
   });