@rollup/plugin-eslint - npm Package Compare versions

Comparing version 8.0.3 to 8.0.4

dist/cjs/index.js

@@ -9,8 +9,27 @@ 'use strict';
 
+function _interopNamespaceDefault(e) {
+    var n = Object.create(null);
+    if (e) {
+        Object.keys(e).forEach(function (k) {
+            if (k !== 'default') {
+                var d = Object.getOwnPropertyDescriptor(e, k);
+                Object.defineProperty(n, k, d.get ? d : {
+                    enumerable: true,
+                    get: function () { return e[k]; }
+                });
+            }
+        });
+    }
+    n.default = e;
+    return Object.freeze(n);
+}
+
+var path__namespace = /*#__PURE__*/_interopNamespaceDefault(path);
+
 function normalizePath(id) {
-    return path.relative(process.cwd(), id).split(path.sep).join('/');
+    return path__namespace.relative(process.cwd(), id).split(path__namespace.sep).join('/');
 }
 function eslint(options = {}) {
     if (typeof options === 'string') {
-        const configFile = path.resolve(process.cwd(), options);
+        const configFile = path__namespace.resolve(process.cwd(), options);
         // eslint-disable-next-line global-require, import/no-dynamic-require, no-param-reassign
@@ -22,39 +41,46 @@ options = require(configFile);
     }
-    const { include, exclude = /node_modules/, throwOnWarning = false, throwOnError = false, formatter = 'stylish', ...eslintOptions } = options;
-    const eslintInstance = new eslint$1.ESLint(eslintOptions);
-    const filter = pluginutils.createFilter(include, exclude);
+    const cli = new eslint$1.CLIEngine(options);
+    let formatter;
+    switch (typeof options.formatter) {
+        case 'string':
+            formatter = cli.getFormatter(options.formatter);
+            break;
+        case 'function':
+            ({ formatter } = options);
+            break;
+        default:
+            formatter = cli.getFormatter('stylish');
+    }
+    const filter = pluginutils.createFilter(options.include, options.exclude || /node_modules/);
     return {
         name: 'eslint',
-        async transform(_, id) {
+        // eslint-disable-next-line consistent-return
+        transform(code, id) {
             const file = normalizePath(id);
-            if (!filter(id) || (await eslintInstance.isPathIgnored(file))) {
+            if (!filter(id) || cli.isPathIgnored(file)) {
                 return null;
             }
-            const results = await eslintInstance.lintFiles(file);
-            const [result] = results;
-            if (eslintOptions.fix) {
-                await eslint$1.ESLint.outputFixes(results);
+            const report = cli.executeOnText(code, file);
+            const hasWarnings = options.throwOnWarning && report.warningCount !== 0;
+            const hasErrors = options.throwOnError && report.errorCount !== 0;
+            if (options.fix && report) {
+                eslint$1.CLIEngine.outputFixes(report);
             }
-            if (result.warningCount === 0 && result.errorCount === 0) {
+            if (report.warningCount === 0 && report.errorCount === 0) {
                 return null;
             }
-            const eslintFormatter = typeof formatter === 'string'
-                ? await eslintInstance.loadFormatter(formatter)
-                : { format: formatter };
-            const output = eslintFormatter.format(results);
-            if (output) {
+            const result = formatter(report.results);
+            if (result) {
                 // eslint-disable-next-line no-console
-                console.log(output);
+                console.log(result);
             }
-            const errorMessages = [];
-            if (result.warningCount > 0 && throwOnWarning) {
-                errorMessages.push(`${result.warningCount} warning${result.warningCount > 1 ? 's' : ''}`);
+            if (hasWarnings && hasErrors) {
+                throw Error('Warnings or errors were found');
             }
-            if (result.errorCount > 0 && throwOnError) {
-                errorMessages.push(`${result.errorCount} error${result.errorCount > 1 ? 's' : ''}`);
+            if (hasWarnings) {
+                throw Error('Warnings were found');
             }
-            if (errorMessages.length > 0) {
-                throw new Error(`Found ${errorMessages.join(' and ')} in ${path.relative('.', result.filePath)}`);
+            if (hasErrors) {
+                throw Error('Errors were found');
             }
-            return null;
         }
@@ -61,0 +87,0 @@ };

dist/es/index.js

@@ -1,11 +0,11 @@
-import { resolve, relative, sep } from 'path';
+import * as path from 'path';
 import { createFilter } from '@rollup/pluginutils';
-import { ESLint } from 'eslint';
+import { CLIEngine } from 'eslint';
 
 function normalizePath(id) {
-    return relative(process.cwd(), id).split(sep).join('/');
+    return path.relative(process.cwd(), id).split(path.sep).join('/');
 }
 function eslint(options = {}) {
     if (typeof options === 'string') {
-        const configFile = resolve(process.cwd(), options);
+        const configFile = path.resolve(process.cwd(), options);
         // eslint-disable-next-line global-require, import/no-dynamic-require, no-param-reassign
@@ -17,39 +17,46 @@ options = require(configFile);
     }
-    const { include, exclude = /node_modules/, throwOnWarning = false, throwOnError = false, formatter = 'stylish', ...eslintOptions } = options;
-    const eslintInstance = new ESLint(eslintOptions);
-    const filter = createFilter(include, exclude);
+    const cli = new CLIEngine(options);
+    let formatter;
+    switch (typeof options.formatter) {
+        case 'string':
+            formatter = cli.getFormatter(options.formatter);
+            break;
+        case 'function':
+            ({ formatter } = options);
+            break;
+        default:
+            formatter = cli.getFormatter('stylish');
+    }
+    const filter = createFilter(options.include, options.exclude || /node_modules/);
     return {
         name: 'eslint',
-        async transform(_, id) {
+        // eslint-disable-next-line consistent-return
+        transform(code, id) {
             const file = normalizePath(id);
-            if (!filter(id) || (await eslintInstance.isPathIgnored(file))) {
+            if (!filter(id) || cli.isPathIgnored(file)) {
                 return null;
             }
-            const results = await eslintInstance.lintFiles(file);
-            const [result] = results;
-            if (eslintOptions.fix) {
-                await ESLint.outputFixes(results);
+            const report = cli.executeOnText(code, file);
+            const hasWarnings = options.throwOnWarning && report.warningCount !== 0;
+            const hasErrors = options.throwOnError && report.errorCount !== 0;
+            if (options.fix && report) {
+                CLIEngine.outputFixes(report);
             }
-            if (result.warningCount === 0 && result.errorCount === 0) {
+            if (report.warningCount === 0 && report.errorCount === 0) {
                 return null;
             }
-            const eslintFormatter = typeof formatter === 'string'
-                ? await eslintInstance.loadFormatter(formatter)
-                : { format: formatter };
-            const output = eslintFormatter.format(results);
-            if (output) {
+            const result = formatter(report.results);
+            if (result) {
                 // eslint-disable-next-line no-console
-                console.log(output);
+                console.log(result);
             }
-            const errorMessages = [];
-            if (result.warningCount > 0 && throwOnWarning) {
-                errorMessages.push(`${result.warningCount} warning${result.warningCount > 1 ? 's' : ''}`);
+            if (hasWarnings && hasErrors) {
+                throw Error('Warnings or errors were found');
             }
-            if (result.errorCount > 0 && throwOnError) {
-                errorMessages.push(`${result.errorCount} error${result.errorCount > 1 ? 's' : ''}`);
+            if (hasWarnings) {
+                throw Error('Warnings were found');
             }
-            if (errorMessages.length > 0) {
-                throw new Error(`Found ${errorMessages.join(' and ')} in ${relative('.', result.filePath)}`);
+            if (hasErrors) {
+                throw Error('Errors were found');
             }
-            return null;
         }
@@ -56,0 +63,0 @@ };

package.json

 {
   "name": "@rollup/plugin-eslint",
-  "version": "8.0.3",
+  "version": "8.0.4",
   "publishConfig": {
@@ -16,11 +16,6 @@ "access": "public"
   "bugs": "https://github.com/rollup/plugins/issues",
-  "main": "./dist/cjs/index.js",
-  "module": "./dist/es/index.js",
-  "exports": {
-    "import": "./dist/es/index.js",
-    "types": "./types/index.d.ts",
-    "default": "./dist/cjs/index.js"
-  },
+  "main": "dist/index.js",
+  "module": "dist/index.es.js",
   "engines": {
-    "node": ">=14.0.0"
+    "node": ">= 10.0.0"
   },
@@ -43,3 +38,2 @@ "scripts": {
     "dist",
-    "!dist/**/*.map",
     "types",
@@ -59,22 +53,26 @@ "README.md",
   "peerDependencies": {
-    "rollup": "^1.20.0||^2.0.0||^3.0.0"
+    "rollup": "^1.20.0||^2.0.0"
   },
-  "peerDependenciesMeta": {
-    "rollup": {
-      "optional": true
-    }
-  },
   "dependencies": {
-    "@rollup/pluginutils": "^4.2.1",
-    "eslint": "^8.24.0"
+    "@rollup/pluginutils": "^4.0.0",
+    "eslint": "^7.12.0"
   },
   "devDependencies": {
-    "@rollup/plugin-node-resolve": "^14.1.0",
-    "@rollup/plugin-typescript": "^8.5.0",
-    "@types/eslint": "^8.4.6",
-    "rollup": "^3.0.0-7",
-    "typescript": "^4.8.3"
+    "@rollup/plugin-node-resolve": "^9.0.0",
+    "@rollup/plugin-typescript": "^6.0.0",
+    "@types/eslint": "^7.2.2",
+    "rollup": "^2.67.3",
+    "typescript": "^4.1.2"
   },
-  "types": "./types/index.d.ts",
+  "types": "types/index.d.ts",
   "ava": {
+    "babel": {
+      "compileEnhancements": false
+    },
+    "extensions": [
+      "ts"
+    ],
+    "require": [
+      "ts-node/register"
+    ],
     "files": [
@@ -81,0 +79,0 @@ "!**/fixtures/**",

README.md

@@ -1,5 +0,5 @@
-[npm]: https://img.shields.io/npm/v/@rollup/plugin-eslint
-[npm-url]: https://www.npmjs.com/package/@rollup/plugin-eslint
-[size]: https://packagephobia.now.sh/badge?p=@rollup/plugin-eslint
-[size-url]: https://packagephobia.now.sh/result?p=@rollup/plugin-eslint
+[npm]: https://img.shields.io/npm/v/@rollup/plugin-alias
+[npm-url]: https://www.npmjs.com/package/@rollup/plugin-alias
+[size]: https://packagephobia.now.sh/badge?p=@rollup/plugin-alias
+[size-url]: https://packagephobia.now.sh/result?p=@rollup/plugin-alias
 
@@ -14,6 +14,2 @@ [![npm][npm]][npm-url]
 
-## Requirements
-
-This plugin requires an [LTS](https://github.com/nodejs/Release) Node version (v14.0.0+) and Rollup v1.20.0+.
-
 ## Install
@@ -46,14 +42,14 @@
 
-This plugin takes a configuration object intended for the [ESLint constructor](https://eslint.org/docs/developer-guide/nodejs-api#-new-eslintoptions) with the addition of a `throwOnWarning`, `throwOnError`, `formatter`, `include` and `exclude` prop.
+See more options here [eslint-config](http://eslint.org/docs/developer-guide/nodejs-api#cliengine).
 
 You can also use eslint configuration in the form of a `.eslintrc.*` file in your project's root. It will be loaded automatically.
 
-### exclude
+### fix
 
-Type: `String | String[]`<br>
-Default: `node_modules/**`
+Type: `Boolean`<br>
+Default: `false`
 
-A single [`picomatch`](https://github.com/micromatch/picomatch) pattern or an array of patterns controlling which files this plugin should explicitly include. Gets forwarded to the [`createFilter`](https://github.com/rollup/plugins/tree/master/packages/pluginutils#createfilter) method of `@rollup/pluginutils`.
+If true, will auto fix source code.
 
-### fix
+### throwOnError
 
@@ -63,31 +59,31 @@ Type: `Boolean`<br>
 
-If true, will auto fix source code.
+If true, will throw an error if any errors were found.
 
-### formatter
+### throwOnWarning
 
-Type: `Function | String`<br>
-Default: `stylish`
+Type: `Boolean`<br>
+Default: `false`
 
-Custom error formatter, the name of a built-in formatter, or the path to a custom formatter.
+If true, will throw an error if any warnings were found.
 
 ### include
 
-Type: `String | String[]`<br>
+Type: `Array | String`<br>
 Default: `[]`
 
-A single [`picomatch`](https://github.com/micromatch/picomatch) pattern or an array of patterns controlling which files this plugin should explicitly include. Gets forwarded to the [`createFilter`](https://github.com/rollup/plugins/tree/master/packages/pluginutils#createfilter) method of `@rollup/pluginutils`.
+A single file, or array of files, to include when linting.
 
-### throwOnError
+### exclude
 
-Type: `Boolean`<br>
-Default: `false`
+Type: `Array | String`<br>
+Default: `node_modules/**`
 
-If true, will throw an error and exit the process when ESLint reports any errors.
+A single file, or array of files, to exclude when linting.
 
-### throwOnWarning
+### formatter
 
-Type: `Boolean`<br>
-Default: `false`
+Type: `Function | String`<br>
+Default: `stylish`
 
-If true, will throw an error and exit the process when ESLint reports any warnings.
+Custom error formatter or the name of a built-in formatter.
 
@@ -94,0 +90,0 @@ ## Meta

types/index.d.ts

 import { Plugin } from 'rollup';
-import { CLIEngine, ESLint } from 'eslint';
+import { CLIEngine } from 'eslint';
 import { CreateFilter } from '@rollup/pluginutils';
 
-export interface RollupEslintOptions extends ESLint.Options {
+export interface RollupEslintOptions extends CLIEngine.Options {
   /**
@@ -7,0 +7,0 @@ * If true, will auto fix source code.

New alerts

New author

Supply chain risk

A new npm collaborator published a version of the package for the first time. New collaborators are usually benign additions to a project, but do indicate a change to the security surface area of a package.

Found 1 instance in 1 package

Fixed alerts

Major refactor

Supply chain risk

Package has recently undergone a major refactor. It may be unstable or indicate significant internal changes. Use caution when updating to versions that include significant changes.

Found 1 instance in 1 package

Improved metrics

Total package byte prevSize

14794

increased by31.95%

Number of package files

8

increased by33.33%

Lines of code

187

increased by19.87%

Worsened metrics

Number of lines in readme file

92

decreased by-4.17%

Dependency changes

• + Added@babel/code-frame@7.12.11(transitive)

• + Added@babel/helper-validator-identifier@7.24.7(transitive)

• + Added@babel/highlight@7.24.7(transitive)

• + Added@eslint/eslintrc@0.4.3(transitive)

• + Added@humanwhocodes/config-array@0.5.0(transitive)

• + Added@humanwhocodes/object-schema@1.2.1(transitive)

• + Addedacorn@7.4.1(transitive)

• + Addedajv@8.16.0(transitive)

• + Addedansi-colors@4.1.3(transitive)

• + Addedansi-styles@3.2.1(transitive)

• + Addedargparse@1.0.10(transitive)

• + Addedastral-regex@2.0.0(transitive)

• + Addedchalk@2.4.2(transitive)

• + Addedcolor-convert@1.9.3(transitive)

• + Addedcolor-name@1.1.3(transitive)

• + Addedemoji-regex@8.0.0(transitive)

• + Addedenquirer@2.4.1(transitive)

• + Addedescape-string-regexp@1.0.5(transitive)

• + Addedeslint@7.32.0(transitive)

• + Addedeslint-scope@5.1.1(transitive)

• + Addedeslint-utils@2.1.0(transitive)

• + Addedeslint-visitor-keys@1.3.02.1.0(transitive)

• + Addedespree@7.3.1(transitive)

• + Addedesprima@4.0.1(transitive)

• + Addedestraverse@4.3.0(transitive)

• + Addedfunctional-red-black-tree@1.0.1(transitive)

• + Addedglob-parent@5.1.2(transitive)

• + Addedhas-flag@3.0.0(transitive)

• + Addedignore@4.0.6(transitive)

• + Addedis-fullwidth-code-point@3.0.0(transitive)

• + Addedjs-tokens@4.0.0(transitive)

• + Addedjs-yaml@3.14.1(transitive)

• + Addedjson-schema-traverse@1.0.0(transitive)

• + Addedlodash.truncate@4.4.2(transitive)

• + Addedpicocolors@1.0.1(transitive)

• + Addedprogress@2.0.3(transitive)

• + Addedregexpp@3.2.0(transitive)

• + Addedrequire-from-string@2.0.2(transitive)

• + Addedrollup@2.79.1(transitive)

• + Addedsemver@7.6.2(transitive)

• + Addedslice-ansi@4.0.0(transitive)

• + Addedsprintf-js@1.0.3(transitive)

• + Addedstring-width@4.2.3(transitive)

• + Addedsupports-color@5.5.0(transitive)

• + Addedtable@6.8.2(transitive)

• + Addedv8-compile-cache@2.4.0(transitive)

• - Removed@eslint-community/eslint-utils@4.4.0(transitive)

• - Removed@eslint-community/regexpp@4.11.0(transitive)

• - Removed@eslint/eslintrc@2.1.4(transitive)

• - Removed@eslint/js@8.57.0(transitive)

• - Removed@humanwhocodes/config-array@0.11.14(transitive)

• - Removed@humanwhocodes/module-importer@1.0.1(transitive)

• - Removed@humanwhocodes/object-schema@2.0.3(transitive)

• - Removed@nodelib/fs.scandir@2.1.5(transitive)

• - Removed@nodelib/fs.stat@2.0.5(transitive)

• - Removed@nodelib/fs.walk@1.2.8(transitive)

• - Removed@ungap/structured-clone@1.2.0(transitive)

• - Removedacorn@8.12.1(transitive)

• - Removedargparse@2.0.1(transitive)

• - Removedeslint@8.57.0(transitive)

• - Removedeslint-scope@7.2.2(transitive)

• - Removedeslint-visitor-keys@3.4.3(transitive)

• - Removedespree@9.6.1(transitive)

• - Removedfastq@1.17.1(transitive)

• - Removedfind-up@5.0.0(transitive)

• - Removedglob-parent@6.0.2(transitive)

• - Removedgraphemer@1.4.0(transitive)

• - Removedignore@5.3.1(transitive)

• - Removedis-path-inside@3.0.3(transitive)

• - Removedjs-yaml@4.1.0(transitive)

• - Removedlocate-path@6.0.0(transitive)

• - Removedp-limit@3.1.0(transitive)

• - Removedp-locate@5.0.0(transitive)

• - Removedpath-exists@4.0.0(transitive)

• - Removedqueue-microtask@1.2.3(transitive)

• - Removedreusify@1.0.4(transitive)

• - Removedrollup@3.29.4(transitive)

• - Removedrun-parallel@1.2.0(transitive)

• - Removedyocto-queue@0.1.0(transitive)

• Updated@rollup/pluginutils@^4.0.0

• Updatedeslint@^7.12.0