fix: properly determine end of snippet parameters with TS annotations

fixes #455

Author: dummdidumm

CHANGELOG.md

@@ -3,6 +3,7 @@
 ## 3.2.7 (unreleased)
 
 -   (fix) force quote style inside style directives
+-   (fix) Svelte 5: properly determine end of snippet parameters with TS annotations
 
 ## 3.2.6
 

src/embed.ts

@@ -100,16 +100,16 @@ export function embed(path: FastPath, _options: Options) {
             printSvelteBlockJS('key');
             break;
         case 'SnippetBlock':
-            // We merge the two parts into one expression, which future-proofs this for template TS support
+            // We merge the two parts into one expression to then treat it like a function
             if (node === parent.expression) {
                 parent.expression.end =
                     options.originalText.indexOf(
                         ')',
-                        parent.context?.end ?? // TODO: remove at some point, snippet API changed in .next-..
+                        (parent.parameters?.[parent.parameters.length - 1] as any)?.typeAnnotation
+                            ?.end ??
                             parent.parameters?.[parent.parameters.length - 1]?.end ??
                             parent.expression.end,
                     ) + 1;
-                parent.context = null;
                 parent.parameters = null;
                 node.isJS = true;
                 node.asFunction = true;

test/printer/samples/snippet-ts.html.skip

@@ -0,0 +1,14 @@
+<script lang="ts">
+</script>
+
+{#snippet test1(arg: string)}
+    foo
+{/snippet}
+
+{#snippet test2(arg: (typeof obj)["key"])}
+    foo
+{/snippet}
+
+{#snippet test3({ onselect }: { onselect: (value: string) => void })}
+    foo
+{/snippet}