Data races in sample after parallel store update calls

[zerobias]

What is the current behavior:

In a complex edge case with two attach with arrays in source old store value is leaking. Curiously, this could be fixed when one of targets is effect, not event

export const initApp = createEvent<number>();
export const reInitApp = createEvent<number>();
export const startRequest = createEvent();
export const resetApp = createEvent();

export const $ownerId = createStore<number>(-1);

const requestFx = createEffect((value) => {
  console.log("ownerId in calling requestFx: ", value, $ownerId.getState());
});

$ownerId.reset(resetApp)

export const fetchFx = createEffect(() => ({}));

export const fetchMoreAFx = attach({effect: fetchFx, source: [$ownerId]});
export const fetchMoreBFx = attach({effect: fetchFx, source: [$ownerId]});

sample({
  clock: startRequest,
  source: [$ownerId], // <--- incorrect value
  fn: ([ownerId]) => {
    console.log(ownerId, $ownerId.getState())
    return ownerId
  },
  target: requestFx
});

sample({
  clock: reInitApp,
  source: $ownerId,
  filter: (ownerId, newOwnerId) => ownerId !== newOwnerId,
  fn: (_, payload) => payload,
  target: [resetApp, initApp]
});

$ownerId.on(initApp, (_, ownerId) => ownerId);

initApp(1);
reInitApp(3);
startRequest();

Bug

No bug

[zerobias]

Looks like fixing it may be a breaking change so scheduled it for effector 24