[✨] Create a lint rule for sync$()

[mhevery]

Is your feature request related to a problem?

Qwik now has a new feature, sync$(), however, it comes with limitations. It would be great if there was a lint rule which could flag when the developer is using it improperly

Describe the solution you'd like

A lint rule qwik/sync$ which would flag whenever the function inside the sync$() would close over any variables (other than browser API) and flag it.

Describe alternatives you've considered

We will create a Rust check as well, but that one will not be instant, and can't provide that much detail.

Additional context

import {foo} from '...

<button onClick$(sync$(() => {
    foo(); // <= lint error. Can't close over 'foo'
  }))>
    click
  </button>

[Lemon2311]

Hello, I`d like to fix this issue.

[gioboa]

Thanks @Lemon2311 🎉 if you need an help don't hesitate to contact me 👍

[ScriptShah]

hello, i am new to Open Source i'd like to work on this please assign me.

[gioboa]

We moved this issue to qwik-evolution repo to create a RFC discussion for this.
Here is our Qwik RFC process thanks.