fix: ignore generic type arguments while creating AST (#15659)

* fix: ignore generic type arguments while creating AST

bumps esrap and fixes a resulting error: because esrap now uses a more compliant TSESTree AST, it has `typeArguments` instead of `typeParameters` in some places - those were not deleted while stripping types, which caused an error.

* fix $$render_inner declaration

Author: dummdidumm

.changeset/happy-snakes-pump.md

@@ -0,0 +1,5 @@
+---
+'svelte': patch
+---
+
+fix: ignore generic type arguments while creating AST

packages/svelte/package.json

@@ -156,7 +156,7 @@
     "axobject-query": "^4.1.0",
     "clsx": "^2.1.1",
     "esm-env": "^1.2.1",
-    "esrap": "^1.4.3",
+    "esrap": "^1.4.6",
     "is-reference": "^3.0.3",
     "locate-character": "^3.0.0",
     "magic-string": "^0.30.11",

packages/svelte/src/compiler/phases/1-parse/remove_typescript_nodes.js

@@ -24,6 +24,7 @@ const visitors = {
 		// until that day comes, we just delete them so they don't confuse esrap
 		delete n.typeAnnotation;
 		delete n.typeParameters;
+		delete n.typeArguments;
 		delete n.returnType;
 		delete n.accessibility;
 	},

packages/svelte/src/compiler/phases/3-transform/server/transform-server.js

@@ -186,12 +186,10 @@ export function server_component(analysis, options) {
 			...snippets,
 			b.let('$$settled', b.true),
 			b.let('$$inner_payload'),
-			b.stmt(
-				b.function(
-					b.id('$$render_inner'),
-					[b.id('$$payload')],
-					b.block(/** @type {Statement[]} */ (rest))
-				)
+			b.function_declaration(
+				b.id('$$render_inner'),
+				[b.id('$$payload')],
+				b.block(/** @type {Statement[]} */ (rest))
 			),
 			b.do_while(
 				b.unary('!', b.id('$$settled')),

packages/svelte/tests/runtime-runes/samples/typescript/main.svelte

@@ -8,7 +8,7 @@
 		console.log(this);
 	}
 
-	function foo(): string {
+	function foo<T>(): string {
 		return ""!;
 	}
 
@@ -46,6 +46,7 @@
 
 	const TypedFoo = Foo<true>;
 	const typeAssertion = <boolean>true;
+	const x = foo<true>();
 </script>
 
 <script>

packages/svelte/tests/snapshot/samples/bind-component-snippet/_expected/server/index.svelte.js

@@ -23,7 +23,7 @@ export default function Bind_component_snippet($$payload) {
 		});
 
 		$$payload.out += `<!----> value: ${$.escape(value)}`;
-	};
+	}
 
 	do {
 		$$settled = true;