refactor(macros): remove unused props for defineComponent

zhiyuanzmj · zhiyuanzmj · commit 1e84dfe4b65d · 2025-07-22T13:59:47.000+08:00
diff --git a/packages/macros/src/core/define-component/index.ts b/packages/macros/src/core/define-component/index.ts
@@ -53,7 +53,7 @@ export function transformDefineComponent(
       s.overwrite(
         root.params[0].start!,
         root.params[0].end!,
-        `${HELPER_PREFIX}props`,
+        root.params.length > 1 ? `${HELPER_PREFIX}props` : '',
       )
     } else if (root.params[0].type === 'ObjectPattern') {
       const restructuredProps = root.params[0]
diff --git a/packages/macros/src/core/index.ts b/packages/macros/src/core/index.ts
@@ -97,7 +97,7 @@ export function transformJsxMacros(
           )
         }
       }
-    } else {
+    } else if (macros.defineModel?.length) {
       s.appendRight(getParamsStart(root, s.original), propsName)
     }
 
diff --git a/packages/macros/tests/__snapshots__/fixtures.spec.ts.snap b/packages/macros/tests/__snapshots__/fixtures.spec.ts.snap
@@ -39,7 +39,7 @@ const Comp = defineComponent(
 }, ...{ Comp: Object }} },
 )
 
-const Comp1 = defineComponent((__MACROS_props) => {const props = __MACROS_useFullProps();
+const Comp1 = defineComponent(() => {const props = __MACROS_useFullProps();
   const foo = __MACROS_useModel(props, 'foo')
   return () => <div>
       {[foo.value, props['bar'], props['onUpdate:bar']]}
@@ -54,7 +54,7 @@ const Comp1 = defineComponent((__MACROS_props) => {const props = __MACROS_useFul
 'fooModifiers': null
 } })
 
-const Comp2 = defineComponent(async (__MACROS_props) => {
+const Comp2 = defineComponent(async () => {
 let __temp, __restore
 
   ;(
@@ -140,7 +140,7 @@ const Comp = defineComponent(
 }, ...{ Comp: Object }} },
 )
 
-const Comp1 = defineComponent((__MACROS_props) => {const props = __MACROS_useFullProps();
+const Comp1 = defineComponent(() => {const props = __MACROS_useFullProps();
   const foo = __MACROS_useModel(props, 'foo')
   return <div>
       {[foo.value, props['bar'], props['onUpdate:bar']]}
@@ -155,7 +155,7 @@ const Comp1 = defineComponent((__MACROS_props) => {const props = __MACROS_useFul
 'fooModifiers': null
 } })
 
-const Comp2 = defineComponent(async (__MACROS_props) => {
+const Comp2 = defineComponent(async () => {
 let __temp, __restore
 
   ;(
@@ -204,7 +204,7 @@ defineComponent((__MACROS_props) => {
 
 exports[`fixtures > ./fixtures/define-expose.tsx 1`] = `
 "
-import { getCurrentInstance as __MACROS_getCurrentInstance } from "vue-jsx-vapor";export function Comp(__MACROS_props) {
+import { getCurrentInstance as __MACROS_getCurrentInstance } from "vue-jsx-vapor";export function Comp() {
   ;(__MACROS_getCurrentInstance().exposed = {
     foo: 1,
   })
@@ -257,14 +257,14 @@ export default function (__MACROS_props) {
 
 exports[`fixtures > ./fixtures/define-slots.tsx 1`] = `
 "
-import { useSlots as __MACROS_useSlots } from "vue";export const Comp = (__MACROS_props) => {
+import { useSlots as __MACROS_useSlots } from "vue";export const Comp = () => {
   const slots = Object.assign<{
     default: () => any
   }>({}, __MACROS_useSlots())
   return <div>{slots.default?.()}</div>
 }
 
-export default function (__MACROS_props) {
+export default function () {
   const slots = Object.assign({
     default: () => <div>default</div>,
   },__MACROS_useSlots())
@@ -277,13 +277,13 @@ exports[`fixtures > ./fixtures/define-style.tsx 1`] = `
 "import "vue-jsx-vapor/macros/define-style/0?scopeId=4e9d5cd0&scoped=true&lang.css";import style1 from "vue-jsx-vapor/macros/define-style/1?scopeId=13f8dd3a&scoped=true&lang.module.scss";
 import { useSlots as __MACROS_useSlots } from "vue";import "vue-jsx-vapor/macros/define-style/2?scopeId=63a7910c&scoped=false&lang.scss";import { defineComponent, ref } from 'vue'
 
-export const Comp = (__MACROS_props) => {
+export const Comp = () => {
   const color = ref('red')
   
   return <div {...{style:{'--4e9d5cd0-color-value': color.value}}} data-v-4e9d5cd0="" class="foo">foo</div>
 }
 
-export default defineComponent((__MACROS_props) => {
+export default defineComponent(() => {
   const color = ref('red')
   const styles = style1
   const { default: Default, ...slots } = Object.assign({}, __MACROS_useSlots())

Original file line number	Diff line number	Diff line change
`@@ -53,7 +53,7 @@ export function transformDefineComponent(`
`53`	`53`	`s.overwrite(`
`54`	`54`	`root.params[0].start!,`
`55`	`55`	`root.params[0].end!,`
`56`		- `${HELPER_PREFIX}props`,
	`56`	+ root.params.length > 1 ? `${HELPER_PREFIX}props` : '',
`57`	`57`	`)`
`58`	`58`	`} else if (root.params[0].type === 'ObjectPattern') {`
`59`	`59`	`const restructuredProps = root.params[0]`
Original file line number	Diff line number	Diff line change
`@@ -97,7 +97,7 @@ export function transformJsxMacros(`
`97`	`97`	`)`
`98`	`98`	`}`
`99`	`99`	`}`
`100`		`- } else {`
	`100`	`+ } else if (macros.defineModel?.length) {`
`101`	`101`	`s.appendRight(getParamsStart(root, s.original), propsName)`
`102`	`102`	`}`
`103`	`103`