chore: only cache return value for setStyle and setDynamicProps

The value of setStyle and setDynamicProps may be a reactive object

diff --git a/packages/compiler-vapor/src/generators/prop.ts b/packages/compiler-vapor/src/generators/prop.ts
index bce64af4029d7a649f1a73f9ef28be7b37b4678b..35716537dd8479f94062800f27f407ed00bb1745 100644 (file)
--- a/packages/compiler-vapor/src/generators/prop.ts
+++ b/packages/compiler-vapor/src/generators/prop.ts
@@ -1,5 +1,4 @@
 import {
-  BindingTypes,
   NewlineType,
   type SimpleExpressionNode,
   isSimpleIdentifier,
@@ -236,6 +235,8 @@ const helpersNeedCachedReturnValue = [
   'setDynamicProps',
 ]
 
+const helpersNoNeedCachedDeps = ['setStyle', 'setDynamicProps']
+
 function processPropValues(
   context: CodegenContext,
   helperName: string,
@@ -247,28 +248,26 @@ function processPropValues(
   // e.g. _foo === _ctx.foo && (_prev_foo = _setStyle(...))
   let shouldWrapInParentheses: boolean = false
   let prevValueName
-  const needReturnValue = helpersNeedCachedReturnValue.includes(helperName)
   if (shouldCacheRenderEffectDeps()) {
-    const { declareNames, operations, identifiers } = processingRenderEffect!
-    // if render effect rely on any reactive object, it should not cache
-    const canCache = identifiers.every(name =>
-      isBindingCacheable(context, name),
-    )
-    processValues(context, values, canCache)
+    const { declareNames, operations } = processingRenderEffect!
+    const needReturnValue = helpersNeedCachedReturnValue.includes(helperName)
+    processValues(context, values, !needReturnValue)
     // if the operation needs to cache the return value and has multiple declareNames,
     // combine them into a single name as the return value name.
     if (declareNames.size > 0 && needReturnValue) {
       const names = [...declareNames]
       prevValueName =
         declareNames.size === 1 ? `_prev${names[0]}` : names.join('')
-      if (!canCache) {
-        declareNames.clear()
-        processingRenderEffect!.earlyCheckExps = []
-      }
       declareNames.add(prevValueName)
     }
-    shouldWrapInParentheses = operations.length === 1 && canCache
+    shouldWrapInParentheses = operations.length === 1
+
+    if (helpersNoNeedCachedDeps.includes(helperName)) {
+      declareNames.clear()
+      processingRenderEffect!.earlyCheckExps = []
+    }
   }
+
   return { prevValueName, shouldWrapInParentheses }
 }
 
@@ -330,14 +329,3 @@ function processValue(
     return [...new Set(earlyCheckExps)]
   }
 }
-
-function isBindingCacheable(context: CodegenContext, name: string): boolean {
-  const {
-    options: { bindingMetadata },
-  } = context
-  const bindingType = bindingMetadata[name]
-  return (
-    bindingType !== BindingTypes.SETUP_REACTIVE_CONST &&
-    bindingType !== BindingTypes.SETUP_CONST
-  )
-}