Rejig the cache logic, using Bun APIs as recommended

Author: eastwood

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

@@ -11,7 +11,7 @@ import path from "path"
 import { CopilotModels } from "./models"
 
 const log = Log.create({ service: "plugin.copilot" })
-const ttl = 5 * 60 * 1000
+const ttl = 60 * 60 * 1000
 
 function cachefile(url: string) {
   return path.join(Global.Path.cache, `copilot-models-${Hash.fast(url)}.json`)
@@ -67,15 +67,22 @@ export async function CopilotAuthPlugin(input: PluginInput): Promise<Hooks> {
           "User-Agent": `opencode/${Installation.VERSION}`,
         }
 
-        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 }))
+        if (fresh(file)) {
+          const cached = await Bun.file(file)
+            .json()
+            .catch(() => undefined)
+          if (cached) return cached as Record<string, Model>
         }
 
-        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)]))
+        return CopilotModels.get(url, headers, provider.models)
+          .then(async (result) => {
+            await Bun.write(file, JSON.stringify(result))
+            return result
+          })
+          .catch((error) => {
+            log.error("failed to fetch copilot models", { error })
+            return Object.fromEntries(Object.entries(provider.models).map(([id, model]) => [id, fix(model)]))
+          })
       },
     },
     auth: {