@lousy-agents/cli 2.9.0 → 2.10.0

package/dist/index.js

@@ -33041,6 +33041,17 @@ function defaultExec(command, args, options) {
  */ function createScriptDiscoveryGateway() {
     return new FileSystemScriptDiscoveryGateway();
 }
+/**
+ * Creates a FeedbackLoopCommandsGateway that discovers mandatory commands from package.json scripts.
+ */ function createFeedbackLoopCommandsGateway(scriptGateway) {
+    const gateway = scriptGateway ?? createScriptDiscoveryGateway();
+    return {
+        async getMandatoryCommands (targetDir) {
+            const scripts = await gateway.discoverScripts(targetDir);
+            return scripts.filter((s)=>s.isMandatory).map((s)=>s.name);
+        }
+    };
+}
 
 ;// CONCATENATED MODULE: ./src/gateways/skill-file-gateway.ts
 /**
@@ -55168,24 +55179,43 @@ const remark = unified().use(remarkParse).use(remarkStringify).freeze()
                     overallQualityScore: 0,
                     suggestions: [
                         "No agent instruction files found. Supported formats: .github/copilot-instructions.md, .github/instructions/*.md, .github/agents/*.md, AGENTS.md, CLAUDE.md"
-                    ]
+                    ],
+                    parsingErrors: []
                 },
                 diagnostics: []
             };
         }
         // Analyze each file
         const fileStructures = new Map();
+        const parsingErrors = [];
         for (const file of discoveredFiles){
             try {
                 const structure = await this.astGateway.parseFile(file.filePath);
                 fileStructures.set(file.filePath, structure);
-            } catch  {
-            // Skip files that can't be parsed
+            } catch (error) {
+                const errorMessage = error instanceof Error ? error.message : "Unknown parsing error";
+                parsingErrors.push({
+                    filePath: file.filePath,
+                    error: errorMessage
+                });
             }
         }
+        // Sort parsing errors for deterministic output
+        parsingErrors.sort((a, b)=>a.filePath.localeCompare(b.filePath));
         // Score each mandatory command across all files
         const commandScores = [];
         const diagnostics = [];
+        // Emit diagnostics for parsing errors
+        for (const pe of parsingErrors){
+            diagnostics.push({
+                filePath: pe.filePath,
+                line: 1,
+                severity: "warning",
+                message: `Failed to parse file: ${pe.error}`,
+                ruleId: "instruction/parse-error",
+                target: "instruction"
+            });
+        }
         for (const command of mandatoryCommands){
             const bestAnalysis = this.findBestScore(command, discoveredFiles, fileStructures, headingPatterns, proximityWindow, diagnostics);
             if (bestAnalysis) {
@@ -55213,12 +55243,17 @@ const remark = unified().use(remarkParse).use(remarkStringify).freeze()
         const overallQualityScore = commandScores.length > 0 ? Math.round(commandScores.reduce((sum, s)=>sum + s.compositeScore, 0) / commandScores.length * 100) : 0;
         // Generate suggestions
         const suggestions = this.generateSuggestions(commandScores);
+        if (parsingErrors.length > 0) {
+            const skippedFiles = parsingErrors.map((pe)=>pe.filePath).join(", ");
+            suggestions.push(`${parsingErrors.length} file(s) could not be parsed and were skipped: ${skippedFiles}. Analysis may be incomplete.`);
+        }
         return {
             result: {
                 discoveredFiles,
                 commandScores,
                 overallQualityScore,
-                suggestions
+                suggestions,
+                parsingErrors
             },
             diagnostics
         };
@@ -55861,13 +55896,7 @@ const remark = unified().use(remarkParse).use(remarkStringify).freeze()
  */ async function lintInstructions(targetDir) {
     const discoveryGateway = createInstructionFileDiscoveryGateway();
     const astGateway = createMarkdownAstGateway();
-    const scriptGateway = createScriptDiscoveryGateway();
-    const commandsGateway = {
-        async getMandatoryCommands (dir) {
-            const scripts = await scriptGateway.discoverScripts(dir);
-            return scripts.filter((s)=>s.isMandatory).map((s)=>s.name);
-        }
-    };
+    const commandsGateway = createFeedbackLoopCommandsGateway();
     const useCase = new AnalyzeInstructionQualityUseCase(discoveryGateway, astGateway, commandsGateway);
     const output = await useCase.execute({
         targetDir
@@ -55991,7 +56020,11 @@ const remark = unified().use(remarkParse).use(remarkStringify).freeze()
             }
         }
         if (totalErrors > 0) {
-            throw new Error(`lint failed: ${totalErrors} error(s), ${totalWarnings} warning(s)`);
+            process.exitCode = 1;
+            if (format === "human") {
+                consola.error(`lint failed: ${totalErrors} error(s), ${totalWarnings} warning(s)`);
+            }
+            return;
         }
         if (format === "human") {
             if (totalWarnings > 0) {