@angular-eslint/builder 19.5.1-alpha.6 → 19.5.1-alpha.8

package/dist/lint.impl.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"lint.impl.d.ts","sourceRoot":"","sources":["../src/lint.impl.ts"],"names":[],"mappings":"AAIA,OAAO,KAAK,EAAE,MAAM,EAAE,MAAM,UAAU,CAAC;;AAMvC,wBA8LE"}
+{"version":3,"file":"lint.impl.d.ts","sourceRoot":"","sources":["../src/lint.impl.ts"],"names":[],"mappings":"AAIA,OAAO,KAAK,EAAE,MAAM,EAAE,MAAM,UAAU,CAAC;;AAMvC,wBA0ME"}

package/dist/lint.impl.js

@@ -5,135 +5,147 @@ const fs_1 = require("fs");
 const path_1 = require("path");
 const eslint_utils_1 = require("./utils/eslint-utils");
 exports.default = (0, architect_1.createBuilder)(async (options, context) => {
-    const systemRoot = context.workspaceRoot;
-    // eslint resolves files relative to the current working directory.
-    // We want these paths to always be resolved relative to the workspace
-    // root to be able to run the lint executor from any subfolder.
-    process.chdir(systemRoot);
-    const projectName = context.target?.project ?? '<???>';
-    const printInfo = options.format && !options.silent;
-    if (printInfo) {
-        console.info(`\nLinting ${JSON.stringify(projectName)}...`);
-    }
-    const eslintConfigPath = options.eslintConfig
-        ? (0, path_1.resolve)(systemRoot, options.eslintConfig)
-        : undefined;
-    options.cacheLocation = options.cacheLocation
-        ? (0, path_1.join)(options.cacheLocation, projectName)
-        : null;
-    /**
-     * Until ESLint v9 is released and the new so called flat config is the default
-     * we only want to support it if the user has explicitly opted into it by converting
-     * their root ESLint config to use a supported flat config file name.
-     */
-    const useFlatConfig = eslint_utils_1.supportedFlatConfigNames.some((name) => (0, fs_1.existsSync)((0, path_1.join)(systemRoot, name)));
-    const { eslint, ESLint } = await (0, eslint_utils_1.resolveAndInstantiateESLint)(eslintConfigPath, options, useFlatConfig);
-    const version = ESLint?.version?.split('.');
-    if (!version ||
-        version.length < 2 ||
-        Number(version[0]) < 7 ||
-        (Number(version[0]) === 7 && Number(version[1]) < 6)) {
-        throw new Error('ESLint must be version 7.6 or higher.');
-    }
-    let lintResults = [];
     try {
-        lintResults = await eslint.lintFiles(options.lintFilePatterns);
-    }
-    catch (err) {
-        if (err instanceof Error &&
-            err.message.includes('You must therefore provide a value for the "parserOptions.project" property for @typescript-eslint/parser')) {
-            let eslintConfigPathForError = `for ${projectName}`;
-            const projectMetadata = await context.getProjectMetadata(projectName);
-            if (projectMetadata?.root) {
-                const { root } = projectMetadata;
-                eslintConfigPathForError =
-                    resolveESLintConfigPath(root) ?? '';
-            }
-            console.error(`
+        const systemRoot = context.workspaceRoot;
+        // eslint resolves files relative to the current working directory.
+        // We want these paths to always be resolved relative to the workspace
+        // root to be able to run the lint executor from any subfolder.
+        process.chdir(systemRoot);
+        const projectName = context.target?.project ?? '<???>';
+        const printInfo = options.format && !options.silent;
+        if (printInfo) {
+            console.info(`\nLinting ${JSON.stringify(projectName)}...`);
+        }
+        const eslintConfigPath = options.eslintConfig
+            ? (0, path_1.resolve)(systemRoot, options.eslintConfig)
+            : undefined;
+        options.cacheLocation = options.cacheLocation
+            ? (0, path_1.join)(options.cacheLocation, projectName)
+            : null;
+        /**
+         * Until ESLint v9 is released and the new so called flat config is the default
+         * we only want to support it if the user has explicitly opted into it by converting
+         * their root ESLint config to use a supported flat config file name.
+         */
+        const useFlatConfig = eslint_utils_1.supportedFlatConfigNames.some((name) => (0, fs_1.existsSync)((0, path_1.join)(systemRoot, name)));
+        const { eslint, ESLint } = await (0, eslint_utils_1.resolveAndInstantiateESLint)(eslintConfigPath, options, useFlatConfig);
+        const version = ESLint?.version?.split('.');
+        if (!version ||
+            version.length < 2 ||
+            Number(version[0]) < 7 ||
+            (Number(version[0]) === 7 && Number(version[1]) < 6)) {
+            throw new Error('ESLint must be version 7.6 or higher.');
+        }
+        let lintResults = [];
+        try {
+            lintResults = await eslint.lintFiles(options.lintFilePatterns);
+        }
+        catch (err) {
+            if (err instanceof Error &&
+                err.message.includes('You must therefore provide a value for the "parserOptions.project" property for @typescript-eslint/parser')) {
+                let eslintConfigPathForError = `for ${projectName}`;
+                const projectMetadata = await context.getProjectMetadata(projectName);
+                if (projectMetadata?.root) {
+                    const { root } = projectMetadata;
+                    eslintConfigPathForError =
+                        resolveESLintConfigPath(root) ?? '';
+                }
+                console.error(`
 Error: You have attempted to use a lint rule which requires the full TypeScript type-checker to be available, but you do not have \`parserOptions.project\` configured to point at your project tsconfig.json files in the relevant TypeScript file "overrides" block of your project ESLint config ${eslintConfigPath || eslintConfigPathForError}
 
 For full guidance on how to resolve this issue, please see https://github.com/angular-eslint/angular-eslint/blob/main/docs/RULES_REQUIRING_TYPE_INFORMATION.md
 `);
-            return {
-                success: false,
-            };
+                return {
+                    success: false,
+                };
+            }
+            // If some unexpected error, rethrow
+            throw err;
         }
-        // If some unexpected error, rethrow
-        throw err;
-    }
-    if (lintResults.length === 0) {
-        const ignoredPatterns = (await Promise.all(options.lintFilePatterns.map(async (pattern) => (await eslint.isPathIgnored(pattern)) ? pattern : null)))
-            .filter((pattern) => !!pattern)
-            .map((pattern) => `- '${pattern}'`);
-        if (ignoredPatterns.length) {
-            throw new Error(`All files matching the following patterns are ignored:\n${ignoredPatterns.join('\n')}\n\nPlease check your '.eslintignore' file.`);
+        if (lintResults.length === 0) {
+            const ignoredPatterns = (await Promise.all(options.lintFilePatterns.map(async (pattern) => (await eslint.isPathIgnored(pattern)) ? pattern : null)))
+                .filter((pattern) => !!pattern)
+                .map((pattern) => `- '${pattern}'`);
+            if (ignoredPatterns.length) {
+                throw new Error(`All files matching the following patterns are ignored:\n${ignoredPatterns.join('\n')}\n\nPlease check your '.eslintignore' file.`);
+            }
+            throw new Error('Invalid lint configuration. Nothing to lint. Please check your lint target pattern(s).');
         }
-        throw new Error('Invalid lint configuration. Nothing to lint. Please check your lint target pattern(s).');
-    }
-    // output fixes to disk, if applicable based on the options
-    await ESLint.outputFixes(lintResults);
-    const formatter = await eslint.loadFormatter(options.format);
-    let totalErrors = 0;
-    let totalWarnings = 0;
-    const reportOnlyErrors = options.quiet;
-    const maxWarnings = options.maxWarnings;
-    /**
-     * Depending on user configuration we may not want to report on all the
-     * results, so we need to adjust them before formatting.
-     */
-    const finalLintResults = lintResults
-        .map((result) => {
-        totalErrors += result.errorCount;
-        totalWarnings += result.warningCount;
-        if (result.errorCount || (result.warningCount && !reportOnlyErrors)) {
-            if (reportOnlyErrors) {
-                // Collect only errors (Linter.Severity === 2)
-                result.messages = result.messages.filter(({ severity }) => severity === 2);
+        // output fixes to disk, if applicable based on the options
+        await ESLint.outputFixes(lintResults);
+        const formatter = await eslint.loadFormatter(options.format);
+        let totalErrors = 0;
+        let totalWarnings = 0;
+        const reportOnlyErrors = options.quiet;
+        const maxWarnings = options.maxWarnings;
+        /**
+         * Depending on user configuration we may not want to report on all the
+         * results, so we need to adjust them before formatting.
+         */
+        const finalLintResults = lintResults
+            .map((result) => {
+            totalErrors += result.errorCount;
+            totalWarnings += result.warningCount;
+            if (result.errorCount || (result.warningCount && !reportOnlyErrors)) {
+                if (reportOnlyErrors) {
+                    // Collect only errors (Linter.Severity === 2)
+                    result.messages = result.messages.filter(({ severity }) => severity === 2);
+                }
+                return result;
             }
-            return result;
+            return null;
+        })
+            // Filter out the null values
+            .filter(Boolean);
+        const hasWarningsToPrint = totalWarnings > 0 && !reportOnlyErrors;
+        const hasErrorsToPrint = totalErrors > 0;
+        /**
+         * It's important that we format all results together so that custom
+         * formatters, such as checkstyle, can provide a valid output for the
+         * whole project being linted.
+         *
+         * Additionally, apart from when outputting to a file, we want to always
+         * log (even when no results) because different formatters handled the
+         * "no results" case differently.
+         */
+        const formattedResults = await formatter.format(finalLintResults);
+        if (options.outputFile) {
+            const pathToOutputFile = (0, path_1.join)(systemRoot, options.outputFile);
+            (0, fs_1.mkdirSync)((0, path_1.dirname)(pathToOutputFile), { recursive: true });
+            (0, fs_1.writeFileSync)(pathToOutputFile, formattedResults);
         }
-        return null;
-    })
-        // Filter out the null values
-        .filter(Boolean);
-    const hasWarningsToPrint = totalWarnings > 0 && !reportOnlyErrors;
-    const hasErrorsToPrint = totalErrors > 0;
-    /**
-     * It's important that we format all results together so that custom
-     * formatters, such as checkstyle, can provide a valid output for the
-     * whole project being linted.
-     *
-     * Additionally, apart from when outputting to a file, we want to always
-     * log (even when no results) because different formatters handled the
-     * "no results" case differently.
-     */
-    const formattedResults = await formatter.format(finalLintResults);
-    if (options.outputFile) {
-        const pathToOutputFile = (0, path_1.join)(systemRoot, options.outputFile);
-        (0, fs_1.mkdirSync)((0, path_1.dirname)(pathToOutputFile), { recursive: true });
-        (0, fs_1.writeFileSync)(pathToOutputFile, formattedResults);
-    }
-    else {
-        console.info(formattedResults);
-    }
-    if (hasWarningsToPrint && printInfo) {
-        console.warn('Lint warnings found in the listed files.\n');
-    }
-    if (hasErrorsToPrint && printInfo) {
-        console.error('Lint errors found in the listed files.\n');
-    }
-    if ((totalWarnings === 0 || reportOnlyErrors) &&
-        totalErrors === 0 &&
-        printInfo) {
-        console.info('All files pass linting.\n');
+        else {
+            console.info(formattedResults);
+        }
+        if (hasWarningsToPrint && printInfo) {
+            console.warn('Lint warnings found in the listed files.\n');
+        }
+        if (hasErrorsToPrint && printInfo) {
+            console.error('Lint errors found in the listed files.\n');
+        }
+        if ((totalWarnings === 0 || reportOnlyErrors) &&
+            totalErrors === 0 &&
+            printInfo) {
+            console.info('All files pass linting.\n');
+        }
+        const tooManyWarnings = maxWarnings >= 0 && totalWarnings > maxWarnings;
+        if (tooManyWarnings && printInfo) {
+            console.error(`Found ${totalWarnings} warnings, which exceeds your configured limit (${options.maxWarnings}). Either increase your maxWarnings limit or fix some of the lint warnings.`);
+        }
+        return {
+            success: options.force || (totalErrors === 0 && !tooManyWarnings),
+        };
     }
-    const tooManyWarnings = maxWarnings >= 0 && totalWarnings > maxWarnings;
-    if (tooManyWarnings && printInfo) {
-        console.error(`Found ${totalWarnings} warnings, which exceeds your configured limit (${options.maxWarnings}). Either increase your maxWarnings limit or fix some of the lint warnings.`);
+    catch (err) {
+        let errorMessage = 'Unknown error';
+        if (err instanceof Error) {
+            errorMessage = `Error when running ESLint: ${err.message}`;
+        }
+        return {
+            success: false,
+            error: String(errorMessage),
+        };
     }
-    return {
-        success: options.force || (totalErrors === 0 && !tooManyWarnings),
-    };
 });
 function resolveESLintConfigPath(projectRoot) {
     const rcPath = (0, path_1.join)(projectRoot, '.eslintrc.json');

package/dist/utils/eslint-utils.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"eslint-utils.d.ts","sourceRoot":"","sources":["../../src/utils/eslint-utils.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,MAAM,EAAE,MAAM,QAAQ,CAAC;AACrC,OAAO,KAAK,EAAE,MAAM,EAAE,MAAM,WAAW,CAAC;AAExC,eAAO,MAAM,wBAAwB,UAIpC,CAAC;AAuBF,wBAAsB,2BAA2B,CAC/C,gBAAgB,EAAE,MAAM,GAAG,SAAS,EACpC,OAAO,EAAE,MAAM,EACf,aAAa,UAAQ;;;GA6FtB"}
+{"version":3,"file":"eslint-utils.d.ts","sourceRoot":"","sources":["../../src/utils/eslint-utils.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,MAAM,EAAE,MAAM,QAAQ,CAAC;AACrC,OAAO,KAAK,EAAE,MAAM,EAAE,MAAM,WAAW,CAAC;AAExC,eAAO,MAAM,wBAAwB,UAOpC,CAAC;AAsBF,wBAAsB,2BAA2B,CAC/C,gBAAgB,EAAE,MAAM,GAAG,SAAS,EACpC,OAAO,EAAE,MAAM,EACf,aAAa,UAAQ;;;GA6FtB"}

package/dist/utils/eslint-utils.js

@@ -6,6 +6,9 @@ exports.supportedFlatConfigNames = [
     'eslint.config.js',
     'eslint.config.mjs',
     'eslint.config.cjs',
+    'eslint.config.ts',
+    'eslint.config.mts',
+    'eslint.config.cts',
 ];
 async function resolveESLintClass(useFlatConfig = false) {
     try {
@@ -18,7 +21,6 @@ async function resolveESLintClass(useFlatConfig = false) {
         if (!useFlatConfig) {
             return eslint.ESLint;
         }
-        // eslint-disable-next-line @typescript-eslint/no-var-requires
         const { FlatESLint } = require('eslint/use-at-your-own-risk');
         return FlatESLint;
     }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@angular-eslint/builder",
-  "version": "19.5.1-alpha.6",
+  "version": "19.5.1-alpha.8",
   "description": "Angular CLI builder for ESLint",
   "license": "MIT",
   "main": "dist/index.js",