fix: address CodeRabbit review — tighten error handling and add tests

- Add `AbortError` type check to catch block: only suppress errors that
  are AbortErrors AND from our timeout AND not user-initiated aborts
- Add test for ENOENT propagation (non-abort errors not masked as timeout)
- Add test for normal completion without timeout on small directories

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

Author: anandgupta42

packages/opencode/src/tool/glob.ts

@@ -83,7 +83,11 @@ export const GlobTool = Tool.define("glob", {
         })
       }
     } catch (err: any) {
-      if (timeout.signal.aborted) {
+      if (
+        err?.name === "AbortError" &&
+        timeout.signal.aborted &&
+        !ctx.abort?.aborted
+      ) {
         // Our timeout fired — return partial results
         timedOut = true
       } else {

packages/opencode/test/tool/glob.test.ts

@@ -197,6 +197,39 @@ describe("tool.glob", () => {
     })
   })
 
+  test("propagates non-abort errors instead of treating as timeout", async () => {
+    await using tmp = await tmpdir()
+    await Instance.provide({
+      directory: tmp.path,
+      fn: async () => {
+        const glob = await GlobTool.init()
+        // Searching a non-existent path should throw ENOENT, not return "timed out"
+        const badPath = path.join(tmp.path, "nonexistent-dir-12345")
+        await expect(
+          glob.execute({ pattern: "*.txt", path: badPath }, ctx),
+        ).rejects.toThrow()
+      },
+    })
+  })
+
+  test("completes without timeout on small directories", async () => {
+    await using tmp = await tmpdir({
+      init: async (dir) => {
+        await Bun.write(path.join(dir, "a.txt"), "hello")
+      },
+    })
+    await Instance.provide({
+      directory: tmp.path,
+      fn: async () => {
+        const glob = await GlobTool.init()
+        const result = await glob.execute({ pattern: "*.txt" }, { ...ctx, abort: undefined })
+        expect(result.metadata.count).toBe(1)
+        expect(result.metadata.truncated).toBe(false)
+        expect(result.output).not.toContain("timed out")
+      },
+    })
+  })
+
   test("excludes dist and build by default", async () => {
     await using tmp = await tmpdir({
       init: async (dir) => {