fix(opencode): ignore node_modules during config and skill discovery

Closes #30337

Config and skill discovery use glob scans rooted at .opencode/
with recursive patterns (**/*.md, **/SKILL.md) and follow every
directory — including node_modules/ installed by @opencode-ai/plugin.

Add ignore: ["**/node_modules/**"] to all .opencode/ glob scans
and thread the ignore option through Glob.scan's Options interface.

Author: ulises-jeremias

packages/core/src/util/glob.ts

@@ -8,6 +8,7 @@ export namespace Glob {
     include?: "file" | "all"
     dot?: boolean
     symlink?: boolean
+    ignore?: string | string[]
   }
 
   function toGlobOptions(options: Options): GlobOptions {
@@ -17,6 +18,7 @@ export namespace Glob {
       dot: options.dot,
       follow: options.symlink ?? false,
       nodir: options.include !== "all",
+      ignore: options.ignore,
     }
   }
 

packages/opencode/src/config/agent.ts

@@ -18,6 +18,7 @@ export async function load(dir: string) {
     absolute: true,
     dot: true,
     symlink: true,
+    ignore: ["**/node_modules/**"],
   })) {
     const md = await ConfigMarkdown.parse(item).catch((err) => {
       log.error("failed to load agent", { agent: item, err })
@@ -44,6 +45,7 @@ export async function loadMode(dir: string) {
     absolute: true,
     dot: true,
     symlink: true,
+    ignore: ["**/node_modules/**"],
   })) {
     const md = await ConfigMarkdown.parse(item).catch((err) => {
       log.error("failed to load mode", { mode: item, err })

packages/opencode/src/config/command.ts

@@ -20,6 +20,7 @@ export async function load(dir: string) {
     absolute: true,
     dot: true,
     symlink: true,
+    ignore: ["**/node_modules/**"],
   })) {
     const md = await ConfigMarkdown.parse(item).catch((err) => {
       log.error("failed to load command", { command: item, err })

packages/opencode/src/config/plugin.ts

@@ -23,6 +23,7 @@ export async function load(dir: string) {
     absolute: true,
     dot: true,
     symlink: true,
+    ignore: ["**/node_modules/**"],
   })) {
     plugins.push(pathToFileURL(item).href)
   }

packages/opencode/src/skill/index.ts

@@ -153,6 +153,7 @@ const scan = Effect.fnUntraced(function* (
         include: "file",
         symlink: true,
         dot: opts?.dot,
+        ignore: ["**/node_modules/**"],
       }),
     catch: (error) => error,
   }).pipe(

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

@@ -1989,3 +1989,52 @@ test("parseManagedPlist handles empty config", async () => {
   )
   expect(config.$schema).toBe("https://opencode.ai/config.json")
 })
+
+it.instance("ignores markdown files under .opencode/node_modules during config discovery", () =>
+  Effect.gen(function* () {
+    const test = yield* TestInstance
+    yield* FSUtil.use.writeWithDirs(
+      path.join(test.directory, ".opencode", "node_modules", "pkg", "agents", "ignored.md"),
+      `---
+model: test/model
+---
+Ignored agent prompt`,
+    )
+    yield* FSUtil.use.writeWithDirs(
+      path.join(test.directory, ".opencode", "agents", "loaded.md"),
+      `---
+model: test/model
+---
+Loaded agent prompt`,
+    )
+    yield* FSUtil.use.writeWithDirs(
+      path.join(test.directory, ".opencode", "node_modules", "pkg", "commands", "ignored.md"),
+      `---
+description: ignored
+---
+Ignored command template`,
+    )
+    yield* FSUtil.use.writeWithDirs(
+      path.join(test.directory, ".opencode", "commands", "loaded.md"),
+      `---
+description: loaded
+---
+Loaded command template`,
+    )
+
+    const config = yield* Config.use.get()
+
+    expect(config.agent?.["loaded"]).toMatchObject({
+      name: "loaded",
+      model: "test/model",
+      prompt: "Loaded agent prompt",
+    })
+    expect(config.agent?.["pkg/agents/ignored"]).toBeUndefined()
+
+    expect(config.command?.["loaded"]).toEqual({
+      description: "loaded",
+      template: "Loaded command template",
+    })
+    expect(config.command?.["pkg/commands/ignored"]).toBeUndefined()
+  }),
+)

packages/opencode/test/skill/skill.test.ts

@@ -197,6 +197,44 @@ Just some content without YAML frontmatter.
     ),
   )
 
+  it.live("ignores skills under .opencode/node_modules", () =>
+    provideTmpdirInstance(
+      (dir) =>
+        Effect.gen(function* () {
+          yield* Effect.promise(() =>
+            Bun.write(
+              path.join(dir, ".opencode", "skill", "kept-skill", "SKILL.md"),
+              `---
+name: kept-skill
+description: Should be discovered.
+---
+
+# Kept Skill
+`,
+            ),
+          )
+          yield* Effect.promise(() =>
+            Bun.write(
+              path.join(dir, ".opencode", "node_modules", "pkg", "skill", "ignored-skill", "SKILL.md"),
+              `---
+name: ignored-skill
+description: Should be ignored.
+---
+
+# Ignored Skill
+`,
+            ),
+          )
+
+          const skill = yield* Skill.Service
+          const list = (yield* skill.all()).filter((s) => s.location !== "<built-in>")
+          expect(list.find((x) => x.name === "kept-skill")).toBeDefined()
+          expect(list.find((x) => x.name === "ignored-skill")).toBeUndefined()
+        }),
+      { git: true },
+    ),
+  )
+
   it.live("discovers skills without descriptions", () =>
     provideTmpdirInstance(
       (dir) =>

packages/opencode/test/util/glob.test.ts

@@ -115,6 +115,17 @@ describe("Glob", () => {
 
       expect(results).toEqual(["visible"])
     })
+
+    test("supports ignore patterns", async () => {
+      await using tmp = await tmpdir()
+      await fs.mkdir(path.join(tmp.path, "node_modules", "pkg"), { recursive: true })
+      await fs.writeFile(path.join(tmp.path, "node_modules", "pkg", "blocked.md"), "", "utf-8")
+      await fs.writeFile(path.join(tmp.path, "allowed.md"), "", "utf-8")
+
+      const results = await Glob.scan("**/*.md", { cwd: tmp.path, ignore: ["**/node_modules/**"] })
+
+      expect(results).toEqual(["allowed.md"])
+    })
   })
 
   describe("scanSync()", () => {