@packmind/cli 0.13.0 → 0.13.1

package/main.cjs

@@ -3852,7 +3852,7 @@ var require_package = __commonJS({
   "apps/cli/package.json"(exports2, module2) {
     module2.exports = {
       name: "@packmind/cli",
-      version: "0.13.0",
+      version: "0.13.1",
       description: "A command-line interface for Packmind linting and code quality checks",
       private: false,
       bin: {
@@ -4195,6 +4195,13 @@ var RuleAddedEvent = class extends UserEvent {
   }
 };
 
+// packages/types/src/standards/events/RuleDeletedEvent.ts
+var RuleDeletedEvent = class extends UserEvent {
+  static {
+    this.eventName = "standards.rule.deleted";
+  }
+};
+
 // packages/types/src/standards/events/RuleUpdatedEvent.ts
 var RuleUpdatedEvent = class extends UserEvent {
   static {
@@ -4766,6 +4773,18 @@ var ExecuteSingleFileAstUseCase = class _ExecuteSingleFileAstUseCase {
 // apps/cli/src/application/services/GitService.ts
 var import_child_process = require("child_process");
 var path = __toESM(require("path"));
+
+// apps/cli/src/application/utils/pathUtils.ts
+function normalizePath(p) {
+  return p.replace(/\\/g, "/");
+}
+function pathStartsWith(filePath, prefix) {
+  const normalizedFile = normalizePath(filePath);
+  const normalizedPrefix = normalizePath(prefix);
+  return normalizedFile.startsWith(normalizedPrefix + "/") || normalizedFile === normalizedPrefix;
+}
+
+// apps/cli/src/application/services/GitService.ts
 var origin = "GitService";
 var GitService = class {
   constructor(logger2 = new PackmindLogger(origin), gitRunner = (cmd, opts) => {
@@ -4965,7 +4984,7 @@ ${error.message}`
       const { stdout } = this.gitRunner("diff --name-only HEAD", {
         cwd: gitRoot
       });
-      return stdout.trim().split("\n").filter((line) => line.length > 0).map((relativePath) => path.join(gitRoot, relativePath));
+      return stdout.trim().split("\n").filter((line) => line.length > 0).map((relativePath) => normalizePath(path.join(gitRoot, relativePath)));
     } catch (error) {
       if (error instanceof Error && error.message.includes("unknown revision")) {
         this.logger.debug(
@@ -4983,7 +5002,7 @@ ${error.message}`
     const { stdout } = this.gitRunner("diff --cached --name-only", {
       cwd: gitRoot
     });
-    return stdout.trim().split("\n").filter((line) => line.length > 0).map((relativePath) => path.join(gitRoot, relativePath));
+    return stdout.trim().split("\n").filter((line) => line.length > 0).map((relativePath) => normalizePath(path.join(gitRoot, relativePath)));
   }
   /**
    * Gets untracked files (new files not yet added to git).
@@ -4994,7 +5013,7 @@ ${error.message}`
     const { stdout } = this.gitRunner("ls-files --others --exclude-standard", {
       cwd: gitRoot
     });
-    return stdout.trim().split("\n").filter((line) => line.length > 0).map((relativePath) => path.join(gitRoot, relativePath));
+    return stdout.trim().split("\n").filter((line) => line.length > 0).map((relativePath) => normalizePath(path.join(gitRoot, relativePath)));
   }
   /**
    * Gets line-level diff information for modified files.
@@ -5066,7 +5085,7 @@ ${error.message}`
     for (const line of lines) {
       const fileMatch = line.match(/^diff --git a\/(.+) b\/(.+)$/);
       if (fileMatch) {
-        currentFile = path.join(gitRoot, fileMatch[2]);
+        currentFile = normalizePath(path.join(gitRoot, fileMatch[2]));
         continue;
       }
       const hunkMatch = line.match(/^@@ -\d+(?:,\d+)? \+(\d+)(?:,(\d+))? @@/);
@@ -5326,7 +5345,7 @@ var LintFilesInDirectoryUseCase = class {
       directoryForGitOps
     );
     const absoluteLintPath = absoluteUserPath;
-    if (!absoluteLintPath.startsWith(gitRepoRoot)) {
+    if (!pathStartsWith(absoluteLintPath, gitRepoRoot)) {
       throw new Error(
         `The path "${absoluteLintPath}" is not within the git repository at "${gitRepoRoot}"`
       );
@@ -5484,7 +5503,7 @@ var LintFilesInDirectoryUseCase = class {
     const violations = [];
     for (const file of files) {
       const fileViolations = [];
-      const relativeFilePath = file.path.startsWith(gitRepoRoot) ? file.path.substring(gitRepoRoot.length) : file.path;
+      const relativeFilePath = pathStartsWith(file.path, gitRepoRoot) ? file.path.substring(gitRepoRoot.length) : file.path;
       const normalizedFilePath = relativeFilePath.startsWith("/") ? relativeFilePath : "/" + relativeFilePath;
       this.logger.debug(
         `Processing file: absolute="${file.path}", relative="${normalizedFilePath}"`
@@ -5771,7 +5790,7 @@ var LintFilesLocallyUseCase = class {
     const allStandardsChecked = /* @__PURE__ */ new Set();
     for (const file of files) {
       const fileViolations = [];
-      const relativeFilePath = file.path.startsWith(basePath) ? file.path.substring(basePath.length) : file.path;
+      const relativeFilePath = pathStartsWith(file.path, basePath) ? file.path.substring(basePath.length) : file.path;
       const normalizedFilePath = relativeFilePath.startsWith("/") ? relativeFilePath : "/" + relativeFilePath;
       this.logger.debug(
         `Processing file: absolute="${file.path}", relative="${normalizedFilePath}"`
@@ -5889,7 +5908,7 @@ var LintFilesLocallyUseCase = class {
    */
   findMatchingTargets(absoluteFilePath, configs) {
     return configs.filter(
-      (config) => absoluteFilePath.startsWith(config.absoluteTargetPath + "/") || absoluteFilePath === config.absoluteTargetPath
+      (config) => pathStartsWith(absoluteFilePath, config.absoluteTargetPath)
     );
   }
   /**
@@ -5940,7 +5959,8 @@ var LintFilesLocallyUseCase = class {
 // apps/cli/src/infra/utils/readSkillDirectory.ts
 var import_promises = __toESM(require("fs/promises"));
 var import_path = __toESM(require("path"));
-function normalizePath(filePath) {
+var import_minimatch3 = require("minimatch");
+function normalizePath2(filePath) {
   let normalized = filePath.replace(/\\/g, "/");
   if (normalized.startsWith("/") || normalized.startsWith("\\")) {
     normalized = normalized.substring(1);
@@ -6018,6 +6038,13 @@ function isBinaryFile(filePath, buffer) {
 }
 var MAX_FILE_SIZE_BYTES = 10 * 1024 * 1024;
 var MAX_FILE_SIZE_MB = 10;
+var BLACKLIST_PATTERNS = ["**/.DS_Store"];
+function isBlacklisted(relativePath) {
+  const normalizedPath = relativePath.replace(/\\/g, "/");
+  return BLACKLIST_PATTERNS.some(
+    (pattern) => (0, import_minimatch3.minimatch)(normalizedPath, pattern, { dot: true })
+  );
+}
 async function readSkillDirectory(dirPath) {
   const files = [];
   async function readDir(currentPath, basePath) {
@@ -6025,6 +6052,9 @@ async function readSkillDirectory(dirPath) {
     for (const entry of entries) {
       const fullPath = import_path.default.join(currentPath, entry.name);
       const relativePath = import_path.default.relative(basePath, fullPath);
+      if (isBlacklisted(relativePath)) {
+        continue;
+      }
       if (entry.isDirectory()) {
         await readDir(fullPath, basePath);
       } else if (entry.isFile()) {
@@ -6037,7 +6067,7 @@ async function readSkillDirectory(dirPath) {
         }
         const buffer = await import_promises.default.readFile(fullPath);
         const isBinary = isBinaryFile(fullPath, buffer);
-        const normalizedPath = normalizePath(relativePath);
+        const normalizedPath = normalizePath2(relativePath);
         let content;
         if (isBinary) {
           content = buffer.toString("base64");
@@ -10531,7 +10561,7 @@ var ConfigFileRepository = class {
    * @returns Array of directory paths that contain a packmind.json file
    */
   async findDescendantConfigs(directory) {
-    const normalizedDir = path9.resolve(directory);
+    const normalizedDir = normalizePath(path9.resolve(directory));
     const results = [];
     const searchRecursively = async (currentDir) => {
       let entries;
@@ -10547,7 +10577,7 @@ var ConfigFileRepository = class {
         if (this.EXCLUDED_DIRECTORIES.includes(entry.name)) {
           continue;
         }
-        const entryPath = path9.join(currentDir, entry.name);
+        const entryPath = normalizePath(path9.join(currentDir, entry.name));
         const config = await this.readConfig(entryPath);
         if (config) {
           results.push(entryPath);
@@ -10569,19 +10599,21 @@ var ConfigFileRepository = class {
   async readHierarchicalConfig(startDirectory, stopDirectory) {
     const configs = [];
     const configPaths = [];
-    const normalizedStart = path9.resolve(startDirectory);
-    const normalizedStop = stopDirectory ? path9.resolve(stopDirectory) : null;
+    const normalizedStart = normalizePath(path9.resolve(startDirectory));
+    const normalizedStop = stopDirectory ? normalizePath(path9.resolve(stopDirectory)) : null;
     let currentDir = normalizedStart;
     while (true) {
       const config = await this.readConfig(currentDir);
       if (config) {
         configs.push(config);
-        configPaths.push(path9.join(currentDir, this.CONFIG_FILENAME));
+        configPaths.push(
+          normalizePath(path9.join(currentDir, this.CONFIG_FILENAME))
+        );
       }
       if (normalizedStop !== null && currentDir === normalizedStop) {
         break;
       }
-      const parentDir = path9.dirname(currentDir);
+      const parentDir = normalizePath(path9.dirname(currentDir));
       if (parentDir === currentDir) {
         break;
       }
@@ -10610,8 +10642,8 @@ var ConfigFileRepository = class {
    * @returns All configs found with their target paths
    */
   async findAllConfigsInTree(startDirectory, stopDirectory) {
-    const normalizedStart = path9.resolve(startDirectory);
-    const normalizedStop = stopDirectory ? path9.resolve(stopDirectory) : null;
+    const normalizedStart = normalizePath(path9.resolve(startDirectory));
+    const normalizedStop = stopDirectory ? normalizePath(path9.resolve(stopDirectory)) : null;
     const basePath = normalizedStop ?? normalizedStart;
     const configsMap = /* @__PURE__ */ new Map();
     let currentDir = normalizedStart;
@@ -10628,7 +10660,7 @@ var ConfigFileRepository = class {
       if (normalizedStop !== null && currentDir === normalizedStop) {
         break;
       }
-      const parentDir = path9.dirname(currentDir);
+      const parentDir = normalizePath(path9.dirname(currentDir));
       if (parentDir === currentDir) {
         break;
       }
@@ -10637,18 +10669,19 @@ var ConfigFileRepository = class {
     const searchRoot = normalizedStop ?? normalizedStart;
     const descendantDirs = await this.findDescendantConfigs(searchRoot);
     for (const descendantDir of descendantDirs) {
-      if (configsMap.has(descendantDir)) {
+      const normalizedDescendantDir = normalizePath(descendantDir);
+      if (configsMap.has(normalizedDescendantDir)) {
         continue;
       }
-      const config = await this.readConfig(descendantDir);
+      const config = await this.readConfig(normalizedDescendantDir);
       if (config) {
         const targetPath = this.computeRelativeTargetPath(
-          descendantDir,
+          normalizedDescendantDir,
           basePath
         );
-        configsMap.set(descendantDir, {
+        configsMap.set(normalizedDescendantDir, {
           targetPath,
-          absoluteTargetPath: descendantDir,
+          absoluteTargetPath: normalizedDescendantDir,
           packages: config.packages
         });
       }
@@ -10671,10 +10704,12 @@ var ConfigFileRepository = class {
     };
   }
   computeRelativeTargetPath(absolutePath, basePath) {
-    if (absolutePath === basePath) {
+    const normalizedAbsolute = normalizePath(absolutePath);
+    const normalizedBase = normalizePath(basePath);
+    if (normalizedAbsolute === normalizedBase) {
       return "/";
     }
-    const relativePath = absolutePath.substring(basePath.length);
+    const relativePath = normalizedAbsolute.substring(normalizedBase.length);
     return relativePath.startsWith("/") ? relativePath : "/" + relativePath;
   }
 };

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@packmind/cli",
-  "version": "0.13.0",
+  "version": "0.13.1",
   "description": "A command-line interface for Packmind linting and code quality checks",
   "private": false,
   "bin": {