@cparra/apexdocs 3.0.0-alpha.5 → 3.0.0-alpha.7

package/dist/cli/generate.js

@@ -13,7 +13,6 @@ var Handlebars = require('handlebars');
 var defaults = require('../defaults-DGKfeZq-.js');
 var fs = require('fs');
 var chalk = require('chalk');
-var ora = require('ora');
 var cosmiconfig = require('cosmiconfig');
 var yargs = require('yargs');
 var cosmiconfigTypescriptLoader = require('cosmiconfig-typescript-loader');
@@ -1690,7 +1689,7 @@ class FileWriter {
       const { outputDocPath, content } = this.getTargetLocation(file, outputDir);
       fs__namespace.mkdirSync(path__namespace.dirname(outputDocPath), { recursive: true });
       fs__namespace.writeFileSync(outputDocPath, content, "utf8");
-      onWriteCallback(file);
+      onWriteCallback == null ? void 0 : onWriteCallback(file);
     });
   }
   static getTargetLocation(file, outputDir) {
@@ -1734,16 +1733,8 @@ class Logger {
   static getChalkFn(color) {
     return color === "green" ? chalk.green : chalk.red;
   }
-  static startSpinner(text) {
-    this.spinner.text = text;
-    this.spinner.start();
-  }
-  static succeedSpinner(text) {
-    this.spinner.succeed(text);
-  }
 }
 Logger.currentFrame = 0;
-Logger.spinner = ora();
 
 const referenceGuideTemplate = `
 # Apex Reference Guide
@@ -1779,16 +1770,28 @@ var __spreadValues$5 = (a, b) => {
   return a;
 };
 var __spreadProps$5 = (a, b) => __defProps$5(a, __getOwnPropDescs$5(b));
+class FileWritingError {
+  constructor(message, error) {
+    this.message = message;
+    this.error = error;
+    this._tag = "FileWritingError";
+  }
+}
 function generate(bundles, config) {
   return _function.pipe(
     generateDocumentationBundle(bundles, config),
-    TE__namespace.map((files) => writeFilesToSystem(files, config.targetDir)),
+    TE__namespace.flatMap((files) => writeFilesToSystem(files, config.targetDir)),
     TE__namespace.mapLeft((error) => {
       if (error._tag === "HookError") {
         Logger.error("Error(s) occurred while processing hooks. Please review the following issues:");
         Logger.error(error.error);
         return;
       }
+      if (error._tag === "FileWritingError") {
+        Logger.error(error.message);
+        Logger.error(error.error);
+        return;
+      }
       const errorMessages = [
         "Error(s) occurred while parsing files. Please review the following issues:",
         ...error.errors.map(formatReflectionError)
@@ -1803,12 +1806,15 @@ function generateDocumentationBundle(bundles, config) {
   }));
 }
 function writeFilesToSystem(files, outputDir) {
-  FileWriter.write(
-    [files.referenceGuide, ...files.docs].filter((file) => !isSkip(file)),
-    outputDir,
-    () => {
-    }
-  );
+  try {
+    FileWriter.write(
+      [files.referenceGuide, ...files.docs].filter((file) => !isSkip(file)),
+      outputDir
+    );
+    return TE__namespace.right(void 0);
+  } catch (error) {
+    return TE__namespace.left(new FileWritingError("An error occurred while writing files to the system.", error));
+  }
 }
 function formatReflectionError(error) {
   return `Source file: ${error.file}
@@ -2867,8 +2873,7 @@ class Apexdocs {
    */
   static generate(config) {
     return __async$1(this, null, function* () {
-      Logger.logSingle("Initializing...");
-      Logger.startSpinner("Processing files...");
+      Logger.logSingle(`Generating ${config.targetGenerator} documentation...`);
       try {
         const fileBodies = ApexFileReader.processFiles(new DefaultFileSystem(), config.sourceDir, config.includeMetadata);
         switch (config.targetGenerator) {
@@ -2879,8 +2884,9 @@ class Apexdocs {
             openApi(fileBodies, config);
             break;
         }
-      } finally {
-        Logger.succeedSpinner("Files processed.");
+        Logger.logSingle("\u2714\uFE0F Documentation generated successfully!");
+      } catch (error) {
+        Logger.logSingle("\u274C An error occurred while generating the documentation.", "red");
       }
     });
   }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@cparra/apexdocs",
-  "version": "3.0.0-alpha.5",
+  "version": "3.0.0-alpha.7",
   "description": "Library with CLI capabilities to generate documentation for Salesforce Apex classes.",
   "keywords": [
     "apex",
@@ -68,7 +68,6 @@
     "fp-ts": "^2.16.8",
     "handlebars": "^4.7.8",
     "js-yaml": "^4.1.0",
-    "ora": "^5.4.1",
     "type-fest": "^4.23.0",
     "yargs": "^17.7.2"
   },

package/src/application/Apexdocs.ts

@@ -14,8 +14,7 @@ export class Apexdocs {
    * Generates documentation out of Apex source files.
    */
   static async generate(config: UserDefinedConfig): Promise<void> {
-    Logger.logSingle('Initializing...');
-    Logger.startSpinner('Processing files...');
+    Logger.logSingle(`Generating ${config.targetGenerator} documentation...`);
 
     try {
       const fileBodies = ApexFileReader.processFiles(new DefaultFileSystem(), config.sourceDir, config.includeMetadata);
@@ -28,9 +27,10 @@ export class Apexdocs {
           openApi(fileBodies, config);
           break;
       }
-    } finally {
-      // TODO: Allow for failures
-      Logger.succeedSpinner('Files processed.');
+
+      Logger.logSingle('✔️ Documentation generated successfully!');
+    } catch (error) {
+      Logger.logSingle('❌ An error occurred while generating the documentation.', 'red');
     }
   }
 }

package/src/application/file-writer.ts

@@ -3,12 +3,12 @@ import * as path from 'path';
 import { PageData } from '../core/shared/types';
 
 export class FileWriter {
-  static write(files: PageData[], outputDir: string, onWriteCallback: (file: PageData) => void) {
+  static write(files: PageData[], outputDir: string, onWriteCallback?: (file: PageData) => void) {
     files.forEach((file) => {
       const { outputDocPath, content } = this.getTargetLocation(file, outputDir);
       fs.mkdirSync(path.dirname(outputDocPath), { recursive: true });
       fs.writeFileSync(outputDocPath, content, 'utf8');
-      onWriteCallback(file);
+      onWriteCallback?.(file);
     });
   }
 

package/src/application/generators/markdown.ts

@@ -13,10 +13,18 @@ import { referenceGuideTemplate } from '../../core/markdown/templates/reference-
 import * as TE from 'fp-ts/TaskEither';
 import { isSkip } from '../../core/shared/utils';
 
+class FileWritingError {
+  readonly _tag = 'FileWritingError';
+  constructor(
+    public message: string,
+    public error: unknown,
+  ) {}
+}
+
 export default function generate(bundles: UnparsedSourceFile[], config: UserDefinedMarkdownConfig) {
   return pipe(
     generateDocumentationBundle(bundles, config),
-    TE.map((files) => writeFilesToSystem(files, config.targetDir)),
+    TE.flatMap((files) => writeFilesToSystem(files, config.targetDir)),
     TE.mapLeft((error) => {
       if (error._tag === 'HookError') {
         Logger.error('Error(s) occurred while processing hooks. Please review the following issues:');
@@ -24,6 +32,12 @@ export default function generate(bundles: UnparsedSourceFile[], config: UserDefi
         return;
       }
 
+      if (error._tag === 'FileWritingError') {
+        Logger.error(error.message);
+        Logger.error(error.error);
+        return;
+      }
+
       const errorMessages = [
         'Error(s) occurred while parsing files. Please review the following issues:',
         ...error.errors.map(formatReflectionError),
@@ -42,16 +56,17 @@ function generateDocumentationBundle(bundles: UnparsedSourceFile[], config: User
 }
 
 function writeFilesToSystem(files: PostHookDocumentationBundle, outputDir: string) {
-  FileWriter.write(
-    [files.referenceGuide, ...files.docs]
-      // Filter out any files that should be skipped
-      .filter((file) => !isSkip(file)) as PageData[],
-    outputDir,
-    () => {
-      // TODO: Don't do this here, send things back up.
-      //Logger.logSingle(`${file.outputDocPath} processed.`, 'green');
-    },
-  );
+  try {
+    FileWriter.write(
+      [files.referenceGuide, ...files.docs]
+        // Filter out any files that should be skipped
+        .filter((file) => !isSkip(file)) as PageData[],
+      outputDir,
+    );
+    return TE.right(undefined);
+  } catch (error) {
+    return TE.left(new FileWritingError('An error occurred while writing files to the system.', error));
+  }
 }
 
 function formatReflectionError(error: ReflectionError) {

package/src/util/logger.ts

@@ -1,11 +1,9 @@
 import chalk from 'chalk';
-import ora from 'ora';
 /**
  * Logs messages to the console.
  */
 export class Logger {
   static currentFrame = 0;
-  static spinner = ora();
 
   /**
    * Logs a message with optional arguments.
@@ -43,13 +41,4 @@ export class Logger {
   private static getChalkFn(color: 'green' | 'red') {
     return color === 'green' ? chalk.green : chalk.red;
   }
-
-  public static startSpinner(text: string) {
-    this.spinner.text = text;
-    this.spinner.start();
-  }
-
-  public static succeedSpinner(text: string) {
-    this.spinner.succeed(text);
-  }
 }