perf: improve performance when spec lint [INS-3724]

[CurryYangxx]

Close #4653 #4241

Move spec lint logic to web worker, can improve both opening file and editing file

Changes:

• move spectral lint login to web worker both in the Design component and router action
  • add dedicate worker file
  • delete all the spectral in main process
• open electron nodeIntegrationInWorker option
• add vite worker plugin config so we can use electron require magic in worker
• wrap code-editor with memo to reduce rerender

How to test:

• trigger lint multiple times and only the latest task result will be updated
• use custom spectral ruleset and it it works

[jackkav]

@CurryYangxx very elegant approach. With node access in webworkers we have opened up some interesting performance options.

[jackkav]

Should we spawn a new worker on every lint command or use a shared worker?

it depends on the error handling story

Does this function handle race conditions?

no if you always spawn a new one, although since it can happen on keystroke we might need to throttle the creation of workers, or resetting a shared one.

Proper error handling from the worker and here.

the simplest way I can imagine is wrapping the worker in a promise and using abort and reject when listening on the worker error event.

Don't reload the spectral ruleset on every lint command
agreed, but perhaps we can consider this is an optimisation for later.

[CurryYangxx]

Should we spawn a new worker on every lint command or use a shared worker?

Now it will create just one worker when component first time render.And use a unique id to distinguish worker message.

Does this function handle race conditions?

Use the unique id for every message, only the result which id match the latested id will be updated.

Don't reload the spectral ruleset on every lint command

use a cache object in worker, only reload when ruleset path change

[CurryYangxx]

We still need to save spectralRun IPC because it is used in this action:

insomnia/packages/insomnia/src/ui/routes/actions.tsx

Line 736 in cc421d1

const results = (await window.main.spectralRun({ contents: apiSpec.contents, rulesetPath })).filter(isLintError);

[ihexxa]

Discussed offline and resolved several concerns, just added some minor comments. Overall it looks good to me but let's wait a bit as there might be some comments from other reviewers.

[CurryYangxx]

Discussed offline and resolved several concerns, just added some minor comments. Overall it looks good to me but let's wait a bit as there might be some comments from other reviewers.

@ihexxa Good catch, update more logs and necessary termination