add comment explaining why we need it at the top level

Author: dummdidumm

packages/kit/test/apps/basics/src/routes/remote/batch/+page.svelte

@@ -7,6 +7,8 @@
 	} from './batch.remote.js';
 
 	const todoIds = ['1', '2', '1', 'error'];
+	// Need to write this outside at the top level to ensure tests succeed in non-async-mode
+	// Else updates are not coming through properly because of state-created-inside-effects-not-updating logic in non-async mode
 	const todos = todoIds.map((id) => ({ id, promise: get_todo(id) }));
 </script>