fix: marking as non-hoistable events that are also referenced on "OnDirective"s

Author: caiquetorres

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

@@ -103,6 +103,10 @@ function get_delegated_event(event_name, handler, context) {
 		}
 
 		if (binding != null) {
+			if (binding.references.some(({ path }) => path.some(({ type }) => type === 'OnDirective'))) {
+				return non_hoistable;
+			}
+
 			for (const { path } of binding.references) {
 				const parent = path.at(-1);
 				if (parent == null) {

packages/svelte/tests/runtime-runes/samples/event-attribute-2/_config.js

@@ -0,0 +1,17 @@
+import { test } from '../../test';
+
+export default test({
+	html: `<button>0</button><button>0</button>`,
+
+	async test({ assert, target }) {
+		const [b1, b2] = target.querySelectorAll('button');
+
+		b1?.click();
+		await Promise.resolve();
+		assert.htmlEqual(target.innerHTML, '<button>1</button><button>1</button>');
+
+		b2?.click();
+		await Promise.resolve();
+		assert.htmlEqual(target.innerHTML, '<button>2</button><button>2</button>');
+	}
+});

packages/svelte/tests/runtime-runes/samples/event-attribute-2/main.svelte

@@ -0,0 +1,7 @@
+<script>
+	let count = $state(0);
+	const onclick = () => count++;
+</script>
+
+<button on:click={onclick}>{count}</button>
+<button onclick={onclick}>{count}</button>