rename

Author: Rich-Harris

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

@@ -1,7 +1,7 @@
 /** @import { Expression, Identifier } from 'estree' */
 /** @import { Context } from '../types' */
 import is_reference from 'is-reference';
-import { should_proxy_or_freeze } from '../../3-transform/client/utils.js';
+import { should_proxy } from '../../3-transform/client/utils.js';
 import * as e from '../../../errors.js';
 import * as w from '../../../warnings.js';
 import { is_rune } from '../../../../utils.js';
@@ -136,7 +136,7 @@ export function Identifier(node, context) {
 					(binding.initial?.type === 'CallExpression' &&
 						binding.initial.arguments.length === 1 &&
 						binding.initial.arguments[0].type !== 'SpreadElement' &&
-						!should_proxy_or_freeze(binding.initial.arguments[0], context.state.scope)))) ||
+						!should_proxy(binding.initial.arguments[0], context.state.scope)))) ||
 				binding.kind === 'raw_state' ||
 				binding.kind === 'derived') &&
 			// We're only concerned with reads here

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

@@ -238,7 +238,7 @@ export function is_prop_source(binding, state) {
  * @param {Expression} node
  * @param {Scope | null} scope
  */
-export function should_proxy_or_freeze(node, scope) {
+export function should_proxy(node, scope) {
 	if (
 		!node ||
 		node.type === 'Literal' ||
@@ -263,7 +263,7 @@ export function should_proxy_or_freeze(node, scope) {
 			binding.initial.type !== 'ImportDeclaration' &&
 			binding.initial.type !== 'EachBlock'
 		) {
-			return should_proxy_or_freeze(binding.initial, null);
+			return should_proxy(binding.initial, null);
 		}
 	}
 	return true;

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

@@ -3,7 +3,7 @@
 import * as b from '../../../../utils/builders.js';
 import { build_assignment_value } from '../../../../utils/ast.js';
 import { is_ignored } from '../../../../state.js';
-import { build_proxy_reassignment, should_proxy_or_freeze } from '../utils.js';
+import { build_proxy_reassignment, should_proxy } from '../utils.js';
 import { visit_assignment_expression } from '../../shared/assignments.js';
 
 /**
@@ -37,7 +37,7 @@ export function build_assignment(operator, left, right, context) {
 					context.visit(build_assignment_value(operator, left, right))
 				);
 
-				if (should_proxy_or_freeze(value, context.state.scope)) {
+				if (should_proxy(value, context.state.scope)) {
 					transformed = true;
 					value =
 						private_state.kind === 'raw_state'
@@ -54,7 +54,7 @@ export function build_assignment(operator, left, right, context) {
 		} else if (left.property.type === 'Identifier' && context.state.in_constructor) {
 			const public_state = context.state.public_state.get(left.property.name);
 
-			if (public_state !== undefined && should_proxy_or_freeze(right, context.state.scope)) {
+			if (public_state !== undefined && should_proxy(right, context.state.scope)) {
 				const value = /** @type {Expression} */ (context.visit(right));
 
 				return b.assignment(
@@ -100,7 +100,7 @@ export function build_assignment(operator, left, right, context) {
 			!is_primitive &&
 			binding.kind !== 'prop' &&
 			context.state.analysis.runes &&
-			should_proxy_or_freeze(value, context.state.scope)
+			should_proxy(value, context.state.scope)
 		) {
 			value = binding.kind === 'raw_state' ? value : build_proxy_reassignment(value, object.name);
 		}

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

@@ -5,7 +5,7 @@ import { dev, is_ignored } from '../../../../state.js';
 import * as b from '../../../../utils/builders.js';
 import { regex_invalid_identifier_chars } from '../../../patterns.js';
 import { get_rune } from '../../../scope.js';
-import { build_proxy_reassignment, should_proxy_or_freeze } from '../utils.js';
+import { build_proxy_reassignment, should_proxy } from '../utils.js';
 
 /**
  * @param {ClassBody} node
@@ -114,7 +114,7 @@ export function ClassBody(node, context) {
 						field.kind === 'state'
 							? b.call(
 									'$.source',
-									should_proxy_or_freeze(init, context.state.scope) ? b.call('$.proxy', init) : init
+									should_proxy(init, context.state.scope) ? b.call('$.proxy', init) : init
 								)
 							: field.kind === 'raw_state'
 								? b.call('$.source', init)

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

@@ -7,12 +7,7 @@ import { extract_paths } from '../../../../utils/ast.js';
 import * as b from '../../../../utils/builders.js';
 import * as assert from '../../../../utils/assert.js';
 import { get_rune } from '../../../scope.js';
-import {
-	get_prop_source,
-	is_prop_source,
-	is_state_source,
-	should_proxy_or_freeze
-} from '../utils.js';
+import { get_prop_source, is_prop_source, is_state_source, should_proxy } from '../utils.js';
 import { is_hoisted_function } from '../../utils.js';
 
 /**
@@ -86,7 +81,7 @@ export function VariableDeclaration(node, context) {
 							if (
 								initial &&
 								binding.kind === 'bindable_prop' &&
-								should_proxy_or_freeze(initial, context.state.scope)
+								should_proxy(initial, context.state.scope)
 							) {
 								initial = b.call('$.proxy', initial);
 							}
@@ -128,7 +123,7 @@ export function VariableDeclaration(node, context) {
 					const binding = /** @type {import('#compiler').Binding} */ (
 						context.state.scope.get(id.name)
 					);
-					if (rune === '$state' && should_proxy_or_freeze(value, context.state.scope)) {
+					if (rune === '$state' && should_proxy(value, context.state.scope)) {
 						value = b.call('$.proxy', value);
 					}
 					if (is_state_source(binding, context.state)) {

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

@@ -1,6 +1,6 @@
 /** @import { BinaryOperator, Expression, Identifier } from 'estree' */
 /** @import { ComponentContext, Context } from '../../types' */
-import { build_proxy_reassignment, is_state_source, should_proxy_or_freeze } from '../../utils.js';
+import { build_proxy_reassignment, is_state_source, should_proxy } from '../../utils.js';
 import * as b from '../../../../../utils/builders.js';
 
 /**