pi-toggle-skills 0.1.2 → 0.1.4

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "pi-toggle-skills",
-  "version": "0.1.2",
+  "version": "0.1.4",
   "description": "Toggle skill visibility in pi's system prompt — flip disable-model-invocation on skills via an interactive TUI",
   "type": "module",
   "author": "Tom X Nguyen",

package/src/skill-discovery.ts

@@ -14,7 +14,8 @@ import {
 } from "./index.js";
 import { getAgentDir } from "@earendil-works/pi-coding-agent";
 import matter from "gray-matter";
-import { readFileSync, writeFileSync, existsSync, readdirSync, statSync } from "node:fs";
+import { readFileSync, writeFileSync, existsSync, readdirSync, statSync, realpathSync } from "node:fs";
+import { homedir } from "node:os";
 import { join, basename, dirname } from "node:path";
 
 // Skill directories to scan (in priority order, matching pi's discovery)
@@ -24,7 +25,7 @@ function getGlobalPiSkillDirs(): string[] {
 }
 
 function getGlobalAgentsSkillDirs(): string[] {
-  return [join(getAgentDir(), "..", "agents", "skills")];
+  return [join(homedir(), ".agents", "skills")];
 }
 
 function getProjectSkillDirs(cwd: string): string[] {
@@ -51,11 +52,14 @@ function getAllSkillDirs(cwd: string): SkillDir[] {
   for (const path of getProjectSkillDirs(cwd)) {
     dirs.push({ path, source: "project-pi" });
   }
-  // Deduplicate by path
+  // Deduplicate by resolved (real) path so that symlinked directories
+  // (e.g. ~/.agents/skills → ~/.pi/agent/skills) are not scanned twice.
   const seen = new Set<string>();
   return dirs.filter((d) => {
-    if (seen.has(d.path)) return false;
-    seen.add(d.path);
+    let resolved = d.path;
+    try { resolved = realpathSync(d.path); } catch { /* path doesn't exist yet */ }
+    if (seen.has(resolved)) return false;
+    seen.add(resolved);
     return true;
   });
 }
@@ -171,11 +175,18 @@ function loadSkillFromFile(filePath: string): ToggleSkill | null {
     // Skills without description are not loaded by pi — skip them
     if (!description || description.trim() === "") return null;
 
+    // Resolve symlinks so that the same physical SKILL.md reached via
+    // different alias paths (e.g. ~/.agents/skills/x vs ~/.pi/agent/skills/x)
+    // gets the same filePath and is correctly deduplicated.
+    let realFilePath = filePath;
+    try { realFilePath = realpathSync(filePath); } catch { /* keep as-is */ }
+    const realBaseDir = dirname(realFilePath);
+
     return {
       name,
       description,
-      filePath,
-      baseDir: skillDir,
+      filePath: realFilePath,
+      baseDir: realBaseDir,
       disabled: isDisabled(frontmatter),
     };
   } catch {

package/src/skill-selector.ts

@@ -297,7 +297,7 @@ export class ToggleSkillSelectorComponent implements Component {
         ? this.theme.fg("accent", item.name)
         : item.name;
       const status = item.disabled
-        ? this.theme.fg("warning", " ✗")
+        ? this.theme.fg("dim", " ✗")
         : this.theme.fg("success", " ✓");
 
       this.listContainer.addChild(