fix: remove unwanted properties from both replaced and unreplaced nod…

…es (#14744)
sveltejs · Dec 18, 2024 · e38975e · e38975e
1 parent 00ff60e
commit e38975e
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 5 deletions.
diff --git a/.changeset/famous-mugs-rhyme.md b/.changeset/famous-mugs-rhyme.md
@@ -0,0 +1,5 @@
+---
+'svelte': patch
+---
+
+fix: remove unwanted properties from both replaced and unreplaced nodes
diff --git a/packages/svelte/src/compiler/phases/1-parse/remove_typescript_nodes.js b/packages/svelte/src/compiler/phases/1-parse/remove_typescript_nodes.js
@@ -18,14 +18,14 @@ function remove_this_param(node, context) {
 /** @type {Visitors<any, null>} */
 const visitors = {
 	_(node, context) {
-		context.next();
+		const n = context.next() ?? node;
 
 		// TODO there may come a time when we decide to preserve type annotations.
 		// until that day comes, we just delete them so they don't confuse esrap
-		delete node.typeAnnotation;
-		delete node.typeParameters;
-		delete node.returnType;
-		delete node.accessibility;
+		delete n.typeAnnotation;
+		delete n.typeParameters;
+		delete n.returnType;
+		delete n.accessibility;
 	},
 	Decorator(node) {
 		e.typescript_invalid_feature(node, 'decorators (related TSC proposal is not stage 4 yet)');