@cparra/apexdocs 3.17.1-beta.1 → 3.17.1-beta.3

package/dist/cli/generate.js

@@ -1,7 +1,7 @@
 #!/usr/bin/env node
 'use strict';
 
-var Apexdocs = require('../Apexdocs-DPBWi8Yn.js');
+var logger$1 = require('../logger-Vr_GM9Dn.js');
 var cosmiconfig = require('cosmiconfig');
 var yargs = require('yargs');
 var E = require('fp-ts/Either');
@@ -24,9 +24,9 @@ require('fp-ts/boolean');
 require('fp-ts/Array');
 require('fs');
 require('fp-ts/lib/TaskEither');
-require('chalk');
 require('minimatch');
 require('@salesforce/source-deploy-retrieve');
+require('chalk');
 
 function _interopNamespaceDefault(e) {
   var n = Object.create(null);
@@ -79,42 +79,42 @@ const markdownOptions = {
   targetDir: {
     type: "string",
     alias: "t",
-    default: Apexdocs.markdownDefaults.targetDir,
+    default: logger$1.markdownDefaults.targetDir,
     describe: "The directory location where documentation will be generated to."
   },
   scope: {
     type: "string",
     array: true,
     alias: "p",
-    default: Apexdocs.markdownDefaults.scope,
+    default: logger$1.markdownDefaults.scope,
     describe: "A list of scopes to document. Values should be separated by a space, e.g --scope global public namespaceaccessible. Annotations are supported and should be passed lowercased and without the @ symbol, e.g. namespaceaccessible auraenabled."
   },
   customObjectVisibility: {
     type: "string",
     array: true,
     alias: "v",
-    default: Apexdocs.markdownDefaults.customObjectVisibility,
+    default: logger$1.markdownDefaults.customObjectVisibility,
     choices: ["public", "protected", "packageprotected"],
     describe: "Controls which custom objects are documented. Values should be separated by a space."
   },
   defaultGroupName: {
     type: "string",
-    default: Apexdocs.markdownDefaults.defaultGroupName,
+    default: logger$1.markdownDefaults.defaultGroupName,
     describe: "Defines the @group name to be used when a file does not specify it."
   },
   customObjectsGroupName: {
     type: "string",
-    default: Apexdocs.markdownDefaults.customObjectsGroupName,
+    default: logger$1.markdownDefaults.customObjectsGroupName,
     describe: "The name under which custom objects will be grouped in the Reference Guide"
   },
   triggersGroupName: {
     type: "string",
-    default: Apexdocs.markdownDefaults.triggersGroupName,
+    default: logger$1.markdownDefaults.triggersGroupName,
     describe: "The name under which triggers will be grouped in the Reference Guide"
   },
   lwcGroupName: {
     type: "string",
-    default: Apexdocs.markdownDefaults.lwcGroupName,
+    default: logger$1.markdownDefaults.lwcGroupName,
     describe: "The name under which Lightning Web Components will be grouped in the Reference Guide"
   },
   namespace: {
@@ -124,28 +124,28 @@ const markdownOptions = {
   sortAlphabetically: {
     type: "boolean",
     describe: "Whether to sort files and members alphabetically.",
-    default: Apexdocs.markdownDefaults.sortAlphabetically
+    default: logger$1.markdownDefaults.sortAlphabetically
   },
   includeMetadata: {
     type: "boolean",
     describe: "Whether to include the file's meta.xml information: Whether it is active and and the API version",
-    default: Apexdocs.markdownDefaults.includeMetadata
+    default: logger$1.markdownDefaults.includeMetadata
   },
   linkingStrategy: {
     type: "string",
     describe: "The strategy to use when linking to other documentation pages.",
     choices: ["relative", "no-link", "none"],
-    default: Apexdocs.markdownDefaults.linkingStrategy
+    default: logger$1.markdownDefaults.linkingStrategy
   },
   referenceGuideTitle: {
     type: "string",
     describe: "The title of the reference guide.",
-    default: Apexdocs.markdownDefaults.referenceGuideTitle
+    default: logger$1.markdownDefaults.referenceGuideTitle
   },
   includeFieldSecurityMetadata: {
     type: "boolean",
     describe: "Whether to include the compliance category and security classification for fields in the generated files.",
-    default: Apexdocs.markdownDefaults.includeFieldSecurityMetadata
+    default: logger$1.markdownDefaults.includeFieldSecurityMetadata
   },
   includeInlineHelpTextMetadata: {
     type: "boolean",
@@ -154,17 +154,17 @@ const markdownOptions = {
   experimentalLwcSupport: {
     type: "boolean",
     describe: "Enable experimental support for documenting Lightning Web Components (LWC).",
-    default: Apexdocs.markdownDefaults.experimentalLwcSupport
+    default: logger$1.markdownDefaults.experimentalLwcSupport
   },
   parallelReflection: {
     type: "boolean",
     describe: "Parallelize CPU-heavy reflection via worker threads.",
-    default: Apexdocs.markdownDefaults.parallelReflection
+    default: logger$1.markdownDefaults.parallelReflection
   },
   parallelReflectionMaxWorkers: {
     type: "number",
     describe: "Maximum number of worker threads to use for parallel reflection. Defaults to a reasonable value based on CPU count.",
-    default: Apexdocs.markdownDefaults.parallelReflectionMaxWorkers
+    default: logger$1.markdownDefaults.parallelReflectionMaxWorkers
   }
 };
 
@@ -200,12 +200,12 @@ const openApiOptions = {
   targetDir: {
     type: "string",
     alias: "t",
-    default: Apexdocs.openApiDefaults.targetDir,
+    default: logger$1.openApiDefaults.targetDir,
     describe: "The directory location where the OpenApi file will be generated."
   },
   fileName: {
     type: "string",
-    default: Apexdocs.openApiDefaults.fileName,
+    default: logger$1.openApiDefaults.fileName,
     describe: "The name of the OpenApi file to be generated."
   },
   namespace: {
@@ -214,23 +214,23 @@ const openApiOptions = {
   },
   title: {
     type: "string",
-    default: Apexdocs.openApiDefaults.title,
+    default: logger$1.openApiDefaults.title,
     describe: "The title of the OpenApi file."
   },
   apiVersion: {
     type: "string",
-    default: Apexdocs.openApiDefaults.apiVersion,
+    default: logger$1.openApiDefaults.apiVersion,
     describe: "The version of the OpenApi file."
   },
   parallelReflection: {
     type: "boolean",
     describe: "Parallelize CPU-heavy reflection via worker threads.",
-    default: Apexdocs.openApiDefaults.parallelReflection
+    default: logger$1.openApiDefaults.parallelReflection
   },
   parallelReflectionMaxWorkers: {
     type: "number",
     describe: "Maximum number of worker threads to use for parallel reflection. Defaults to a reasonable value based on CPU count.",
-    default: Apexdocs.openApiDefaults.parallelReflectionMaxWorkers
+    default: logger$1.openApiDefaults.parallelReflectionMaxWorkers
   }
 };
 
@@ -249,12 +249,12 @@ const changeLogOptions = {
   parallelReflection: {
     type: "boolean",
     describe: "Parallelize CPU-heavy reflection via worker threads.",
-    default: Apexdocs.markdownDefaults.parallelReflection
+    default: logger$1.markdownDefaults.parallelReflection
   },
   parallelReflectionMaxWorkers: {
     type: "number",
     describe: "Maximum number of worker threads to use for parallel reflection. Defaults to a reasonable value based on CPU count.",
-    default: Apexdocs.markdownDefaults.parallelReflectionMaxWorkers
+    default: logger$1.markdownDefaults.parallelReflectionMaxWorkers
   },
   previousVersionDir: {
     type: "string",
@@ -273,32 +273,32 @@ const changeLogOptions = {
   targetDir: {
     type: "string",
     alias: "t",
-    default: Apexdocs.changeLogDefaults.targetDir,
+    default: logger$1.changeLogDefaults.targetDir,
     describe: "The directory location where the changelog file will be generated."
   },
   fileName: {
     type: "string",
-    default: Apexdocs.changeLogDefaults.fileName,
+    default: logger$1.changeLogDefaults.fileName,
     describe: "The name of the changelog file to be generated."
   },
   scope: {
     type: "string",
     array: true,
     alias: "s",
-    default: Apexdocs.changeLogDefaults.scope,
+    default: logger$1.changeLogDefaults.scope,
     describe: "The list of scope to respect when generating the changelog. Values should be separated by a space, e.g --scope global public namespaceaccessible. Annotations are supported and should be passed lowercased and without the @ symbol, e.g. namespaceaccessible auraenabled."
   },
   customObjectVisibility: {
     type: "string",
     array: true,
     alias: "v",
-    default: Apexdocs.changeLogDefaults.customObjectVisibility,
+    default: logger$1.changeLogDefaults.customObjectVisibility,
     choices: ["public", "protected", "packageprotected"],
     describe: "Controls which custom objects are documented. Values should be separated by a space."
   },
   skipIfNoChanges: {
     type: "boolean",
-    default: Apexdocs.changeLogDefaults.skipIfNoChanges,
+    default: logger$1.changeLogDefaults.skipIfNoChanges,
     describe: "Skip the changelog generation if there are no changes between the previous and current version."
   }
 };
@@ -406,13 +406,13 @@ function extractArgsForCommandProvidedThroughCli(extractFromProcessFn, config) {
   switch (mergedConfig.targetGenerator) {
     case "markdown":
       return _function.pipe(
-        Apexdocs.validateSourceDirectoryConfig(extractSourceDirectoryConfig(mergedConfig)),
+        logger$1.validateSourceDirectoryConfig(extractSourceDirectoryConfig(mergedConfig)),
         E__namespace.mapLeft((error) => new Error(`Invalid markdown configuration: ${error.message}`)),
         E__namespace.map(() => __spreadValues(__spreadValues({}, configOnlyMarkdownDefaults), mergedConfig))
       );
     case "openapi":
       return _function.pipe(
-        Apexdocs.validateSourceDirectoryConfig(extractSourceDirectoryConfig(mergedConfig)),
+        logger$1.validateSourceDirectoryConfig(extractSourceDirectoryConfig(mergedConfig)),
         E__namespace.mapLeft((error) => new Error(`Invalid openapi configuration: ${error.message}`)),
         E__namespace.map(() => __spreadValues(__spreadValues({}, configOnlyOpenApiDefaults), mergedConfig))
       );
@@ -440,7 +440,7 @@ function extractArgsForCommandsProvidedInConfig(extractFromProcessFn, config) {
           E__namespace.flatMap((cliArgs) => {
             const mergedConfig = __spreadValues(__spreadValues(__spreadValues({}, configOnlyMarkdownDefaults), generatorConfig), cliArgs);
             return _function.pipe(
-              Apexdocs.validateSourceDirectoryConfig(extractSourceDirectoryConfig(mergedConfig)),
+              logger$1.validateSourceDirectoryConfig(extractSourceDirectoryConfig(mergedConfig)),
               E__namespace.mapLeft((error) => new Error(`Invalid markdown configuration: ${error.message}`)),
               E__namespace.map(() => mergedConfig)
             );
@@ -452,7 +452,7 @@ function extractArgsForCommandsProvidedInConfig(extractFromProcessFn, config) {
           E__namespace.flatMap((cliArgs) => {
             const mergedConfig = __spreadValues(__spreadValues(__spreadValues({}, configOnlyOpenApiDefaults), generatorConfig), cliArgs);
             return _function.pipe(
-              Apexdocs.validateSourceDirectoryConfig(extractSourceDirectoryConfig(mergedConfig)),
+              logger$1.validateSourceDirectoryConfig(extractSourceDirectoryConfig(mergedConfig)),
               E__namespace.mapLeft((error) => new Error(`Invalid openapi configuration: ${error.message}`)),
               E__namespace.map(() => mergedConfig)
             );
@@ -557,8 +557,8 @@ function validateChangelogConfig(config) {
   };
   return _function.pipe(
     E__namespace.Do,
-    E__namespace.bind("previousValid", () => Apexdocs.validateSourceDirectoryConfig(previousVersionConfig)),
-    E__namespace.bind("currentValid", () => Apexdocs.validateSourceDirectoryConfig(currentVersionConfig)),
+    E__namespace.bind("previousValid", () => logger$1.validateSourceDirectoryConfig(previousVersionConfig)),
+    E__namespace.bind("currentValid", () => logger$1.validateSourceDirectoryConfig(currentVersionConfig)),
     E__namespace.map(() => config)
   );
 }
@@ -586,11 +586,9 @@ var __async = (__this, __arguments, generator) => {
 function isDebugEnabledFromProcessArgs() {
   return process.argv.includes("--debug");
 }
-const logger = new Apexdocs.StdOutLogger();
+const logger = new logger$1.StdOutLogger();
 logger.setDebug(isDebugEnabledFromProcessArgs());
-if (logger.isDebugEnabled()) {
-  process.env.APEXDOCS_DEBUG = "true";
-}
+const reflectionDebugLogger = logger$1.createReflectionDebugLogger(logger);
 function main() {
   const aggregatedFailures = [];
   let commandsRun = 0;
@@ -632,7 +630,7 @@ function main() {
         if (logger.isDebugEnabled()) {
           logger.debug(`Currently processing generator: ${config.targetGenerator}`);
         }
-        const result = yield Apexdocs.Apexdocs.generate(config, logger);
+        const result = yield logger$1.Apexdocs.generate(config, { logger, reflectionDebugLogger });
         if (logger.isDebugEnabled()) {
           logger.logSingle(
             `${config.targetGenerator}: ${E__namespace.isRight(result) ? "success" : "failure"}`,

package/dist/index.js

@@ -1,7 +1,7 @@
 #!/usr/bin/env node
 'use strict';
 
-var Apexdocs = require('./Apexdocs-DPBWi8Yn.js');
+var logger = require('./logger-Vr_GM9Dn.js');
 var E = require('fp-ts/Either');
 require('fp-ts/function');
 require('fp-ts/TaskEither');
@@ -21,9 +21,9 @@ require('fp-ts/boolean');
 require('fp-ts/Array');
 require('fs');
 require('fp-ts/lib/TaskEither');
-require('chalk');
 require('minimatch');
 require('@salesforce/source-deploy-retrieve');
+require('chalk');
 
 function _interopNamespaceDefault(e) {
   var n = Object.create(null);
@@ -82,12 +82,13 @@ var __async = (__this, __arguments, generator) => {
 };
 function process(config) {
   return __async(this, null, function* () {
-    const logger = new Apexdocs.NoLogger();
+    const logger$1 = new logger.NoLogger();
+    const reflectionDebugLogger = logger.createReflectionDebugLogger(logger$1);
     const configWithDefaults = __spreadValues$1(__spreadValues$1({}, getDefault(config)), config);
     if (!configWithDefaults.sourceDir) {
       throw new Error("sourceDir is required");
     }
-    const result = yield Apexdocs.Apexdocs.generate(configWithDefaults, logger);
+    const result = yield logger.Apexdocs.generate(configWithDefaults, { logger: logger$1, reflectionDebugLogger });
     E__namespace.match(
       (errors) => {
         throw errors;
@@ -100,11 +101,11 @@ function process(config) {
 function getDefault(config) {
   switch (config.targetGenerator) {
     case "markdown":
-      return Apexdocs.markdownDefaults;
+      return logger.markdownDefaults;
     case "openapi":
-      return Apexdocs.openApiDefaults;
+      return logger.openApiDefaults;
     case "changelog":
-      return Apexdocs.changeLogDefaults;
+      return logger.changeLogDefaults;
     default:
       throw new Error("Unknown target generator");
   }
@@ -130,23 +131,23 @@ var __spreadValues = (a, b) => {
 };
 var __spreadProps = (a, b) => __defProps(a, __getOwnPropDescs(b));
 function defineMarkdownConfig(config) {
-  return __spreadProps(__spreadValues(__spreadValues({}, Apexdocs.markdownDefaults), config), {
+  return __spreadProps(__spreadValues(__spreadValues({}, logger.markdownDefaults), config), {
     targetGenerator: "markdown"
   });
 }
 function defineOpenApiConfig(config) {
-  return __spreadProps(__spreadValues(__spreadValues({}, Apexdocs.openApiDefaults), config), {
+  return __spreadProps(__spreadValues(__spreadValues({}, logger.openApiDefaults), config), {
     targetGenerator: "openapi"
   });
 }
 function defineChangelogConfig(config) {
-  return __spreadProps(__spreadValues(__spreadValues({}, Apexdocs.changeLogDefaults), config), {
+  return __spreadProps(__spreadValues(__spreadValues({}, logger.changeLogDefaults), config), {
     targetGenerator: "changelog"
   });
 }
 
-exports.skip = Apexdocs.skip;
-exports.templateHelpers = Apexdocs.templateHelpers;
+exports.skip = logger.skip;
+exports.templateHelpers = logger.templateHelpers;
 exports.defineChangelogConfig = defineChangelogConfig;
 exports.defineMarkdownConfig = defineMarkdownConfig;
 exports.defineOpenApiConfig = defineOpenApiConfig;

package/dist/{Apexdocs-DPBWi8Yn.js → logger-Vr_GM9Dn.js}

@@ -19,9 +19,9 @@ var boolean = require('fp-ts/boolean');
 var A$1 = require('fp-ts/Array');
 var fs$1 = require('fs');
 var TE$1 = require('fp-ts/lib/TaskEither');
-var chalk = require('chalk');
 var minimatch = require('minimatch');
 var sourceDeployRetrieve = require('@salesforce/source-deploy-retrieve');
+var chalk = require('chalk');
 
 function _interopNamespaceDefault(e) {
   var n = Object.create(null);
@@ -1420,14 +1420,13 @@ var __async$5 = (__this, __arguments, generator) => {
     step((generator = generator.apply(__this, __arguments)).next());
   });
 };
-const noopDebugLogger = {
-  onStart: () => {
-  },
+const noopReflectionDebugLogger = {
   onSuccess: () => {
   },
   onFailure: () => {
   }
 };
+const noopDebugLogger = noopReflectionDebugLogger;
 function reflectAsync$1(rawSource) {
   return __async$5(this, null, function* () {
     return new Promise((resolve, reject) => {
@@ -1484,7 +1483,6 @@ function reflectApexSourceBestEffort(apexBundles, config, debugLogger = noopDebu
     return _function.pipe(
       apexBundles,
       A__namespace.traverse(Ap)((bundle) => {
-        debugLogger.onStart(bundle.filePath);
         return _function.pipe(
           reflectBundle$2(bundle),
           TE__namespace.map((parsed) => {
@@ -1514,9 +1512,7 @@ function reflectApexSourceBestEffort(apexBundles, config, debugLogger = noopDebu
         try {
           const results = yield Promise.allSettled(
             apexBundles.map((bundle) => __async$5(null, null, function* () {
-              debugLogger.onStart(bundle.filePath);
-              const typeMirror = yield pool.run({ content: bundle.content });
-              return typeMirror;
+              return yield pool.run({ content: bundle.content });
             }))
           );
           const errors = [];
@@ -3656,13 +3652,24 @@ function generateForApex(apexBundles, config, debugLogger) {
       },
       debugLogger
     ),
-    TE__namespace.map(({ successes }) => {
-      return successes;
-    }),
-    TE__namespace.map(filterOutOfScope),
-    TE__namespace.map(addInheritedMembersToTypes),
-    TE__namespace.map(addInheritanceChainToTypes),
-    TE__namespace.map(removeExcluded)
+    TE__namespace.flatMap(({ successes, errors }) => {
+      const filtered = _function.pipe(
+        TE__namespace.right(successes),
+        TE__namespace.map(filterOutOfScope),
+        TE__namespace.map(addInheritedMembersToTypes),
+        TE__namespace.map(addInheritanceChainToTypes),
+        TE__namespace.map(removeExcluded)
+      );
+      return _function.pipe(
+        filtered,
+        TE__namespace.flatMap((parsed) => {
+          if (errors.errors.length > 0) {
+            return TE__namespace.left(errors);
+          }
+          return TE__namespace.right(parsed);
+        })
+      );
+    })
   );
 }
 function transformReferenceHook(config) {
@@ -4777,6 +4784,21 @@ class OpenApiDocsProcessor {
   }
 }
 
+function createReflectionDebugLogger(logger) {
+  if (!logger.isDebugEnabled()) {
+    return noopReflectionDebugLogger;
+  }
+  return {
+    onSuccess: (filePath) => {
+      logger.debug(`Parsing: ${filePath} OK`);
+    },
+    onFailure: (filePath, errorMessage) => {
+      logger.debug(`Parsing: ${filePath} FAILED`);
+      logger.debug(`Parsing error: ${filePath} - ${errorMessage}`);
+    }
+  };
+}
+
 var __defProp$2 = Object.defineProperty;
 var __getOwnPropSymbols$2 = Object.getOwnPropertySymbols;
 var __hasOwnProp$2 = Object.prototype.hasOwnProperty;
@@ -4824,18 +4846,7 @@ function openApi(logger, fileBodies, config) {
       namespace: config.namespace,
       version: config.apiVersion
     });
-    const debugLogger = {
-      onStart: (filePath) => {
-        logger.debug(`Parsing: ${filePath}`);
-      },
-      onSuccess: (filePath) => {
-        logger.logSingle(`Parsing: ${filePath} OK`, "green");
-      },
-      onFailure: (filePath, errorMessage) => {
-        logger.logSingle(`Parsing: ${filePath} FAILED`, "red");
-        logger.debug(`Parsing error: ${filePath} - ${errorMessage}`);
-      }
-    };
+    const debugLogger = createReflectionDebugLogger(logger);
     const reflectedEither = yield reflectApexSourceBestEffort(
       fileBodies,
       {
@@ -4845,9 +4856,7 @@ function openApi(logger, fileBodies, config) {
       debugLogger
     )();
     if (E__namespace.isLeft(reflectedEither)) {
-      const errors = reflectedEither.left;
-      logger.error(errors);
-      return;
+      return E__namespace.left(reflectedEither.left);
     }
     const { successes: parsedFiles, errors: recoverableErrors } = reflectedEither.right;
     if (recoverableErrors.errors.length > 0) {
@@ -4882,13 +4891,24 @@ function openApi(logger, fileBodies, config) {
     const processor = new OpenApiDocsProcessor(logger);
     Transpiler.generate(filteredTypes, processor);
     const generatedFiles = processor.fileBuilder().files();
-    yield _function.pipe(
+    const writeResult = yield _function.pipe(
       writeFiles(generatedFiles, config.targetDir, (file) => {
         logger.logSingle(`${file.outputDocPath} processed.`, "green");
       }),
-      TE__namespace.mapError((error) => logger.error(error))
+      TE__namespace.mapLeft((error) => new FileWritingError("An error occurred while writing files to the system.", error))
     )();
+    if (E__namespace.isLeft(writeResult)) {
+      return E__namespace.left(writeResult.left);
+    }
     ErrorLogger.logErrors(logger, filteredTypes);
+    if (recoverableErrors.errors.length > 0) {
+      return E__namespace.left(
+        new ReflectionErrors(
+          recoverableErrors.errors.map((e) => __spreadValues$2({}, e))
+        )
+      );
+    }
+    return E__namespace.right(void 0);
   });
 }
 function filterByScopes(logger, manifest) {
@@ -5505,71 +5525,6 @@ function changelogToSourceChangelog(changelog) {
   };
 }
 
-class StdOutLogger {
-  constructor(options) {
-    this.debugEnabled = false;
-    var _a;
-    this.debugEnabled = (_a = options == null ? void 0 : options.debug) != null ? _a : false;
-  }
-  setDebug(enabled) {
-    this.debugEnabled = enabled;
-  }
-  isDebugEnabled() {
-    return this.debugEnabled;
-  }
-  debug(message) {
-    if (!this.debugEnabled) {
-      return;
-    }
-    this.logSingle(`[debug] ${String(message)}`);
-  }
-  /**
-   * Logs a message with optional arguments.
-   * @param message The message to log.
-   * @param args Optional arguments.
-   */
-  log(message, ...args) {
-    this.logSingle(message);
-    args.forEach((arg) => {
-      this.logSingle(arg);
-    });
-  }
-  /**
-   * Logs an error message with optional arguments.
-   * @param message The error message to log.
-   * @param args Optional arguments.
-   */
-  error(message, ...args) {
-    this.logSingle(message, "red");
-    args.forEach(() => {
-      this.logSingle(message, "red");
-    });
-  }
-  logSingle(text, color = "green") {
-    const logMessage = `${this.getChalkFn(color)((/* @__PURE__ */ new Date()).toLocaleString() + ": ")}${text}
-`;
-    process.stdout.write(logMessage);
-  }
-  getChalkFn(color) {
-    return color === "green" ? chalk.green : chalk.red;
-  }
-}
-class NoLogger {
-  setDebug(enabled) {
-  }
-  isDebugEnabled() {
-    return false;
-  }
-  debug(message) {
-  }
-  log(message, ...args) {
-  }
-  error(message, ...args) {
-  }
-  logSingle(text, color) {
-  }
-}
-
 var __defProp$1 = Object.defineProperty;
 var __defProps$1 = Object.defineProperties;
 var __getOwnPropDescs$1 = Object.getOwnPropertyDescriptors;
@@ -5615,34 +5570,7 @@ function changelogReflectionConfig(config) {
     parallelReflectionMaxWorkers: config.parallelReflectionMaxWorkers
   };
 }
-function getDebugLogger() {
-  var _a, _b;
-  const enabled = ((_a = process.env.APEXDOCS_DEBUG) != null ? _a : "").toLowerCase() === "true" || ((_b = process.env.APEXDOCS_DEBUG) != null ? _b : "").toLowerCase() === "1";
-  const logger = new StdOutLogger({ debug: enabled });
-  if (!enabled) {
-    return {
-      onStart: () => {
-      },
-      onSuccess: () => {
-      },
-      onFailure: () => {
-      }
-    };
-  }
-  return {
-    onStart: (filePath) => {
-      logger.debug(`Parsing: ${filePath}`);
-    },
-    onSuccess: (filePath) => {
-      logger.logSingle(`Parsing: ${filePath} OK`, "green");
-    },
-    onFailure: (filePath, errorMessage) => {
-      logger.logSingle(`Parsing: ${filePath} FAILED`, "red");
-      logger.debug(`Parsing error: ${filePath} - ${errorMessage}`);
-    }
-  };
-}
-function generateChangeLog(oldBundles, newBundles, config) {
+function generateChangeLog(oldBundles, newBundles, config, debugLogger) {
   const convertToPageData = apply(toPageData, config.fileName);
   function handleConversion({ changelog, newManifest }) {
     if (config.skipIfNoChanges && !hasChanges(changelog)) {
@@ -5657,12 +5585,10 @@ function generateChangeLog(oldBundles, newBundles, config) {
   }
   const reflectVersionBestEffort = (bundles) => {
     const filterOutOfScopeApex = apply(filterScope, config.scope);
-    const debugLogger = getDebugLogger();
     const apexConfig = changelogReflectionConfig(config);
     return _function.pipe(
       TE__namespace.right(bundles),
       TE__namespace.bindTo("bundles"),
-      // 1) Apex
       TE__namespace.bind(
         "apex",
         ({ bundles: bundles2 }) => _function.pipe(
@@ -5675,11 +5601,9 @@ function generateChangeLog(oldBundles, newBundles, config) {
             parsedFiles: filterOutOfScopeApex(successes),
             errors
           })),
-          // Best-effort: if the reflection system itself fails unexpectedly, treat that as a single error and continue.
           TE__namespace.orElseW((errs) => TE__namespace.right({ parsedFiles: [], errors: errs }))
         )
       ),
-      // 2) Custom objects / fields / metadata records
       TE__namespace.bind(
         "objects",
         ({ bundles: bundles2, apex }) => _function.pipe(
@@ -5691,13 +5615,12 @@ function generateChangeLog(oldBundles, newBundles, config) {
             parsedFiles: [...apex.parsedFiles, ...parsedObjectFiles],
             errors: apex.errors
           })),
-          // For best-effort semantics we swallow failures here and keep going, because
+          // We swallow failures here and keep going, because
           // this helper's left type is `never`. Any such failures are treated as if
           // this step produced no additional parsed files.
           TE__namespace.orElseW(() => TE__namespace.right({ parsedFiles: apex.parsedFiles, errors: apex.errors }))
         )
       ),
-      // 3) Triggers
       TE__namespace.bind(
         "all",
         ({ objects, bundles: bundles2 }) => _function.pipe(
@@ -5706,7 +5629,7 @@ function generateChangeLog(oldBundles, newBundles, config) {
             parsedFiles: [...objects.parsedFiles, ...parsedTriggerFiles],
             errors: objects.errors
           })),
-          // For best-effort semantics we swallow failures here and keep going, because
+          // We swallow failures here and keep going, because
           // this helper's left type is `never`. Any such failures are treated as if
           // no triggers were parsed.
           TE__namespace.orElseW(() => TE__namespace.right({ parsedFiles: objects.parsedFiles, errors: objects.errors }))
@@ -5822,14 +5745,14 @@ function postHookCompile(page) {
   });
 }
 
-function generate(oldBundles, newBundles, config) {
+function generate(oldBundles, newBundles, config, debugLogger) {
   function handleFile(file) {
     if (isSkip(file)) {
       return TE__namespace.right("\u2714\uFE0F Done! Skipped writing files to the system.");
     }
     return writeFilesToSystem(file, config.targetDir);
   }
-  return _function.pipe(generateChangeLog(oldBundles, newBundles, config), TE__namespace.flatMap(handleFile));
+  return _function.pipe(generateChangeLog(oldBundles, newBundles, config, debugLogger), TE__namespace.flatMap(handleFile));
 }
 function writeFilesToSystem(pageData, outputDir) {
   return _function.pipe(
@@ -6245,19 +6168,15 @@ var __async = (__this, __arguments, generator) => {
 class Apexdocs {
   /**
    * Generates documentation out of Apex source files.
-   *
-   * NOTE:
-   * - Markdown + Changelog generators are now expected to be "best-effort" and may return both successes and
-   *   recoverable parse/reflection failures (to be presented at the end of the run).
-   * - OpenAPI is handled separately and already has its own logging/reporting behavior.
    */
-  static generate(config, logger) {
+  static generate(config, deps) {
     return __async(this, null, function* () {
+      const { logger, reflectionDebugLogger } = deps;
       logger.logSingle(`Generating ${config.targetGenerator} documentation...`);
       try {
         switch (config.targetGenerator) {
           case "markdown": {
-            return (yield processMarkdown(config, logger))();
+            return (yield processMarkdown(config, logger, reflectionDebugLogger))();
           }
           case "openapi": {
             const task = yield processOpenApi(config, logger);
@@ -6268,7 +6187,7 @@ class Apexdocs {
             return E__namespace.right("\u2714\uFE0F Documentation generated successfully!");
           }
           case "changelog": {
-            return (yield processChangeLog(config, logger))();
+            return (yield processChangeLog(config, logger, reflectionDebugLogger))();
           }
         }
       } catch (error) {
@@ -6278,23 +6197,9 @@ class Apexdocs {
   }
 }
 const readFiles = apply(processFiles, new DefaultFileSystem());
-function processMarkdown(config, logger) {
+function processMarkdown(config, logger, reflectionDebugLogger) {
   return __async(this, null, function* () {
-    const debugLogger = {
-      onStart: (filePath) => {
-        if (!logger.isDebugEnabled()) return;
-        logger.debug(`Parsing: ${filePath}`);
-      },
-      onSuccess: (filePath) => {
-        if (!logger.isDebugEnabled()) return;
-        logger.logSingle(`Parsing: ${filePath} OK`, "green");
-      },
-      onFailure: (filePath, errorMessage) => {
-        if (!logger.isDebugEnabled()) return;
-        logger.logSingle(`Parsing: ${filePath} FAILED`, "red");
-        logger.debug(`Parsing error: ${filePath} - ${errorMessage}`);
-      }
-    };
+    const debugLogger = reflectionDebugLogger;
     return _function.pipe(
       resolveAndValidateSourceDirectories(config),
       E__namespace.mapLeft((error) => new FileReadingError(`Failed to resolve source directories: ${error.message}`, error)),
@@ -6309,8 +6214,6 @@ function processMarkdown(config, logger) {
       TE__namespace.fromEither,
       TE__namespace.flatMap((fileBodies) => generate$1(fileBodies, config, debugLogger)),
       TE__namespace.map(() => "\u2714\uFE0F Documentation generated successfully!"),
-      // Best-effort behavior: if there were recoverable parse/reflection failures, they should be returned here
-      // (for presentation to the user at the end of the run), not thrown earlier.
       TE__namespace.mapLeft((err) => {
         const errors = toErrors(err);
         if (logger.isDebugEnabled()) {
@@ -6342,7 +6245,7 @@ function processOpenApi(config, logger) {
     );
   });
 }
-function processChangeLog(config, logger) {
+function processChangeLog(config, logger, reflectionDebugLogger) {
   return __async(this, null, function* () {
     function loadFiles() {
       const previousVersionConfig = {
@@ -6380,9 +6283,7 @@ function processChangeLog(config, logger) {
     return _function.pipe(
       loadFiles(),
       TE__namespace.fromEither,
-      TE__namespace.flatMap(([previous, current]) => generate(previous, current, config)),
-      // Best-effort behavior: if there were recoverable parse/reflection failures, they should be returned here
-      // (for presentation to the user at the end of the run), not thrown earlier.
+      TE__namespace.flatMap(([previous, current]) => generate(previous, current, config, reflectionDebugLogger)),
       TE__namespace.mapLeft((err) => {
         const errors = toErrors(err);
         if (logger.isDebugEnabled()) {
@@ -6414,10 +6315,76 @@ ${error.message}
 `;
 }
 
+class StdOutLogger {
+  constructor(options) {
+    this.debugEnabled = false;
+    var _a;
+    this.debugEnabled = (_a = options == null ? void 0 : options.debug) != null ? _a : false;
+  }
+  setDebug(enabled) {
+    this.debugEnabled = enabled;
+  }
+  isDebugEnabled() {
+    return this.debugEnabled;
+  }
+  debug(message) {
+    if (!this.debugEnabled) {
+      return;
+    }
+    this.logSingle(`[debug] ${String(message)}`);
+  }
+  /**
+   * Logs a message with optional arguments.
+   * @param message The message to log.
+   * @param args Optional arguments.
+   */
+  log(message, ...args) {
+    this.logSingle(message);
+    args.forEach((arg) => {
+      this.logSingle(arg);
+    });
+  }
+  /**
+   * Logs an error message with optional arguments.
+   * @param message The error message to log.
+   * @param args Optional arguments.
+   */
+  error(message, ...args) {
+    this.logSingle(message, "red");
+    args.forEach(() => {
+      this.logSingle(message, "red");
+    });
+  }
+  logSingle(text, color = "green") {
+    const logMessage = `${this.getChalkFn(color)((/* @__PURE__ */ new Date()).toLocaleString() + ": ")}${text}
+`;
+    process.stdout.write(logMessage);
+  }
+  getChalkFn(color) {
+    return color === "green" ? chalk.green : chalk.red;
+  }
+}
+class NoLogger {
+  setDebug(enabled) {
+  }
+  isDebugEnabled() {
+    return false;
+  }
+  debug(message) {
+  }
+  log(message, ...args) {
+  }
+  error(message, ...args) {
+  }
+  logSingle(text, color) {
+  }
+}
+
 exports.Apexdocs = Apexdocs;
 exports.NoLogger = NoLogger;
 exports.StdOutLogger = StdOutLogger;
 exports.changeLogDefaults = changeLogDefaults;
+exports.createReflectionDebugLogger = createReflectionDebugLogger;
 exports.markdownDefaults = markdownDefaults;
 exports.openApiDefaults = openApiDefaults;
 exports.skip = skip;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@cparra/apexdocs",
-  "version": "3.17.1-beta.1",
+  "version": "3.17.1-beta.3",
   "description": "Library with CLI capabilities to generate documentation for Salesforce Apex classes.",
   "engines": {
     "node": ">=18"