Add support --fix-type

1) Add the type field in rule's meta
2) Fix the fixable field

Author: Vlad Esin

lib/rules/boolean-prop-naming.js

@@ -21,6 +21,7 @@ const messages = {
 
 module.exports = {
   meta: {
+    type: 'suggestion',
     docs: {
       category: 'Stylistic Issues',
       description: 'Enforces consistent naming for boolean props',

lib/rules/button-has-type.js

@@ -30,6 +30,7 @@ const messages = {
 
 module.exports = {
   meta: {
+    type: 'problem',
     docs: {
       description: 'Forbid "button" element without an explicit "type" attribute',
       category: 'Possible Errors',

lib/rules/default-props-match-prop-types.js

@@ -21,6 +21,7 @@ const messages = {
 
 module.exports = {
   meta: {
+    type: 'suggestion',
     docs: {
       description: 'Enforce all defaultProps are defined and not "required" in propTypes.',
       category: 'Best Practices',

lib/rules/destructuring-assignment.js

@@ -54,6 +54,7 @@ const messages = {
 
 module.exports = {
   meta: {
+    type: 'suggestion',
     docs: {
       description: 'Enforce consistent usage of destructuring assignment of props, state, and context',
       category: 'Stylistic Issues',

lib/rules/display-name.js

@@ -23,6 +23,7 @@ const messages = {
 
 module.exports = {
   meta: {
+    type: 'suggestion',
     docs: {
       description: 'Prevent missing displayName in a React component definition',
       category: 'Best Practices',

lib/rules/forbid-component-props.js

@@ -24,6 +24,7 @@ const messages = {
 
 module.exports = {
   meta: {
+    type: 'suggestion',
     docs: {
       description: 'Forbid certain props on components',
       category: 'Best Practices',

lib/rules/forbid-dom-props.js

@@ -24,6 +24,7 @@ const messages = {
 
 module.exports = {
   meta: {
+    type: 'suggestion',
     docs: {
       description: 'Forbid certain props on DOM Nodes',
       category: 'Best Practices',

lib/rules/forbid-elements.js

@@ -21,6 +21,7 @@ const messages = {
 
 module.exports = {
   meta: {
+    type: 'suggestion',
     docs: {
       description: 'Forbid certain elements',
       category: 'Best Practices',

lib/rules/forbid-foreign-prop-types.js

@@ -15,6 +15,7 @@ const messages = {
 
 module.exports = {
   meta: {
+    type: 'suggestion',
     docs: {
       description: 'Forbid using another component\'s propTypes',
       category: 'Best Practices',

lib/rules/forbid-prop-types.js

@@ -27,6 +27,7 @@ const messages = {
 
 module.exports = {
   meta: {
+    type: 'suggestion',
     docs: {
       description: 'Forbid certain propTypes',
       category: 'Best Practices',

lib/rules/function-component-definition.js

@@ -100,6 +100,7 @@ const messages = {
 
 module.exports = {
   meta: {
+    type: 'suggestion',
     docs: {
       description: 'Standardize the way function component get defined',
       category: 'Stylistic Issues',

lib/rules/jsx-boolean-value.js

@@ -56,6 +56,7 @@ const messages = {
 
 module.exports = {
   meta: {
+    type: 'suggestion',
     docs: {
       description: 'Enforce boolean attributes notation in JSX',
       category: 'Stylistic Issues',

lib/rules/jsx-child-element-spacing.js

@@ -46,13 +46,14 @@ const messages = {
 
 module.exports = {
   meta: {
+    type: 'layout',
     docs: {
       description: 'Ensures inline tags are not rendered without spaces between them',
       category: 'Stylistic Issues',
       recommended: false,
       url: docsUrl('jsx-child-element-spacing'),
     },
-    fixable: null,
+    // fixable: null,
 
     messages,
 

lib/rules/jsx-closing-bracket-location.js

@@ -19,13 +19,14 @@ const messages = {
 
 module.exports = {
   meta: {
+    type: 'layout',
     docs: {
       description: 'Validate closing bracket location in JSX',
       category: 'Stylistic Issues',
       recommended: false,
       url: docsUrl('jsx-closing-bracket-location'),
     },
-    fixable: 'code',
+    fixable: 'whitespace',
 
     messages,
 

lib/rules/jsx-closing-tag-location.js

@@ -20,6 +20,7 @@ const messages = {
 
 module.exports = {
   meta: {
+    type: 'layout',
     docs: {
       description: 'Validate closing tag location for multiline JSX',
       category: 'Stylistic Issues',

lib/rules/jsx-curly-brace-presence.js

@@ -38,6 +38,7 @@ const messages = {
 
 module.exports = {
   meta: {
+    type: 'suggestion',
     docs: {
       description:
         'Disallow unnecessary JSX expressions when literals alone are sufficient '

lib/rules/jsx-curly-newline.js

@@ -44,16 +44,13 @@ const messages = {
 module.exports = {
   meta: {
     type: 'layout',
-
     docs: {
       description: 'Enforce consistent line breaks inside jsx curly',
       category: 'Stylistic Issues',
       recommended: false,
       url: docsUrl('jsx-curly-newline'),
     },
-
     fixable: 'whitespace',
-
     schema: [
       {
         oneOf: [

lib/rules/jsx-curly-spacing.js

@@ -36,13 +36,14 @@ const messages = {
 
 module.exports = {
   meta: {
+    type: 'layout',
     docs: {
       description: 'Enforce or disallow spaces inside of curly braces in JSX attributes',
       category: 'Stylistic Issues',
       recommended: false,
       url: docsUrl('jsx-curly-spacing'),
     },
-    fixable: 'code',
+    fixable: 'whitespace',
 
     messages,
 

lib/rules/jsx-equals-spacing.js

@@ -21,13 +21,14 @@ const messages = {
 
 module.exports = {
   meta: {
+    type: 'layout',
     docs: {
       description: 'Disallow or enforce spaces around equal signs in JSX attributes',
       category: 'Stylistic Issues',
       recommended: false,
       url: docsUrl('jsx-equals-spacing'),
     },
-    fixable: 'code',
+    fixable: 'whitespace',
 
     messages,
 

lib/rules/jsx-filename-extension.js

@@ -29,6 +29,7 @@ const messages = {
 
 module.exports = {
   meta: {
+    type: 'suggestion',
     docs: {
       description: 'Restrict file extensions that may contain JSX',
       category: 'Stylistic Issues',

lib/rules/jsx-first-prop-new-line.js

@@ -19,13 +19,14 @@ const messages = {
 
 module.exports = {
   meta: {
+    type: 'layout',
     docs: {
       description: 'Ensure proper position of the first property in JSX',
       category: 'Stylistic Issues',
       recommended: false,
       url: docsUrl('jsx-first-prop-new-line'),
     },
-    fixable: 'code',
+    fixable: 'whitespace',
 
     messages,
 

lib/rules/jsx-fragments.js

@@ -29,6 +29,7 @@ const messages = {
 
 module.exports = {
   meta: {
+    type: 'suggestion',
     docs: {
       description: 'Enforce shorthand or standard form for React fragments',
       category: 'Stylistic Issues',

lib/rules/jsx-handler-names.js

@@ -19,6 +19,7 @@ const messages = {
 
 module.exports = {
   meta: {
+    type: 'suggestion',
     docs: {
       description: 'Enforce event handler naming conventions in JSX',
       category: 'Stylistic Issues',

lib/rules/jsx-indent-props.js

@@ -44,13 +44,14 @@ const messages = {
 
 module.exports = {
   meta: {
+    type: 'layout',
     docs: {
       description: 'Validate props indentation in JSX',
       category: 'Stylistic Issues',
       recommended: false,
       url: docsUrl('jsx-indent-props'),
     },
-    fixable: 'code',
+    fixable: 'whitespace',
 
     messages,
 

lib/rules/jsx-indent.js

@@ -46,6 +46,7 @@ const messages = {
 
 module.exports = {
   meta: {
+    type: 'layout',
     docs: {
       description: 'Validate JSX indentation',
       category: 'Stylistic Issues',

lib/rules/jsx-key.js

@@ -30,6 +30,7 @@ const messages = {
 
 module.exports = {
   meta: {
+    type: 'problem',
     docs: {
       description: 'Report missing `key` props in iterators/collection literals',
       category: 'Possible Errors',

lib/rules/jsx-max-depth.js

@@ -22,6 +22,7 @@ const messages = {
 
 module.exports = {
   meta: {
+    type: 'suggestion',
     docs: {
       description: 'Validate JSX maximum depth',
       category: 'Stylistic Issues',

lib/rules/jsx-max-props-per-line.js

@@ -25,13 +25,14 @@ const messages = {
 
 module.exports = {
   meta: {
+    type: 'layout',
     docs: {
       description: 'Limit maximum of props on a single line in JSX',
       category: 'Stylistic Issues',
       recommended: false,
       url: docsUrl('jsx-max-props-per-line'),
     },
-    fixable: 'code',
+    fixable: 'whitespace',
 
     messages,
 

lib/rules/jsx-no-bind.js

@@ -26,6 +26,7 @@ const messages = {
 
 module.exports = {
   meta: {
+    type: 'suggestion',
     docs: {
       description: 'Prevents usage of Function.prototype.bind and arrow functions in React component props',
       category: 'Best Practices',

lib/rules/jsx-no-comment-textnodes.js

@@ -35,6 +35,7 @@ function checkText(node, context) {
 
 module.exports = {
   meta: {
+    type: 'suggestion',
     docs: {
       description: 'Comments inside children section of tag should be placed inside braces',
       category: 'Possible Errors',

lib/rules/jsx-no-duplicate-props.js

@@ -19,6 +19,7 @@ const messages = {
 
 module.exports = {
   meta: {
+    type: 'problem',
     docs: {
       description: 'Enforce no duplicate props',
       category: 'Possible Errors',

lib/rules/jsx-no-literals.js

@@ -26,6 +26,7 @@ const messages = {
 
 module.exports = {
   meta: {
+    type: 'suggestion',
     docs: {
       description: 'Prevent using string literals in React component definition',
       category: 'Stylistic Issues',

lib/rules/jsx-no-script-url.js

@@ -49,6 +49,7 @@ const messages = {
 
 module.exports = {
   meta: {
+    type: 'suggestion',
     docs: {
       description: 'Forbid `javascript:` URLs',
       category: 'Best Practices',

lib/rules/jsx-no-target-blank.js

@@ -103,6 +103,7 @@ const messages = {
 
 module.exports = {
   meta: {
+    type: 'suggestion',
     fixable: 'code',
     docs: {
       description: 'Forbid `target="_blank"` attribute without `rel="noreferrer"`',

lib/rules/jsx-no-undef.js

@@ -19,6 +19,7 @@ const messages = {
 
 module.exports = {
   meta: {
+    type: 'problem',
     docs: {
       description: 'Disallow undeclared variables in JSX',
       category: 'Possible Errors',

lib/rules/jsx-one-expression-per-line.js

@@ -23,6 +23,7 @@ const messages = {
 
 module.exports = {
   meta: {
+    type: 'layout',
     docs: {
       description: 'Limit to one expression per line in JSX',
       category: 'Stylistic Issues',

lib/rules/jsx-pascal-case.js

@@ -78,6 +78,7 @@ const messages = {
 
 module.exports = {
   meta: {
+    type: 'suggestion',
     docs: {
       description: 'Enforce PascalCase for user-defined JSX components',
       category: 'Stylistic Issues',

lib/rules/jsx-props-no-multi-spaces.js

@@ -19,13 +19,14 @@ const messages = {
 
 module.exports = {
   meta: {
+    type: 'layout',
     docs: {
       description: 'Disallow multiple spaces between inline JSX props',
       category: 'Stylistic Issues',
       recommended: false,
       url: docsUrl('jsx-props-no-multi-spaces'),
     },
-    fixable: 'code',
+    fixable: 'whitespace',
 
     messages,
 

lib/rules/jsx-props-no-spreading.js

@@ -40,6 +40,7 @@ const messages = {
 
 module.exports = {
   meta: {
+    type: 'suggestion',
     docs: {
       description: 'Prevent JSX prop spreading',
       category: 'Best Practices',

lib/rules/jsx-sort-default-props.js

@@ -21,6 +21,7 @@ const messages = {
 
 module.exports = {
   meta: {
+    type: 'suggestion',
     docs: {
       description: 'Enforce default props alphabetical sorting',
       category: 'Stylistic Issues',