perf(agera): speed up effect runs by splitting warmup and rerun paths (#178)

Split `runEffect` into small `warmupEffect` and `runEffect` specializations
so V8 can inline them into the effect creation and flush paths, and gate the
`noMount` and `notifyMounted` lifecycle hooks behind `isMountableUsed`.
Effect benchmark improves from ~345 to ~320 ns/op (~7% faster).
Bump size limits by 10-20 B in `agera`, `kida` and `store` to account for
the duplicated run bodies.

Author: dangreen

packages/agera/.size-limit.json

@@ -3,7 +3,7 @@
     "name": "All publics",
     "path": "dist/index.js",
     "import": "*",
-    "limit": "2.46 kB"
+    "limit": "2.47 kB"
   },
   {
     "name": "Signal",
@@ -21,6 +21,6 @@
     "name": "Popular set",
     "path": "dist/index.js",
     "import": "{ signal, computed, effect, mountable, onMounted }",
-    "limit": "1.87 kB"
+    "limit": "1.88 kB"
   }
 ]

packages/agera/src/effect.spec.ts

@@ -459,7 +459,7 @@ describe('agera', () => {
 
       $num(1)
 
-      expect(fn).toHaveBeenCalledWith(undefined)
+      expect(fn).toHaveBeenLastCalledWith()
 
       stop()
     })

packages/agera/src/internals/system.ts

@@ -523,7 +523,7 @@ export function effect(fn: EffectCallback, noDefer = false): Destroy {
     }
   }
 
-  runEffect(e, true)
+  warmupEffect(e)
 
   return effectOper.bind(e)
 }
@@ -654,22 +654,41 @@ function updateSignal(s: SignalNode): boolean {
   return s.value !== (s.value = s.pendingValue)
 }
 
-function runEffect(e: EffectNode, warmup?: true): void {
+function warmupEffect(e: EffectNode): void {
   const prevSub = pushActiveSub(e)
   const prevNoMount = isMountableUsed ? pushNoMount(e.noMount) : undefined
 
   try {
-    e.destroy = e.fn(warmup) || undefined
+    e.destroy = e.fn(true) || undefined
   } finally {
-    popNoMount(prevNoMount)
     popActiveSub(prevSub)
     e.flags &= ~RecursedCheckFlag
 
-    if (warmup === undefined) {
-      purgeDeps(e)
+    if (isMountableUsed) {
+      popNoMount(prevNoMount)
+      notifyMounted(activeSub)
     }
+  }
+}
 
-    notifyMounted(activeSub)
+function runEffect(e: EffectNode): void {
+  const prevSub = pushActiveSub(e)
+  const prevNoMount = isMountableUsed ? pushNoMount(e.noMount) : undefined
+
+  try {
+    e.destroy = e.fn() || undefined
+  } finally {
+    if (isMountableUsed) {
+      popNoMount(prevNoMount)
+    }
+
+    popActiveSub(prevSub)
+    e.flags &= ~RecursedCheckFlag
+    purgeDeps(e)
+
+    if (isMountableUsed) {
+      notifyMounted(activeSub)
+    }
   }
 }
 
@@ -840,7 +859,7 @@ function runDeferredEffects(link: Link): void {
           runDeferredEffects(dep.deps)
         }
       } else {
-        runEffect(dep as EffectNode, true)
+        warmupEffect(dep as EffectNode)
       }
     }
 

packages/kida/.size-limit.json

@@ -15,6 +15,6 @@
     "name": "Popular set",
     "path": "dist/index.js",
     "import": "{ signal, record, computed, effect, mountable, onMount }",
-    "limit": "2.21 kB"
+    "limit": "2.23 kB"
   }
 ]

packages/store/.size-limit.json

@@ -15,6 +15,6 @@
     "name": "Popular set",
     "path": "dist/index.js",
     "import": "{ signal, record, computed, effect, mountable, onMount }",
-    "limit": "2.21 kB"
+    "limit": "2.23 kB"
   }
 ]