From 7907db1e1643a01e488a8f83ebf08118ced68aec Mon Sep 17 00:00:00 2001
From: ota-meshi <otameshiyo23@gmail.com>
Date: Wed, 10 Jan 2024 14:40:35 +0900
Subject: [PATCH] Add support for defineModel

---
 docs/rules/define-macros-order.md             | 13 ++--
 docs/rules/no-unsupported-features.md         |  1 +
 lib/rules/define-macros-order.js              | 12 ++-
 lib/rules/no-undef-properties.js              | 20 +++++
 lib/rules/no-unsupported-features.js          |  3 +
 lib/rules/no-unused-emit-declarations.js      | 11 +++
 lib/rules/no-unused-properties.js             | 35 ++++++++-
 lib/rules/require-prop-types.js               | 57 +++++++++-----
 lib/rules/syntaxes/define-model.js            | 22 ++++++
 lib/utils/index.js                            | 53 +++++++++++++
 tests/lib/rules/define-macros-order.js        | 34 ++++++++-
 tests/lib/rules/no-undef-properties.js        | 27 +++++++
 .../no-unsupported-features/define-model.js   | 59 +++++++++++++++
 .../lib/rules/no-unused-emit-declarations.js  | 26 +++++++
 tests/lib/rules/no-unused-properties.js       | 35 +++++++++
 tests/lib/rules/require-prop-types.js         | 74 +++++++++++++++++++
 typings/eslint-plugin-vue/util-types/utils.ts | 13 ++++
 17 files changed, 465 insertions(+), 30 deletions(-)
 create mode 100644 lib/rules/syntaxes/define-model.js
 create mode 100644 tests/lib/rules/no-unsupported-features/define-model.js

diff --git a/docs/rules/define-macros-order.md b/docs/rules/define-macros-order.md
index d46fc5108..7df8f42c7 100644
--- a/docs/rules/define-macros-order.md
+++ b/docs/rules/define-macros-order.md
@@ -27,7 +27,7 @@ This rule reports the `defineProps` and `defineEmits` compiler macros when they
 }
 ```
 
-- `order` (`string[]`) ... The order of defineEmits and defineProps macros. You can also add `"defineOptions"` and `"defineSlots"`.
+- `order` (`string[]`) ... The order of defineEmits and defineProps macros. You can also add `"defineOptions"`, `"defineSlots"`, and `"defineModel"`.
 - `defineExposeLast` (`boolean`) ... Force `defineExpose` at the end.
 
 ### `{ "order": ["defineProps", "defineEmits"] }` (default)
@@ -69,14 +69,15 @@ defineEmits(/* ... */)
 
 </eslint-code-block>
 
-### `{ "order": ["defineOptions", "defineProps", "defineEmits", "defineSlots"] }`
+### `{ "order": ["defineOptions", "defineModel", "defineProps", "defineEmits", "defineSlots"] }`
 
-<eslint-code-block fix :rules="{'vue/define-macros-order': ['error', {order: ['defineOptions', 'defineProps', 'defineEmits', 'defineSlots']}]}">
+<eslint-code-block fix :rules="{'vue/define-macros-order': ['error', {order: ['defineOptions', 'defineModel', 'defineProps', 'defineEmits', 'defineSlots']}]}">
 
 ```vue
 <!-- ✓ GOOD -->
 <script setup>
 defineOptions({/* ... */})
+const model = defineModel()
 defineProps(/* ... */)
 defineEmits(/* ... */)
 const slots = defineSlots()
@@ -85,7 +86,7 @@ const slots = defineSlots()
 
 </eslint-code-block>
 
-<eslint-code-block fix :rules="{'vue/define-macros-order': ['error', {order: ['defineOptions', 'defineProps', 'defineEmits', 'defineSlots']}]}">
+<eslint-code-block fix :rules="{'vue/define-macros-order': ['error', {order: ['defineOptions', 'defineModel', 'defineProps', 'defineEmits', 'defineSlots']}]}">
 
 ```vue
 <!-- ✗ BAD -->
@@ -94,18 +95,20 @@ defineEmits(/* ... */)
 const slots = defineSlots()
 defineProps(/* ... */)
 defineOptions({/* ... */})
+const model = defineModel()
 </script>
 ```
 
 </eslint-code-block>
 
-<eslint-code-block fix :rules="{'vue/define-macros-order': ['error', {order: ['defineOptions', 'defineProps', 'defineEmits', 'defineSlots']}]}">
+<eslint-code-block fix :rules="{'vue/define-macros-order': ['error', {order: ['defineOptions', 'defineModel', 'defineProps', 'defineEmits', 'defineSlots']}]}">
 
 ```vue
 <!-- ✗ BAD -->
 <script setup>
 const bar = ref()
 defineOptions({/* ... */})
+const model = defineModel()
 defineProps(/* ... */)
 defineEmits(/* ... */)
 const slots = defineSlots()
diff --git a/docs/rules/no-unsupported-features.md b/docs/rules/no-unsupported-features.md
index 393a30476..2f6356517 100644
--- a/docs/rules/no-unsupported-features.md
+++ b/docs/rules/no-unsupported-features.md
@@ -30,6 +30,7 @@ This rule reports unsupported Vue.js syntax on the specified version.
 - `ignores` ... You can use this `ignores` option to ignore the given features.
 The `"ignores"` option accepts an array of the following strings.
   - Vue.js 3.4.0+
+    - `"define-model"` ... `defineModel()` macro.
     - `"v-bind-same-name-shorthand"` ... `v-bind` same-name shorthand.
   - Vue.js 3.3.0+
     - `"define-slots"` ... `defineSlots()` macro.
diff --git a/lib/rules/define-macros-order.js b/lib/rules/define-macros-order.js
index b55c5e8c7..7c915ca80 100644
--- a/lib/rules/define-macros-order.js
+++ b/lib/rules/define-macros-order.js
@@ -10,7 +10,14 @@ const MACROS_EMITS = 'defineEmits'
 const MACROS_PROPS = 'defineProps'
 const MACROS_OPTIONS = 'defineOptions'
 const MACROS_SLOTS = 'defineSlots'
-const ORDER_SCHEMA = [MACROS_EMITS, MACROS_PROPS, MACROS_OPTIONS, MACROS_SLOTS]
+const MACROS_MODEL = 'defineModel'
+const ORDER_SCHEMA = [
+  MACROS_EMITS,
+  MACROS_PROPS,
+  MACROS_OPTIONS,
+  MACROS_SLOTS,
+  MACROS_MODEL
+]
 const DEFAULT_ORDER = [MACROS_PROPS, MACROS_EMITS]
 
 /**
@@ -116,6 +123,9 @@ function create(context) {
       onDefineSlotsExit(node) {
         macrosNodes.set(MACROS_SLOTS, getDefineMacrosStatement(node))
       },
+      onDefineModelExit(node) {
+        macrosNodes.set(MACROS_MODEL, getDefineMacrosStatement(node))
+      },
       onDefineExposeExit(node) {
         defineExposeNode = getDefineMacrosStatement(node)
       }
diff --git a/lib/rules/no-undef-properties.js b/lib/rules/no-undef-properties.js
index d7026eb62..09b45594c 100644
--- a/lib/rules/no-undef-properties.js
+++ b/lib/rules/no-undef-properties.js
@@ -321,6 +321,26 @@ module.exports = {
           const propertyReferences =
             propertyReferenceExtractor.extractFromPattern(pattern)
           ctx.verifyReferences(propertyReferences)
+        },
+        onDefineModelEnter(node, model) {
+          const ctx = getVueComponentContext(programNode)
+
+          ctx.defineProperties.set(model.name.modelName, {
+            isProps: true
+          })
+
+          if (
+            !node.parent ||
+            node.parent.type !== 'VariableDeclarator' ||
+            node.parent.init !== node
+          ) {
+            return
+          }
+
+          const pattern = node.parent.id
+          const propertyReferences =
+            propertyReferenceExtractor.extractFromPattern(pattern)
+          ctx.verifyReferences(propertyReferences)
         }
       }),
       utils.defineVueVisitor(context, {
diff --git a/lib/rules/no-unsupported-features.js b/lib/rules/no-unsupported-features.js
index e40d55ebe..d5ff8b48a 100644
--- a/lib/rules/no-unsupported-features.js
+++ b/lib/rules/no-unsupported-features.js
@@ -37,6 +37,7 @@ const FEATURES = {
   'define-options': require('./syntaxes/define-options'),
   'define-slots': require('./syntaxes/define-slots'),
   // Vue.js 3.4.0+
+  'define-model': require('./syntaxes/define-model'),
   'v-bind-same-name-shorthand': require('./syntaxes/v-bind-same-name-shorthand')
 }
 
@@ -128,6 +129,8 @@ module.exports = {
       forbiddenDefineSlots:
         '`defineSlots()` macros are not supported until Vue.js "3.3.0".',
       // Vue.js 3.4.0+
+      forbiddenDefineModel:
+        '`defineModel()` macros are not supported until Vue.js "3.4.0".',
       forbiddenVBindSameNameShorthand:
         '`v-bind` same-name shorthand is not supported until Vue.js "3.4.0".'
     }
diff --git a/lib/rules/no-unused-emit-declarations.js b/lib/rules/no-unused-emit-declarations.js
index ee43248cb..5704775de 100644
--- a/lib/rules/no-unused-emit-declarations.js
+++ b/lib/rules/no-unused-emit-declarations.js
@@ -332,6 +332,17 @@ module.exports = {
             emitReferenceIds
           })
         },
+        onDefineModelEnter(node, model) {
+          if (
+            node.parent &&
+            node.parent.type === 'VariableDeclarator' &&
+            node.parent.init === node
+          ) {
+            // If store the return of defineModel() in a variable, we can mark the 'update:modelName' event as used if use that variable.
+            // If that variable is unused it will already be reported by `no-unused-var` rule.
+            emitCalls.set(`update:${model.name.modelName}`, node)
+          }
+        },
         ...callVisitor
       }),
       {
diff --git a/lib/rules/no-unused-properties.js b/lib/rules/no-unused-properties.js
index b64c42d36..0a5c1fe6d 100644
--- a/lib/rules/no-unused-properties.js
+++ b/lib/rules/no-unused-properties.js
@@ -30,7 +30,7 @@ const {
  * @typedef {object} ComponentNonObjectPropertyData
  * @property {string} name
  * @property {GroupName} groupName
- * @property {'array' | 'type' | 'infer-type'} type
+ * @property {'array' | 'type' | 'infer-type' | 'model' } type
  * @property {ASTNode} node
  *
  * @typedef { ComponentNonObjectPropertyData | ComponentObjectPropertyData } ComponentPropertyData
@@ -406,7 +406,9 @@ module.exports = {
           if (!groups.has('props')) {
             return
           }
-          const container = getVueComponentPropertiesContainer(node)
+          const container = getVueComponentPropertiesContainer(
+            context.getSourceCode().ast
+          )
 
           for (const prop of props) {
             if (!prop.propName) {
@@ -452,6 +454,35 @@ module.exports = {
           const propertyReferences =
             propertyReferenceExtractor.extractFromPattern(pattern)
           container.propertyReferencesForProps.push(propertyReferences)
+        },
+        onDefineModelEnter(node, model) {
+          if (!groups.has('props')) {
+            return
+          }
+          const container = getVueComponentPropertiesContainer(
+            context.getSourceCode().ast
+          )
+          if (
+            node.parent &&
+            node.parent.type === 'VariableDeclarator' &&
+            node.parent.init === node
+          ) {
+            // If store the return of defineModel() in a variable, we can mark the model prop as used if use that variable.
+            // If that variable is unused it will already be reported by `no-unused-var` rule.
+            container.propertyReferences.push(
+              propertyReferenceExtractor.extractFromName(
+                model.name.modelName,
+                model.name.node || node
+              )
+            )
+            return
+          }
+          container.properties.push({
+            type: 'model',
+            name: model.name.modelName,
+            groupName: 'props',
+            node: model.name.node || node
+          })
         }
       }),
       utils.defineVueVisitor(context, {
diff --git a/lib/rules/require-prop-types.js b/lib/rules/require-prop-types.js
index 326095e93..9a394084a 100644
--- a/lib/rules/require-prop-types.js
+++ b/lib/rules/require-prop-types.js
@@ -26,6 +26,29 @@ module.exports = {
   },
   /** @param {RuleContext} context */
   create(context) {
+    /**
+     * @param {Expression} node
+     * @returns {boolean|null}
+     */
+    function optionHasType(node) {
+      switch (node.type) {
+        case 'ObjectExpression': {
+          // foo: {
+          return objectHasType(node)
+        }
+        case 'ArrayExpression': {
+          // foo: [
+          return node.elements.length > 0
+        }
+        case 'FunctionExpression':
+        case 'ArrowFunctionExpression': {
+          return false
+        }
+      }
+
+      // Unknown
+      return null
+    }
     /**
      * @param {ObjectExpression} node
      * @returns {boolean}
@@ -44,7 +67,6 @@ module.exports = {
       )
       return Boolean(typeProperty || validatorProperty)
     }
-
     /**
      * @param {ComponentProp} prop
      */
@@ -52,29 +74,13 @@ module.exports = {
       if (prop.type !== 'object' && prop.type !== 'array') {
         return
       }
-      let hasType = true
+      let hasType
 
       if (prop.type === 'array') {
         hasType = false
       } else {
         const { value } = prop
-        switch (value.type) {
-          case 'ObjectExpression': {
-            // foo: {
-            hasType = objectHasType(value)
-            break
-          }
-          case 'ArrayExpression': {
-            // foo: [
-            hasType = value.elements.length > 0
-            break
-          }
-          case 'FunctionExpression':
-          case 'ArrowFunctionExpression': {
-            hasType = false
-            break
-          }
-        }
+        hasType = optionHasType(value) ?? true
       }
 
       if (!hasType) {
@@ -99,6 +105,19 @@ module.exports = {
           for (const prop of props) {
             checkProperty(prop)
           }
+        },
+        onDefineModelEnter(node, model) {
+          if (model.typeNode) return
+          if (model.options && (optionHasType(model.options) ?? true)) {
+            return
+          }
+          context.report({
+            node: model.options || node,
+            messageId: 'requireType',
+            data: {
+              name: model.name.modelName
+            }
+          })
         }
       }),
       utils.executeOnVue(context, (obj) => {
diff --git a/lib/rules/syntaxes/define-model.js b/lib/rules/syntaxes/define-model.js
new file mode 100644
index 000000000..188aae52e
--- /dev/null
+++ b/lib/rules/syntaxes/define-model.js
@@ -0,0 +1,22 @@
+/**
+ * @author Yosuke Ota
+ * See LICENSE file in root directory for full license.
+ */
+'use strict'
+
+const utils = require('../../utils/index')
+
+module.exports = {
+  supported: '>=3.4.0',
+  /** @param {RuleContext} context @returns {RuleListener} */
+  createScriptVisitor(context) {
+    return utils.defineScriptSetupVisitor(context, {
+      onDefineModelEnter(node) {
+        context.report({
+          node,
+          messageId: 'forbiddenDefineModel'
+        })
+      }
+    })
+  }
+}
diff --git a/lib/utils/index.js b/lib/utils/index.js
index 0cd842e13..4fec4e1fa 100644
--- a/lib/utils/index.js
+++ b/lib/utils/index.js
@@ -26,6 +26,8 @@ const { getScope } = require('./scope')
  * @typedef {import('../../typings/eslint-plugin-vue/util-types/utils').ComponentInferTypeEmit} ComponentInferTypeEmit
  * @typedef {import('../../typings/eslint-plugin-vue/util-types/utils').ComponentUnknownEmit} ComponentUnknownEmit
  * @typedef {import('../../typings/eslint-plugin-vue/util-types/utils').ComponentEmit} ComponentEmit
+ * @typedef {import('../../typings/eslint-plugin-vue/util-types/utils').ComponentModelName} ComponentModelName
+ * @typedef {import('../../typings/eslint-plugin-vue/util-types/utils').ComponentModel} ComponentModel
  */
 /**
  * @typedef { {key: string | null, value: BlockStatement | null} } ComponentComputedProperty
@@ -1318,6 +1320,8 @@ module.exports = {
    * - `onDefineSlotsExit` ... Event when defineSlots visit ends.
    * - `onDefineExposeEnter` ... Event when defineExpose is found.
    * - `onDefineExposeExit` ... Event when defineExpose visit ends.
+   * - `onDefineModelEnter` ... Event when defineModel is found.
+   * - `onDefineModelExit` ... Event when defineModel visit ends.
    *
    * @param {RuleContext} context The ESLint rule context object.
    * @param {ScriptSetupVisitor} visitor The visitor to traverse the AST nodes.
@@ -1412,6 +1416,13 @@ module.exports = {
         'onDefineExposeExit',
         (candidateMacro, node) => candidateMacro === node,
         () => undefined
+      ),
+      new MacroListener(
+        'defineModel',
+        'onDefineModelEnter',
+        'onDefineModelExit',
+        (candidateMacro, node) => candidateMacro === node,
+        getComponentModelFromDefineModel
       )
     ].filter((m) => m.hasListener)
     if (macroListenerList.length > 0) {
@@ -3225,6 +3236,48 @@ function getComponentEmitsFromDefineEmits(context, node) {
     }
   ]
 }
+/**
+ * Get model info from `defineModel` call expression.
+ * @param {RuleContext} _context The rule context object.
+ * @param {CallExpression} node `defineModel` call expression
+ * @return {ComponentModel} Object of component model
+ */
+function getComponentModelFromDefineModel(_context, node) {
+  /** @type {ComponentModelName} */
+  let name = {
+    modelName: 'modelValue',
+    node: null
+  }
+  /** @type {Expression|null} */
+  let options =
+    node.arguments[0]?.type === 'SpreadElement' ? null : node.arguments[0]
+  if (node.arguments.length > 0) {
+    const nameNodeCandidate = skipTSAsExpression(node.arguments[0])
+    if (nameNodeCandidate.type === 'Literal') {
+      name = {
+        modelName: String(nameNodeCandidate.value),
+        node: nameNodeCandidate
+      }
+      options =
+        node.arguments[1]?.type === 'SpreadElement' ? null : node.arguments[1]
+    }
+  }
+
+  const typeArguments =
+    'typeArguments' in node ? node.typeArguments : node.typeParameters
+  if (typeArguments && typeArguments.params.length > 0) {
+    return {
+      name,
+      options,
+      typeNode: typeArguments.params[0]
+    }
+  }
+  return {
+    name,
+    options,
+    typeNode: null
+  }
+}
 /**
  * Get all props by looking at all component's properties
  * @param {ObjectExpression|ArrayExpression} propsNode Object with props definition
diff --git a/tests/lib/rules/define-macros-order.js b/tests/lib/rules/define-macros-order.js
index c58fd516f..878c0ea93 100644
--- a/tests/lib/rules/define-macros-order.js
+++ b/tests/lib/rules/define-macros-order.js
@@ -168,6 +168,8 @@ tester.run('define-macros-order', rule, {
           import Foo from 'foo'
           /** options */
           defineOptions({})
+          /** model */
+          const model = defineModel()
           /** emits */
           defineEmits(['update:foo'])
           /** props */
@@ -179,7 +181,13 @@ tester.run('define-macros-order', rule, {
       `,
       options: [
         {
-          order: ['defineOptions', 'defineEmits', 'defineProps', 'defineSlots']
+          order: [
+            'defineOptions',
+            'defineModel',
+            'defineEmits',
+            'defineProps',
+            'defineSlots'
+          ]
         }
       ]
     },
@@ -611,6 +619,8 @@ tester.run('define-macros-order', rule, {
           const slots = defineSlots()
           /** options */
           defineOptions({})
+          /** model */
+          const model = defineModel()
         </script>
       `,
       output: `
@@ -618,6 +628,8 @@ tester.run('define-macros-order', rule, {
           import Foo from 'foo'
           /** options */
           defineOptions({})
+          /** model */
+          const model = defineModel()
           /** emits */
           defineEmits(['update:foo'])
           /** props */
@@ -629,7 +641,13 @@ tester.run('define-macros-order', rule, {
       `,
       options: [
         {
-          order: ['defineOptions', 'defineEmits', 'defineProps', 'defineSlots']
+          order: [
+            'defineOptions',
+            'defineModel',
+            'defineEmits',
+            'defineProps',
+            'defineSlots'
+          ]
         }
       ],
       errors: [
@@ -651,12 +669,16 @@ tester.run('define-macros-order', rule, {
           defineEmits(['update:foo'])
           /** props */
           const props = defineProps(['foo'])
+          /** model */
+          const model = defineModel()
         </script>
       `,
       output: `
         <script setup>
           /** options */
           defineOptions({})
+          /** model */
+          const model = defineModel()
           /** emits */
           defineEmits(['update:foo'])
           /** props */
@@ -667,7 +689,13 @@ tester.run('define-macros-order', rule, {
       `,
       options: [
         {
-          order: ['defineOptions', 'defineEmits', 'defineProps', 'defineSlots']
+          order: [
+            'defineOptions',
+            'defineModel',
+            'defineEmits',
+            'defineProps',
+            'defineSlots'
+          ]
         }
       ],
       errors: [
diff --git a/tests/lib/rules/no-undef-properties.js b/tests/lib/rules/no-undef-properties.js
index 7ba344197..66b928b76 100644
--- a/tests/lib/rules/no-undef-properties.js
+++ b/tests/lib/rules/no-undef-properties.js
@@ -1176,6 +1176,33 @@ tester.run('no-undef-properties', rule, {
         }
       ],
       ...getTypeScriptFixtureTestOptions()
+    },
+
+    {
+      // defineModel
+      filename: 'test.vue',
+      code: `
+      <script setup>
+      const m = defineModel()
+      const a = defineModel('A')
+      const [withModifiers, modifiers] = defineModel('withModifiers')
+      </script>
+      <template>
+        <div>
+          {{ m }} {{ modelValue }}
+          {{ a }} {{ A }}
+          {{withModifiers}} {{modifiers}}
+        </div>
+
+        <div>{{ undef }}</div>
+      </template>
+      `,
+      errors: [
+        {
+          message: "'undef' is not defined.",
+          line: 14
+        }
+      ]
     }
   ]
 })
diff --git a/tests/lib/rules/no-unsupported-features/define-model.js b/tests/lib/rules/no-unsupported-features/define-model.js
new file mode 100644
index 000000000..0307d1f9f
--- /dev/null
+++ b/tests/lib/rules/no-unsupported-features/define-model.js
@@ -0,0 +1,59 @@
+/**
+ * @author Yosuke Ota
+ * See LICENSE file in root directory for full license.
+ */
+'use strict'
+
+const RuleTester = require('eslint').RuleTester
+const rule = require('../../../../lib/rules/no-unsupported-features')
+const utils = require('./utils')
+
+const buildOptions = utils.optionsBuilder('define-model', '^3.3.0')
+const tester = new RuleTester({
+  parser: require.resolve('vue-eslint-parser'),
+  parserOptions: {
+    ecmaVersion: 2019
+  }
+})
+
+tester.run('no-unsupported-features/define-model', rule, {
+  valid: [
+    {
+      code: `
+      <script setup>
+        const model = defineModel()
+      </script>`,
+      options: buildOptions({ version: '^3.4.0' })
+    },
+    {
+      code: `
+      <script setup>
+        defineSlots({})
+      </script>`,
+      options: buildOptions()
+    },
+    {
+      code: `
+      <script setup>
+        const model = defineModel()
+      </script>`,
+      options: buildOptions({ version: '^3.3.0', ignores: ['define-model'] })
+    }
+  ],
+  invalid: [
+    {
+      code: `
+      <script setup>
+        const model = defineModel()
+      </script>`,
+      options: buildOptions(),
+      errors: [
+        {
+          message:
+            '`defineModel()` macros are not supported until Vue.js "3.4.0".',
+          line: 3
+        }
+      ]
+    }
+  ]
+})
diff --git a/tests/lib/rules/no-unused-emit-declarations.js b/tests/lib/rules/no-unused-emit-declarations.js
index 10bf3b33a..f3189ea82 100644
--- a/tests/lib/rules/no-unused-emit-declarations.js
+++ b/tests/lib/rules/no-unused-emit-declarations.js
@@ -357,6 +357,16 @@ tester.run('no-unused-emit-declarations', rule, {
       const change = () => emit('foo');
       `,
       ...getTypeScriptFixtureTestOptions()
+    },
+    {
+      // defineModel
+      filename: 'test.vue',
+      code: `
+      <script setup>
+      defineEmits({'update:foo'() {}})
+      const m = defineModel('foo')
+      </script>
+      `
     }
   ],
   invalid: [
@@ -718,6 +728,22 @@ tester.run('no-unused-emit-declarations', rule, {
         }
       ],
       ...getTypeScriptFixtureTestOptions()
+    },
+    {
+      // defineModel
+      filename: 'test.vue',
+      code: `
+      <script setup>
+      defineEmits({'update:foo'() {}})
+      defineModel('foo')
+      </script>
+      `,
+      errors: [
+        {
+          message: '`update:foo` is defined as emit but never used.',
+          line: 3
+        }
+      ]
     }
   ]
 })
diff --git a/tests/lib/rules/no-unused-properties.js b/tests/lib/rules/no-unused-properties.js
index f71a26519..e47f346aa 100644
--- a/tests/lib/rules/no-unused-properties.js
+++ b/tests/lib/rules/no-unused-properties.js
@@ -2152,6 +2152,19 @@ tester.run('no-unused-properties', rule, {
         },
       };
       </script>`
+    },
+    {
+      // defineModel
+      filename: 'test.vue',
+      code: `
+      <script setup>
+      defineProps(['a'])
+      const foo = defineModel('a')
+      </script>
+      <template>
+        {{ foo }}
+      </template>
+      `
     }
   ],
   invalid: [
@@ -3811,6 +3824,28 @@ tester.run('no-unused-properties', rule, {
         }
       ],
       ...getTypeScriptFixtureTestOptions()
+    },
+
+    {
+      // defineModel
+      filename: 'test.vue',
+      code: `
+      <script setup>
+      const m = defineModel()
+      console.log(m.value)
+      defineModel('a')
+      defineModel('unused')
+      </script>
+      <template>
+        {{ a }}
+      </template>
+      `,
+      errors: [
+        {
+          message: "'unused' of property found, but never used.",
+          line: 6
+        }
+      ]
     }
   ]
 })
diff --git a/tests/lib/rules/require-prop-types.js b/tests/lib/rules/require-prop-types.js
index fdb91dd30..f50b4b116 100644
--- a/tests/lib/rules/require-prop-types.js
+++ b/tests/lib/rules/require-prop-types.js
@@ -187,6 +187,40 @@ ruleTester.run('require-prop-types', rule, {
       defineProps<Props>()
       </script>`,
       ...getTypeScriptFixtureTestOptions()
+    },
+    {
+      // defineModel
+      code: `
+      <script setup>
+      const m = defineModel({type:String})
+      const foo = defineModel('foo', {type:String})
+      </script>
+      `,
+      parser: require.resolve('vue-eslint-parser')
+    },
+    {
+      // defineModel
+      code: `
+      <script setup>
+      const m = defineModel(String)
+      const foo = defineModel('foo', String)
+      </script>
+      `,
+      parser: require.resolve('vue-eslint-parser')
+    },
+    {
+      code: `
+      <script setup lang="ts">
+      const m = defineModel<string>()
+      const foo = defineModel<string>('foo')
+      </script>
+      `,
+      parser: require.resolve('vue-eslint-parser'),
+      parserOptions: {
+        ecmaVersion: 6,
+        sourceType: 'module',
+        parser: require.resolve('@typescript-eslint/parser')
+      }
     }
   ],
 
@@ -368,6 +402,46 @@ ruleTester.run('require-prop-types', rule, {
           line: 3
         }
       ]
+    },
+    {
+      // defineModel
+      code: `
+      <script setup>
+      const m = defineModel()
+      const foo = defineModel('foo')
+      </script>
+      `,
+      parser: require.resolve('vue-eslint-parser'),
+      errors: [
+        {
+          message: 'Prop "modelValue" should define at least its type.',
+          line: 3
+        },
+        {
+          message: 'Prop "foo" should define at least its type.',
+          line: 4
+        }
+      ]
+    },
+    {
+      // defineModel
+      code: `
+      <script setup>
+      const m = defineModel({})
+      const foo = defineModel('foo',{})
+      </script>
+      `,
+      parser: require.resolve('vue-eslint-parser'),
+      errors: [
+        {
+          message: 'Prop "modelValue" should define at least its type.',
+          line: 3
+        },
+        {
+          message: 'Prop "foo" should define at least its type.',
+          line: 4
+        }
+      ]
     }
   ]
 })
diff --git a/typings/eslint-plugin-vue/util-types/utils.ts b/typings/eslint-plugin-vue/util-types/utils.ts
index 4d8384d66..3e9184262 100644
--- a/typings/eslint-plugin-vue/util-types/utils.ts
+++ b/typings/eslint-plugin-vue/util-types/utils.ts
@@ -46,10 +46,13 @@ export interface ScriptSetupVisitor extends ScriptSetupVisitorBase {
   onDefineSlotsExit?(node: CallExpression): void
   onDefineExposeEnter?(node: CallExpression): void
   onDefineExposeExit?(node: CallExpression): void
+  onDefineModelEnter?(node: CallExpression, model: ComponentModel): void
+  onDefineModelExit?(node: CallExpression, model: ComponentModel): void
   [query: string]:
     | ((node: VAST.ParamNode) => void)
     | ((node: CallExpression, props: ComponentProp[]) => void)
     | ((node: CallExpression, emits: ComponentEmit[]) => void)
+    | ((node: CallExpression, model: ComponentModel) => void)
     | undefined
 }
 
@@ -187,3 +190,13 @@ export type ComponentEmit =
   | ComponentTypeEmit
   | ComponentInferTypeEmit
   | ComponentUnknownEmit
+
+export type ComponentModelName = {
+  modelName: string
+  node: Literal | null
+}
+export type ComponentModel = {
+  name: ComponentModelName
+  options: Expression | null
+  typeNode: TypeNode | null
+}