update comment — it _does_ feel slightly wrong, but no wronger than the rest of this cursed function

Rich-Harris · Rich-Harris · commit 82e431fb771e · 2025-07-22T20:33:53.000-04:00
diff --git a/packages/svelte/src/internal/client/reactivity/batch.js b/packages/svelte/src/internal/client/reactivity/batch.js
@@ -461,7 +461,7 @@ export function flushSync(fn) {
 			if (queued_root_effects.length === 0) {
 				current_batch?.flush();
 
-				// TODO this feels wrong
+				// we need to check again, in case we just updated an `$effect.pending()`
 				if (queued_root_effects.length === 0) {
 					// this would be reset in `flush_effects()` but since we are early returning here,
 					// we need to reset it here as well in case the first time there's 0 queued root effects
