Skip to content

Commit 32a1453

Browse files
fix: include method definitions in class private fields (#14365)
* fix: include method definitions in class private fields * Update packages/svelte/tests/runtime-runes/samples/class-disabinguate-private-method-definition/_config.js --------- Co-authored-by: Rich Harris <[email protected]>
1 parent ce47131 commit 32a1453

File tree

4 files changed

+17
-1
lines changed

4 files changed

+17
-1
lines changed

.changeset/unlucky-icons-sit.md

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,5 @@
1+
---
2+
'svelte': patch
3+
---
4+
5+
fix: include method definitions in class private fields

packages/svelte/src/compiler/phases/3-transform/client/visitors/ClassBody.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -28,7 +28,7 @@ export function ClassBody(node, context) {
2828

2929
for (const definition of node.body) {
3030
if (
31-
definition.type === 'PropertyDefinition' &&
31+
(definition.type === 'PropertyDefinition' || definition.type === 'MethodDefinition') &&
3232
(definition.key.type === 'Identifier' ||
3333
definition.key.type === 'PrivateIdentifier' ||
3434
definition.key.type === 'Literal')
Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,3 @@
1+
import { test } from '../../test';
2+
3+
export default test({});
Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,8 @@
1+
<script>
2+
class Repo {
3+
tree = $state();
4+
async #tree() {
5+
}
6+
}
7+
const repo = new Repo();
8+
</script>

0 commit comments

Comments
 (0)