apparently not?

Rich-Harris · Rich-Harris · commit c9ee8df1a5b5 · 2025-06-07T07:56:15.000-04:00
diff --git a/packages/svelte/src/compiler/phases/3-transform/client/visitors/shared/utils.js b/packages/svelte/src/compiler/phases/3-transform/client/visitors/shared/utils.js
@@ -387,10 +387,7 @@ export function build_expression(context, expression, metadata) {
 
 		var getter = build_getter({ ...binding.node }, context.state);
 
-		// TODO do we need all this?
-		if (binding.kind === 'rest_prop') {
-			getter = b.call('Object.keys', getter);
-		} else if (binding.kind === 'bindable_prop') {
+		if (binding.kind === 'bindable_prop') {
 			getter = b.call('$.deep_read_state', getter);
 		}
 

Original file line number	Diff line number	Diff line change
`@@ -387,10 +387,7 @@ export function build_expression(context, expression, metadata) {`
`387`	`387`
`388`	`388`	`var getter = build_getter({ ...binding.node }, context.state);`
`389`	`389`
`390`		`- // TODO do we need all this?`
`391`		`- if (binding.kind === 'rest_prop') {`
`392`		`- getter = b.call('Object.keys', getter);`
`393`		`- } else if (binding.kind === 'bindable_prop') {`
	`390`	`+ if (binding.kind === 'bindable_prop') {`
`394`	`391`	`getter = b.call('$.deep_read_state', getter);`
`395`	`392`	`}`
`396`	`393`