@henryqw/pi-add-dir 0.1.4 → 0.1.6

package/extensions/add-dir-helpers.ts

@@ -1,4 +1,4 @@
-import { readdirSync, readFileSync, realpathSync, statSync } from "node:fs";
+import { closeSync, openSync, readdirSync, readFileSync, readSync, realpathSync, statSync } from "node:fs";
 import { readdir } from "node:fs/promises";
 import { homedir } from "node:os";
 import * as path from "node:path";
@@ -128,6 +128,29 @@ function skillDescription(content: string): string {
 	return value.replace(/^("|')|("|')$/g, "").trim() || "No description";
 }
 
+// ponytail: 8KB head is enough for SKILL.md frontmatter descriptions; full-body reads stay in scanDirContext
+function readHead(filePath: string, bytes = 8192): string | null {
+	let fd: number | undefined;
+	try {
+		fd = openSync(filePath, "r");
+		const buffer = Buffer.alloc(bytes);
+		const bytesRead = readSync(fd, buffer, 0, bytes, 0);
+		return buffer.toString("utf8", 0, bytesRead);
+	} catch {
+		return null;
+	} finally {
+		if (fd !== undefined) closeSync(fd);
+	}
+}
+
+// ponytail: 8KB head covers typical frontmatter; falls back to full read only when opener present but closer missing (rare)
+function readSkillFrontmatter(filePath: string): string | null {
+	const head = readHead(filePath, 8192);
+	if (head === null || !/^---\r?\n/.test(head)) return head;
+	if (/^---\r?\n[\s\S]*\r?\n---(?:\r?\n|$)/.test(head)) return head;
+	return readFileSafe(filePath);
+}
+
 export function buildContextInjection(dirs: AddedDir[]): string {
 	if (dirs.length === 0) return "";
 
@@ -138,23 +161,20 @@ export function buildContextInjection(dirs: AddedDir[]): string {
 
 	const registeredSkills = new Set<string>();
 	for (const dir of dirs) {
-		const ctx = scanDirContext(dir.absolutePath);
-		const skills = [...ctx.skills].filter(([name]) => {
-			if (registeredSkills.has(name)) return false;
-			registeredSkills.add(name);
-			return true;
-		});
 		sections.push(`### ${dir.label} - \`${dir.absolutePath}\``);
 
-		if (ctx.agentsMd) sections.push(`\n#### AGENTS.md (from ${dir.label})\n${ctx.agentsMd}`);
-		if (ctx.claudeMd) sections.push(`\n#### CLAUDE.md (from ${dir.label})\n${ctx.claudeMd}`);
+		const agentsMd = readContextFile(dir.absolutePath, "AGENTS.md");
+		if (agentsMd) sections.push(`\n#### AGENTS.md (from ${dir.label})\n${agentsMd}`);
+		const claudeMd = readContextFile(dir.absolutePath, "CLAUDE.md");
+		if (claudeMd) sections.push(`\n#### CLAUDE.md (from ${dir.label})\n${claudeMd}`);
 
+		const skills = skillFiles(dir.absolutePath).filter((skill) => !registeredSkills.has(skill.name));
 		if (skills.length > 0) {
 			sections.push(`\n#### Skills from ${dir.label} (registered as /skill:name commands):`);
-			for (const [name, content] of skills) {
-				sections.push(
-					`- **${name}**: ${skillDescription(content)} - use \`/skill:${name}\` or read \`${ctx.skillPaths.get(name)}\``,
-				);
+			for (const skill of skills) {
+				registeredSkills.add(skill.name);
+				const description = skillDescription(readSkillFrontmatter(skill.path) ?? "");
+				sections.push(`- **${skill.name}**: ${description} - use \`/skill:${skill.name}\` or read \`${skill.path}\``);
 			}
 		}
 	}

package/extensions/add-dir.ts

@@ -170,6 +170,16 @@ export default function addDirExtension(pi: ExtensionAPI): void {
 				hasNewSkills: false,
 			};
 		}
+		const overlap = addedDirs.find(
+			(dir) => isWithinDir(dir.absolutePath, absolutePath) || isWithinDir(absolutePath, dir.absolutePath),
+		);
+		if (overlap) {
+			return {
+				ok: false,
+				message: `Directory overlaps already-added directory: ${overlap.absolutePath}`,
+				hasNewSkills: false,
+			};
+		}
 
 		const context = scanDirContext(absolutePath);
 		const label = basename(absolutePath) || absolutePath;
@@ -382,7 +392,6 @@ export default function addDirExtension(pi: ExtensionAPI): void {
 		renderCall(args, theme) {
 			let text = theme.fg("toolTitle", theme.bold("search_external_files "));
 			text += theme.fg("accent", `"${cleanPath(args.pattern)}"`);
-			text += theme.fg("dim", ` across ${addedDirs.length} dir(s)`);
 			return new Text(text, 0, 0);
 		},
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@henryqw/pi-add-dir",
-  "version": "0.1.4",
+  "version": "0.1.6",
   "description": "Add external directories to a Pi session with context, skills, and file search.",
   "keywords": [
     "pi-package",
@@ -21,7 +21,7 @@
   ],
   "scripts": {
     "test": "node --test test/*.test.ts",
-    "typecheck": "tsc --noEmit --allowImportingTsExtensions --target ES2022 --module NodeNext --moduleResolution NodeNext --skipLibCheck extensions/*.ts test/*.test.ts",
+    "typecheck": "tsc --noEmit --allowImportingTsExtensions --target ES2022 --module NodeNext --moduleResolution NodeNext --skipLibCheck extensions/*.ts",
     "pack:check": "npm pack --dry-run"
   },
   "peerDependencies": {