fix: ensure props internally untracks current_value on sets (#13859)

* fix: ensure props internally untracks current_value on sets

* Update packages/svelte/src/internal/client/reactivity/props.js

---------

Co-authored-by: Simon H <[email protected]>

Author: trueadm

.changeset/short-jokes-speak.md

@@ -0,0 +1,5 @@
+---
+'svelte': patch
+---
+
+fix: ensure props internally untracks current_value on sets

packages/svelte/src/internal/client/reactivity/props.js

@@ -373,8 +373,6 @@ export function prop(props, key, flags, fallback) {
 	if (!immutable) current_value.equals = safe_equals;
 
 	return function (/** @type {any} */ value, /** @type {boolean} */ mutation) {
-		var current = get(current_value);
-
 		// legacy nonsense — need to ensure the source is invalidated when necessary
 		// also needed for when handling inspect logic so we can inspect the correct source signal
 		if (is_signals_recorded) {
@@ -398,12 +396,11 @@ export function prop(props, key, flags, fallback) {
 				if (fallback_used && fallback_value !== undefined) {
 					fallback_value = new_value;
 				}
-				get(current_value); // force a synchronisation immediately
+				untrack(() => get(current_value)); // force a synchronisation immediately
 			}
 
 			return value;
 		}
-
-		return current;
+		return get(current_value);
 	};
 }

packages/svelte/tests/runtime-runes/samples/props-assignment-tracking/_config.js

@@ -0,0 +1,16 @@
+import { flushSync } from 'svelte';
+import { test } from '../../test';
+
+export default test({
+	test({ assert, logs, target }) {
+		const btn = target.querySelector('button');
+
+		flushSync(() => {
+			btn?.click();
+			btn?.click();
+			btn?.click();
+		});
+
+		assert.deepEqual(logs, ['effect']);
+	}
+});

packages/svelte/tests/runtime-runes/samples/props-assignment-tracking/main.svelte

@@ -0,0 +1,10 @@
+<script>
+	let {display = true} = $props();
+
+	$effect(()=>{
+		display = true;
+		console.log("effect")
+	});
+</script>
+
+<button onclick={() => display = !display} >display</button>