feat: add lint trim option

Author: zdm

lib/lint/file.js

@@ -15,6 +15,7 @@ import Table from "#core/text/table";
 import * as utils from "#core/utils";
 import uuid from "#core/uuid";
 import Markdown from "#lib/markdown";
+import { getCliConfig } from "#lib/utils";
 
 // NOTE: status codes:
 const STATUSES = {
@@ -411,9 +412,7 @@ export class LintFile {
     // private
     async #lint () {
         const configs = await this.#getConfigs( this.#cwd ),
-            cliConfig = configs.cli
-                ? await this.#getCliConfig( configs.cli )
-                : null,
+            cliConfig = this.#getCliConfig( configs.cli ),
             editorConfig = await this.#getEditorConfig( {
                 "fallbackToDefault": configs.package
                     ? Boolean( configs.cli )
@@ -1184,15 +1183,13 @@ export class LintFile {
         }
     }
 
-    async #getCliConfig ( path ) {
+    #getCliConfig ( path ) {
         if ( !path ) return;
 
         this.#cache.cliConfigs ||= new Map();
 
         if ( !this.#cache.cliConfigs.has( path ) ) {
-            this.#cache.cliConfigs.set( path, null );
-
-            // XXX
+            this.#cache.cliConfigs.set( path, getCliConfig( path ) );
         }
 
         return this.#cache.cliConfigs.get( path );

lib/package.js

@@ -2,7 +2,6 @@ import "#core/temporal";
 import childProcess from "node:child_process";
 import fs from "node:fs";
 import path from "node:path";
-import Ajv from "#core/ajv";
 import ansi from "#core/ansi";
 import GitHubApi from "#core/api/github";
 import { readConfig, readConfigSync, writeConfigSync } from "#core/config";
@@ -15,17 +14,15 @@ import GlobPatterns from "#core/glob/patterns";
 import Locale from "#core/locale";
 import SemanticVersion from "#core/semantic-version";
 import Table from "#core/text/table";
-import { confirm, mergeObjects, objectIsEmpty, repeatAction } from "#core/utils";
+import { confirm, objectIsEmpty, repeatAction } from "#core/utils";
 import yaml from "#core/yaml";
 import Git from "#lib/git";
 import lintFile from "#lib/lint/file";
 import Dependencies from "#lib/package/dependencies";
 import Docs from "#lib/package/docs";
 import Localization from "#lib/package/localization";
 import Wiki from "#lib/package/wiki";
-
-const validateCliConfig = new Ajv().compileFile( import.meta.resolve( "#resources/schemas/cli.config.schema.yaml" ) ),
-    defaultCliConfig = await readConfig( "#resources/cli.config.yaml", { "resolve": import.meta.url } );
+import { getCliConfig } from "#lib/utils";
 
 export default class Package {
     #root;
@@ -153,15 +150,9 @@ export default class Package {
 
     get cliConfig () {
         if ( this.#cliConfig === undefined ) {
-            this.#cliConfig = null;
-
-            if ( fs.existsSync( this.root + "/cli.config.yaml" ) ) {
-                const cliConfig = mergeObjects( {}, defaultCliConfig, readConfigSync( this.root + "/cli.config.yaml" ) );
-
-                if ( !validateCliConfig( cliConfig ) ) throw `CLI config is not valid:\n${ validateCliConfig.errors }`;
-
-                this.#cliConfig = cliConfig;
-            }
+            this.#cliConfig = getCliConfig( this.root + "/cli.config.yaml", {
+                "validate": true,
+            } );
         }
 
         return this.#cliConfig;

lib/utils.js

@@ -1,12 +1,19 @@
+import fs from "node:fs";
+import Ajv from "#core/ajv";
 import ansi from "#core/ansi";
+import { readConfig, readConfigSync } from "#core/config";
 import env from "#core/env";
 import { globSync } from "#core/glob";
 import GlobPatterns from "#core/glob/patterns";
 import Table from "#core/text/table";
+import { mergeObjects } from "#core/utils";
 import Package from "#lib/package";
 
 env.loadUserEnv();
 
+const validateCliConfig = new Ajv().compileFile( import.meta.resolve( "#resources/schemas/cli.config.schema.yaml" ) ),
+    defaultCliConfig = await readConfig( "#resources/cli.config.yaml", { "resolve": import.meta.url } );
+
 export function getLintReportTable ( options = {} ) {
     return new Table( {
         "style": "borderless",
@@ -89,3 +96,15 @@ export function findWorkspacePackages ( { patterns, git = true, "package": isPac
 
     return result( 200, packages );
 }
+
+export function getCliConfig ( path, { validate = true } = {} ) {
+    if ( !fs.existsSync( path ) ) return null;
+
+    const cliConfig = mergeObjects( {}, defaultCliConfig, readConfigSync( path ) );
+
+    if ( validate ) {
+        if ( !validateCliConfig( cliConfig ) ) throw `CLI config is not valid:\n${ validateCliConfig.errors }`;
+    }
+
+    return cliConfig;
+}