move _op check out of the fiber hot path

Effect-TS · Oct 31, 2024 · 3457750 · 3457750
1 parent 62d365d
commit 3457750
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 5 deletions.
diff --git a/.changeset/hip-dogs-invent.md b/.changeset/hip-dogs-invent.md
@@ -0,0 +1,5 @@
+---
+"effect": patch
+---
+
+move \_op check out of the fiber hot path
diff --git a/packages/effect/src/internal/fiberRuntime.ts b/packages/effect/src/internal/fiberRuntime.ts
@@ -1330,11 +1330,6 @@ export class FiberRuntime<in out A, in out E = never> extends Effectable.Class<A
         }
       }
       try {
-        if (!("_op" in cur) || !((cur as core.Primitive)._op in this)) {
-          // @ts-expect-error
-          absurd(cur)
-        }
-
         // @ts-expect-error
         cur = this.currentTracer.context(
           () => {
@@ -1369,6 +1364,10 @@ export class FiberRuntime<in out A, in out E = never> extends Effectable.Class<A
             core.exitFailCause(internalCause.die(op))
         }
       } catch (e) {
+        if (cur !== YieldedOp && !Predicate.hasProperty(cur, "_op") || !((cur as core.Primitive)._op in this)) {
+          // @ts-expect-error
+          absurd(`Not a valid effect: ${Inspectable.toStringUnknown(cur)}`)
+        }
         if (core.isEffectError(e)) {
           cur = core.exitFailCause(e.cause)
         } else if (core.isInterruptedException(e)) {