@angular-eslint/builder 17.5.3-alpha.1 → 17.5.3

package/README.md

@@ -1,7 +1 @@
 # @angular-eslint/builder
-
-Please see https://github.com/angular-eslint/angular-eslint for full usage instructions and guidance.
-
-The `@angular-eslint/builder` package is a custom Angular CLI builder that allows you to run ESLint on your Angular CLI projects.
-
-It wraps the ESLint programmatic node API (https://eslint.org/docs/latest/integrate/nodejs-api) to provide a seamless experience via `ng lint` that is closely equivalent to using the `eslint` CLI directly.

package/dist/lint.impl.d.ts

@@ -1,3 +1,4 @@
-declare const _default: any;
+import type { Schema } from './schema';
+declare const _default: import("@angular-devkit/architect/src/internal").Builder<Schema & import("@angular-devkit/core").JsonObject>;
 export default _default;
 //# sourceMappingURL=lint.impl.d.ts.map

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

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

package/dist/lint.impl.js

@@ -1,16 +1,16 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
-const devkit_1 = require("@nx/devkit");
+const architect_1 = require("@angular-devkit/architect");
 const fs_1 = require("fs");
 const path_1 = require("path");
 const eslint_utils_1 = require("./utils/eslint-utils");
-exports.default = (0, devkit_1.convertNxExecutor)(async (options, context) => {
-    const systemRoot = context.root;
+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.projectName || '<???>';
+    const projectName = context.target?.project ?? '<???>';
     const printInfo = options.format && !options.silent;
     if (printInfo) {
         console.info(`\nLinting ${JSON.stringify(projectName)}...`);
@@ -30,7 +30,7 @@ exports.default = (0, devkit_1.convertNxExecutor)(async (options, context) => {
      * we only want to support it if the user has explicitly opted into it by converting
      * their root ESLint config to use eslint.config.js
      */
-    const useFlatConfig = (0, fs_1.existsSync)((0, devkit_1.joinPathFragments)(devkit_1.workspaceRoot, 'eslint.config.js'));
+    const useFlatConfig = (0, fs_1.existsSync)((0, path_1.join)(systemRoot, 'eslint.config.js'));
     const { eslint, ESLint } = await (0, eslint_utils_1.resolveAndInstantiateESLint)(eslintConfigPath, options, useFlatConfig);
     const version = ESLint?.version?.split('.');
     if (!version ||
@@ -47,8 +47,9 @@ exports.default = (0, devkit_1.convertNxExecutor)(async (options, context) => {
         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}`;
-            if (context.projectsConfigurations?.projects?.[projectName]?.root) {
-                const { root } = context.projectsConfigurations.projects[projectName];
+            const projectMetadata = await context.getProjectMetadata(projectName);
+            if (projectMetadata?.root) {
+                const { root } = projectMetadata;
                 eslintConfigPathForError = `\`${root}/.eslintrc.json\``;
             }
             console.error(`
@@ -111,7 +112,7 @@ For full guidance on how to resolve this issue, please see https://github.com/an
      */
     const formattedResults = await formatter.format(finalLintResults);
     if (options.outputFile) {
-        const pathToOutputFile = (0, path_1.join)(context.root, 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);
     }

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;AAuBxC,wBAAsB,2BAA2B,CAC/C,gBAAgB,EAAE,MAAM,GAAG,SAAS,EACpC,OAAO,EAAE,MAAM,EACf,aAAa,UAAQ;;;GAyEtB"}
+{"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;AAiBxC,wBAAsB,2BAA2B,CAC/C,gBAAgB,EAAE,MAAM,GAAG,SAAS,EACpC,OAAO,EAAE,MAAM,EACf,aAAa,UAAQ;;;GAoEtB"}

package/dist/utils/eslint-utils.js

@@ -3,14 +3,8 @@ Object.defineProperty(exports, "__esModule", { value: true });
 exports.resolveAndInstantiateESLint = void 0;
 async function resolveESLintClass(useFlatConfig = false) {
     try {
-        // In eslint 8.57.0 (the final v8 version), a dedicated API was added for resolving the correct ESLint class.
-        const eslint = await import('eslint');
-        if (typeof eslint.loadESLint === 'function') {
-            return await eslint.loadESLint({ useFlatConfig });
-        }
-        // If that API is not available (an older version of v8), we need to use the old way of resolving the ESLint class.
         if (!useFlatConfig) {
-            return eslint.ESLint;
+            return (await import('eslint')).ESLint;
         }
         // eslint-disable-next-line @typescript-eslint/no-var-requires
         const { FlatESLint } = require('eslint/use-at-your-own-risk');
@@ -44,6 +38,7 @@ async function resolveAndInstantiateESLint(eslintConfigPath, options, useFlatCon
          * not be any html files in the project, so keeping it true would break linting every time
          */
         errorOnUnmatchedPattern: false,
+        reportUnusedDisableDirectives: options.reportUnusedDisableDirectives || undefined,
     };
     if (useFlatConfig) {
         if (typeof options.useEslintrc !== 'undefined') {
@@ -55,9 +50,6 @@ async function resolveAndInstantiateESLint(eslintConfigPath, options, useFlatCon
         if (options.ignorePath !== undefined) {
             throw new Error('For Flat Config, ESLint removed `ignorePath` and so it is not supported as an option. See https://eslint.org/docs/latest/use/configure/configuration-files-new');
         }
-        if (options.reportUnusedDisableDirectives) {
-            throw new Error('For Flat Config, ESLint removed `reportedUnusedDisableDirectives` and so it is not supported as an option. See https://eslint.org/docs/latest/use/configure/configuration-files-new');
-        }
     }
     else {
         eslintOptions.rulePaths = options.rulesdir || [];
@@ -69,8 +61,6 @@ async function resolveAndInstantiateESLint(eslintConfigPath, options, useFlatCon
          * merge the provided config with others it finds automatically.
          */
         eslintOptions.useEslintrc = !options.noEslintrc;
-        eslintOptions.reportUnusedDisableDirectives =
-            options.reportUnusedDisableDirectives || undefined;
     }
     const eslint = new ESLint(eslintOptions);
     return {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@angular-eslint/builder",
-  "version": "17.5.3-alpha.1",
+  "version": "17.5.3",
   "description": "Angular CLI builder for ESLint",
   "license": "MIT",
   "main": "dist/index.js",
@@ -18,13 +18,9 @@
     "LICENSE",
     "builders.json"
   ],
-  "dependencies": {
-    "@nx/devkit": "^19.0.6",
-    "nx": "^19.0.6"
-  },
   "builders": "./builders.json",
   "peerDependencies": {
-    "eslint": "^8.57.0 || ^9.0.0",
+    "eslint": "^7.20.0 || ^8.0.0",
     "typescript": "*"
   },
   "gitHead": "e2006e5e9c99e5a943d1a999e0efa5247d29ec24"