svelte 5.39.8 → 5.39.10

package/package.json

@@ -2,7 +2,7 @@
   "name": "svelte",
   "description": "Cybernetically enhanced web apps",
   "license": "MIT",
-  "version": "5.39.8",
+  "version": "5.39.10",
   "type": "module",
   "types": "./types/index.d.ts",
   "engines": {

package/src/compiler/phases/2-analyze/visitors/VariableDeclarator.js

@@ -46,6 +46,21 @@ export function VariableDeclarator(node, context) {
 								: path.is_rest
 									? 'rest_prop'
 									: 'prop';
+				if (rune === '$props' && binding.kind === 'rest_prop' && node.id.type === 'ObjectPattern') {
+					const { properties } = node.id;
+					/** @type {string[]} */
+					const exclude_props = [];
+					for (const property of properties) {
+						if (property.type === 'RestElement') {
+							continue;
+						}
+						const key = /** @type {Identifier | Literal & { value: string | number }} */ (
+							property.key
+						);
+						exclude_props.push(key.type === 'Identifier' ? key.name : key.value.toString());
+					}
+					(binding.metadata ??= {}).exclude_props = exclude_props;
+				}
 			}
 		}
 

package/src/compiler/phases/3-transform/client/visitors/Identifier.js

@@ -32,7 +32,11 @@ export function Identifier(node, context) {
 				grand_parent?.type !== 'AssignmentExpression' &&
 				grand_parent?.type !== 'UpdateExpression'
 			) {
-				return b.id('$$props');
+				const key = /** @type {Identifier} */ (parent.property);
+
+				if (!binding.metadata?.exclude_props?.includes(key.name)) {
+					return b.id('$$props');
+				}
 			}
 		}
 

package/src/compiler/phases/3-transform/client/visitors/shared/fragment.js

@@ -99,7 +99,14 @@ export function process_children(nodes, initial, is_element, context) {
 
 			if (is_static_element(node, context.state)) {
 				skipped += 1;
-			} else if (node.type === 'EachBlock' && nodes.length === 1 && is_element) {
+			} else if (
+				node.type === 'EachBlock' &&
+				nodes.length === 1 &&
+				is_element &&
+				// In case it's wrapped in async the async logic will want to skip sibling nodes up until the end, hence we cannot make this controlled
+				// TODO switch this around and instead optimize for elements with a single block child and not require extra comments (neither for async nor normally)
+				!(node.body.metadata.has_await || node.metadata.expression.has_await)
+			) {
 				node.metadata.is_controlled = true;
 			} else {
 				const id = flush_node(false, node.type === 'RegularElement' ? node.name : 'node');

package/src/compiler/phases/3-transform/server/visitors/IfBlock.js

@@ -23,7 +23,11 @@ export function IfBlock(node, context) {
 	/** @type {Statement} */
 	let statement = b.if(test, consequent, alternate);
 
-	if (node.metadata.expression.has_await) {
+	if (
+		node.metadata.expression.has_await ||
+		node.consequent.metadata.has_await ||
+		node.alternate?.metadata.has_await
+	) {
 		statement = create_async_block(b.block([statement]));
 	}
 

package/src/compiler/phases/3-transform/server/visitors/MemberExpression.js

@@ -7,11 +7,7 @@ import * as b from '#compiler/builders';
  * @param {Context} context
  */
 export function MemberExpression(node, context) {
-	if (
-		context.state.analysis.runes &&
-		node.object.type === 'ThisExpression' &&
-		node.property.type === 'PrivateIdentifier'
-	) {
+	if (context.state.analysis.runes && node.property.type === 'PrivateIdentifier') {
 		const field = context.state.state_fields?.get(`#${node.property.name}`);
 
 		if (field?.type === '$derived' || field?.type === '$derived.by') {

package/src/compiler/phases/scope.js

@@ -122,7 +122,7 @@ export class Binding {
 
 	/**
 	 * Additional metadata, varies per binding type
-	 * @type {null | { inside_rest?: boolean; is_template_declaration?: boolean }}
+	 * @type {null | { inside_rest?: boolean; is_template_declaration?: boolean; exclude_props?: string[] }}
 	 */
 	metadata = null;
 
@@ -263,6 +263,7 @@ class Evaluation {
 					if (binding.initial?.type === 'SnippetBlock') {
 						this.is_defined = true;
 						this.is_known = false;
+						this.values.add(UNKNOWN);
 						break;
 					}
 

package/src/internal/client/dom/blocks/boundary.js

@@ -285,6 +285,13 @@ export class Boundary {
 				this.#anchor.before(this.#offscreen_fragment);
 				this.#offscreen_fragment = null;
 			}
+
+			// TODO this feels like a little bit of a kludge, but until we
+			// overhaul the boundary/batch relationship it's probably
+			// the most pragmatic solution available to us
+			queue_micro_task(() => {
+				Batch.ensure().flush();
+			});
 		}
 	}
 

package/src/version.js

@@ -4,5 +4,5 @@
  * The current version, as set in package.json.
  * @type {string}
  */
-export const VERSION = '5.39.8';
+export const VERSION = '5.39.10';
 export const PUBLIC_VERSION = '5';