fix: improve code generation

Author: trueadm

.changeset/odd-buckets-lie.md

@@ -217,9 +217,10 @@ function is_expression_async(expression) {
  * @param {import('estree').AssignmentExpression} node
  * @param {import('zimmerframe').Context<import('#compiler').SvelteNode, State>} context
  * @param {() => any} fallback
+ * @param {{skip_proxy_and_freeze?: boolean}} [options]
  * @returns {import('estree').Expression}
  */
-export function serialize_set_binding(node, context, fallback) {
+export function serialize_set_binding(node, context, fallback, options) {
 	const { state, visit } = context;
 
 	if (
@@ -242,7 +243,7 @@ export function serialize_set_binding(node, context, fallback) {
 			const value = path.expression?.(b.id(tmp_id));
 			const assignment = b.assignment('=', path.node, value);
 			original_assignments.push(assignment);
-			assignments.push(serialize_set_binding(assignment, context, () => assignment));
+			assignments.push(serialize_set_binding(assignment, context, () => assignment, options));
 		}
 
 		if (assignments.every((assignment, i) => assignment === original_assignments[i])) {
@@ -288,7 +289,11 @@ export function serialize_set_binding(node, context, fallback) {
 			if (private_state !== undefined) {
 				if (state.in_constructor) {
 					// See if we should wrap value in $.proxy
-					if (context.state.analysis.runes && should_proxy_or_freeze(value)) {
+					if (
+						context.state.analysis.runes &&
+						!options?.skip_proxy_and_freeze &&
+						should_proxy_or_freeze(value)
+					) {
 						const assignment = fallback();
 						if (assignment.type === 'AssignmentExpression') {
 							assignment.right =
@@ -302,7 +307,9 @@ export function serialize_set_binding(node, context, fallback) {
 					return b.call(
 						'$.set',
 						left,
-						context.state.analysis.runes && should_proxy_or_freeze(value)
+						context.state.analysis.runes &&
+							!options?.skip_proxy_and_freeze &&
+							should_proxy_or_freeze(value)
 							? private_state.kind === 'frozen_state'
 								? b.call('$.freeze', value)
 								: b.call('$.proxy', value)
@@ -321,6 +328,7 @@ export function serialize_set_binding(node, context, fallback) {
 			if (
 				context.state.analysis.runes &&
 				public_state !== undefined &&
+				!options?.skip_proxy_and_freeze &&
 				should_proxy_or_freeze(value)
 			) {
 				const assignment = fallback();
@@ -387,15 +395,19 @@ export function serialize_set_binding(node, context, fallback) {
 				return b.call(
 					'$.set',
 					b.id(left_name),
-					context.state.analysis.runes && should_proxy_or_freeze(value)
+					context.state.analysis.runes &&
+						!options?.skip_proxy_and_freeze &&
+						should_proxy_or_freeze(value)
 						? b.call('$.proxy', value)
 						: value
 				);
 			} else if (binding.kind === 'frozen_state') {
 				return b.call(
 					'$.set',
 					b.id(left_name),
-					context.state.analysis.runes && should_proxy_or_freeze(value)
+					context.state.analysis.runes &&
+						!options?.skip_proxy_and_freeze &&
+						should_proxy_or_freeze(value)
 						? b.call('$.freeze', value)
 						: value
 				);
@@ -624,6 +636,5 @@ export function should_proxy_or_freeze(node) {
 	) {
 		return false;
 	}
-
 	return true;
 }

packages/svelte/src/compiler/phases/3-transform/client/utils.js

@@ -2604,7 +2604,10 @@ export const template_visitors = {
 			serialize_set_binding(
 				assignment,
 				context,
-				() => /** @type {import('estree').Expression} */ (context.visit(assignment))
+				() => /** @type {import('estree').Expression} */ (context.visit(assignment)),
+				{
+					skip_proxy_and_freeze: true,
+				}
 			)
 		);
 

packages/svelte/src/compiler/phases/3-transform/client/visitors/template.js

@@ -31,4 +31,4 @@
 			}
 		]
 	}
-}
+}

packages/svelte/tests/parser-legacy/samples/action-duplicate/output.json

@@ -73,4 +73,4 @@
 			}
 		]
 	}
-}
+}

packages/svelte/tests/parser-legacy/samples/action-with-call/output.json

@@ -38,4 +38,4 @@
 			}
 		]
 	}
-}
+}

packages/svelte/tests/parser-legacy/samples/action-with-identifier/output.json

@@ -39,4 +39,4 @@
 			}
 		]
 	}
-}
+}

packages/svelte/tests/parser-legacy/samples/action-with-literal/output.json

@@ -23,4 +23,4 @@
 			}
 		]
 	}
-}
+}

packages/svelte/tests/parser-legacy/samples/action/output.json

@@ -76,4 +76,4 @@
 			}
 		]
 	}
-}
+}

packages/svelte/tests/parser-legacy/samples/animation/output.json

@@ -38,4 +38,4 @@
 			}
 		]
 	}
-}
+}