Merge branch 'main' into scope-evaluate

Author: Rich-Harris

.changeset/brown-rockets-shake.md

@@ -0,0 +1,5 @@
+---
+'svelte': patch
+---
+
+fix: invalidate parent effects when child effects update parent dependencies

.changeset/curvy-countries-flow.md

@@ -0,0 +1,5 @@
+---
+'svelte': patch
+---
+
+fix: replace `undefined` with `void 0` to avoid edge case

.changeset/plenty-bats-lay.md

@@ -0,0 +1,5 @@
+---
+'svelte': patch
+---
+
+fix: add `files` and `group` to HTMLInputAttributes in elements.d.ts

packages/svelte/elements.d.ts

@@ -1076,6 +1076,7 @@ export interface HTMLInputAttributes extends HTMLAttributes<HTMLInputElement> {
 	checked?: boolean | undefined | null;
 	dirname?: string | undefined | null;
 	disabled?: boolean | undefined | null;
+	files?: FileList | undefined | null;
 	form?: string | undefined | null;
 	formaction?: string | undefined | null;
 	formenctype?:
@@ -1087,6 +1088,7 @@ export interface HTMLInputAttributes extends HTMLAttributes<HTMLInputElement> {
 	formmethod?: 'dialog' | 'get' | 'post' | 'DIALOG' | 'GET' | 'POST' | undefined | null;
 	formnovalidate?: boolean | undefined | null;
 	formtarget?: string | undefined | null;
+	group?: any | undefined | null;
 	height?: number | string | undefined | null;
 	indeterminate?: boolean | undefined | null;
 	list?: string | undefined | null;

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

@@ -116,8 +116,7 @@ export function VariableDeclaration(node, context) {
 			}
 
 			const args = /** @type {CallExpression} */ (init).arguments;
-			const value =
-				args.length === 0 ? b.id('undefined') : /** @type {Expression} */ (context.visit(args[0]));
+			const value = args.length > 0 ? /** @type {Expression} */ (context.visit(args[0])) : b.void0;
 
 			if (rune === '$state' || rune === '$state.raw') {
 				/**

packages/svelte/src/compiler/phases/3-transform/server/visitors/CallExpression.js

@@ -13,11 +13,11 @@ export function CallExpression(node, context) {
 	const rune = get_rune(node, context.state.scope);
 
 	if (rune === '$host') {
-		return b.id('undefined');
+		return b.void0;
 	}
 
 	if (rune === '$effect.tracking') {
-		return b.literal(false);
+		return b.false;
 	}
 
 	if (rune === '$effect.root') {

packages/svelte/src/compiler/phases/3-transform/server/visitors/VariableDeclaration.js

@@ -45,7 +45,7 @@ export function VariableDeclaration(node, context) {
 						) {
 							const right = node.right.arguments.length
 								? /** @type {Expression} */ (context.visit(node.right.arguments[0]))
-								: b.id('undefined');
+								: b.void0;
 							return b.assignment_pattern(node.left, right);
 						}
 					}
@@ -75,8 +75,7 @@ export function VariableDeclaration(node, context) {
 			}
 
 			const args = /** @type {CallExpression} */ (init).arguments;
-			const value =
-				args.length === 0 ? b.id('undefined') : /** @type {Expression} */ (context.visit(args[0]));
+			const value = args.length > 0 ? /** @type {Expression} */ (context.visit(args[0])) : b.void0;
 
 			if (rune === '$derived.by') {
 				declarations.push(

packages/svelte/src/compiler/utils/builders.js

@@ -154,6 +154,8 @@ export function unary(operator, argument) {
 	return { type: 'UnaryExpression', argument, operator, prefix: true };
 }
 
+export const void0 = unary('void', literal(0));
+
 /**
  * @param {ESTree.Expression} test
  * @param {ESTree.Expression} consequent

packages/svelte/src/internal/client/runtime.js

@@ -460,6 +460,14 @@ export function update_reaction(reaction) {
 		// the same version
 		if (previous_reaction !== null) {
 			read_version++;
+
+			if (untracked_writes !== null) {
+				if (previous_untracked_writes === null) {
+					previous_untracked_writes = untracked_writes;
+				} else {
+					previous_untracked_writes.push(.../** @type {Source[]} */ (untracked_writes));
+				}
+			}
 		}
 
 		return result;

packages/svelte/tests/runtime-runes/samples/untracked-write-pre/_config.js

@@ -0,0 +1,7 @@
+import { test } from '../../test';
+
+export default test({
+	test({ assert, target, logs }) {
+		assert.deepEqual(logs, ['Outer', 'Inner', 'Outer', 'Inner']);
+	}
+});

packages/svelte/tests/runtime-runes/samples/untracked-write-pre/main.svelte

@@ -0,0 +1,13 @@
+<script>
+	let value = $state(0);
+
+	$effect.pre(() => {
+		console.log("Outer");
+		value;
+
+		$effect.pre(() => {
+			console.log("Inner");
+			value = 10;
+		});
+	});
+</script>