tidy up

Author: Rich-Harris

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

@@ -130,11 +130,6 @@ export function Identifier(node, context) {
 			context.state.expression.has_state = true;
 		}
 
-		// TODO it would be better to just bail out when we hit the ExportSpecifier node but that's
-		// not currently possibly because of our visitor merging, which I desperately want to nuke
-		const is_export_specifier =
-			/** @type {SvelteNode} */ (context.path.at(-1)).type === 'ExportSpecifier';
-
 		if (
 			context.state.analysis.runes &&
 			node !== binding.node &&
@@ -149,7 +144,6 @@ export function Identifier(node, context) {
 						!should_proxy_or_freeze(binding.initial.arguments[0], context.state.scope)))) ||
 				binding.kind === 'frozen_state' ||
 				binding.kind === 'derived') &&
-			!is_export_specifier &&
 			// We're only concerned with reads here
 			(parent.type !== 'AssignmentExpression' || parent.left !== node) &&
 			parent.type !== 'UpdateExpression'

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

@@ -66,8 +66,6 @@ export function LabeledStatement(node, context) {
 
 			context.state.reactive_statements.set(node, reactive_statement);
 
-			// Ideally this would be in the validation file, but that isn't possible because this visitor
-			// calls "next" before setting the reactive statements.
 			if (
 				reactive_statement.dependencies.length &&
 				reactive_statement.dependencies.every(

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

@@ -12,7 +12,6 @@ export function StyleDirective(node, context) {
 		e.style_directive_invalid_modifier(node);
 	}
 
-	// the case for node.value different from true is already covered by the Identifier visitor
 	if (node.value === true) {
 		// get the binding for node.name and change the binding to state
 		let binding = context.state.scope.get(node.name);