fix: ensure deps optimizer first run, fixes #8750

[patak-dev]

Description

Alternative to:

• fix(optimizer): always queue run at least once (fixes #8750) #8760

@sapphi-red thanks for the research on #8760, I think we could go with this implementation. What do you think?

---

What is the purpose of this pull request?

• Bug fix
• New Feature
• Documentation update
• Other

[netlify]

✅ Deploy Preview for vite-docs-main canceled.

Name	Link
🔨 Latest commit	f381c7d
🔍 Latest deploy log	https://app.netlify.com/sites/vite-docs-main/deploys/62b87089aee0a700095c3e36

[sapphi-red]

Nice! I didn't notice firstRunCalled.
LGTM

[patak-dev]

The windows fails looks legit, I'll check if we need to move this later (when resolving the ids, instead on buildStart)

[sapphi-red]

If we are going to do this in resolveId, IIRC this would work. (this is the first way I tried)

    resolveId(id, importer) {
      if (getDepsOptimizer(config)?.isOptimizedDepFile(id)) {
        return id
      }
      // entrypoints
      if (importer === undefined) {
        getDepsOptimizer(config)?.delayDepsOptimizerUntil(id, async () => {})
      }
    },

[patak-dev]

Oh, right, getDepsOptimizer(config)?.delayDepsOptimizerUntil(id, async () => {}) is ok because there is a debounce after the registered ids are empty. It should be an else though 🤔