fix: remove 'implements' from class declaration (#14749)

* fix: remove implements from class declaration * add a smoke test
sveltejs · Dec 18, 2024 · c79b909 · c79b909
1 parent e38975e
commit c79b909
Show file tree

Hide file tree

Showing 4 changed files with 14 additions and 0 deletions.
diff --git a/.changeset/calm-maps-cover.md b/.changeset/calm-maps-cover.md
@@ -0,0 +1,5 @@
+---
+'svelte': patch
+---
+
+fix: remove implements from class declarations
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
@@ -115,6 +115,7 @@ const visitors = {
 		if (node.declare) {
 			return b.empty;
 		}
+		delete node.implements;
 		return context.next();
 	},
 	VariableDeclaration(node, context) {

diff --git a/packages/svelte/tests/runtime-browser/samples/typescript-class-and-interface/_config.js b/packages/svelte/tests/runtime-browser/samples/typescript-class-and-interface/_config.js
@@ -0,0 +1,3 @@
+import { test } from '../../assert';
+
+export default test({});
diff --git a/packages/svelte/tests/runtime-browser/samples/typescript-class-and-interface/main.svelte b/packages/svelte/tests/runtime-browser/samples/typescript-class-and-interface/main.svelte
@@ -0,0 +1,5 @@
+<script lang="ts">
+  // TypeScript syntax should not cause compilation failure
+  interface MyInterface {}
+  class MyClass implements MyInterface {}
+</script>
Original file line number	Diff line number	Diff line change
		@@ -0,0 +1,3 @@
		import { test } from '../../assert';

		export default test({});