@vaclav-synacek/pi-coding-agent-termux 0.50.3-0 → 0.50.7-0

package/dist/core/package-manager.js

@@ -2,7 +2,8 @@ import { spawn, spawnSync } from "node:child_process";
 import { createHash } from "node:crypto";
 import { existsSync, mkdirSync, readdirSync, readFileSync, rmSync, statSync, writeFileSync } from "node:fs";
 import { homedir, tmpdir } from "node:os";
-import { basename, dirname, join, relative, resolve } from "node:path";
+import { basename, dirname, join, relative, resolve, sep } from "node:path";
+import ignore from "ignore";
 import { minimatch } from "minimatch";
 import { CONFIG_DIR_NAME } from "../config.js";
 import { looksLikeGitUrl } from "../utils/git.js";
@@ -13,6 +14,51 @@ const FILE_PATTERNS = {
     prompts: /\.md$/,
     themes: /\.json$/,
 };
+const IGNORE_FILE_NAMES = [".gitignore", ".ignore", ".fdignore"];
+function toPosixPath(p) {
+    return p.split(sep).join("/");
+}
+function prefixIgnorePattern(line, prefix) {
+    const trimmed = line.trim();
+    if (!trimmed)
+        return null;
+    if (trimmed.startsWith("#") && !trimmed.startsWith("\\#"))
+        return null;
+    let pattern = line;
+    let negated = false;
+    if (pattern.startsWith("!")) {
+        negated = true;
+        pattern = pattern.slice(1);
+    }
+    else if (pattern.startsWith("\\!")) {
+        pattern = pattern.slice(1);
+    }
+    if (pattern.startsWith("/")) {
+        pattern = pattern.slice(1);
+    }
+    const prefixed = prefix ? `${prefix}${pattern}` : pattern;
+    return negated ? `!${prefixed}` : prefixed;
+}
+function addIgnoreRules(ig, dir, rootDir) {
+    const relativeDir = relative(rootDir, dir);
+    const prefix = relativeDir ? `${toPosixPath(relativeDir)}/` : "";
+    for (const filename of IGNORE_FILE_NAMES) {
+        const ignorePath = join(dir, filename);
+        if (!existsSync(ignorePath))
+            continue;
+        try {
+            const content = readFileSync(ignorePath, "utf-8");
+            const patterns = content
+                .split(/\r?\n/)
+                .map((line) => prefixIgnorePattern(line, prefix))
+                .filter((line) => Boolean(line));
+            if (patterns.length > 0) {
+                ig.add(patterns);
+            }
+        }
+        catch { }
+    }
+}
 function isPattern(s) {
     return s.startsWith("!") || s.startsWith("+") || s.startsWith("-") || s.includes("*") || s.includes("?");
 }
@@ -29,10 +75,13 @@ function splitPatterns(entries) {
     }
     return { plain, patterns };
 }
-function collectFiles(dir, filePattern, skipNodeModules = true) {
+function collectFiles(dir, filePattern, skipNodeModules = true, ignoreMatcher, rootDir) {
     const files = [];
     if (!existsSync(dir))
         return files;
+    const root = rootDir ?? dir;
+    const ig = ignoreMatcher ?? ignore();
+    addIgnoreRules(ig, dir, root);
     try {
         const entries = readdirSync(dir, { withFileTypes: true });
         for (const entry of entries) {
@@ -53,8 +102,12 @@ function collectFiles(dir, filePattern, skipNodeModules = true) {
                     continue;
                 }
             }
+            const relPath = toPosixPath(relative(root, fullPath));
+            const ignorePath = isDir ? `${relPath}/` : relPath;
+            if (ig.ignores(ignorePath))
+                continue;
             if (isDir) {
-                files.push(...collectFiles(fullPath, filePattern, skipNodeModules));
+                files.push(...collectFiles(fullPath, filePattern, skipNodeModules, ig, root));
             }
             else if (isFile && filePattern.test(entry.name)) {
                 files.push(fullPath);
@@ -66,10 +119,13 @@ function collectFiles(dir, filePattern, skipNodeModules = true) {
     }
     return files;
 }
-function collectSkillEntries(dir, includeRootFiles = true) {
+function collectSkillEntries(dir, includeRootFiles = true, ignoreMatcher, rootDir) {
     const entries = [];
     if (!existsSync(dir))
         return entries;
+    const root = rootDir ?? dir;
+    const ig = ignoreMatcher ?? ignore();
+    addIgnoreRules(ig, dir, root);
     try {
         const dirEntries = readdirSync(dir, { withFileTypes: true });
         for (const entry of dirEntries) {
@@ -90,8 +146,12 @@ function collectSkillEntries(dir, includeRootFiles = true) {
                     continue;
                 }
             }
+            const relPath = toPosixPath(relative(root, fullPath));
+            const ignorePath = isDir ? `${relPath}/` : relPath;
+            if (ig.ignores(ignorePath))
+                continue;
             if (isDir) {
-                entries.push(...collectSkillEntries(fullPath, false));
+                entries.push(...collectSkillEntries(fullPath, false, ig, root));
             }
             else if (isFile) {
                 const isRootMd = includeRootFiles && entry.name.endsWith(".md");
@@ -114,6 +174,8 @@ function collectAutoPromptEntries(dir) {
     const entries = [];
     if (!existsSync(dir))
         return entries;
+    const ig = ignore();
+    addIgnoreRules(ig, dir, dir);
     try {
         const dirEntries = readdirSync(dir, { withFileTypes: true });
         for (const entry of dirEntries) {
@@ -131,6 +193,9 @@ function collectAutoPromptEntries(dir) {
                     continue;
                 }
             }
+            const relPath = toPosixPath(relative(dir, fullPath));
+            if (ig.ignores(relPath))
+                continue;
             if (isFile && entry.name.endsWith(".md")) {
                 entries.push(fullPath);
             }
@@ -145,6 +210,8 @@ function collectAutoThemeEntries(dir) {
     const entries = [];
     if (!existsSync(dir))
         return entries;
+    const ig = ignore();
+    addIgnoreRules(ig, dir, dir);
     try {
         const dirEntries = readdirSync(dir, { withFileTypes: true });
         for (const entry of dirEntries) {
@@ -162,6 +229,9 @@ function collectAutoThemeEntries(dir) {
                     continue;
                 }
             }
+            const relPath = toPosixPath(relative(dir, fullPath));
+            if (ig.ignores(relPath))
+                continue;
             if (isFile && entry.name.endsWith(".json")) {
                 entries.push(fullPath);
             }
@@ -213,6 +283,8 @@ function collectAutoExtensionEntries(dir) {
     const entries = [];
     if (!existsSync(dir))
         return entries;
+    const ig = ignore();
+    addIgnoreRules(ig, dir, dir);
     try {
         const dirEntries = readdirSync(dir, { withFileTypes: true });
         for (const entry of dirEntries) {
@@ -233,6 +305,10 @@ function collectAutoExtensionEntries(dir) {
                     continue;
                 }
             }
+            const relPath = toPosixPath(relative(dir, fullPath));
+            const ignorePath = isDir ? `${relPath}/` : relPath;
+            if (ig.ignores(ignorePath))
+                continue;
             if (isFile && (entry.name.endsWith(".ts") || entry.name.endsWith(".js"))) {
                 entries.push(fullPath);
             }
@@ -249,6 +325,19 @@ function collectAutoExtensionEntries(dir) {
     }
     return entries;
 }
+/**
+ * Collect resource files from a directory based on resource type.
+ * Extensions use smart discovery (index.ts in subdirs), others use recursive collection.
+ */
+function collectResourceFiles(dir, resourceType) {
+    if (resourceType === "skills") {
+        return collectSkillEntries(dir);
+    }
+    if (resourceType === "extensions") {
+        return collectAutoExtensionEntries(dir);
+    }
+    return collectFiles(dir, FILE_PATTERNS[resourceType]);
+}
 function matchesAnyPattern(filePath, patterns, baseDir) {
     const rel = relative(baseDir, filePath);
     const name = basename(filePath);
@@ -918,7 +1007,7 @@ export class DefaultPackageManager {
             const dir = join(packageRoot, resourceType);
             if (existsSync(dir)) {
                 // Collect all files from the directory (all enabled by default)
-                const files = resourceType === "skills" ? collectSkillEntries(dir) : collectFiles(dir, FILE_PATTERNS[resourceType]);
+                const files = collectResourceFiles(dir, resourceType);
                 for (const f of files) {
                     this.addResource(this.getTargetMap(accumulator, resourceType), f, metadata, true);
                 }
@@ -937,7 +1026,7 @@ export class DefaultPackageManager {
         const dir = join(packageRoot, resourceType);
         if (existsSync(dir)) {
             // Collect all files from the directory (all enabled by default)
-            const files = resourceType === "skills" ? collectSkillEntries(dir) : collectFiles(dir, FILE_PATTERNS[resourceType]);
+            const files = collectResourceFiles(dir, resourceType);
             for (const f of files) {
                 this.addResource(target, f, metadata, true);
             }
@@ -977,9 +1066,7 @@ export class DefaultPackageManager {
         if (!existsSync(conventionDir)) {
             return { allFiles: [], enabledByManifest: new Set() };
         }
-        const allFiles = resourceType === "skills"
-            ? collectSkillEntries(conventionDir)
-            : collectFiles(conventionDir, FILE_PATTERNS[resourceType]);
+        const allFiles = collectResourceFiles(conventionDir, resourceType);
         return { allFiles, enabledByManifest: new Set(allFiles) };
     }
     readPiManifest(packageRoot) {
@@ -1091,12 +1178,7 @@ export class DefaultPackageManager {
                     files.push(p);
                 }
                 else if (stats.isDirectory()) {
-                    if (resourceType === "skills") {
-                        files.push(...collectSkillEntries(p));
-                    }
-                    else {
-                        files.push(...collectFiles(p, FILE_PATTERNS[resourceType]));
-                    }
+                    files.push(...collectResourceFiles(p, resourceType));
                 }
             }
             catch {