fix: correctly differentiate static fields before emitting `duplicate_class_field` (#16526)

* fix: correctly differentiate static fields before emitting `duplicate_class_field`

* remove unnecessary `#` concatenation

* add test

.changeset/nine-cups-film.md

@@ -0,0 +1,5 @@
+---
+'svelte': patch
+---
+
+fix: correctly differentiate static fields before emitting `duplicate_class_field`

packages/svelte/src/compiler/phases/2-analyze/visitors/ClassBody.js

@@ -57,7 +57,7 @@ export function ClassBody(node, context) {
 				e.state_field_duplicate(node, name);
 			}
 
-			const _key = (key.type === 'PrivateIdentifier' ? '#' : '') + name;
+			const _key = (node.type === 'AssignmentExpression' || !node.static ? '' : '@') + name;
 			const field = fields.get(_key);
 
 			// if there's already a method or assigned field, error
@@ -78,7 +78,7 @@ export function ClassBody(node, context) {
 	for (const child of node.body) {
 		if (child.type === 'PropertyDefinition' && !child.computed && !child.static) {
 			handle(child, child.key, child.value);
-			const key = (child.key.type === 'PrivateIdentifier' ? '#' : '') + get_name(child.key);
+			const key = /** @type {string} */ (get_name(child.key));
 			const field = fields.get(key);
 			if (!field) {
 				fields.set(key, [child.value ? 'assigned_prop' : 'prop']);
@@ -91,7 +91,7 @@ export function ClassBody(node, context) {
 			if (child.kind === 'constructor') {
 				constructor = child;
 			} else if (!child.computed) {
-				const key = (child.key.type === 'PrivateIdentifier' ? '#' : '') + get_name(child.key);
+				const key = (child.static ? '@' : '') + get_name(child.key);
 				const field = fields.get(key);
 				if (!field) {
 					fields.set(key, [child.kind]);

packages/svelte/tests/validator/samples/class-state-constructor-9/input.svelte.js

@@ -1,6 +1,6 @@
 export class Counter {
 	count = -1;
-
+	static count() {}
 	constructor() {
 		this.count = $state(0);
 	}