Revert "feat: Add allowVariableNamePattern to valid-define-options"

This reverts commit d523648.

Author: FrontEndDog

docs/rules/valid-define-options.md

@@ -108,18 +108,7 @@ defineOptions({ props: { msg: String } })
 
 ## :wrench: Options
 
-```json
-{
-  "vue/valid-define-options": [
-    "error",
-    {
-      "allowVariableNamePattern": "^[A-Z_]+$"
-    }
-  ]
-}
-```
-
-- `allowVariableNamePattern` Allow reference to declared variables if variable name matches patterns, default `null`.
+Nothing.
 
 ## :couple: Related Rules
 

lib/rules/valid-define-options.js

@@ -16,17 +16,7 @@ module.exports = {
       url: 'https://eslint.vuejs.org/rules/valid-define-options.html'
     },
     fixable: null,
-    schema: [
-      {
-        type: 'object',
-        properties: {
-          allowVariableNamePattern: {
-            type: 'string'
-          }
-        },
-        additionalProperties: false
-      }
-    ],
+    schema: [],
     messages: {
       referencingLocally:
         '`defineOptions` is referencing locally declared variables.',
@@ -39,13 +29,6 @@ module.exports = {
   },
   /** @param {RuleContext} context */
   create(context) {
-    const option = context.options[0] || {}
-    const allowVariableNamePattern = option.allowVariableNamePattern
-    /** @type {RegExp | null} */
-    let allowVariableNamePatternRegEx = null
-    if (allowVariableNamePattern) {
-      allowVariableNamePatternRegEx = new RegExp(allowVariableNamePattern, 'u')
-    }
     const scriptSetup = utils.getScriptSetupElement(context)
     if (!scriptSetup) {
       return {}
@@ -110,9 +93,7 @@ module.exports = {
                   (def) =>
                     def.type !== 'ImportBinding' &&
                     utils.inRange(scriptSetup.range, def.name) &&
-                    !utils.inRange(defineOptions.range, def.name) &&
-                    (allowVariableNamePatternRegEx === null ||
-                      !allowVariableNamePatternRegEx.test(variable.name))
+                    !utils.inRange(defineOptions.range, def.name)
                 )
               ) {
                 if (utils.withinTypeNode(node)) {

tests/lib/rules/valid-define-options.js

@@ -74,15 +74,6 @@ tester.run('valid-define-options', rule, {
 
       defineOptions(def);
       </script>`
-    },
-    {
-      filename: 'test.vue',
-      code: `
-      <script setup>
-        const COMPONENT_NAME = 'TestComponent'
-        defineOptions({ name: COMPONENT_NAME })
-      </script>`,
-      options: [{ allowVariableNamePattern: '^[A-Z_]+$' }]
     }
   ],
   invalid: [
@@ -214,21 +205,6 @@ tester.run('valid-define-options', rule, {
           line: 3
         }
       ]
-    },
-    {
-      filename: 'test.vue',
-      code: `
-      <script setup>
-        const componentName = "TestComponent"
-        defineOptions({ name: componentName });
-      </script>`,
-      options: [{ allowVariableNamePattern: '^[A-Z_]+$' }],
-      errors: [
-        {
-          message: '`defineOptions` is referencing locally declared variables.',
-          line: 4
-        }
-      ]
     }
   ]
 })