eslint-plugin-prettier 3.3.0 → 4.0.0

package/CHANGELOG.md

@@ -1,5 +1,32 @@
 # Changelog
 
+## v4.0.0 (2021-08-30)
+
+This breaking change drops support for old versions of ESLint, Prettier and
+Node. You must use at least ESLint v7.28.0, Prettier v2.0.0 and Node v12.0.0.
+Aside from that, usage of this plugin remains identical.
+
+* v4 - Drop support for eslint 5/6, prettier 1, node 6/8 ([#429](git@github.com:prettier/eslint-plugin-prettier/issues/429)) ([acb56f3](git@github.com:prettier/eslint-plugin-prettier/commit/acb56f3b2891b2a6998a75a7d4406183d452ba16))
+
+## v3.4.1 (2021-08-20)
+
+* build(deps): Bump glob-parent from 5.0.0 to 5.1.2 ([#420](git@github.com:prettier/eslint-plugin-prettier/issues/420)) ([b6d075c](git@github.com:prettier/eslint-plugin-prettier/commit/b6d075cf7111468e8af4161c306c7f37f09f220e))
+* build(deps): Bump path-parse from 1.0.6 to 1.0.7 ([#425](git@github.com:prettier/eslint-plugin-prettier/issues/425)) ([24f957e](git@github.com:prettier/eslint-plugin-prettier/commit/24f957ee2a5476bb9cc8e64921b9841fc751391e))
+* feat: support `@graphql-eslint/eslint-plugin` out of box ([#413](git@github.com:prettier/eslint-plugin-prettier/issues/413)) ([ec6fbb1](git@github.com:prettier/eslint-plugin-prettier/commit/ec6fbb159e2454c6e145db55480932dc953cf7c1))
+* chore: add tests for Node 16 ([#410](git@github.com:prettier/eslint-plugin-prettier/issues/410)) ([76bd45e](git@github.com:prettier/eslint-plugin-prettier/commit/76bd45ece6d56eb52f75db6b4a1efdd2efb56392))
+
+## v3.4.0 (2021-04-15)
+
+* feat: support processor virtual filename ([#401](git@github.com:prettier/eslint-plugin-prettier/issues/401)) ([ee0ccc6](git@github.com:prettier/eslint-plugin-prettier/commit/ee0ccc6ac06d13cd546e78b444e53164f59eb27f))
+* Simplify report logic ([#380](git@github.com:prettier/eslint-plugin-prettier/issues/380)) ([d993f24](git@github.com:prettier/eslint-plugin-prettier/commit/d993f247b5661683af031ab3b93955a0dfe448fa))
+* Update: README.md ([#375](git@github.com:prettier/eslint-plugin-prettier/issues/375)) ([3ea4242](git@github.com:prettier/eslint-plugin-prettier/commit/3ea4242a8d4acdb76eb7e7dca9e44d3e87db70e3))
+
+## v3.3.1 (2021-01-04)
+
+* fix: add eslint-config-prettier as an optional peer dependency ([#374](git@github.com:prettier/eslint-plugin-prettier/issues/374)) ([d59df27](git@github.com:prettier/eslint-plugin-prettier/commit/d59df27890aaffec9e528ceb3155831a0261848d))
+* build(deps-dev): bump eslint from 7.16.0 to 7.17.0 ([b87985d](git@github.com:prettier/eslint-plugin-prettier/commit/b87985d8b1986743374b56691bcc1633df8f4eae))
+* build(deps-dev): bump eslint from 7.15.0 to 7.16.0 ([11e427e](git@github.com:prettier/eslint-plugin-prettier/commit/11e427e5d6cedeb26e3e03c8143be3496a24955a))
+
 ## v3.3.0 (2020-12-13)
 
 * Minor: Perf improvement: Do not clear the config cache on each run ([#368](git@github.com:prettier/eslint-plugin-prettier/issues/368)) ([1b90ea7](git@github.com:prettier/eslint-plugin-prettier/commit/1b90ea752636959babb27ebca5d67093c346dab9))

package/README.md

@@ -4,6 +4,8 @@ Runs [Prettier](https://github.com/prettier/prettier) as an [ESLint](http://esli
 
 If your desired formatting does not match Prettier’s output, you should use a different tool such as [prettier-eslint](https://github.com/prettier/prettier-eslint) instead.
 
+Please read [Integrating with linters](https://prettier.io/docs/en/integrating-with-linters.html) before installing.
+
 ## Sample
 
 ```js
@@ -74,20 +76,6 @@ This plugin ships with a `plugin:prettier/recommended` config that sets up both
 
    You can then set Prettier's own options inside a `.prettierrc` file.
 
-3. Some ESLint plugins (such as [eslint-plugin-react](https://github.com/yannickcr/eslint-plugin-react)) also contain rules that conflict with Prettier. Add extra exclusions for the plugins you use like so:
-
-   ```json
-   {
-     "extends": [
-       "plugin:prettier/recommended",
-       "prettier/flowtype",
-       "prettier/react"
-     ]
-   }
-   ```
-
-   For the list of every available exclusion rule set, please see the [readme of eslint-config-prettier](https://github.com/prettier/eslint-config-prettier/blob/master/README.md).
-
 Exactly what does `plugin:prettier/recommended` do? Well, this is what it expands to:
 
 ```json
@@ -102,7 +90,7 @@ Exactly what does `plugin:prettier/recommended` do? Well, this is what it expand
 }
 ```
 
-- `"extends": ["prettier"]` enables the main config from `eslint-config-prettier`, which turns off some ESLint core rules that conflict with Prettier.
+- `"extends": ["prettier"]` enables the config from `eslint-config-prettier`, which turns off some ESLint rules that conflict with Prettier.
 - `"plugins": ["prettier"]` registers this plugin.
 - `"prettier/prettier": "error"` turns on the rule provided by this plugin, which runs Prettier from within ESLint.
 - `"arrow-body-style": "off"` and `"prefer-arrow-callback": "off"` turns off two ESLint core rules that unfortunately are problematic with this plugin – see the next section.

package/eslint-plugin-prettier.js

@@ -11,7 +11,7 @@
 
 const {
   showInvisibles,
-  generateDifferences
+  generateDifferences,
 } = require('prettier-linter-helpers');
 
 // ------------------------------------------------------------------------------
@@ -25,6 +25,9 @@ const { INSERT, DELETE, REPLACE } = generateDifferences;
 // ------------------------------------------------------------------------------
 
 // Lazily-loaded Prettier.
+/**
+ * @type {import('prettier')}
+ */
 let prettier;
 
 // ------------------------------------------------------------------------------
@@ -32,71 +35,26 @@ let prettier;
 // ------------------------------------------------------------------------------
 
 /**
- * Reports an "Insert ..." issue where text must be inserted.
- * @param {RuleContext} context - The ESLint rule context.
- * @param {number} offset - The source offset where to insert text.
- * @param {string} text - The text to be inserted.
- * @returns {void}
- */
-function reportInsert(context, offset, text) {
-  const pos = context.getSourceCode().getLocFromIndex(offset);
-  const range = [offset, offset];
-  context.report({
-    message: 'Insert `{{ code }}`',
-    data: { code: showInvisibles(text) },
-    loc: { start: pos, end: pos },
-    fix(fixer) {
-      return fixer.insertTextAfterRange(range, text);
-    }
-  });
-}
-
-/**
- * Reports a "Delete ..." issue where text must be deleted.
- * @param {RuleContext} context - The ESLint rule context.
- * @param {number} offset - The source offset where to delete text.
- * @param {string} text - The text to be deleted.
+ * Reports a difference.
+ * @param {import('eslint').Rule.RuleContext} context - The ESLint rule context.
+ * @param {import('prettier-linter-helpers').Difference} difference - The difference object.
  * @returns {void}
  */
-function reportDelete(context, offset, text) {
-  const start = context.getSourceCode().getLocFromIndex(offset);
-  const end = context.getSourceCode().getLocFromIndex(offset + text.length);
-  const range = [offset, offset + text.length];
-  context.report({
-    message: 'Delete `{{ code }}`',
-    data: { code: showInvisibles(text) },
-    loc: { start, end },
-    fix(fixer) {
-      return fixer.removeRange(range);
-    }
-  });
-}
-
-/**
- * Reports a "Replace ... with ..." issue where text must be replaced.
- * @param {RuleContext} context - The ESLint rule context.
- * @param {number} offset - The source offset where to replace deleted text
- with inserted text.
- * @param {string} deleteText - The text to be deleted.
- * @param {string} insertText - The text to be inserted.
- * @returns {void}
- */
-function reportReplace(context, offset, deleteText, insertText) {
-  const start = context.getSourceCode().getLocFromIndex(offset);
-  const end = context
-    .getSourceCode()
-    .getLocFromIndex(offset + deleteText.length);
+function reportDifference(context, difference) {
+  const { operation, offset, deleteText = '', insertText = '' } = difference;
   const range = [offset, offset + deleteText.length];
+  const [start, end] = range.map((index) =>
+    context.getSourceCode().getLocFromIndex(index)
+  );
+
   context.report({
-    message: 'Replace `{{ deleteCode }}` with `{{ insertCode }}`',
+    messageId: operation,
     data: {
-      deleteCode: showInvisibles(deleteText),
-      insertCode: showInvisibles(insertText)
+      deleteText: showInvisibles(deleteText),
+      insertText: showInvisibles(insertText),
     },
     loc: { start, end },
-    fix(fixer) {
-      return fixer.replaceTextRange(range, insertText);
-    }
+    fix: (fixer) => fixer.replaceTextRange(range, insertText),
   });
 }
 
@@ -112,15 +70,15 @@ module.exports = {
       rules: {
         'prettier/prettier': 'error',
         'arrow-body-style': 'off',
-        'prefer-arrow-callback': 'off'
-      }
-    }
+        'prefer-arrow-callback': 'off',
+      },
+    },
   },
   rules: {
     prettier: {
       meta: {
         docs: {
-          url: 'https://github.com/prettier/eslint-plugin-prettier#options'
+          url: 'https://github.com/prettier/eslint-plugin-prettier#options',
         },
         type: 'layout',
         fixable: 'code',
@@ -129,7 +87,7 @@ module.exports = {
           {
             type: 'object',
             properties: {},
-            additionalProperties: true
+            additionalProperties: true,
           },
           {
             type: 'object',
@@ -138,12 +96,17 @@ module.exports = {
               fileInfoOptions: {
                 type: 'object',
                 properties: {},
-                additionalProperties: true
-              }
+                additionalProperties: true,
+              },
             },
-            additionalProperties: true
-          }
-        ]
+            additionalProperties: true,
+          },
+        ],
+        messages: {
+          [INSERT]: 'Insert `{{ insertText }}`',
+          [DELETE]: 'Delete `{{ deleteText }}`',
+          [REPLACE]: 'Replace `{{ deleteText }}` with `{{ insertText }}`',
+        },
       },
       create(context) {
         const usePrettierrc =
@@ -152,6 +115,12 @@ module.exports = {
           (context.options[1] && context.options[1].fileInfoOptions) || {};
         const sourceCode = context.getSourceCode();
         const filepath = context.getFilename();
+        // Processors that extract content from a file, such as the markdown
+        // plugin extracting fenced code blocks may choose to specify virtual
+        // file paths. If this is the case then we need to resolve prettier
+        // config and file info using the on-disk path instead of the virtual
+        // path.
+        const onDiskFilepath = context.getPhysicalFilename();
         const source = sourceCode.text;
 
         return {
@@ -164,13 +133,13 @@ module.exports = {
             const eslintPrettierOptions = context.options[0] || {};
 
             const prettierRcOptions = usePrettierrc
-              ? prettier.resolveConfig.sync(filepath, {
-                  editorconfig: true
+              ? prettier.resolveConfig.sync(onDiskFilepath, {
+                  editorconfig: true,
                 })
               : null;
 
-            const prettierFileInfo = prettier.getFileInfo.sync(
-              filepath,
+            const { ignored, inferredParser } = prettier.getFileInfo.sync(
+              onDiskFilepath,
               Object.assign(
                 {},
                 { resolveConfig: true, ignorePath: '.prettierignore' },
@@ -179,13 +148,13 @@ module.exports = {
             );
 
             // Skip if file is ignored using a .prettierignore file
-            if (prettierFileInfo.ignored) {
+            if (ignored) {
               return;
             }
 
             const initialOptions = {};
 
-            // ESLint suppports processors that let you extract and lint JS
+            // ESLint supports processors that let you extract and lint JS
             // fragments within a non-JS language. In the cases where prettier
             // supports the same language as a processor, we want to process
             // the provided source code as javascript (as ESLint provides the
@@ -193,9 +162,14 @@ module.exports = {
             // based off the filename. Otherwise, for instance, on a .md file we
             // end up trying to run prettier over a fragment of JS using the
             // markdown parser, which throws an error.
-            // If we can't infer the parser from from the filename, either
-            // because no filename was provided or because there is no parser
-            // found for the filename, use javascript.
+            // Processors may set virtual filenames for these extracted blocks.
+            // If they do so then we want to trust the file extension they
+            // provide, and no override is needed.
+            // If the processor does not set any virtual filename (signified by
+            // `filepath` and `onDiskFilepath` being equal) AND we can't
+            // infer the parser from the filename, either because no filename
+            // was provided or because there is no parser found for the
+            // filename, use javascript.
             // This is added to the options first, so that
             // prettierRcOptions and eslintPrettierOptions can still override
             // the parser.
@@ -205,17 +179,22 @@ module.exports = {
             // * Prettier supports parsing the file type
             // * There is an ESLint processor that extracts JavaScript snippets
             //   from the file type.
-            const parserBlocklist = [null, 'graphql', 'markdown', 'html'];
+            const parserBlocklist = [null, 'markdown', 'html'];
+
+            let inferParserToBabel =
+              parserBlocklist.indexOf(inferredParser) !== -1;
+
             if (
-              parserBlocklist.indexOf(prettierFileInfo.inferredParser) !== -1
+              // it could be processed by `@graphql-eslint/eslint-plugin` or `eslint-plugin-graphql`
+              inferredParser === 'graphql' &&
+              // for `eslint-plugin-graphql`, see https://github.com/apollographql/eslint-plugin-graphql/blob/master/src/index.js#L416
+              source.startsWith('ESLintPluginGraphQLFile`')
             ) {
-              // Prettier v1.16.0 renamed the `babylon` parser to `babel`
-              // Use the modern name if available
-              const supportBabelParser = prettier
-                .getSupportInfo()
-                .languages.some(language => language.parsers.includes('babel'));
+              inferParserToBabel = true;
+            }
 
-              initialOptions.parser = supportBabelParser ? 'babel' : 'babylon';
+            if (filepath === onDiskFilepath && inferParserToBabel) {
+              initialOptions.parser = 'babel';
             }
 
             const prettierOptions = Object.assign(
@@ -227,7 +206,7 @@ module.exports = {
             );
 
             // prettier.format() may throw a SyntaxError if it cannot parse the
-            // source code it is given. Ususally for JS files this isn't a
+            // source code it is given. Usually for JS files this isn't a
             // problem as ESLint will report invalid syntax before trying to
             // pass it to the prettier plugin. However this might be a problem
             // for non-JS languages that are handled by a plugin. Notably Vue
@@ -245,7 +224,7 @@ module.exports = {
               let message = 'Parsing error: ' + err.message;
 
               // Prettier's message contains a codeframe style preview of the
-              // invalid code and the line/column at which the error occured.
+              // invalid code and the line/column at which the error occurred.
               // ESLint shows those pieces of information elsewhere already so
               // remove them from the message
               if (err.codeFrame) {
@@ -263,36 +242,13 @@ module.exports = {
             if (source !== prettierSource) {
               const differences = generateDifferences(source, prettierSource);
 
-              differences.forEach(difference => {
-                switch (difference.operation) {
-                  case INSERT:
-                    reportInsert(
-                      context,
-                      difference.offset,
-                      difference.insertText
-                    );
-                    break;
-                  case DELETE:
-                    reportDelete(
-                      context,
-                      difference.offset,
-                      difference.deleteText
-                    );
-                    break;
-                  case REPLACE:
-                    reportReplace(
-                      context,
-                      difference.offset,
-                      difference.deleteText,
-                      difference.insertText
-                    );
-                    break;
-                }
-              });
+              for (const difference of differences) {
+                reportDifference(context, difference);
+              }
             }
-          }
+          },
         };
-      }
-    }
-  }
+      },
+    },
+  },
 };

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "eslint-plugin-prettier",
-  "version": "3.3.0",
+  "version": "4.0.0",
   "description": "Runs prettier as an eslint rule",
   "keywords": [
     "eslint",
@@ -31,23 +31,25 @@
     "prettier-linter-helpers": "^1.0.0"
   },
   "peerDependencies": {
-    "eslint": ">=5.0.0",
-    "prettier": ">=1.13.0"
+    "eslint": ">=7.28.0",
+    "prettier": ">=2.0.0"
   },
   "devDependencies": {
+    "@graphql-eslint/eslint-plugin": "^2.0.1",
     "@not-an-aardvark/node-release-script": "^0.1.0",
-    "eslint": "^7.0.0",
+    "eslint": "^7.28.0",
     "eslint-config-not-an-aardvark": "^2.1.0",
     "eslint-config-prettier": "^6.0.0",
     "eslint-plugin-eslint-plugin": "^2.0.0",
     "eslint-plugin-node": "^8.0.0",
     "eslint-plugin-self": "^1.1.0",
+    "graphql": "^15.5.1",
     "mocha": "^6.0.0",
-    "prettier": "^1.15.3",
+    "prettier": "^2.3.0",
     "vue-eslint-parser": "^6.0.0"
   },
   "peerDependenciesMeta": {
-    "eslint-plugin-prettier": {
+    "eslint-config-prettier": {
       "optional": true
     }
   },