fix: fire afterUpdate even if dependencies are not used in template

.changeset/dirty-dancers-hide.md

@@ -0,0 +1,5 @@
+---
+'svelte': patch
+---
+
+fix: fire `afterUpdate` even if dependencies are not used in template

packages/svelte/rollup.config.js

@@ -13,5 +13,5 @@ export default defineConfig({
 		format: 'umd',
 		name: 'svelte'
 	},
-	plugins: [resolve(), commonjs(), terser()]
+	plugins: [resolve(), commonjs()]
 });

packages/svelte/src/internal/client/types.d.ts

@@ -39,7 +39,7 @@ export type ComponentContext = {
 	s: MaybeSignal<Record<string, unknown>>;
 	/** accessors */
 	a: Record<string, any> | null;
-	/** effectgs */
+	/** effects */
 	e: null | Array<EffectSignal>;
 	/** mounted */
 	m: boolean;

packages/svelte/src/main/main-client.js

@@ -177,15 +177,15 @@ function init_update_callbacks() {
 				// TODO somehow beforeUpdate ran twice on mount in Svelte 4 if it causes a render
 				// possibly strategy to get this back if needed: analyse beforeUpdate function for assignements to state,
 				// if yes, add a call to the component to force-run beforeUpdate once.
-				untrack(() => update_callbacks.b.forEach(/** @param {any} c */ (c) => c()));
+				update_callbacks.b.forEach((c) => c());
 				flush_local_render_effects();
 				// beforeUpdate can run again once if afterUpdate causes another update,
 				// but afterUpdate shouldn't be called again in that case to prevent infinite loops
 				if (!called_after) {
 					user_effect(() => {
 						called_before = false;
 						called_after = true;
-						untrack(() => update_callbacks.a.forEach(/** @param {any} c */ (c) => c()));
+						update_callbacks.a.forEach((c) => c());
 						// managed_effect so that it's not cleaned up when the parent effect is cleaned up
 						const managed = managed_effect(() => {
 							destroy_signal(managed);

packages/svelte/tests/runtime-legacy/samples/lifecycle-render-afterUpdate2/_config.js

@@ -0,0 +1,38 @@
+import { test, ok } from '../../test';
+import { flushSync } from 'svelte';
+
+const log = console.log;
+/** @type {string[]} */
+let messages = [];
+
+export default test({
+	before_test: () => {
+		console.log = (msg) => messages.push(msg);
+		messages = [];
+	},
+	after_test: () => {
+		console.log = log;
+	},
+	test({ assert, target, window }) {
+		const [button1, button2] = target.querySelectorAll('button');
+		ok(button1);
+		ok(button2);
+
+		button1.click();
+		flushSync();
+		button2.click();
+		flushSync();
+
+		assert.deepEqual(messages, ['after update 0, 0', 'after update 1, 0', 'after update 1, 1']);
+		assert.htmlEqual(
+			target.innerHTML,
+			`
+			<button>count:
+			1</button><button>count
+			2:
+			1</button><hr>
+			1
+		`
+		);
+	}
+});

packages/svelte/tests/runtime-legacy/samples/lifecycle-render-afterUpdate2/main.svelte

@@ -0,0 +1,21 @@
+<script>
+	import AfterUpdate from "./sub.svelte";
+	let count = 0;
+	let count2 = 0;
+
+	function increment() {
+		count += 1;
+	}
+	function increment2() {
+		count2 += 1;
+	}
+</script>
+
+<button on:click={increment}>
+	count: {count}
+</button>
+<button on:click={increment2}>
+	count 2: {count2}
+</button>
+<hr />
+<AfterUpdate {count} {count2}/>

packages/svelte/tests/runtime-legacy/samples/lifecycle-render-afterUpdate2/sub.svelte

@@ -0,0 +1,12 @@
+<script>
+	import { afterUpdate } from "svelte";
+
+	export let count;
+	export let count2;
+
+	afterUpdate(()=>{
+		console.log(`after update ${count}, ${count2}`);
+	});
+</script>
+
+{count}