fix: prevent last_propagated_event from being DCE'd (#16538)

Rich-Harris · web-flow · commit c04975d3db74 · 2025-07-31T15:28:01.000-07:00
* fix: prevent last_propagated_event from being DCE'd

* changeset
diff --git a/.changeset/serious-cars-hear.md b/.changeset/serious-cars-hear.md
@@ -0,0 +1,5 @@
+---
+'svelte': patch
+---
+
+fix: prevent last_propagated_event from being DCE'd
diff --git a/packages/svelte/src/internal/client/dom/elements/events.js b/packages/svelte/src/internal/client/dom/elements/events.js
@@ -168,8 +168,11 @@ export function handle_event_propagation(event) {
 	// mounted apps. In this case we don't want to trigger events multiple times.
 	var path_idx = 0;
 
+	// the `last_propagated_event === event` check is redundant, but
+	// without it the variable will be DCE'd and things will
+	// fail mysteriously in Firefox
 	// @ts-expect-error is added below
-	var handled_at = event.__root;
+	var handled_at = last_propagated_event === event && event.__root;
 
 	if (handled_at) {
 		var at_idx = path.indexOf(handled_at);

-Original file line number
+Diff line change
@@ @@ -0,0 +1,5 @@ @@
 +---
 +'svelte': patch
 +---
++
 +fix: prevent last_propagated_event from being DCE'd