desktop: path links skip directories and paths with spaces #55863

Activity

1. opencode-agent commented on Oct 15, 2026

   

   opencode-agentBot

   on Oct 15, 2026 – with OpenCode Agent

   Contributor

   More actions

   Thanks for the detailed report. I reproduced all three problems on 2.0.29, and they still happen on the latest dev (a448d5b).

   I couldn't run the desktop app here, so I ran the code it relies on against a real opencode serve:

   • Folders: /api/fs/list returns a folder as type: "directory". The desktop's exists() check accepts only type === "file", so it answers false and the folder never becomes a link.
   • Paths with spaces: the inline-code classifier returns nothing for /…/my folder/file.txt and C:\Program Files\app\file.txt. That's because of its whitespace check, even though the existence check would find the file.
   • Clicking a folder link: GET /api/fs/read/<folder> returns 404 FileNotFoundError ("File not found: "), which is the text of the toast you saw.

   The script below reproduces this. Related issues about desktop path links: #50390, #51872, #54540, #51516, and #37891, which asks for clickable paths that open or reveal.

   Reproduction script (repro.ts)

   // Reproduces anomalyco/opencode#55863 without the desktop shell.
   // Usage: start `opencode serve --port 4567` (prints a password), then
   //   OC_PASSWORD=<password> OC_REPO=/workspace/repo bun /workspace/repro/repro.ts
   // It (1) runs the desktop's inline-code classifier on paths, and (2) replays the
   // desktop `exists()` check (packages/app/src/workspaces/files/model.tsx) and the
   // click-to-open read (/api/fs/read) against a real server, for a directory and a
   // file whose path contains a space.
   import { mkdirSync, writeFileSync } from "node:fs"
   import { join, dirname, basename } from "node:path"
   
   const repo = process.env.OC_REPO ?? "/workspace/repo"
   const { inlineCodeKind } = await import(`${repo}/packages/session-ui/src/components/markdown-inline-code-kind.ts`)
   
   const base = process.env.OC_URL ?? "http://127.0.0.1:4567"
   const auth = "Basic " + Buffer.from(`opencode:${process.env.OC_PASSWORD ?? ""}`).toString("base64")
   const proj = "/workspace/repro/proj"
   mkdirSync(join(proj, "sub"), { recursive: true })
   mkdirSync(join(proj, "my folder"), { recursive: true })
   writeFileSync(join(proj, "my folder/file.txt"), "hi\n")
   
   const q = (extra: Record<string, string>) =>
     new URLSearchParams({ "location[directory]": proj, ...extra }).toString()
   
   // Same logic as the desktop `exists()`: list parent, accept only type === "file".
   async function exists(p: string) {
     const res = await fetch(`${base}/api/fs/list?${q({ path: dirname(p) })}`, { headers: { authorization: auth } })
     if (!res.ok) return { ok: false, status: res.status }
     const body: any = await res.json()
     const entries: any[] = body.data ?? body
     const match = entries.find((e) => basename(e.path) === basename(p))
     return { listedType: match?.type, existsResult: !!match && match.type === "file" }
   }
   
   async function read(p: string) {
     const res = await fetch(`${base}/api/fs/read/${encodeURI(p)}?${q({})}`, { headers: { authorization: auth } })
     return { status: res.status, body: (await res.text()).slice(0, 200) }
   }
   
   for (const p of [join(proj, "sub"), join(proj, "my folder/file.txt"), "C:\\Program Files\\app\\file.txt", join(proj, "sub/a.txt")]) {
     console.log(JSON.stringify(p), "inlineCodeKind =", inlineCodeKind(p))
   }
   console.log("directory  exists():", await exists(join(proj, "sub")))
   console.log("spaced file exists():", await exists(join(proj, "my folder/file.txt")))
   console.log("read(directory) on click:", await read("sub"))

   Run it: bun repro.ts
2. opencode-agent

   added

   reproduced

   on Oct 15, 2026
3. Finn763 commented on Oct 19, 2026

   

   Finn763

   on Oct 19, 2026

   Contributor

   More actions

   I'd like to work on this — the file-link existence check only accepts files and the inline-code classifier rejects whitespace, so directories and paths with spaces never linkify. I'll fix both and cover them with tests.