pi-interactive-shell 0.5.1 → 0.5.2

package/CHANGELOG.md

@@ -2,6 +2,12 @@
 
 All notable changes to the `pi-interactive-shell` extension will be documented in this file.
 
+## [0.5.2] - 2026-01-23
+
+### Fixed
+- **npx installation missing files** - The install script had a hardcoded file list that was missing 4 critical files (`key-encoding.ts`, `types.ts`, `tool-schema.ts`, `reattach-overlay.ts`). Now reads from `package.json`'s `files` array as the single source of truth, ensuring all files are always copied.
+- **Broken symlink handling** - Fixed skill symlink creation failing when a broken symlink already existed at the target path. `existsSync()` returns `false` for broken symlinks, causing the old code to skip removal. Now unconditionally attempts removal, correctly handling broken symlinks.
+
 ## [0.5.1] - 2026-01-22
 
 ### Fixed

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "pi-interactive-shell",
-  "version": "0.5.1",
+  "version": "0.5.2",
   "description": "Run AI coding agents as foreground subagents in pi TUI overlays with hands-free monitoring",
   "type": "module",
   "bin": {

package/scripts/install.js

@@ -1,6 +1,6 @@
 #!/usr/bin/env node
 
-import { existsSync, mkdirSync, cpSync, symlinkSync, unlinkSync, readFileSync } from "node:fs";
+import { existsSync, mkdirSync, cpSync, symlinkSync, unlinkSync, readFileSync, statSync } from "node:fs";
 import { join, dirname } from "node:path";
 import { homedir } from "node:os";
 import { execSync } from "node:child_process";
@@ -24,36 +24,34 @@ function main() {
 	log(`Creating ${EXTENSION_DIR}`);
 	mkdirSync(EXTENSION_DIR, { recursive: true });
 
-	// Files to copy
-	const files = [
-		"package.json",
-		"index.ts",
-		"config.ts",
-		"overlay-component.ts",
-		"pty-session.ts",
-		"session-manager.ts",
-		"README.md",
-		"SKILL.md",
-		"CHANGELOG.md",
-	];
+	// Read files list from package.json (single source of truth)
+	// Include package.json itself (npm auto-includes it but it's not in the files array)
+	const files = ["package.json", ...(pkg.files || [])];
 
-	// Copy files
-	for (const file of files) {
-		const src = join(packageRoot, file);
-		const dest = join(EXTENSION_DIR, file);
-		if (existsSync(src)) {
-			cpSync(src, dest);
-			log(`Copied ${file}`);
+	// Copy files and directories
+	for (const rawEntry of files) {
+		// Normalize: remove trailing slashes for consistent handling
+		const entry = rawEntry.replace(/\/+$/, "");
+		const src = join(packageRoot, entry);
+		const dest = join(EXTENSION_DIR, entry);
+
+		if (!existsSync(src)) {
+			continue;
 		}
-	}
 
-	// Copy scripts directory
-	const scriptsDir = join(packageRoot, "scripts");
-	const destScriptsDir = join(EXTENSION_DIR, "scripts");
-	if (existsSync(scriptsDir)) {
-		mkdirSync(destScriptsDir, { recursive: true });
-		cpSync(scriptsDir, destScriptsDir, { recursive: true });
-		log("Copied scripts/");
+		try {
+			const stat = statSync(src);
+			if (stat.isDirectory()) {
+				mkdirSync(dest, { recursive: true });
+				cpSync(src, dest, { recursive: true });
+				log(`Copied ${entry}/`);
+			} else {
+				cpSync(src, dest);
+				log(`Copied ${entry}`);
+			}
+		} catch (error) {
+			log(`Warning: Could not copy ${entry}: ${error.message}`);
+		}
 	}
 
 	// Run npm install in extension directory
@@ -72,8 +70,12 @@ function main() {
 	const skillTarget = join(EXTENSION_DIR, "SKILL.md");
 
 	try {
-		if (existsSync(skillLink)) {
+		// Remove existing entry if present (handles regular files, symlinks, and broken symlinks)
+		// Note: existsSync returns false for broken symlinks, so we unconditionally try unlink
+		try {
 			unlinkSync(skillLink);
+		} catch (e) {
+			if (e.code !== "ENOENT") throw e;
 		}
 		symlinkSync(skillTarget, skillLink);
 		log("Skill symlink created");