fix: improve internal_set versioning mechanic (#15724)

.changeset/pretty-planes-visit.md

@@ -0,0 +1,5 @@
+---
+'svelte': patch
+---
+
+fix: improve internal_set versioning mechanic

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

@@ -162,7 +162,6 @@ export function internal_set(source, value) {
 		}
 
 		source.v = value;
-		source.wv = increment_write_version();
 
 		if (DEV && tracing_mode_flag) {
 			source.updated = get_stack('UpdatedAt');
@@ -180,6 +179,8 @@ export function internal_set(source, value) {
 			set_signal_status(source, (source.f & UNOWNED) === 0 ? CLEAN : MAYBE_DIRTY);
 		}
 
+		source.wv = increment_write_version();
+
 		mark_reactions(source, DIRTY);
 
 		// It's possible that the current reaction might not have up-to-date dependencies

packages/svelte/tests/runtime-runes/samples/writable-derived-2/_config.js

@@ -0,0 +1,7 @@
+import { test } from '../../test';
+
+export default test({
+	html: `true true`,
+
+	test({ assert, target, window }) {}
+});

packages/svelte/tests/runtime-runes/samples/writable-derived-2/main.svelte

@@ -0,0 +1,9 @@
+<script>
+	import { expect2, createAppState } from "./util.svelte.js"
+
+  const result = createAppState({ source: () => "wrong" });
+  result.onChange("right");
+  const expect1 = result.value === "right";
+</script>
+
+{expect1} {expect2}

packages/svelte/tests/runtime-runes/samples/writable-derived-2/util.svelte.js

@@ -0,0 +1,17 @@
+export const createAppState = (options) => {
+	const source = $derived(options.source());
+	let value = $derived(source);
+
+	return {
+		get value() {
+			return value;
+		},
+		onChange(nextValue) {
+			value = nextValue;
+		}
+	};
+};
+
+const result = createAppState({ source: () => 'wrong' });
+result.onChange('right');
+export const expect2 = result.value === 'right';