effect 2.3.0 → 2.3.2

package/src/Schedule.ts

@@ -160,11 +160,11 @@ export const addDelay: {
  */
 export const addDelayEffect: {
   <Out, Env2>(
-    f: (out: Out) => Effect.Effect<Env2, never, Duration.DurationInput>
+    f: (out: Out) => Effect.Effect<Duration.DurationInput, never, Env2>
   ): <Env, In>(self: Schedule<Env, In, Out>) => Schedule<Env2 | Env, In, Out>
   <Env, In, Out, Env2>(
     self: Schedule<Env, In, Out>,
-    f: (out: Out) => Effect.Effect<Env2, never, Duration.DurationInput>
+    f: (out: Out) => Effect.Effect<Duration.DurationInput, never, Env2>
   ): Schedule<Env | Env2, In, Out>
 } = internal.addDelayEffect
 
@@ -268,11 +268,11 @@ export const check: {
  */
 export const checkEffect: {
   <In, Out, Env2>(
-    test: (input: In, output: Out) => Effect.Effect<Env2, never, boolean>
+    test: (input: In, output: Out) => Effect.Effect<boolean, never, Env2>
   ): <Env>(self: Schedule<Env, In, Out>) => Schedule<Env2 | Env, In, Out>
   <Env, In, Out, Env2>(
     self: Schedule<Env, In, Out>,
-    test: (input: In, output: Out) => Effect.Effect<Env2, never, boolean>
+    test: (input: In, output: Out) => Effect.Effect<boolean, never, Env2>
   ): Schedule<Env | Env2, In, Out>
 } = internal.checkEffect
 
@@ -390,11 +390,11 @@ export const mapInputContext: {
  */
 export const mapInputEffect: {
   <In, Env2, In2>(
-    f: (in2: In2) => Effect.Effect<Env2, never, In>
+    f: (in2: In2) => Effect.Effect<In, never, Env2>
   ): <Env, Out>(self: Schedule<Env, In, Out>) => Schedule<Env2 | Env, In2, Out>
   <Env, In, Out, Env2, In2>(
     self: Schedule<Env, In, Out>,
-    f: (in2: In2) => Effect.Effect<Env2, never, In>
+    f: (in2: In2) => Effect.Effect<In, never, Env2>
   ): Schedule<Env | Env2, In2, Out>
 } = internal.mapInputEffect
 
@@ -469,11 +469,11 @@ export const delayed: {
  */
 export const delayedEffect: {
   <Env2>(
-    f: (duration: Duration.Duration) => Effect.Effect<Env2, never, Duration.DurationInput>
+    f: (duration: Duration.Duration) => Effect.Effect<Duration.DurationInput, never, Env2>
   ): <Env, In, Out>(self: Schedule<Env, In, Out>) => Schedule<Env2 | Env, In, Out>
   <Env, In, Out, Env2>(
     self: Schedule<Env, In, Out>,
-    f: (duration: Duration.Duration) => Effect.Effect<Env2, never, Duration.DurationInput>
+    f: (duration: Duration.Duration) => Effect.Effect<Duration.DurationInput, never, Env2>
   ): Schedule<Env | Env2, In, Out>
 } = internal.delayedEffect
 
@@ -528,15 +528,15 @@ export const mapBoth: {
 export const mapBothEffect: {
   <In2, Env2, In, Out, Env3, Out2>(
     options: {
-      readonly onInput: (input: In2) => Effect.Effect<Env2, never, In>
-      readonly onOutput: (out: Out) => Effect.Effect<Env3, never, Out2>
+      readonly onInput: (input: In2) => Effect.Effect<In, never, Env2>
+      readonly onOutput: (out: Out) => Effect.Effect<Out2, never, Env3>
     }
   ): <Env>(self: Schedule<Env, In, Out>) => Schedule<Env2 | Env3 | Env, In2, Out2>
   <Env, In, Out, In2, Env2, Env3, Out2>(
     self: Schedule<Env, In, Out>,
     options: {
-      readonly onInput: (input: In2) => Effect.Effect<Env2, never, In>
-      readonly onOutput: (out: Out) => Effect.Effect<Env3, never, Out2>
+      readonly onInput: (input: In2) => Effect.Effect<In, never, Env2>
+      readonly onOutput: (out: Out) => Effect.Effect<Out2, never, Env3>
     }
   ): Schedule<Env | Env2 | Env3, In2, Out2>
 } = internal.mapBothEffect
@@ -822,11 +822,11 @@ export const map: {
  */
 export const mapEffect: {
   <Out, Env2, Out2>(
-    f: (out: Out) => Effect.Effect<Env2, never, Out2>
+    f: (out: Out) => Effect.Effect<Out2, never, Env2>
   ): <Env, In>(self: Schedule<Env, In, Out>) => Schedule<Env2 | Env, In, Out2>
   <Env, In, Out, Env2, Out2>(
     self: Schedule<Env, In, Out>,
-    f: (out: Out) => Effect.Effect<Env2, never, Out2>
+    f: (out: Out) => Effect.Effect<Out2, never, Env2>
   ): Schedule<Env | Env2, In, Out2>
 } = internal.mapEffect
 
@@ -1026,12 +1026,12 @@ export const reduce: {
 export const reduceEffect: {
   <Out, Env1, Z>(
     zero: Z,
-    f: (z: Z, out: Out) => Effect.Effect<Env1, never, Z>
+    f: (z: Z, out: Out) => Effect.Effect<Z, never, Env1>
   ): <Env, In>(self: Schedule<Env, In, Out>) => Schedule<Env1 | Env, In, Z>
   <Env, In, Out, Env1, Z>(
     self: Schedule<Env, In, Out>,
     zero: Z,
-    f: (z: Z, out: Out) => Effect.Effect<Env1, never, Z>
+    f: (z: Z, out: Out) => Effect.Effect<Z, never, Env1>
   ): Schedule<Env | Env1, In, Z>
 } = internal.reduceEffect
 
@@ -1151,11 +1151,11 @@ export const sync: <A>(evaluate: LazyArg<A>) => Schedule<never, unknown, A> = in
  */
 export const tapInput: {
   <Env2, In2, X>(
-    f: (input: In2) => Effect.Effect<Env2, never, X>
+    f: (input: In2) => Effect.Effect<X, never, Env2>
   ): <Env, In, Out>(self: Schedule<Env, In, Out>) => Schedule<Env2 | Env, In & In2, Out>
   <Env, In, Out, Env2, In2, X>(
     self: Schedule<Env, In, Out>,
-    f: (input: In2) => Effect.Effect<Env2, never, X>
+    f: (input: In2) => Effect.Effect<X, never, Env2>
   ): Schedule<Env | Env2, In & In2, Out>
 } = internal.tapInput
 
@@ -1168,11 +1168,11 @@ export const tapInput: {
  */
 export const tapOutput: {
   <Out, XO extends Out, Env2, X>(
-    f: (out: XO) => Effect.Effect<Env2, never, X>
+    f: (out: XO) => Effect.Effect<X, never, Env2>
   ): <Env, In>(self: Schedule<Env, In, Out>) => Schedule<Env2 | Env, In, Out>
   <Env, In, Out, XO extends Out, Env2, X>(
     self: Schedule<Env, In, Out>,
-    f: (out: XO) => Effect.Effect<Env2, never, X>
+    f: (out: XO) => Effect.Effect<X, never, Env2>
   ): Schedule<Env | Env2, In, Out>
 } = internal.tapOutput
 
@@ -1243,11 +1243,11 @@ export const untilInput: {
  */
 export const untilInputEffect: {
   <In, Env2>(
-    f: (input: In) => Effect.Effect<Env2, never, boolean>
+    f: (input: In) => Effect.Effect<boolean, never, Env2>
   ): <Env, Out>(self: Schedule<Env, In, Out>) => Schedule<Env2 | Env, In, Out>
   <Env, In, Out, Env2>(
     self: Schedule<Env, In, Out>,
-    f: (input: In) => Effect.Effect<Env2, never, boolean>
+    f: (input: In) => Effect.Effect<boolean, never, Env2>
   ): Schedule<Env | Env2, In, Out>
 } = internal.untilInputEffect
 
@@ -1272,11 +1272,11 @@ export const untilOutput: {
  */
 export const untilOutputEffect: {
   <Out, Env2>(
-    f: (out: Out) => Effect.Effect<Env2, never, boolean>
+    f: (out: Out) => Effect.Effect<boolean, never, Env2>
   ): <Env, In>(self: Schedule<Env, In, Out>) => Schedule<Env2 | Env, In, Out>
   <Env, In, Out, Env2>(
     self: Schedule<Env, In, Out>,
-    f: (out: Out) => Effect.Effect<Env2, never, boolean>
+    f: (out: Out) => Effect.Effect<boolean, never, Env2>
   ): Schedule<Env | Env2, In, Out>
 } = internal.untilOutputEffect
 
@@ -1312,11 +1312,11 @@ export const whileInput: {
  */
 export const whileInputEffect: {
   <In, Env2>(
-    f: (input: In) => Effect.Effect<Env2, never, boolean>
+    f: (input: In) => Effect.Effect<boolean, never, Env2>
   ): <Env, Out>(self: Schedule<Env, In, Out>) => Schedule<Env2 | Env, In, Out>
   <Env, In, Out, Env2>(
     self: Schedule<Env, In, Out>,
-    f: (input: In) => Effect.Effect<Env2, never, boolean>
+    f: (input: In) => Effect.Effect<boolean, never, Env2>
   ): Schedule<Env | Env2, In, Out>
 } = internal.whileInputEffect
 
@@ -1341,11 +1341,11 @@ export const whileOutput: {
  */
 export const whileOutputEffect: {
   <Out, Env1>(
-    f: (out: Out) => Effect.Effect<Env1, never, boolean>
+    f: (out: Out) => Effect.Effect<boolean, never, Env1>
   ): <Env, In>(self: Schedule<Env, In, Out>) => Schedule<Env1 | Env, In, Out>
   <Env, In, Out, Env1>(
     self: Schedule<Env, In, Out>,
-    f: (out: Out) => Effect.Effect<Env1, never, boolean>
+    f: (out: Out) => Effect.Effect<boolean, never, Env1>
   ): Schedule<Env | Env1, In, Out>
 } = internal.whileOutputEffect
 

package/src/Scheduler.ts

@@ -8,7 +8,6 @@ import type { FiberRef } from "./FiberRef.js"
 import { dual } from "./Function.js"
 import { globalValue } from "./GlobalValue.js"
 import * as core from "./internal/core.js"
-import * as timeout from "./internal/timeout.js"
 
 /**
  * @since 2.0.0
@@ -107,7 +106,7 @@ export class MixedScheduler implements Scheduler {
    */
   private starve(depth = 0) {
     if (depth >= this.maxNextTickBeforeTimer) {
-      timeout.set(() => this.starveInternal(0), 0)
+      setTimeout(() => this.starveInternal(0), 0)
     } else {
       Promise.resolve(void 0).then(() => this.starveInternal(depth + 1))
     }
@@ -337,14 +336,14 @@ export const makeBatched = (
  * @category constructors
  */
 export const timer = (ms: number, shouldYield: Scheduler["shouldYield"] = defaultShouldYield) =>
-  make((task) => timeout.set(task, ms), shouldYield)
+  make((task) => setTimeout(task, ms), shouldYield)
 
 /**
  * @since 2.0.0
  * @category constructors
  */
 export const timerBatched = (ms: number, shouldYield: Scheduler["shouldYield"] = defaultShouldYield) =>
-  makeBatched((task) => timeout.set(task, ms), shouldYield)
+  makeBatched((task) => setTimeout(task, ms), shouldYield)
 
 /** @internal */
 export const currentScheduler: FiberRef<Scheduler> = globalValue(

package/src/internal/clock.ts

@@ -5,7 +5,6 @@ import type * as Effect from "../Effect.js"
 import * as Either from "../Either.js"
 import { constFalse } from "../Function.js"
 import * as core from "./core.js"
-import * as timeout from "./timeout.js"
 
 /** @internal */
 const ClockSymbolKey = "effect/Clock"
@@ -29,12 +28,12 @@ export const globalClockScheduler: Clock.ClockScheduler = {
       return constFalse
     }
     let completed = false
-    const handle = timeout.set(() => {
+    const handle = setTimeout(() => {
       completed = true
       task()
     }, millis)
     return () => {
-      timeout.clear(handle)
+      clearTimeout(handle)
       return !completed
     }
   }

package/src/internal/fiberRuntime.ts

@@ -2771,7 +2771,12 @@ export const zipLeftOptions = dual<
   ) => Effect.Effect<A, E | E2, R | R2>
 >(
   (args) => core.isEffect(args[1]),
-  (self, that, options) => zipWithOptions(self, that, (a, _) => a, options)
+  (self, that, options) => {
+    if (options?.concurrent !== true && (options?.batching === undefined || options.batching === false)) {
+      return core.zipLeft(self, that)
+    }
+    return zipWithOptions(self, that, (a, _) => a, options)
+  }
 )
 
 /** @internal */
@@ -2794,11 +2799,16 @@ export const zipRightOptions: {
 } = dual((args) => core.isEffect(args[1]), <A, E, R, A2, E2, R2>(
   self: Effect.Effect<A, E, R>,
   that: Effect.Effect<A2, E2, R2>,
-  options: {
+  options?: {
     readonly concurrent?: boolean | undefined
     readonly batching?: boolean | "inherit" | undefined
   }
-): Effect.Effect<A2, E2 | E, R2 | R> => zipWithOptions(self, that, (_, b) => b, options))
+): Effect.Effect<A2, E2 | E, R2 | R> => {
+  if (options?.concurrent !== true && (options?.batching === undefined || options.batching === false)) {
+    return core.zipRight(self, that)
+  }
+  return zipWithOptions(self, that, (_, b) => b, options)
+})
 
 /** @internal */
 export const zipWithOptions: {

package/src/internal/version.ts

@@ -1 +1 @@
-export const moduleVersion = "2.3.0"
+export const moduleVersion = "2.3.2"

package/dist/cjs/internal/timeout.js

@@ -1,24 +0,0 @@
-"use strict";
-
-Object.defineProperty(exports, "__esModule", {
-  value: true
-});
-exports.set = exports.clear = void 0;
-/**
- * Bun currently has a bug where `setTimeout` doesn't behave correctly with a 0ms delay.
- *
- * @see https://github.com/oven-sh/bun/issues/3333
- */
-/** @internal */
-const isBun = typeof process === "undefined" ? false : !!process?.isBun;
-/** @internal */
-const clear = exports.clear = isBun ? id => clearInterval(id) : id => clearTimeout(id);
-/** @internal */
-const set = exports.set = isBun ? (fn, ms) => {
-  const id = setInterval(() => {
-    fn();
-    clearInterval(id);
-  }, ms);
-  return id;
-} : (fn, ms) => setTimeout(fn, ms);
-//# sourceMappingURL=timeout.js.map

package/dist/cjs/internal/timeout.js.map

@@ -1 +0,0 @@
-{"version":3,"file":"timeout.js","names":["isBun","process","clear","exports","id","clearInterval","clearTimeout","set","fn","ms","setInterval","setTimeout"],"sources":["../../../src/internal/timeout.ts"],"sourcesContent":[null],"mappings":";;;;;;AAAA;;;;;AAMA;AACA,MAAMA,KAAK,GAAG,OAAOC,OAAO,KAAK,WAAW,GAAG,KAAK,GAAG,CAAC,CAAGA,OAAe,EAAED,KAAM;AAElF;AACO,MAAME,KAAK,GAAAC,OAAA,CAAAD,KAAA,GAAiCF,KAAK,GAAII,EAAE,IAAKC,aAAa,CAACD,EAAE,CAAC,GAAIA,EAAE,IAAKE,YAAY,CAACF,EAAE,CAAC;AAE/G;AACO,MAAMG,GAAG,GAAAJ,OAAA,CAAAI,GAAA,GAAmDP,KAAK,GACtE,CAACQ,EAAc,EAAEC,EAAU,KAAI;EAC7B,MAAML,EAAE,GAAGM,WAAW,CAAC,MAAK;IAC1BF,EAAE,EAAE;IACJH,aAAa,CAACD,EAAE,CAAC;EACnB,CAAC,EAAEK,EAAE,CAAC;EAEN,OAAOL,EAAE;AACX,CAAC,GACD,CAACI,EAAc,EAAEC,EAAU,KAAKE,UAAU,CAACH,EAAE,EAAEC,EAAE,CAAC"}

package/dist/dts/internal/timeout.d.ts

@@ -1,7 +0,0 @@
-/**
- * Bun currently has a bug where `setTimeout` doesn't behave correctly with a 0ms delay.
- *
- * @see https://github.com/oven-sh/bun/issues/3333
- */
-export {};
-//# sourceMappingURL=timeout.d.ts.map

package/dist/dts/internal/timeout.d.ts.map

@@ -1 +0,0 @@
-{"version":3,"file":"timeout.d.ts","sourceRoot":"","sources":["../../../src/internal/timeout.ts"],"names":[],"mappings":"AAAA;;;;GAIG"}

package/dist/esm/internal/timeout.js

@@ -1,18 +0,0 @@
-/**
- * Bun currently has a bug where `setTimeout` doesn't behave correctly with a 0ms delay.
- *
- * @see https://github.com/oven-sh/bun/issues/3333
- */
-/** @internal */
-const isBun = typeof process === "undefined" ? false : !!process?.isBun;
-/** @internal */
-export const clear = isBun ? id => clearInterval(id) : id => clearTimeout(id);
-/** @internal */
-export const set = isBun ? (fn, ms) => {
-  const id = setInterval(() => {
-    fn();
-    clearInterval(id);
-  }, ms);
-  return id;
-} : (fn, ms) => setTimeout(fn, ms);
-//# sourceMappingURL=timeout.js.map

package/dist/esm/internal/timeout.js.map

@@ -1 +0,0 @@
-{"version":3,"file":"timeout.js","names":["isBun","process","clear","id","clearInterval","clearTimeout","set","fn","ms","setInterval","setTimeout"],"sources":["../../../src/internal/timeout.ts"],"sourcesContent":[null],"mappings":"AAAA;;;;;AAMA;AACA,MAAMA,KAAK,GAAG,OAAOC,OAAO,KAAK,WAAW,GAAG,KAAK,GAAG,CAAC,CAAGA,OAAe,EAAED,KAAM;AAElF;AACA,OAAO,MAAME,KAAK,GAAiCF,KAAK,GAAIG,EAAE,IAAKC,aAAa,CAACD,EAAE,CAAC,GAAIA,EAAE,IAAKE,YAAY,CAACF,EAAE,CAAC;AAE/G;AACA,OAAO,MAAMG,GAAG,GAAmDN,KAAK,GACtE,CAACO,EAAc,EAAEC,EAAU,KAAI;EAC7B,MAAML,EAAE,GAAGM,WAAW,CAAC,MAAK;IAC1BF,EAAE,EAAE;IACJH,aAAa,CAACD,EAAE,CAAC;EACnB,CAAC,EAAEK,EAAE,CAAC;EAEN,OAAOL,EAAE;AACX,CAAC,GACD,CAACI,EAAc,EAAEC,EAAU,KAAKE,UAAU,CAACH,EAAE,EAAEC,EAAE,CAAC"}

package/src/internal/timeout.ts

@@ -1,23 +0,0 @@
-/**
- * Bun currently has a bug where `setTimeout` doesn't behave correctly with a 0ms delay.
- *
- * @see https://github.com/oven-sh/bun/issues/3333
- */
-
-/** @internal */
-const isBun = typeof process === "undefined" ? false : !!((process as any)?.isBun)
-
-/** @internal */
-export const clear: (id: NodeJS.Timeout) => void = isBun ? (id) => clearInterval(id) : (id) => clearTimeout(id)
-
-/** @internal */
-export const set: (fn: () => void, ms: number) => NodeJS.Timeout = isBun ?
-  (fn: () => void, ms: number) => {
-    const id = setInterval(() => {
-      fn()
-      clearInterval(id)
-    }, ms)
-
-    return id
-  } :
-  (fn: () => void, ms: number) => setTimeout(fn, ms)