Rule prefer-map-over-sample

[igorkamyshev]

I found such code:

const sendEvent = sample({ clock: somethingDone, fn: (payload) => mapper(payload) })

In my opinion, it should be rewritten in this way:

const sendEvent = somethingDone.map((payload) => mapper(payload))

[Kelin2025]

I'm not sure if this restriction is good.
Honestly, if time machines existed, I'd probably go back and remove all dot methods from effector units, instead make only operators (like map(unit, data => ...))

1. To make it more consistent (everything through operators rather than switching between two notations)
2. To have more control over codebase in terms of parsing (e.g. babel can always check if map() is imported from effector, but you have no guarantee that store in store.map is actually store). As I remember, @zerobias pointed that out as well
   In the example above, sample might be verbose but you don't need to rewrite the whole thing when you need to add store (you just add source instead of rewriting it)

---

Speaking of this - I'd ban forward({ from, to }) operator, and forced sample({ clock, target }) instead. The only problem is that forward has different execution priority rather than sample

[igorkamyshev]

Yeah, you are right. Could you please open issue about forward({ from, to }) -> sample({ clock, target })?