fix: avoid clean up while committing deps folder

[patak-dev]

Description

Reported error in Discord by @bluwy:

Error: ENOENT: no such file or directory, open '/Users/bjorn/Work/repros/vite-scss-slow/node_modules/.vite/deps_temp_f7b967e0/_metadata.json'
    at Object.openSync (node:fs:590:3)
    at Object.writeFileSync (node:fs:2202:35)
    at Object.commit (file:///Users/bjorn/Work/repros/vite-scss-slow/node_modules/.pnpm/vite@4.3.0-beta.1_sass@1.49.7/node_modules/vite/dist/node/chunks/dep-f365bad6.js:44771:18)

This cancel() could be called after the result is committed.

I added a guard to avoid this issue but also refactored the optimizer code, so the cancel wouldn't be called after this PR.

---

What is the purpose of this pull request?

• Bug fix
• New Feature
• Documentation update
• Other

[stackblitz]

Run & review this pull request in StackBlitz Codeflow.

[bluwy]

I can't seem to repro the old error before, but running this locally a few times seems to work fine too 👍 Nice refactor.