⭐️New: Add vue/require-direct-export rule

docs/rules/require-direct-export.md

@@ -0,0 +1,35 @@
+# require the component to be directly exported (require-direct-export)
+
+- :gear: This rule is included in `"plugin:vue/essentials"`.
+
+## Rule Details
+
+This rule aims to...
+
+:-1: Examples of **incorrect** code:
+
+```js
+const ComponentA = {
+	name: 'ComponentA',
+	data() {
+		return {
+			state: 1
+		}
+	}
+}
+
+export default Component A
+```
+
+:+1: Examples of **correct** code:
+
+```js
+export default {
+	name: 'ComponentA',
+	data() {
+		return {
+			state: 1
+		}
+	}
+}
+```

lib/configs/essential.js

@@ -20,6 +20,7 @@ module.exports = {
     'vue/no-use-v-if-with-v-for': 'error',
     'vue/require-component-is': 'error',
     'vue/require-prop-type-constructor': 'error',
+    'vue/require-direct-export': 'error',
     'vue/require-render-return': 'error',
     'vue/require-v-for-key': 'error',
     'vue/require-valid-default-prop': 'error',

lib/index.js

@@ -43,6 +43,7 @@ module.exports = {
     'prop-name-casing': require('./rules/prop-name-casing'),
     'require-component-is': require('./rules/require-component-is'),
     'require-default-prop': require('./rules/require-default-prop'),
+    'require-direct-export': require('./rules/require-direct-export'),
     'require-prop-type-constructor': require('./rules/require-prop-type-constructor'),
     'require-prop-types': require('./rules/require-prop-types'),
     'require-render-return': require('./rules/require-render-return'),

lib/rules/require-direct-export.js

@@ -0,0 +1,47 @@
+/**
+ * @fileoverview require the component to be directly exported
+ * @author Hiroki Osame <[email protected]>
+ */
+'use strict'
+
+const utils = require('../utils')
+
+// ------------------------------------------------------------------------------
+// Rule Definition
+// ------------------------------------------------------------------------------
+
+module.exports = {
+  meta: {
+    docs: {
+      description: 'require the component to be directly exported',
+      category: 'essential',
+      recommended: false,
+      url: 'https://github.com/vuejs/eslint-plugin-vue/blob/v5.0.0-beta.3/docs/rules/require-direct-export.md'
+    },
+    fixable: null,  // or "code" or "whitespace"
+    schema: []
+  },
+
+  create (context) {
+    const filePath = context.getFilename()
+
+    return {
+      'ExportDefaultDeclaration:exit' (node) {
+        const isVueFile = utils.isVueFile(filePath)
+        if (!isVueFile) { return }
+
+        const isObjectExpression = (
+          node.type === 'ExportDefaultDeclaration' &&
+                    node.declaration.type === 'ObjectExpression'
+        )
+
+        if (!isObjectExpression) {
+          context.report({
+            node,
+            message: `Expected the component literal to be directly exported.`
+          })
+        }
+      }
+    }
+  }
+}

lib/utils/index.js

@@ -404,6 +404,10 @@ module.exports = {
       })
   },
 
+  isVueFile (path) {
+    return path.endsWith('.vue') || path.endsWith('.jsx')
+  },
+
   /**
    * Check whether the given node is a Vue component based
    * on the filename and default export type
@@ -413,8 +417,7 @@ module.exports = {
    * @returns {boolean}
    */
   isVueComponentFile (node, path) {
-    const isVueFile = path.endsWith('.vue') || path.endsWith('.jsx')
-    return isVueFile &&
+    return this.isVueFile(path) &&
       node.type === 'ExportDefaultDeclaration' &&
       node.declaration.type === 'ObjectExpression'
   },

tests/lib/rules/require-direct-export.js

@@ -0,0 +1,56 @@
+/**
+ * @fileoverview require the component to be directly exported
+ * @author Hiroki Osame <[email protected]>
+ */
+'use strict'
+
+// ------------------------------------------------------------------------------
+// Requirements
+// ------------------------------------------------------------------------------
+
+const rule = require('../../../lib/rules/require-direct-export')
+const RuleTester = require('eslint').RuleTester
+
+const parserOptions = {
+  ecmaVersion: 2018,
+  sourceType: 'module',
+  ecmaFeatures: { jsx: true }
+}
+
+// ------------------------------------------------------------------------------
+// Tests
+// ------------------------------------------------------------------------------
+
+const ruleTester = new RuleTester()
+ruleTester.run('require-direct-export', rule, {
+
+  valid: [
+    {
+      filename: 'test.vue',
+      code: ''
+    },
+    {
+      filename: 'test.vue',
+      code: `
+            export default {}
+          `,
+      parserOptions
+    }
+  ],
+
+  invalid: [
+
+    {
+      filename: 'test.vue',
+      code: `
+          const A = {};
+          export default A`,
+      parserOptions,
+      errors: [{
+        message: 'Expected the component literal to be directly exported.',
+        type: 'ExportDefaultDeclaration',
+        line: 3
+      }]
+    }
+  ]
+})