merge main

Author: benmccann

.changeset/eight-jeans-compare.md

@@ -0,0 +1,5 @@
+---
+"svelte": patch
+---
+
+fix: improve reactive Map and Set implementations

.changeset/gentle-ties-fetch.md

@@ -0,0 +1,5 @@
+---
+"svelte": patch
+---
+
+fix: improve controlled each block cleanup performance

.changeset/ninety-rockets-battle.md

@@ -0,0 +1,5 @@
+---
+"svelte": patch
+---
+
+fix: allow for more svelte-ignore to work

.changeset/pre.json

@@ -110,6 +110,7 @@
     "eight-carrots-hunt",
     "eight-cougars-watch",
     "eight-hornets-punch",
+    "eight-jeans-compare",
     "eight-pianos-raise",
     "eight-steaks-shout",
     "eighty-bikes-camp",
@@ -170,6 +171,7 @@
     "gentle-dolls-juggle",
     "gentle-sheep-hug",
     "gentle-spies-happen",
+    "gentle-ties-fetch",
     "gentle-toys-chew",
     "gentle-trees-exercise",
     "gentle-wasps-pull",
@@ -310,6 +312,7 @@
     "nine-cooks-join",
     "ninety-dingos-walk",
     "ninety-dots-train",
+    "ninety-rockets-battle",
     "odd-buckets-lie",
     "odd-needles-joke",
     "odd-schools-wait",
@@ -412,6 +415,7 @@
     "shiny-baboons-play",
     "shiny-mayflies-clean",
     "shiny-melons-love",
+    "shiny-months-tease",
     "shiny-rats-heal",
     "shiny-shrimps-march",
     "short-buses-camp",
@@ -425,11 +429,13 @@
     "silly-lies-film",
     "silly-ways-wash",
     "silver-points-approve",
+    "silver-sheep-knock",
     "six-bears-trade",
     "six-boats-shave",
     "sixty-items-crash",
     "sixty-numbers-hope",
     "sixty-pandas-rush",
+    "sixty-plants-cover",
     "sleepy-cats-eat",
     "slimy-clouds-talk",
     "slimy-hairs-impress",

.changeset/shiny-months-tease.md

@@ -0,0 +1,5 @@
+---
+"svelte": patch
+---
+
+fix: reevaluate namespace in slots

.changeset/silver-sheep-knock.md

@@ -0,0 +1,5 @@
+---
+"svelte": patch
+---
+
+chore: improve $state.frozen performance in prod

.changeset/sixty-plants-cover.md

@@ -0,0 +1,5 @@
+---
+"svelte": patch
+---
+
+breaking: removed deferred event updates

packages/svelte/CHANGELOG.md

@@ -1,5 +1,29 @@
 # svelte
 
+## 5.0.0-next.148
+
+### Patch Changes
+
+- chore: improve $state.frozen performance in prod ([#11852](https://github.com/sveltejs/svelte/pull/11852))
+
+- breaking: removed deferred event updates ([#11855](https://github.com/sveltejs/svelte/pull/11855))
+
+## 5.0.0-next.147
+
+### Patch Changes
+
+- fix: improve reactive Map and Set implementations ([#11827](https://github.com/sveltejs/svelte/pull/11827))
+
+- fix: improve controlled each block cleanup performance ([#11839](https://github.com/sveltejs/svelte/pull/11839))
+
+## 5.0.0-next.146
+
+### Patch Changes
+
+- fix: allow for more svelte-ignore to work ([#11833](https://github.com/sveltejs/svelte/pull/11833))
+
+- fix: reevaluate namespace in slots ([#11849](https://github.com/sveltejs/svelte/pull/11849))
+
 ## 5.0.0-next.145
 
 ### Patch Changes

packages/svelte/package.json

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

packages/svelte/scripts/process-messages/templates/compile-warnings.js

@@ -1,4 +1,4 @@
-import { filename, locator, warnings, ignore_stack } from './state.js';
+import { filename, locator, warnings, ignore_stack, ignore_map } from './state.js';
 
 /** @typedef {{ start?: number, end?: number }} NodeLike */
 
@@ -8,7 +8,11 @@ import { filename, locator, warnings, ignore_stack } from './state.js';
  * @param {string} message
  */
 function w(node, code, message) {
-	if (ignore_stack.at(-1)?.has(code)) return;
+	let stack = ignore_stack;
+	if (node) {
+		stack = ignore_map.get(node) ?? ignore_stack;
+	}
+	if (stack && stack.at(-1)?.has(code)) return;
 
 	warnings.push({
 		code,

packages/svelte/src/compiler/phases/2-analyze/index.js

@@ -30,7 +30,7 @@ import { prune } from './css/css-prune.js';
 import { hash } from './utils.js';
 import { warn_unused } from './css/css-warn.js';
 import { extract_svelte_ignore } from '../../utils/extract_svelte_ignore.js';
-import { pop_ignore, push_ignore } from '../../state.js';
+import { ignore_map, ignore_stack, pop_ignore, push_ignore } from '../../state.js';
 
 /**
  * @param {import('#compiler').Script | null} script
@@ -1107,6 +1107,7 @@ function is_safe_identifier(expression, scope) {
 /** @type {import('./types').Visitors} */
 const common_visitors = {
 	_(node, { state, next, path }) {
+		ignore_map.set(node, structuredClone(ignore_stack));
 		const parent = path.at(-1);
 		if (parent?.type === 'Fragment' && node.type !== 'Comment' && node.type !== 'Text') {
 			const idx = parent.nodes.indexOf(/** @type {any} */ (node));
@@ -1129,6 +1130,7 @@ const common_visitors = {
 
 			if (ignores.length > 0) {
 				push_ignore(ignores);
+				ignore_map.set(node, structuredClone(ignore_stack));
 				next();
 				pop_ignore();
 			}
@@ -1148,6 +1150,7 @@ const common_visitors = {
 				}
 				if (ignores.length > 0) {
 					push_ignore(ignores);
+					ignore_map.set(node, structuredClone(ignore_stack));
 					next();
 					pop_ignore();
 				}

packages/svelte/src/compiler/phases/3-transform/utils.js

@@ -176,7 +176,8 @@ export function infer_namespace(namespace, parent, nodes) {
 			parent.type === 'Component' ||
 			parent.type === 'SvelteComponent' ||
 			parent.type === 'SvelteFragment' ||
-			parent.type === 'SnippetBlock'
+			parent.type === 'SnippetBlock' ||
+			parent.type === 'SlotElement'
 		) {
 			const new_namespace = check_nodes_for_namespace(nodes, 'keep');
 			if (new_namespace !== 'keep' && new_namespace !== 'maybe_html') {

packages/svelte/src/compiler/state.js

@@ -14,9 +14,20 @@ export let filename;
 
 export let locator = getLocator('', { offsetLine: 1 });
 
-/** @type {Set<string>[]} */
+/**
+ * The current stack of ignored warnings
+ * @type {Set<string>[]}
+ */
 export let ignore_stack = [];
 
+/**
+ * For each node the list of warnings that should be ignored for that node.
+ * Exists in addition to `ignore_stack` because not all warnings are emitted
+ * while the stack is being built.
+ * @type {Map<import("./types").SvelteNode | NodeLike, Set<string>[]>}
+ */
+export let ignore_map = new Map();
+
 /**
  * @param {string[]} ignores
  */
@@ -49,4 +60,5 @@ export function reset(source, options) {
 	locator = getLocator(source, { offsetLine: 1 });
 	warnings = [];
 	ignore_stack = [];
+	ignore_map.clear();
 }

packages/svelte/src/compiler/warnings.js

@@ -1,6 +1,12 @@
 /* This file is generated by scripts/process-messages/index.js. Do not edit! */
 
-import { filename, locator, warnings, ignore_stack } from './state.js';
+import {
+	filename,
+	locator,
+	warnings,
+	ignore_stack,
+	ignore_map
+} from './state.js';
 
 /** @typedef {{ start?: number, end?: number }} NodeLike */
 /**
@@ -9,7 +15,13 @@ import { filename, locator, warnings, ignore_stack } from './state.js';
  * @param {string} message
  */
 function w(node, code, message) {
-	if (ignore_stack.at(-1)?.has(code)) return;
+	let stack = ignore_stack;
+
+	if (node) {
+		stack = ignore_map.get(node) ?? ignore_stack;
+	}
+
+	if (stack && stack.at(-1)?.has(code)) return;
 
 	warnings.push({
 		code,

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

@@ -18,4 +18,5 @@ export const EFFECT_TRANSPARENT = 1 << 15;
 export const LEGACY_DERIVED_PROP = 1 << 16;
 
 export const STATE_SYMBOL = Symbol('$state');
+export const STATE_FROZEN_SYMBOL = Symbol('$state.frozen');
 export const LOADING_ATTR_SYMBOL = Symbol('');

packages/svelte/src/internal/client/dev/inspect.js

@@ -62,16 +62,6 @@ export function inspect(get_value, inspector = console.log) {
  */
 function deep_snapshot(value, visited = new Map()) {
 	if (typeof value === 'object' && value !== null && !visited.has(value)) {
-		if (DEV) {
-			// When dealing with ReactiveMap or ReactiveSet, return normal versions
-			// so that console.log provides better output versions
-			if (value instanceof Map && value.constructor !== Map) {
-				return new Map(value);
-			}
-			if (value instanceof Set && value.constructor !== Set) {
-				return new Set(value);
-			}
-		}
 		const unstated = snapshot(value);
 
 		if (unstated !== value) {

packages/svelte/src/internal/client/dom/blocks/each.js

@@ -57,9 +57,9 @@ export function index(_, i) {
  * subsequent destruction. Used in each blocks
  * @param {import('#client').EachItem[]} items
  * @param {null | Node} controlled_anchor
- * @param {() => void} [callback]
+ * @param {Map<any, import("#client").EachItem>} items_map
  */
-function pause_effects(items, controlled_anchor, callback) {
+function pause_effects(items, controlled_anchor, items_map) {
 	/** @type {import('#client').TransitionManager[]} */
 	var transitions = [];
 	var length = items.length;
@@ -68,20 +68,29 @@ function pause_effects(items, controlled_anchor, callback) {
 		pause_children(items[i].e, transitions, true);
 	}
 
+	var is_controlled = length > 0 && transitions.length === 0 && controlled_anchor !== null;
 	// If we have a controlled anchor, it means that the each block is inside a single
 	// DOM element, so we can apply a fast-path for clearing the contents of the element.
-	if (length > 0 && transitions.length === 0 && controlled_anchor !== null) {
-		var parent_node = /** @type {Element} */ (controlled_anchor.parentNode);
+	if (is_controlled) {
+		var parent_node = /** @type {Element} */ (
+			/** @type {Element} */ (controlled_anchor).parentNode
+		);
 		clear_text_content(parent_node);
-		parent_node.append(controlled_anchor);
+		parent_node.append(/** @type {Element} */ (controlled_anchor));
+		items_map.clear();
+		link(items[0].prev, items[length - 1].next);
 	}
 
 	run_out_transitions(transitions, () => {
 		for (var i = 0; i < length; i++) {
-			destroy_effect(items[i].e);
+			var item = items[i];
+			if (!is_controlled) {
+				items_map.delete(item.k);
+				item.o.remove();
+				link(item.prev, item.next);
+			}
+			destroy_effect(item.e, !is_controlled);
 		}
-
-		if (callback !== undefined) callback();
 	});
 }
 
@@ -391,31 +400,27 @@ function reconcile(array, state, anchor, render_fn, flags, get_key) {
 
 	const to_destroy = Array.from(seen);
 
-	while (current) {
+	while (current !== null) {
 		to_destroy.push(current);
 		current = current.next;
 	}
+	var destroy_length = to_destroy.length;
 
-	var controlled_anchor = (flags & EACH_IS_CONTROLLED) !== 0 && length === 0 ? anchor : null;
+	if (destroy_length > 0) {
+		var controlled_anchor = (flags & EACH_IS_CONTROLLED) !== 0 && length === 0 ? anchor : null;
 
-	if (is_animated) {
-		for (i = 0; i < to_destroy.length; i += 1) {
-			to_destroy[i].a?.measure();
-		}
+		if (is_animated) {
+			for (i = 0; i < destroy_length; i += 1) {
+				to_destroy[i].a?.measure();
+			}
 
-		for (i = 0; i < to_destroy.length; i += 1) {
-			to_destroy[i].a?.fix();
+			for (i = 0; i < destroy_length; i += 1) {
+				to_destroy[i].a?.fix();
+			}
 		}
-	}
 
-	pause_effects(to_destroy, controlled_anchor, () => {
-		for (var i = 0; i < to_destroy.length; i += 1) {
-			var item = to_destroy[i];
-			items.delete(item.k);
-			item.o.remove();
-			link(item.prev, item.next);
-		}
-	});
+		pause_effects(to_destroy, controlled_anchor, items);
+	}
 
 	if (is_animated) {
 		effect(() => {