cleanup

Author: trueadm

packages/svelte/src/internal/client/runtime.js

@@ -1454,13 +1454,28 @@ function proxify_object(signal, value, handler, path) {
 	return proxy;
 }
 
+/**
+ * @param {any} value
+ */
+function is_primitive_or_state_object(value) {
+	const type = typeof value;
+	return (
+		value == null ||
+		type === 'string' ||
+		type === 'number' ||
+		type === 'boolean' ||
+		type === 'symbol' ||
+		type === 'bigint' ||
+		STATE_SYMBOL in value
+	);
+}
+
 /**
  * @template V
  * @param {import("./types.js").ComputationSignal<V>} signal
  * @param {V} derived_value
  * @returns {V}
  */
-/*#__NO_SIDE_EFFECTS__*/
 function create_derived_proxy(signal, derived_value) {
 	const handler = {
 		/**
@@ -1484,20 +1499,11 @@ function create_derived_proxy(signal, derived_value) {
 				keys.has(prop) &&
 				is_last_current_dependency(signal)
 			) {
-				const type = typeof value;
+				// Avoid generating the new_path array until we know it needed to avoid CPU cycles.
 				let new_path;
 				// We only track paths for primitives or state objects, to avoid tracking objects
 				// that likely change all the time.
-				if (
-					value === void 0 ||
-					type === 'string' ||
-					type === 'number' ||
-					type === 'boolean' ||
-					type === 'symbol' ||
-					type === 'bigint' ||
-					type === null ||
-					STATE_SYMBOL in value
-				) {
+				if (is_primitive_or_state_object(value)) {
 					new_path = [...path, prop];
 					if (current_derived_proxy_property !== null) {
 						capture_fine_grain_derived_property();