fix: drop existing overrides for filtered-out models

mergeModels previously re-merged every entry from existing provider
model config, so a user override for a model the filter excludes (e.g.
includeModels: ["claude-*"] with an existing gpt-5.6-terra override)
was still exposed. Pass an allowlist of surviving model IDs (only when
a filter is active) so overrides are preserved only for models that
survive filtering. No behavior change when no filter is configured.

Author: fsx8

CHANGELOG.md

@@ -12,7 +12,8 @@ and this project follows [Semantic Versioning](https://semver.org/).
 - `includeModels` and `excludeModels` plugin options to filter the discovered
   CLIProxyAPI catalog so only a subset of models is exposed to OpenCode.
   Entries are glob patterns; `excludeModels` takes precedence over
-  `includeModels`. Filtering to zero models fails loudly at startup.
+  `includeModels`. Existing model overrides are only preserved for models that
+  survive filtering. Filtering to zero models fails loudly at startup.
 
 ## [0.1.2] - 2026-07-28
 

src/index.test.ts

@@ -261,6 +261,45 @@ describe("CLIProxyAPIPlugin", () => {
     }
   })
 
+  test("filtering drops existing overrides for excluded models but keeps included ones", async () => {
+    const originalFetch = globalThis.fetch
+    globalThis.fetch = async () =>
+      Response.json({
+        data: [{ id: "claude-sonnet-4-6" }, { id: "gpt-5.6-terra" }],
+      })
+
+    try {
+      const plugin = await CLIProxyAPIPlugin(
+        {
+          client: { app: { log: async () => ({}) } },
+        } as PluginInput,
+        {
+          baseURL: "http://cliproxy.test:8317",
+          apiKey: "secret",
+          includeModels: ["claude-*"],
+        },
+      )
+      const config: Config = {
+        provider: {
+          cliproxyapi: {
+            models: {
+              "gpt-5.6-terra": { name: "My Terra" },
+              "claude-sonnet-4-6": { name: "My Claude" },
+            },
+          },
+        },
+      }
+
+      await plugin.config?.(config)
+
+      const models = config.provider?.cliproxyapi?.models ?? {}
+      expect(Object.keys(models).sort()).toEqual(["claude-sonnet-4-6"])
+      expect(models["claude-sonnet-4-6"]?.name).toBe("My Claude")
+    } finally {
+      globalThis.fetch = originalFetch
+    }
+  })
+
   test("fails loudly when the filter matches no discovered models", async () => {
     const originalFetch = globalThis.fetch
     globalThis.fetch = async () =>

src/index.ts

@@ -90,6 +90,9 @@ export const CLIProxyAPIPlugin: Plugin = async ({ client }, rawOptions) => {
           protocol: options.protocol ?? "chat",
           catalog: filteredCatalog,
           protocolCatalog: protocolDiscovery.catalog,
+          allowedModelIDs: filtering
+            ? new Set(filteredCatalog.map((model) => model.id))
+            : undefined,
         })
 
         await client.app.log({
@@ -140,6 +143,7 @@ function addProvider(
     protocol: "chat" | "responses"
     catalog: CatalogModel[]
     protocolCatalog: ModelProtocolCatalog
+    allowedModelIDs?: Set<string>
   },
 ) {
   const existing = config.provider?.[input.providerID]
@@ -186,7 +190,7 @@ function addProvider(
         baseURL: input.baseURL,
         ...(input.apiKey ? { apiKey: input.apiKey } : {}),
       },
-      models: mergeModels(discovered, existing?.models),
+      models: mergeModels(discovered, existing?.models, input.allowedModelIDs),
     },
   }
 }
@@ -203,10 +207,12 @@ function resolveModelNpm(
 function mergeModels(
   discovered: Record<string, ModelConfig>,
   existing: ProviderConfig["models"],
+  allowedModelIDs?: Set<string>,
 ): Record<string, ModelConfig> {
   const merged = { ...discovered }
 
   for (const [modelID, model] of Object.entries(existing ?? {})) {
+    if (allowedModelIDs && !allowedModelIDs.has(modelID)) continue
     const discoveredModel = merged[modelID]
     const providerNpm = model.provider?.npm ?? discoveredModel?.provider?.npm
     merged[modelID] = {