`2.0.0-beta.3` `onSettled` in `onSettled` does not execute silently

[mizulu]

Describe the bug

when using nested onSettled inside the app, the inner onSettled does not run
when using it outside the app both inner and outer are run

onSettled(()=>{
 
 onSettled(()=>{
   alert("This block does not run"
  }
})

Your Example Website or App

https://solid2.netlify.app/#solidjs-repl:TAh2XG5mRKCjCjr3tGiuZg

Steps to Reproduce the Bug or Issue

.

Expected behavior

nested onSettled should run, or fail, should not silently be ignored.

Screenshots or Videos

Platform

.

Additional context

No response

[ryansolid]

I did the nested unowned fixed. Should be fixed in signals 0.13.4.

[mizulu]

Uncaught StatusError: Potential Infinite Loop Detected.

why is this repro now shows ?
is this part of another fix?

[ryansolid]

I specifically added a check on flush but I'm not seeing the error here.. I'm not sure why. I have the exact same shape in tests.

[mizulu]

The reproduction had issues with
https://solid2.netlify.app/#solidjs-repl:jgTM6Ly8S7-ezyUs_aN1TQ
signals@0.13.4 signals@0.13.5

 Uncaught Error: Potential Infinite Loop Detected.

in >= signals@0.13.6

we get the expected flush error logged

https://solid2.netlify.app/#solidjs-repl:tQzLSyTPRBmU0JOrnwd3Fg

Uncaught Error: Cannot call flush() from inside onSettled or createTrackedEffect. flush() is not reentrant there.

[brenelz]

So this can be closed @mizulu ?

[ryansolid]

yeah I'm considering this closed.