perf: use has for effect sets

Author: Varixo

packages/qwik/src/core/reactive-primitives/cleanup.ts

@@ -46,7 +46,7 @@ export function clearAllEffects(container: Container, consumer: Consumer): void
 
 function clearSignal(container: Container, producer: SignalImpl, effect: EffectSubscription) {
   const effects = producer.$effects$;
-  if (effects) {
+  if (effects && effects.has(effect)) {
     effects.delete(effect);
   }
 
@@ -61,11 +61,11 @@ function clearAsyncComputedSignal(
   effect: EffectSubscription
 ) {
   const effects = producer.$effects$;
-  if (effects) {
+  if (effects && effects.has(effect)) {
     effects.delete(effect);
   }
   const pendingEffects = producer.$loadingEffects$;
-  if (pendingEffects) {
+  if (pendingEffects && pendingEffects.has(effect)) {
     pendingEffects.delete(effect);
   }
 }
@@ -74,7 +74,9 @@ function clearStore(producer: StoreHandler, effect: EffectSubscription) {
   const effects = producer?.$effects$;
   if (effects) {
     for (const propEffects of effects.values()) {
-      propEffects.delete(effect);
+      if (propEffects.has(effect)) {
+        propEffects.delete(effect);
+      }
     }
   }
 }

packages/qwik/src/core/reactive-primitives/utils.ts

@@ -54,13 +54,14 @@ export const ensureContainsSubscription = (
   array: Set<EffectSubscription>,
   effectSubscription: EffectSubscription
 ) => {
-  array.add(effectSubscription);
+  !array.has(effectSubscription) && array.add(effectSubscription);
 };
 
 /** Ensure the item is in back refs set */
 export const ensureContainsBackRef = (array: EffectSubscription, value: any) => {
   array[EffectSubscriptionProp.BACK_REF] ||= new Set();
-  array[EffectSubscriptionProp.BACK_REF].add(value);
+  !array[EffectSubscriptionProp.BACK_REF].has(value) &&
+    array[EffectSubscriptionProp.BACK_REF].add(value);
 };
 
 export const addQrlToSerializationCtx = (