chore: skip emitting $.proxy() calls for more expressions

Author: jeremy-deutsch

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

@@ -498,6 +498,8 @@ export function should_proxy(node) {
 		node.type === 'Literal' ||
 		node.type === 'ArrowFunctionExpression' ||
 		node.type === 'FunctionExpression' ||
+		node.type === 'UnaryExpression' ||
+		node.type === 'BinaryExpression' ||
 		(node.type === 'Identifier' && node.name === 'undefined')
 	) {
 		return false;

packages/svelte/tests/snapshot/samples/function-prop-no-getter/_expected/client/index.svelte.js

@@ -9,16 +9,18 @@ export default function Function_prop_no_getter($$anchor, $$props) {
 	let count = $.source(0);
 
 	function onmouseup() {
-		$.set(count, $.proxy($.get(count) + 2));
+		$.set(count, $.get(count) + 2);
 	}
 
+	const plusOne = (num) => num + 1;
 	/* Init */
 	var fragment = $.comment($$anchor);
 	var node = $.child_frag(fragment);
 
 	Button(node, {
-		onmousedown: () => $.set(count, $.proxy($.get(count) + 1)),
+		onmousedown: () => $.set(count, $.get(count) + 1),
 		onmouseup,
+		onmouseenter: () => $.set(count, $.proxy(plusOne($.get(count)))),
 		children: ($$anchor, $$slotProps) => {
 			/* Init */
 			var node_1 = $.space($$anchor);

packages/svelte/tests/snapshot/samples/function-prop-no-getter/_expected/server/index.svelte.js

@@ -11,13 +11,15 @@ export default function Function_prop_no_getter($$payload, $$props) {
 		count += 2;
 	}
 
+	const plusOne = (num) => num + 1;
 	const anchor = $.create_anchor($$payload);
 
 	$$payload.out += `${anchor}`;
 
 	Button($$payload, {
 		onmousedown: () => count += 1,
 		onmouseup,
+		onmouseenter: () => count = plusOne(count),
 		children: ($$payload, $$slotProps) => {
 			$$payload.out += `clicks: ${$.escape(count)}`;
 		}

packages/svelte/tests/snapshot/samples/function-prop-no-getter/index.svelte

@@ -4,8 +4,10 @@
 	function onmouseup() {
 		count += 2;
 	}
+
+	const plusOne = (num) => num + 1;
 </script>
 
-<Button onmousedown={() => count += 1} {onmouseup}>
+<Button onmousedown={() => count += 1} {onmouseup} onmouseenter={() => count = plusOne(count)}>
 	clicks: {count}
 </Button>