@rpamis/comet 0.3.7 → 0.3.9

package/dist/utils/file-system.js

@@ -1,17 +1,52 @@
 import { promises as fs } from 'fs';
 import path from 'path';
+/**
+ * Resolve symlinks in a path, handling broken symlinks by following their
+ * readlink target. Falls back to the original path if resolution fails.
+ */
+async function resolveSymlinkPath(filePath) {
+    try {
+        return await fs.realpath(filePath);
+    }
+    catch {
+        // Path doesn't fully exist — walk up to find the deepest existing ancestor
+        const dir = path.dirname(filePath);
+        if (dir === filePath)
+            return filePath; // filesystem root
+        const resolvedDir = await resolveSymlinkPath(dir);
+        const base = path.basename(filePath);
+        // Check if this segment is a broken symlink and follow its target
+        try {
+            const stat = await fs.lstat(path.join(resolvedDir, base));
+            if (stat.isSymbolicLink()) {
+                const target = await fs.readlink(path.join(resolvedDir, base));
+                return path.resolve(resolvedDir, target);
+            }
+        }
+        catch {
+            // Segment doesn't exist — return as-is
+        }
+        return path.join(resolvedDir, base);
+    }
+}
 /**
  * Ensure a directory exists, creating it recursively if needed.
+ * Resolves symlinks so that broken symlink targets are created correctly.
  */
 export async function ensureDir(dir) {
-    await fs.mkdir(dir, { recursive: true });
+    const resolved = await resolveSymlinkPath(dir);
+    await fs.mkdir(resolved, { recursive: true });
 }
 /**
  * Copy a file from src to dest, creating parent directories if needed.
+ * Resolves symlinks in the destination path so files are written to the
+ * actual target location when dest contains symlinks (e.g., skill dirs
+ * symlinked from ~/.claude/skills/ to ~/.agents/skills/).
  */
 export async function copyFile(src, dest) {
-    await ensureDir(path.dirname(dest));
-    await fs.copyFile(src, dest);
+    const resolvedDest = await resolveSymlinkPath(dest);
+    await ensureDir(path.dirname(resolvedDest));
+    await fs.copyFile(src, resolvedDest);
 }
 /**
  * Check if a file or directory exists.
@@ -34,10 +69,12 @@ export async function readJson(filePath) {
 }
 /**
  * Write content to a file, creating parent directories if needed.
+ * Resolves symlinks so files are written to the actual target location.
  */
 export async function writeFile(filePath, content) {
-    await ensureDir(path.dirname(filePath));
-    await fs.writeFile(filePath, content, 'utf-8');
+    const resolved = await resolveSymlinkPath(filePath);
+    await ensureDir(path.dirname(resolved));
+    await fs.writeFile(resolved, content, 'utf-8');
 }
 /**
  * List entries in a directory. Returns empty array if directory doesn't exist.
@@ -54,4 +91,62 @@ export async function readDir(dirPath) {
         throw error;
     }
 }
+/**
+ * Returns true when an error means the path was simply not found. ENOENT is
+ * the only non-fatal outcome for the removal helpers below; all other errors
+ * (permissions, IO) are reported as failures instead of being masked as
+ * "already gone".
+ */
+function isNotFoundError(error) {
+    return error?.code === 'ENOENT';
+}
+/**
+ * Remove a file. Returns true if the file existed and was removed.
+ * Operates on the path directly so a symlink entry is removed rather than its
+ * resolved target (avoids deleting files the symlink merely points at).
+ */
+export async function removeFile(filePath) {
+    try {
+        await fs.unlink(filePath);
+        return true;
+    }
+    catch {
+        // Not found or failed (permissions/IO): nothing was removed.
+        return false;
+    }
+}
+/**
+ * Remove a directory recursively. Returns true if the directory existed and was removed.
+ * Symlinked directories are unlinked directly rather than recursed into, so the
+ * directory a symlink points at is never deleted.
+ */
+export async function removeDir(dirPath) {
+    try {
+        // lstat does not follow symlinks; unlink a symlinked dir instead of rm-ing its target.
+        const stat = await fs.lstat(dirPath);
+        if (stat.isSymbolicLink()) {
+            await fs.unlink(dirPath);
+            return true;
+        }
+        await fs.rm(dirPath, { recursive: true, force: true });
+        return true;
+    }
+    catch (error) {
+        return isNotFoundError(error);
+    }
+}
+/**
+ * Check if a directory is empty. A missing directory is treated as empty;
+ * unreadable directories (permissions/IO) return false so callers never delete
+ * a directory they could not inspect.
+ */
+export async function isDirEmpty(dirPath) {
+    try {
+        const entries = await fs.readdir(dirPath);
+        return entries.length === 0;
+    }
+    catch (error) {
+        return isNotFoundError(error);
+    }
+}
 //# sourceMappingURL=file-system.js.map

package/dist/utils/file-system.js.map

@@ -1 +1 @@
-{"version":3,"file":"file-system.js","sourceRoot":"","sources":["../../src/utils/file-system.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,QAAQ,IAAI,EAAE,EAAE,MAAM,IAAI,CAAC;AACpC,OAAO,IAAI,MAAM,MAAM,CAAC;AAExB;;GAEG;AACH,MAAM,CAAC,KAAK,UAAU,SAAS,CAAC,GAAW;IACzC,MAAM,EAAE,CAAC,KAAK,CAAC,GAAG,EAAE,EAAE,SAAS,EAAE,IAAI,EAAE,CAAC,CAAC;AAC3C,CAAC;AAED;;GAEG;AACH,MAAM,CAAC,KAAK,UAAU,QAAQ,CAAC,GAAW,EAAE,IAAY;IACtD,MAAM,SAAS,CAAC,IAAI,CAAC,OAAO,CAAC,IAAI,CAAC,CAAC,CAAC;IACpC,MAAM,EAAE,CAAC,QAAQ,CAAC,GAAG,EAAE,IAAI,CAAC,CAAC;AAC/B,CAAC;AAED;;GAEG;AACH,MAAM,CAAC,KAAK,UAAU,UAAU,CAAC,QAAgB;IAC/C,IAAI,CAAC;QACH,MAAM,EAAE,CAAC,MAAM,CAAC,QAAQ,CAAC,CAAC;QAC1B,OAAO,IAAI,CAAC;IACd,CAAC;IAAC,MAAM,CAAC;QACP,OAAO,KAAK,CAAC;IACf,CAAC;AACH,CAAC;AAED;;GAEG;AACH,MAAM,CAAC,KAAK,UAAU,QAAQ,CAAc,QAAgB;IAC1D,MAAM,OAAO,GAAG,MAAM,EAAE,CAAC,QAAQ,CAAC,QAAQ,EAAE,OAAO,CAAC,CAAC;IACrD,OAAO,IAAI,CAAC,KAAK,CAAC,OAAO,CAAM,CAAC;AAClC,CAAC;AAED;;GAEG;AACH,MAAM,CAAC,KAAK,UAAU,SAAS,CAAC,QAAgB,EAAE,OAAe;IAC/D,MAAM,SAAS,CAAC,IAAI,CAAC,OAAO,CAAC,QAAQ,CAAC,CAAC,CAAC;IACxC,MAAM,EAAE,CAAC,SAAS,CAAC,QAAQ,EAAE,OAAO,EAAE,OAAO,CAAC,CAAC;AACjD,CAAC;AAED;;GAEG;AACH,MAAM,CAAC,KAAK,UAAU,OAAO,CAAC,OAAe;IAC3C,IAAI,CAAC;QACH,OAAO,MAAM,EAAE,CAAC,OAAO,CAAC,OAAO,CAAC,CAAC;IACnC,CAAC;IAAC,OAAO,KAAK,EAAE,CAAC;QACf,MAAM,IAAI,GAAI,KAA+B,EAAE,IAAI,CAAC;QACpD,IAAI,IAAI,KAAK,QAAQ,IAAI,IAAI,KAAK,SAAS,EAAE,CAAC;YAC5C,OAAO,EAAE,CAAC;QACZ,CAAC;QAED,MAAM,KAAK,CAAC;IACd,CAAC;AACH,CAAC"}
+{"version":3,"file":"file-system.js","sourceRoot":"","sources":["../../src/utils/file-system.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,QAAQ,IAAI,EAAE,EAAE,MAAM,IAAI,CAAC;AACpC,OAAO,IAAI,MAAM,MAAM,CAAC;AAExB;;;GAGG;AACH,KAAK,UAAU,kBAAkB,CAAC,QAAgB;IAChD,IAAI,CAAC;QACH,OAAO,MAAM,EAAE,CAAC,QAAQ,CAAC,QAAQ,CAAC,CAAC;IACrC,CAAC;IAAC,MAAM,CAAC;QACP,2EAA2E;QAC3E,MAAM,GAAG,GAAG,IAAI,CAAC,OAAO,CAAC,QAAQ,CAAC,CAAC;QACnC,IAAI,GAAG,KAAK,QAAQ;YAAE,OAAO,QAAQ,CAAC,CAAC,kBAAkB;QAEzD,MAAM,WAAW,GAAG,MAAM,kBAAkB,CAAC,GAAG,CAAC,CAAC;QAClD,MAAM,IAAI,GAAG,IAAI,CAAC,QAAQ,CAAC,QAAQ,CAAC,CAAC;QAErC,kEAAkE;QAClE,IAAI,CAAC;YACH,MAAM,IAAI,GAAG,MAAM,EAAE,CAAC,KAAK,CAAC,IAAI,CAAC,IAAI,CAAC,WAAW,EAAE,IAAI,CAAC,CAAC,CAAC;YAC1D,IAAI,IAAI,CAAC,cAAc,EAAE,EAAE,CAAC;gBAC1B,MAAM,MAAM,GAAG,MAAM,EAAE,CAAC,QAAQ,CAAC,IAAI,CAAC,IAAI,CAAC,WAAW,EAAE,IAAI,CAAC,CAAC,CAAC;gBAC/D,OAAO,IAAI,CAAC,OAAO,CAAC,WAAW,EAAE,MAAM,CAAC,CAAC;YAC3C,CAAC;QACH,CAAC;QAAC,MAAM,CAAC;YACP,uCAAuC;QACzC,CAAC;QAED,OAAO,IAAI,CAAC,IAAI,CAAC,WAAW,EAAE,IAAI,CAAC,CAAC;IACtC,CAAC;AACH,CAAC;AAED;;;GAGG;AACH,MAAM,CAAC,KAAK,UAAU,SAAS,CAAC,GAAW;IACzC,MAAM,QAAQ,GAAG,MAAM,kBAAkB,CAAC,GAAG,CAAC,CAAC;IAC/C,MAAM,EAAE,CAAC,KAAK,CAAC,QAAQ,EAAE,EAAE,SAAS,EAAE,IAAI,EAAE,CAAC,CAAC;AAChD,CAAC;AAED;;;;;GAKG;AACH,MAAM,CAAC,KAAK,UAAU,QAAQ,CAAC,GAAW,EAAE,IAAY;IACtD,MAAM,YAAY,GAAG,MAAM,kBAAkB,CAAC,IAAI,CAAC,CAAC;IACpD,MAAM,SAAS,CAAC,IAAI,CAAC,OAAO,CAAC,YAAY,CAAC,CAAC,CAAC;IAC5C,MAAM,EAAE,CAAC,QAAQ,CAAC,GAAG,EAAE,YAAY,CAAC,CAAC;AACvC,CAAC;AAED;;GAEG;AACH,MAAM,CAAC,KAAK,UAAU,UAAU,CAAC,QAAgB;IAC/C,IAAI,CAAC;QACH,MAAM,EAAE,CAAC,MAAM,CAAC,QAAQ,CAAC,CAAC;QAC1B,OAAO,IAAI,CAAC;IACd,CAAC;IAAC,MAAM,CAAC;QACP,OAAO,KAAK,CAAC;IACf,CAAC;AACH,CAAC;AAED;;GAEG;AACH,MAAM,CAAC,KAAK,UAAU,QAAQ,CAAc,QAAgB;IAC1D,MAAM,OAAO,GAAG,MAAM,EAAE,CAAC,QAAQ,CAAC,QAAQ,EAAE,OAAO,CAAC,CAAC;IACrD,OAAO,IAAI,CAAC,KAAK,CAAC,OAAO,CAAM,CAAC;AAClC,CAAC;AAED;;;GAGG;AACH,MAAM,CAAC,KAAK,UAAU,SAAS,CAAC,QAAgB,EAAE,OAAe;IAC/D,MAAM,QAAQ,GAAG,MAAM,kBAAkB,CAAC,QAAQ,CAAC,CAAC;IACpD,MAAM,SAAS,CAAC,IAAI,CAAC,OAAO,CAAC,QAAQ,CAAC,CAAC,CAAC;IACxC,MAAM,EAAE,CAAC,SAAS,CAAC,QAAQ,EAAE,OAAO,EAAE,OAAO,CAAC,CAAC;AACjD,CAAC;AAED;;GAEG;AACH,MAAM,CAAC,KAAK,UAAU,OAAO,CAAC,OAAe;IAC3C,IAAI,CAAC;QACH,OAAO,MAAM,EAAE,CAAC,OAAO,CAAC,OAAO,CAAC,CAAC;IACnC,CAAC;IAAC,OAAO,KAAK,EAAE,CAAC;QACf,MAAM,IAAI,GAAI,KAA+B,EAAE,IAAI,CAAC;QACpD,IAAI,IAAI,KAAK,QAAQ,IAAI,IAAI,KAAK,SAAS,EAAE,CAAC;YAC5C,OAAO,EAAE,CAAC;QACZ,CAAC;QAED,MAAM,KAAK,CAAC;IACd,CAAC;AACH,CAAC;AAED;;;;;GAKG;AACH,SAAS,eAAe,CAAC,KAAc;IACrC,OAAQ,KAA2C,EAAE,IAAI,KAAK,QAAQ,CAAC;AACzE,CAAC;AAED;;;;GAIG;AACH,MAAM,CAAC,KAAK,UAAU,UAAU,CAAC,QAAgB;IAC/C,IAAI,CAAC;QACH,MAAM,EAAE,CAAC,MAAM,CAAC,QAAQ,CAAC,CAAC;QAC1B,OAAO,IAAI,CAAC;IACd,CAAC;IAAC,MAAM,CAAC;QACP,6DAA6D;QAC7D,OAAO,KAAK,CAAC;IACf,CAAC;AACH,CAAC;AAED;;;;GAIG;AACH,MAAM,CAAC,KAAK,UAAU,SAAS,CAAC,OAAe;IAC7C,IAAI,CAAC;QACH,uFAAuF;QACvF,MAAM,IAAI,GAAG,MAAM,EAAE,CAAC,KAAK,CAAC,OAAO,CAAC,CAAC;QACrC,IAAI,IAAI,CAAC,cAAc,EAAE,EAAE,CAAC;YAC1B,MAAM,EAAE,CAAC,MAAM,CAAC,OAAO,CAAC,CAAC;YACzB,OAAO,IAAI,CAAC;QACd,CAAC;QACD,MAAM,EAAE,CAAC,EAAE,CAAC,OAAO,EAAE,EAAE,SAAS,EAAE,IAAI,EAAE,KAAK,EAAE,IAAI,EAAE,CAAC,CAAC;QACvD,OAAO,IAAI,CAAC;IACd,CAAC;IAAC,OAAO,KAAK,EAAE,CAAC;QACf,OAAO,eAAe,CAAC,KAAK,CAAC,CAAC;IAChC,CAAC;AACH,CAAC;AAED;;;;GAIG;AACH,MAAM,CAAC,KAAK,UAAU,UAAU,CAAC,OAAe;IAC9C,IAAI,CAAC;QACH,MAAM,OAAO,GAAG,MAAM,EAAE,CAAC,OAAO,CAAC,OAAO,CAAC,CAAC;QAC1C,OAAO,OAAO,CAAC,MAAM,KAAK,CAAC,CAAC;IAC9B,CAAC;IAAC,OAAO,KAAK,EAAE,CAAC;QACf,OAAO,eAAe,CAAC,KAAK,CAAC,CAAC;IAChC,CAAC;AACH,CAAC"}

package/package.json

@@ -1,64 +1,69 @@
-{
-  "name": "@rpamis/comet",
-  "version": "0.3.7",
-  "description": "OpenSpec + Superpowers dual-star development workflow",
-  "keywords": [
-    "comet",
-    "openspec",
-    "superpowers",
-    "skills",
-    "workflow"
-  ],
-  "license": "MIT",
-  "author": "benym",
-  "type": "module",
-  "publishConfig": {
-    "access": "public"
-  },
-  "bin": {
-    "comet": "./bin/comet.js"
-  },
-  "files": [
-    "dist",
-    "bin",
-    "assets",
-    "scripts/postinstall.js",
-    "!dist/**/*.test.js",
-    "!dist/**/__tests__"
-  ],
-  "scripts": {
-    "build": "node build.js",
-    "dev": "tsc --watch",
-    "test": "vitest run",
-    "test:coverage": "vitest run --coverage",
-    "test:watch": "vitest",
-    "lint": "eslint src/",
-    "lint:fix": "eslint src/ --fix",
-    "format": "prettier --write src/",
-    "format:check": "prettier --check src/",
-    "test:shell": "node scripts/run-bats.js test/shell/*.bats",
-    "benchmark:context": "node scripts/context-compression-benchmark.mjs",
-    "benchmark:execution": "node scripts/context-execution-benchmark.mjs",
-    "prepare": "pnpm run build",
-    "prepublishOnly": "node scripts/prepublish-check.js && pnpm run build",
-    "postinstall": "node scripts/postinstall.js"
-  },
-  "packageManager": "pnpm@10.18.3",
-  "engines": {
-    "node": ">=20"
-  },
-  "devDependencies": {
-    "@eslint/js": "^10.0.1",
-    "@types/node": "^24.2.0",
-    "@vitest/coverage-v8": "^4.1.6",
-    "eslint": "^10.4.0",
-    "prettier": "^3.8.3",
-    "typescript": "^5.9.3",
-    "typescript-eslint": "^8.59.3",
-    "vitest": "^4.1.6"
-  },
-  "dependencies": {
-    "@inquirer/prompts": "^8.4.3",
-    "commander": "^14.0.0"
-  }
-}
+{
+  "name": "@rpamis/comet",
+  "version": "0.3.9",
+  "description": "Agent Skill Harness Phase-Guarded Automation From Idea To Archive",
+  "keywords": [
+    "comet",
+    "openspec",
+    "superpowers",
+    "skills",
+    "workflow"
+  ],
+  "license": "MIT",
+  "author": "benym",
+  "type": "module",
+  "publishConfig": {
+    "access": "public"
+  },
+  "bin": {
+    "comet": "./bin/comet.js"
+  },
+  "files": [
+    "dist",
+    "bin",
+    "assets",
+    "scripts/postinstall.js",
+    "!dist/**/*.test.js",
+    "!dist/**/__tests__"
+  ],
+  "scripts": {
+    "build": "node build.js",
+    "dev": "tsc --watch",
+    "test": "vitest run",
+    "test:coverage": "vitest run --coverage",
+    "test:watch": "vitest",
+    "lint": "eslint src/",
+    "lint:fix": "eslint src/ --fix",
+    "format": "prettier --write src/",
+    "format:check": "prettier --check src/",
+    "test:shell": "node scripts/run-bats.js test/shell/*.bats",
+    "benchmark:context": "node scripts/context-compression-benchmark.mjs",
+    "benchmark:execution": "node scripts/context-execution-benchmark.mjs",
+    "prepare": "husky && pnpm run build",
+    "prepublishOnly": "node scripts/prepublish-check.js && pnpm run build",
+    "postinstall": "node scripts/postinstall.js"
+  },
+  "lint-staged": {
+    "src/**/*.{ts,tsx,js,mjs,cjs,json,md,yaml,yml}": "prettier --write"
+  },
+  "packageManager": "pnpm@10.18.3",
+  "engines": {
+    "node": ">=20"
+  },
+  "devDependencies": {
+    "@eslint/js": "^10.0.1",
+    "@types/node": "^24.2.0",
+    "@vitest/coverage-v8": "^4.1.6",
+    "eslint": "^10.4.0",
+    "husky": "^9.1.7",
+    "lint-staged": "^17.0.7",
+    "prettier": "^3.8.3",
+    "typescript": "^5.9.3",
+    "typescript-eslint": "^8.59.3",
+    "vitest": "^4.1.6"
+  },
+  "dependencies": {
+    "@inquirer/prompts": "^8.4.3",
+    "commander": "^14.0.0"
+  }
+}

package/scripts/postinstall.js

@@ -1,44 +1,44 @@
-#!/usr/bin/env node
-
-/**
- * Postinstall script that hints about comet init.
- *
- * The tip is suppressed when:
- * - CI=true environment variable is set
- * - COMET_NO_HINTS=1 environment variable is set
- * - dist/ directory doesn't exist (dev setup scenario)
- */
-
-import { promises as fs } from 'fs';
-import path from 'path';
-import { fileURLToPath } from 'url';
-
-const __filename = fileURLToPath(import.meta.url);
-const __dirname = path.dirname(__filename);
-
-function shouldSkip() {
-  if (process.env.CI === 'true' || process.env.CI === '1') return true;
-  if (process.env.COMET_NO_HINTS === '1') return true;
-  return false;
-}
-
-async function distExists() {
-  try {
-    const stat = await fs.stat(path.join(__dirname, '..', 'dist'));
-    return stat.isDirectory();
-  } catch {
-    return false;
-  }
-}
-
-async function main() {
-  try {
-    if (shouldSkip()) return;
-    if (!(await distExists())) return;
-    console.log(`\nTip: Run 'comet init' to set up Comet workflow in your project`);
-  } catch {
-    // Never break npm install
-  }
-}
-
-main().catch(() => process.exit(0));
+#!/usr/bin/env node
+
+/**
+ * Postinstall script that hints about comet init.
+ *
+ * The tip is suppressed when:
+ * - CI=true environment variable is set
+ * - COMET_NO_HINTS=1 environment variable is set
+ * - dist/ directory doesn't exist (dev setup scenario)
+ */
+
+import { promises as fs } from 'fs';
+import path from 'path';
+import { fileURLToPath } from 'url';
+
+const __filename = fileURLToPath(import.meta.url);
+const __dirname = path.dirname(__filename);
+
+function shouldSkip() {
+  if (process.env.CI === 'true' || process.env.CI === '1') return true;
+  if (process.env.COMET_NO_HINTS === '1') return true;
+  return false;
+}
+
+async function distExists() {
+  try {
+    const stat = await fs.stat(path.join(__dirname, '..', 'dist'));
+    return stat.isDirectory();
+  } catch {
+    return false;
+  }
+}
+
+async function main() {
+  try {
+    if (shouldSkip()) return;
+    if (!(await distExists())) return;
+    console.log(`\nTip: Run 'comet init' to set up Comet workflow in your project`);
+  } catch {
+    // Never break npm install
+  }
+}
+
+main().catch(() => process.exit(0));