Skip to content

fix(eve): support symlinked skill discovery#620

Open
iroiro147 wants to merge 1 commit into
vercel:mainfrom
iroiro147:fix/skill-discovery-symlinks-205
Open

fix(eve): support symlinked skill discovery#620
iroiro147 wants to merge 1 commit into
vercel:mainfrom
iroiro147:fix/skill-discovery-symlinks-205

Conversation

@iroiro147

Copy link
Copy Markdown
Contributor

Summary

  • resolve skill entries that appear as filesystem links before classifying authored skills
  • support flat markdown skill symlinks, package directory symlinks, and packaged SKILL.md symlinks
  • add disk-backed integration coverage and a patch changeset

Fixes #205.

Tests

  • pnpm --filter eve run build:compiled
  • pnpm --filter eve exec vitest run --config vitest.integration.config.ts src/discover/skills.integration.test.ts
  • pnpm exec oxfmt --check packages/eve/src/discover/skills.ts packages/eve/src/discover/skills.integration.test.ts .changeset/skill-discovery-symlinks.md
  • pnpm --filter eve exec oxlint src/discover/skills.ts src/discover/skills.integration.test.ts --deny-warnings
  • pnpm --filter eve run typecheck
  • pnpm guard:invariants
  • pnpm changeset status --since origin/main
  • git diff --check
  • git diff --cached --check

@vercel

vercel Bot commented Jul 8, 2026

Copy link
Copy Markdown
Contributor

@iroiro147 is attempting to deploy a commit to the Vercel Team on Vercel.

A member of the Team first needs to authorize it.

Signed-off-by: iroiro147 <sarthak.singh@juspay.in>
@iroiro147 iroiro147 force-pushed the fix/skill-discovery-symlinks-205 branch from f62de07 to 7528916 Compare July 12, 2026 12:45
@iroiro147

Copy link
Copy Markdown
Contributor Author

Rebased this onto current main and pushed signed head 7528916b. I also hardened compilation so linked skill packages are dereferenced into self-contained workspace resources and included in the resource hash, added a compile scenario that removes the original linked source before validating the artifact, made unsupported link creation an explicit skip, covered Windows junctions, and updated the skill-sharing docs. Verified under Node 24 with 8 discovery integration tests, the focused compile scenario, package typecheck, all documentation checks, invariant checks, and git diff --check. Fresh CI is running.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Skill discovery should support symlinked skill files/directories

1 participant