@dreb/coding-agent 2.4.3 → 2.4.4

package/dist/core/package-manager.js

@@ -68,7 +68,9 @@ function addIgnoreRules(ig, dir, rootDir) {
                 ig.add(patterns);
             }
         }
-        catch { }
+        catch {
+            /* .gitignore may not exist or be unreadable — ignore patterns are optional */
+        }
     }
 }
 function isPattern(s) {
@@ -111,6 +113,7 @@ function collectFiles(dir, filePattern, skipNodeModules = true, ignoreMatcher, r
                     isFile = stats.isFile();
                 }
                 catch {
+                    // Broken symlink or deleted file — skip entry
                     continue;
                 }
             }
@@ -127,7 +130,7 @@ function collectFiles(dir, filePattern, skipNodeModules = true, ignoreMatcher, r
         }
     }
     catch {
-        // Ignore errors
+        // Directory may not exist or be unreadable — return empty list
     }
     return files;
 }
@@ -155,6 +158,7 @@ function collectSkillEntries(dir, includeRootFiles = true, ignoreMatcher, rootDi
                     isFile = stats.isFile();
                 }
                 catch {
+                    // Broken symlink or deleted file — skip entry
                     continue;
                 }
             }
@@ -175,7 +179,7 @@ function collectSkillEntries(dir, includeRootFiles = true, ignoreMatcher, rootDi
         }
     }
     catch {
-        // Ignore errors
+        // Directory may not exist or be unreadable — return empty list
     }
     return entries;
 }
@@ -233,6 +237,7 @@ function collectAutoPromptEntries(dir) {
                     isFile = statSync(fullPath).isFile();
                 }
                 catch {
+                    // Broken symlink — skip entry
                     continue;
                 }
             }
@@ -245,7 +250,7 @@ function collectAutoPromptEntries(dir) {
         }
     }
     catch {
-        // Ignore errors
+        // Directory may not exist or be unreadable — return empty list
     }
     return entries;
 }
@@ -269,6 +274,7 @@ function collectAutoThemeEntries(dir) {
                     isFile = statSync(fullPath).isFile();
                 }
                 catch {
+                    // Broken symlink — skip entry
                     continue;
                 }
             }
@@ -281,7 +287,7 @@ function collectAutoThemeEntries(dir) {
         }
     }
     catch {
-        // Ignore errors
+        // Directory may not exist or be unreadable — return empty list
     }
     return entries;
 }
@@ -292,6 +298,7 @@ function readDrebManifestFile(packageJsonPath) {
         return pkg.dreb ?? null;
     }
     catch {
+        // Malformed package.json — fall back to convention-based discovery
         return null;
     }
 }
@@ -351,6 +358,7 @@ function collectAutoExtensionEntries(dir) {
                     isFile = stats.isFile();
                 }
                 catch {
+                    // Broken symlink or deleted file — skip entry
                     continue;
                 }
             }
@@ -370,7 +378,7 @@ function collectAutoExtensionEntries(dir) {
         }
     }
     catch {
-        // Ignore errors
+        // Directory may not exist or be unreadable — return empty list
     }
     return entries;
 }
@@ -854,6 +862,7 @@ export class DefaultPackageManager {
             }
         }
         catch {
+            // Package resource collection failed — skip this package
             return;
         }
     }
@@ -987,6 +996,7 @@ export class DefaultPackageManager {
             return latestVersion !== installedVersion;
         }
         catch {
+            // Update check failure — conservatively report no updates
             return false;
         }
     }
@@ -1000,6 +1010,7 @@ export class DefaultPackageManager {
             return pkg.version;
         }
         catch {
+            // Version read failed — treat as unknown
             return undefined;
         }
     }
@@ -1025,6 +1036,7 @@ export class DefaultPackageManager {
             return localHead.trim() !== remoteHead.trim();
         }
         catch {
+            // Update check failure — conservatively report no updates
             return false;
         }
     }
@@ -1075,6 +1087,7 @@ export class DefaultPackageManager {
             };
         }
         catch {
+            // Upstream ref detection failed — fall back to origin/HEAD
             await this.runCommand("git", ["remote", "set-head", "origin", "-a"], { cwd: installedPath }).catch(() => { });
             const head = await this.runCommandCapture("git", ["rev-parse", "origin/HEAD"], {
                 cwd: installedPath,
@@ -1083,7 +1096,7 @@ export class DefaultPackageManager {
             const originHeadRef = await this.runCommandCapture("git", ["symbolic-ref", "refs/remotes/origin/HEAD"], {
                 cwd: installedPath,
                 timeoutMs: NETWORK_TIMEOUT_MS,
-            }).catch(() => "");
+            }).catch(() => "" /* Symbolic ref resolution failed — use empty string fallback */);
             const branch = originHeadRef.trim().replace(/^refs\/remotes\/origin\//, "");
             if (branch) {
                 return {
@@ -1119,6 +1132,7 @@ export class DefaultPackageManager {
             return branch ? `refs/heads/${branch}` : undefined;
         }
         catch {
+            // Upstream ref detection failed — return undefined
             return undefined;
         }
     }
@@ -1327,6 +1341,7 @@ export class DefaultPackageManager {
                 rmSync(current, { recursive: true, force: true });
             }
             catch {
+                // rmSync failed — stop pruning
                 break;
             }
             current = dirname(current);
@@ -1535,6 +1550,7 @@ export class DefaultPackageManager {
             return pkg.dreb ?? null;
         }
         catch {
+            // Malformed package.json — fall back to convention-based discovery
             return null;
         }
     }
@@ -1642,7 +1658,7 @@ export class DefaultPackageManager {
                 }
             }
             catch {
-                // Ignore errors
+                // File/dir inaccessible — skip
             }
         }
         return files;