fix(skills): use original path for symlinked skills instead of skipping

hubbyxiao-cpu · hubbyxiao-cpu · commit b2dac58717f6 · 2026-04-10T05:02:20.000+08:00
resolveContainedSkillPath() skipped skills when their realpath resolved
outside the configured root (e.g. pnpm symlinks, macOS /var -&gt; /private/var).
This caused valid skills to be silently dropped. Now falls back to the
original (unresolved) path when it is inside the root, logging a warning
instead of skipping entirely.
diff --git a/src/agents/skills/workspace.ts b/src/agents/skills/workspace.ts
@@ -211,6 +211,15 @@ function resolveContainedSkillPath(params: {
   if (isPathInside(params.rootRealPath, candidateRealPath)) {
     return candidateRealPath;
   }
+  if (isPathInside(params.rootDir, path.resolve(params.candidatePath))) {
+    skillsLogger.warn("Skill path resolves outside root via symlink; using original path.", {
+      source: params.source,
+      rootDir: params.rootDir,
+      path: path.resolve(params.candidatePath),
+      realPath: candidateRealPath,
+    });
+    return path.resolve(params.candidatePath);
+  }
   warnEscapedSkillPath({
     source: params.source,
     rootDir: params.rootDir,
