Use existing file cache instead

Author: eastwood

packages/opencode/src/plugin/github-copilot/copilot.ts

@@ -1,14 +1,25 @@
 import type { Hooks, PluginInput } from "@opencode-ai/plugin"
 import type { Model } from "@opencode-ai/sdk/v2"
+import { Global } from "@/global"
 import { Installation } from "@/installation"
 import { iife } from "@/util/iife"
+import { Filesystem } from "@/util/filesystem"
+import { Hash } from "@/util/hash"
 import { Log } from "../../util/log"
 import { setTimeout as sleep } from "node:timers/promises"
+import path from "path"
 import { CopilotModels } from "./models"
 
 const log = Log.create({ service: "plugin.copilot" })
+const ttl = 5 * 60 * 1000
 
-const modelsCache = new Map<string, Record<string, Model>>()
+function cachefile(url: string) {
+  return path.join(Global.Path.cache, `copilot-models-${Hash.fast(url)}.json`)
+}
+
+function fresh(file: string) {
+  return Date.now() - Number(Filesystem.stat(file)?.mtimeMs ?? 0) < ttl
+}
 
 const CLIENT_ID = "Ov23li8tweQw6odWQebz"
 // Add a small safety buffer when polling to avoid hitting the server
@@ -50,19 +61,19 @@ export async function CopilotAuthPlugin(input: PluginInput): Promise<Hooks> {
         }
 
         const url = base(ctx.auth.enterpriseUrl)
-        const key = `${url}:${ctx.auth.refresh}`
+        const file = cachefile(url)
         const headers = {
           Authorization: `Bearer ${ctx.auth.refresh}`,
           "User-Agent": `opencode/${Installation.VERSION}`,
         }
 
-        // Fire-and-forget background refresh (same pattern as models.dev)
-        CopilotModels.get(url, headers, provider.models)
-          .then((result) => modelsCache.set(key, result))
-          .catch((error) => log.error("failed to fetch copilot models", { error }))
+        if (!fresh(file)) {
+          CopilotModels.get(url, headers, provider.models)
+            .then((result) => Filesystem.write(file, JSON.stringify(result)))
+            .catch((error) => log.error("failed to fetch copilot models", { error }))
+        }
 
-        // Return cached result immediately, falling back to static models
-        const cached = modelsCache.get(key)
+        const cached = await Filesystem.readJson<Record<string, Model>>(file).catch(() => undefined)
         if (cached) return cached
         return Object.fromEntries(Object.entries(provider.models).map(([id, model]) => [id, fix(model)]))
       },