Skip to content

tip: skip dependency dirs during skill discovery (origin #51278) #92

Description

@kvnloo

Origin

anomalyco#51278 — fix(opencode): skip dependency directories during skill discovery

Why (fork leaf)

Skill discovery walks skills/**/SKILL.md before the TUI is ready. Without ignore patterns, cold starts descend into node_modules / .git / dist / .cache, and symlink-aliased skill roots can be scanned twice.

Scope (this fork PR)

  • Glob.Options.ignore passthrough
  • Skill scan: SKIP globs + realpath dedupe of skill roots
  • Regression tests for ignore descent + symlink/dependency skip

Proof

cd packages/opencode
bun test test/util/glob.test.ts test/skill/skill.test.ts -t "does not descend into ignored|skips dependency"
# NEGATIVE (base product + tip tests): 0 pass / 2 fail
# GREEN: 2 pass
# RESTORE: 0 pass / 2 fail

Policy

github_writes_origin=0 — fork leaf only. Codex promote parked.

Activity

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions