@henryqw/pi-add-dir 3.2.2 → 3.2.4

package/extensions/add-dir-helpers.ts

@@ -11,14 +11,14 @@ export interface AddedDir {
 export interface DirContext {
 	agentsMd: string | null;
 	claudeMd: string | null;
-	skills: Set<string>;
+	skills: string[];
 }
 
 const CONTEXT_FILES = ["AGENTS.md", "CLAUDE.md"] as const;
 const SKILL_DIRS = [".pi/skills", ".agents/skills", ".claude/skills"] as const;
 const SKIPPED_SEARCH_DIRS = new Set([".git", "node_modules"]);
 
-function isMissingPathError(error: unknown): boolean {
+export function isMissingPathError(error: unknown): boolean {
 	if (!error || typeof error !== "object" || !("code" in error)) return false;
 	return error.code === "ENOENT" || error.code === "ENOTDIR";
 }
@@ -95,7 +95,7 @@ export function scanDirContext(dir: string): DirContext {
 	return {
 		agentsMd: readContextFile(dir, "AGENTS.md"),
 		claudeMd: readContextFile(dir, "CLAUDE.md"),
-		skills: new Set(skillFiles(dir).map(({ name }) => name)),
+		skills: skillFiles(dir).map(({ name }) => name),
 	};
 }
 

package/extensions/add-dir.ts

@@ -16,6 +16,7 @@ import {
 	collectSkillPaths,
 	dirExists,
 	findFiles,
+	isMissingPathError,
 	resolveDir,
 	scanDirContext,
 	type AddedDir,
@@ -129,9 +130,7 @@ function hasGitMarker(cwd: string): boolean {
 			lstatSync(join(directory, ".git"));
 			return true;
 		} catch (error) {
-			if (!error || typeof error !== "object" || !("code" in error) || (error.code !== "ENOENT" && error.code !== "ENOTDIR")) {
-				throw error;
-			}
+			if (!isMissingPathError(error)) throw error;
 		}
 		const parent = dirname(directory);
 		if (parent === directory) return false;
@@ -148,20 +147,11 @@ function overlaps(a: string, b: string): boolean {
 	return isWithinDir(a, b) || isWithinDir(b, a);
 }
 
-function contextDetails(dirCtx: DirContext, absolutePath: string): AddDirectoryDetails {
-	return {
-		directory: absolutePath,
-		hasAgentsMd: dirCtx.agentsMd !== null,
-		hasClaudeMd: dirCtx.claudeMd !== null,
-		skillNames: [...dirCtx.skills],
-	};
-}
-
 function contextSummary(dirCtx: DirContext): string {
 	const found: string[] = [];
 	if (dirCtx.agentsMd !== null) found.push("AGENTS.md");
 	if (dirCtx.claudeMd !== null) found.push("CLAUDE.md");
-	if (dirCtx.skills.size > 0) found.push(`${dirCtx.skills.size} skill(s)`);
+	if (dirCtx.skills.length > 0) found.push(`${dirCtx.skills.length} skill(s)`);
 	return found.length > 0 ? ` Found: ${found.join(", ")}.` : " No context files found.";
 }
 
@@ -614,15 +604,20 @@ export default function addDirExtension(pi: ExtensionAPI, options: ExtensionOpti
 			if (context.claudeMd !== null) {
 				response.push(`\nCLAUDE.md instructions from ${absolutePath}:\n${context.claudeMd}`);
 			}
-			if (context.skills.size > 0) {
-				response.push(`\nDiscovered skills: ${[...context.skills].join(", ")}`);
+			if (context.skills.length > 0) {
+				response.push(`\nDiscovered skills: ${context.skills.join(", ")}`);
 			}
 			if (result.resourcesChanged) response.push("Run /reload to update external skills.");
 			response.push(`\nYou can now access files at: ${absolutePath}`);
 
 			return {
 				content: [{ type: "text", text: response.join("\n") }],
-				details: contextDetails(context, absolutePath),
+				details: {
+					directory: absolutePath,
+					hasAgentsMd: context.agentsMd !== null,
+					hasClaudeMd: context.claudeMd !== null,
+					skillNames: context.skills,
+				} satisfies AddDirectoryDetails,
 			};
 		},
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@henryqw/pi-add-dir",
-  "version": "3.2.2",
+  "version": "3.2.4",
   "description": "Add external directories to a Pi session with context, skills, and file search.",
   "keywords": [
     "pi-package",