From Discord: (bug?) TypeError when using discard option with Effect.all

[effect-bot]

Summary

Summary:

1. Issue Description:

   • User szethh encountered a TypeError when using the discard option with Effect.allWith on a record of effects. The code compiles but crashes at runtime.
   • The error does not occur if discard is set to false or omitted.

2. Workaround:

   • User whatthebluck found that using an array instead of a record works without causing the error:

     pipe(
       [Effect.succeed(1), Effect.succeed(2)],
       Effect.allWith({ discard: true }),
       Effect.runPromise
     )

3. Further Confirmation:

   • User datner_ confirmed the issue and noted that it also occurs with the more idiomatic form using Effect.all:

     Effect.all(
       { a: Effect.succeed(1) },
       { discard: true },
     )

   • The issue is not specific to allWith and happens regardless of concurrency and batching settings.
   • Using any mode besides default works as expected.

Key Takeaways:

• There appears to be a bug when using the discard option with Effect.all or Effect.allWith on a record of effects.
• The issue does not occur when using an array of effects.
• The problem is not related to concurrency or batching settings but is specific to the default mode.
• Using a different mode can serve as a temporary workaround.

Discord thread

https://discord.com/channels/795981131316985866/1249714696974766150

[datner]

We should get the bot to attach sandbox links automatically I think..

https://effect.website/play#c530ae52e62e