@cparra/apexdocs 3.19.9-beta.6 → 3.19.9-beta.8

package/dist/cli/generate.js

@@ -1,7 +1,7 @@
 #!/usr/bin/env node
 'use strict';
 
-var errorCollector = require('../error-collector-Bsh7lZPF.js');
+var errorCollector = require('../error-collector-CgDY7vBU.js');
 var cosmiconfig = require('cosmiconfig');
 var yargs = require('yargs');
 var E = require('fp-ts/Either');
@@ -589,27 +589,16 @@ function isDebugEnabledFromProcessArgs() {
 const logger = new errorCollector.StdOutLogger();
 logger.setDebug(isDebugEnabledFromProcessArgs());
 function main() {
-  let commandsRun = 0;
   function printFailuresAtEnd(collector, config) {
     if (!collector.hasErrors()) {
       return;
     }
     const count = collector.count();
-    logger.logSingle(
-      `\u26A0\uFE0F ${config.targetGenerator}: ${count} error item(s) occurred. Please review the following:`,
-      "red"
-    );
+    logger.logSingle(`\u26A0\uFE0F ${config.targetGenerator}: ${count} error(s) occurred. Please review the following:`, "red");
     for (const item of collector.all()) {
       logger.error(errorCollector.ErrorCollector.format(item));
     }
   }
-  function printDebugSummary(collector) {
-    if (!logger.isDebugEnabled()) {
-      return;
-    }
-    logger.debug(`commandsRun=${commandsRun}`);
-    logger.debug(`aggregatedFailures=${collector.count()}`);
-  }
   function catchUnexpectedError(error) {
     logger.error(`\u274C An error occurred while processing the request: ${error}`);
     process.exit(1);
@@ -622,7 +611,6 @@ function main() {
   extractArgs().then((maybeConfigs) => __async(null, null, function* () {
     E__namespace.match(catchUnexpectedError, (configs) => __async(null, null, function* () {
       for (const config of configs) {
-        commandsRun++;
         const errorCollector$1 = new errorCollector.ErrorCollector(config.targetGenerator);
         const reflectionDebugLogger = errorCollector.createReflectionDebugLogger(logger, (filePath, errorMessage) => {
           errorCollector$1.addFailure("other", filePath, errorMessage);
@@ -635,7 +623,6 @@ function main() {
           reflectionDebugLogger,
           errorCollector: errorCollector$1
         });
-        printResultMessage(result);
         if (logger.isDebugEnabled()) {
           logger.logSingle(
             `${config.targetGenerator}: ${E__namespace.isRight(result) ? "success" : "failure"}`,
@@ -643,10 +630,10 @@ function main() {
           );
         }
         printFailuresAtEnd(errorCollector$1, config);
-        printDebugSummary(errorCollector$1);
         if (errorCollector$1.hasErrors()) {
           process.exitCode = 1;
         }
+        printResultMessage(result);
       }
     }))(maybeConfigs);
   })).catch(catchUnexpectedError);

package/dist/{error-collector-Bsh7lZPF.js → error-collector-CgDY7vBU.js}

@@ -5734,9 +5734,6 @@ function generateChangeLog(oldBundles, newBundles, config, debugLogger) {
     })),
     // Fail at the very end if we had any recoverable reflection errors,
     // so the CLI can set an exit code after completing the work.
-    //
-    // We intentionally do NOT propagate individual error item details here.
-    // Those are recorded via the per-generator ErrorCollector as they happen.
     TE__namespace.flatMap(({ page, combinedReflectionErrors }) => {
       if (combinedReflectionErrors.errors.length > 0) {
         return TE__namespace.left(new ReflectionErrors([]));
@@ -6287,7 +6284,7 @@ function processMarkdown(config, logger, reflectionDebugLogger, errorCollector)
       TE__namespace.map(() => "\u2714\uFE0F Documentation generated successfully!"),
       TE__namespace.mapLeft((err) => {
         if (logger.isDebugEnabled()) {
-          logger.debug(`markdown generator finished with errors (see collector for details)`);
+          logger.debug(`markdown generator finished with errors`);
         }
         return toErrors(err);
       })
@@ -6384,9 +6381,8 @@ function processChangeLog(config, logger, reflectionDebugLogger, errorCollector)
       TE__namespace.fromEither,
       TE__namespace.flatMap(([previous, current]) => generate(previous, current, config, reflectionDebugLogger)),
       TE__namespace.mapLeft((err) => {
-        errorCollector.addGlobalFailure("other", "Changelog generation completed with errors.", err);
         if (logger.isDebugEnabled()) {
-          logger.debug(`changelog generator finished with errors (see collector for details)`);
+          logger.debug(`changelog generator finished with errors`);
         }
         return toErrors(err);
       })
@@ -6498,28 +6494,14 @@ var __spreadValues = (a, b) => {
 class ErrorCollector {
   constructor(generator) {
     this.items = [];
-    // Basic de-dupe to avoid double-recording the same failure if the same layer reports it twice.
-    // Keyed by generator|stage|filePath|message.
-    this.seen = /* @__PURE__ */ new Set();
     this.generator = generator;
   }
   add(item) {
-    var _a, _b, _c;
+    var _a;
     const fullItem = __spreadValues({
       generator: this.generator,
       at: (_a = item.at) != null ? _a : (/* @__PURE__ */ new Date()).toISOString()
     }, item);
-    const key = [
-      fullItem.generator,
-      fullItem.stage,
-      (_b = fullItem.filePath) != null ? _b : "",
-      // Normalize whitespace a bit so trivial formatting differences don't explode the log.
-      ((_c = fullItem.message) != null ? _c : "").trim().replace(/\s+/g, " ")
-    ].join("|");
-    if (this.seen.has(key)) {
-      return;
-    }
-    this.seen.add(key);
     this.items.push(fullItem);
   }
   addFailure(stage, filePath, message, raw) {
@@ -6540,7 +6522,6 @@ class ErrorCollector {
   hasErrors() {
     return this.items.length > 0;
   }
-  /** Count of collected error items (post de-duplication). */
   count() {
     return this.items.length;
   }
@@ -6548,24 +6529,11 @@ class ErrorCollector {
     return this.items;
   }
   /**
-   * Groups error items by file path. Items with no file path are grouped under `__global__`.
-   */
-  groupByFilePath() {
-    return this.items.reduce((acc, item) => {
-      var _a, _b;
-      const key = (_a = item.filePath) != null ? _a : "__global__";
-      acc[key] = (_b = acc[key]) != null ? _b : [];
-      acc[key].push(item);
-      return acc;
-    }, {});
-  }
-  /**
-   * A compact human-readable representation of the error item.
-   * Useful for CLI output.
+   * Human-readable representation of the error item.
    */
   static format(item) {
-    const location = item.filePath ? `${item.filePath}` : "(no file)";
-    return `[${item.generator}/${item.stage}] ${location}: ${item.message}`;
+    const location = item.filePath ? `${item.filePath}` : "";
+    return `[${item.generator}] ${location}: ${item.message}`;
   }
 }
 

package/dist/index.js

@@ -1,7 +1,7 @@
 #!/usr/bin/env node
 'use strict';
 
-var errorCollector = require('./error-collector-Bsh7lZPF.js');
+var errorCollector = require('./error-collector-CgDY7vBU.js');
 var E = require('fp-ts/Either');
 require('fp-ts/function');
 require('fp-ts/TaskEither');

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@cparra/apexdocs",
-  "version": "3.19.9-beta.6",
+  "version": "3.19.9-beta.8",
   "description": "Library with CLI capabilities to generate documentation for Salesforce Apex classes.",
   "engines": {
     "node": ">=18"
@@ -96,7 +96,7 @@
     ]
   },
   "dependencies": {
-    "@cparra/apex-reflection": "2.23.7",
+    "@cparra/apex-reflection": "2.23.10-dev.20260107145611",
     "@salesforce/source-deploy-retrieve": "^12.20.1",
     "@types/js-yaml": "^4.0.9",
     "@types/yargs": "^17.0.32",