Address feedback

Author: trueadm

packages/svelte/src/compiler/phases/2-analyze/index.js

@@ -201,16 +201,20 @@ export function analyze_module(ast, options) {
 		}
 	}
 
+	/** @type {import('../types').RawWarning[]} */
+	const warnings = [];
+
+	const analysis = {
+		warnings
+	};
+
 	walk(
 		/** @type {import('estree').Node} */ (ast),
-		{ scope },
+		{ scope, analysis },
 		// @ts-expect-error TODO clean this mess up
 		merge(set_scope(scopes), validation_runes_js, runes_scope_js_tweaker)
 	);
 
-	/** @type {import('../types').RawWarning[]} */
-	const warnings = [];
-
 	// If we are in runes mode, then check for possible misuses of state runes
 	for (const [, scope] of scopes) {
 		for (const [name, binding] of scope.declarations) {
@@ -608,7 +612,7 @@ const legacy_scope_tweaker = {
 	}
 };
 
-/** @type {import('zimmerframe').Visitors<import('#compiler').SvelteNode, { scope: Scope }>} */
+/** @type {import('zimmerframe').Visitors<import('#compiler').SvelteNode, { scope: Scope, analysis: { warnings: import('../types').RawWarning[] } }>} */
 const runes_scope_js_tweaker = {
 	VariableDeclarator(node, { state }) {
 		if (node.init?.type !== 'CallExpression') return;

packages/svelte/src/compiler/phases/2-analyze/validation.js

@@ -583,7 +583,14 @@ export const validation_runes_js = {
 	NewExpression(node, context) {
 		const callee = node.callee;
 
-		if (callee.type === 'ClassExpression' && context.state.scope.function_depth !== 0) {
+		const binding = callee.type === 'Identifier' ? context.state.scope.get(callee.name) : null;
+		if (
+			(callee.type === 'ClassExpression' && context.state.scope.function_depth !== 0) ||
+			(binding !== null &&
+				binding.initial !== null &&
+				binding.initial.type === 'ClassDeclaration' &&
+				binding.scope.function_depth !== 0)
+		) {
 			warn(context.state.analysis.warnings, node, context.path, 'inline-new-class');
 		}
 	}

packages/svelte/src/compiler/phases/scope.js

@@ -437,7 +437,7 @@ export function create_scopes(ast, root, allow_reactive_declarations, parent) {
 		SwitchStatement: create_block_scope,
 
 		ClassDeclaration(node, { state, next }) {
-			if (node.id) state.scope.declare(node.id, 'normal', 'const');
+			if (node.id) state.scope.declare(node.id, 'normal', 'const', node);
 			next();
 		},
 

packages/svelte/tests/validator/samples/inline-new-class-2/input.svelte

@@ -0,0 +1,6 @@
+<script>
+	class Foo {
+		foo = $state(0)
+	}
+	const a = new Foo();
+</script>

packages/svelte/tests/validator/samples/inline-new-class-2/warnings.json

@@ -0,0 +1,14 @@
+[
+	{
+		"code": "inline-new-class",
+		"message": "Creating inline classes will likely cause performance issues. Instead, declare the class at the module-level and create new instances from the class reference.",
+		"start": {
+			"column": 11,
+			"line": 5
+		},
+		"end": {
+			"column": 20,
+			"line": 5
+		}
+	}
+]