fix: linting

Fixes error happening with linting and also allows the cli to be ran just for
linting purposes

Signed-off-by: flakey5 <[email protected]>

Author: flakey5

bin/cli.mjs

@@ -33,7 +33,7 @@ program
     new Option(
       '-o, --output <path>',
       'Specify the relative or absolute output directory'
-    ).makeOptionMandatory()
+    )
   )
   .addOption(
     new Option(
@@ -112,18 +112,20 @@ const { getAllMajors } = createNodeReleases(changelog);
 
 linter.lintAll(parsedApiDocs);
 
-await runGenerators({
-  // A list of target modes for the API docs parser
-  generators: target,
-  // Resolved `input` to be used
-  input: input,
-  // Resolved `output` path to be used
-  output: resolve(output),
-  // Resolved SemVer of current Node.js version
-  version: coerce(version),
-  // A list of all Node.js major versions with LTS status
-  releases: await getAllMajors(),
-});
+if (target && output) {
+  await runGenerators({
+    // A list of target modes for the API docs parser
+    generators: target,
+    // Resolved `input` to be used
+    input: input,
+    // Resolved `output` path to be used
+    output: resolve(output),
+    // Resolved SemVer of current Node.js version
+    version: coerce(version),
+    // A list of all Node.js major versions with LTS status
+    releases: await getAllMajors(),
+  });
+}
 
 linter.report(reporter);
 

src/linter/index.mjs

@@ -11,6 +11,17 @@ import rules from './rules/index.mjs';
  * @param {string[]} disabledRules List of disabled rules names
  */
 const createLinter = (dryRun, disabledRules) => {
+  /**
+   * Retrieves all enabled rules
+   *
+   * @returns {import('./types').LintRule[]}
+   */
+  const getEnabledRules = () => {
+    return Object.entries(rules)
+      .filter(([ruleName]) => !disabledRules.includes(ruleName))
+      .map(([, rule]) => rule);
+  };
+
   const engine = createLinterEngine(getEnabledRules(disabledRules));
 
   /**
@@ -56,17 +67,6 @@ const createLinter = (dryRun, disabledRules) => {
     return issues.some(issue => issue.level === 'error');
   };
 
-  /**
-   * Retrieves all enabled rules
-   *
-   * @returns {import('./types').LintRule[]}
-   */
-  const getEnabledRules = () => {
-    return Object.entries(rules)
-      .filter(([ruleName]) => !disabledRules.includes(ruleName))
-      .map(([, rule]) => rule);
-  };
-
   return {
     lintAll,
     report,