Stabilize local .opencode install checks on Windows

Author: Harryoung

packages/app/src/utils/server.test.ts

@@ -0,0 +1,22 @@
+import { describe, expect, test } from "bun:test"
+import { base64Encode } from "@opencode-ai/util/encode"
+import { normalizeDirectory } from "./server"
+
+describe("normalizeDirectory", () => {
+  test("keeps absolute posix directories unchanged", () => {
+    expect(normalizeDirectory("/tmp/demo")).toBe("/tmp/demo")
+  })
+
+  test("decodes posix route slugs into absolute directories", () => {
+    expect(normalizeDirectory(base64Encode("/tmp/demo"))).toBe("/tmp/demo")
+  })
+
+  test("decodes windows route slugs into absolute directories", () => {
+    expect(normalizeDirectory(base64Encode("C:\\Users\\demo\\repo"))).toBe("C:\\Users\\demo\\repo")
+  })
+
+  test("does not rewrite plain relative values", () => {
+    expect(normalizeDirectory("workspace")).toBe("workspace")
+    expect(normalizeDirectory(base64Encode("workspace"))).toBe(base64Encode("workspace"))
+  })
+})

packages/app/src/utils/server.ts

@@ -1,5 +1,17 @@
 import { createOpencodeClient } from "@opencode-ai/sdk/v2/client"
 import type { ServerConnection } from "@/context/server"
+import { decode64 } from "./base64"
+
+function absolute(dir: string) {
+  return dir.startsWith("/") || /^[A-Za-z]:[\\/]/.test(dir) || dir.startsWith("\\\\")
+}
+
+export function normalizeDirectory(dir?: string) {
+  if (!dir || absolute(dir)) return dir
+  const next = decode64(dir)
+  if (!next || !absolute(next)) return dir
+  return next
+}
 
 export function createSdkForServer({
   server,
@@ -16,6 +28,7 @@ export function createSdkForServer({
 
   return createOpencodeClient({
     ...config,
+    directory: normalizeDirectory(config.directory),
     headers: { ...config.headers, ...auth },
     baseUrl: server.url,
   })

packages/opencode/src/config/config.ts

@@ -330,7 +330,25 @@ export namespace Config {
     }
   }
 
+  async function shared(dir: string) {
+    if (!Installation.isLocal()) return false
+
+    const roots = [Instance.directory, Instance.worktree].filter(Boolean)
+    for (const root of roots) {
+      const rel = path.relative(root, dir)
+      if (path.isAbsolute(rel) || rel.startsWith("..")) continue
+      if (await Filesystem.exists(path.join(root, "node_modules", "@opencode-ai", "plugin"))) return true
+    }
+
+    return false
+  }
+
   export async function needsInstall(dir: string) {
+    if (await shared(dir)) {
+      log.debug("config dir can use shared local dependencies, skipping dependency install", { dir })
+      return false
+    }
+
     // Some config dirs may be read-only.
     // Installing deps there will fail; skip installation in that case.
     const writable = await isWritable(dir)

packages/opencode/test/config/config.test.ts

@@ -763,6 +763,37 @@ test("installs dependencies in writable OPENCODE_CONFIG_DIR", async () => {
   }
 })
 
+test("skips installs for project .opencode when local deps are available upstream", async () => {
+  await using tmp = await tmpdir({
+    init: async (dir) => {
+      await fs.mkdir(path.join(dir, "node_modules", "@opencode-ai", "plugin"), { recursive: true })
+      await fs.mkdir(path.join(dir, ".opencode"), { recursive: true })
+    },
+  })
+
+  await Instance.provide({
+    directory: tmp.path,
+    fn: async () => {
+      expect(await Config.needsInstall(path.join(tmp.path, ".opencode"))).toBe(false)
+    },
+  })
+})
+
+test("still installs for project .opencode when shared local deps are missing", async () => {
+  await using tmp = await tmpdir({
+    init: async (dir) => {
+      await fs.mkdir(path.join(dir, ".opencode"), { recursive: true })
+    },
+  })
+
+  await Instance.provide({
+    directory: tmp.path,
+    fn: async () => {
+      expect(await Config.needsInstall(path.join(tmp.path, ".opencode"))).toBe(true)
+    },
+  })
+})
+
 test("resolves scoped npm plugins in config", async () => {
   await using tmp = await tmpdir({
     init: async (dir) => {