fix: use transform.read for ownership_validator.mutation array (#15848)

Author: paoloricciuti

.changeset/gorgeous-experts-perform.md

@@ -0,0 +1,5 @@
+---
+'svelte': patch
+---
+
+fix: use `transform.read` for `ownership_validator.mutation` array

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

@@ -324,15 +324,18 @@ export function validate_mutation(node, context, expression) {
 	const state = /** @type {ComponentClientTransformState} */ (context.state);
 	state.analysis.needs_mutation_validation = true;
 
-	/** @type {Array<Identifier | Literal>} */
+	/** @type {Array<Identifier | Literal | Expression>} */
 	const path = [];
 
 	while (left.type === 'MemberExpression') {
 		if (left.property.type === 'Literal') {
 			path.unshift(left.property);
 		} else if (left.property.type === 'Identifier') {
+			const transform = Object.hasOwn(context.state.transform, left.property.name)
+				? context.state.transform[left.property.name]
+				: null;
 			if (left.computed) {
-				path.unshift(left.property);
+				path.unshift(transform?.read ? transform.read(left.property) : left.property);
 			} else {
 				path.unshift(b.literal(left.property.name));
 			}

packages/svelte/tests/runtime-runes/samples/ownership-invalid-mutation-use-transform/_config.js

@@ -0,0 +1,10 @@
+import { test } from '../../test';
+
+export default test({
+	compileOptions: {
+		dev: true
+	},
+	async test({ assert, errors }) {
+		assert.deepEqual(errors, []);
+	}
+});

packages/svelte/tests/runtime-runes/samples/ownership-invalid-mutation-use-transform/main.svelte

@@ -0,0 +1,5 @@
+<script lang="ts">
+	let { rows = $bindable([]), row } = $props();
+	rows[row] = '';
+</script>
+