Svelte 5: Snippet references do not prevent function from being pulled out of component scope

[brunnerh]

Describe the bug

Snippets at the top level declare a variable in the component, if the variable is referenced in a script function, the compiler should make sure it does not lose the reference by moving the function around. Currently this can happen.

Use cases include e.g. passing a snippet to a modal showing function.

Reproduction

<script>
	function log() {
		console.log(snippet)
	}
</script>
{#snippet snippet()}Hello{/snippet}
<button on:click={log}>
	log snippet
</button>

REPL

Logs

No response

System Info

REPL

Severity

annoyance

[brunnerh]

As a workaround, apparently just making the handler an arrow function is enough to pull the function into the component:

<script>
	function log() {
		console.log(snippet)
	}
</script>
{#snippet snippet()}Hello{/snippet}
<button on:click={() => log()}>
	log snippet
</button>

REPL

[dummdidumm]

My guess is that this is related to the event hoisting logic, which doesn't know about snippet and thinks it's a global, so it's safe to hoist