Abnormal behaviors in building worker modules.

[XiSenao]

Describe the bug

When handling worker modules, Vite uses Rollup for building, and the default target output is iife. However, an interesting issue #15305 was discovered when the target output is iife and there is an exception in handling the statement by Rollup.

// Before compiling worker.js 
const t = import.meta && import.meta.url && new URL(import.meta.url)

// After compiling worker.js
var e = typeof document < "u" ? document.currentScript : null;
const t = {
    url: e && e.src || new URL("assets/worker-mLIARZvR.js",document.baseURI).href
} && self.location.href && new URL(self.location.href);

There will definitely be an issue in worker modules because the document instance cannot be accessed within worker. It behaves correctly when the target output is es. This also creates a differentiation between the development stage and the build stage, so some additional handling may be required in this case.

Reproduction

https://stackblitz.com/edit/vitejs-vite-bryjvn?file=index.html

Steps to reproduce

pnpm install && pnpm build && pnpm preview

System Info

System:
    OS: macOS 14.1.2
    CPU: (8) arm64 Apple M1
    Memory: 48.72 MB / 16.00 GB
    Shell: 5.9 - /bin/zsh
  Binaries:
    Node: 18.12.0 - ~/.nvm/versions/node/v18.12.0/bin/node
    Yarn: 1.22.17 - /opt/homebrew/bin/yarn
    npm: 8.19.2 - ~/.nvm/versions/node/v18.12.0/bin/npm
    pnpm: 8.11.0 - ~/.nvm/versions/node/v18.12.0/bin/pnpm
    bun: 1.0.14 - ~/.bun/bin/bun
  Browsers:
    Chrome: 120.0.6099.71
    Safari: 17.1.2
  npmPackages:
    @vitejs/plugin-react-swc: ^3.5.0 => 3.5.0 
    vite: ^5.0.0 => 5.0.7

Used Package Manager

pnpm

Logs

No response

Validations

• Follow our Code of Conduct
• Read the Contributing Guidelines.
• Read the docs.
• Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
• Make sure this is a Vite issue and not a framework-specific issue. For example, if it's a Vue SFC related bug, it should likely be reported to vuejs/core instead.
• Check that this is a concrete bug. For Q&A open a GitHub Discussion or join our Discord Chat Server.
• The provided reproduction is a minimal reproducible example of the bug.

[stackblitz]

Start a new pull request in StackBlitz Codeflow.

• vitejs-vite-bryjvn (Open reproduction)

[hi-ogawa]

It sounds similar to this closed issue and the corresponding PR:

• Undefined document introduced when using import.meta.url in worker #12611
• fix(worker): worker import.meta.url should not depends on document in iife mode #12629

It looks like that PR handled the exact expression import.meta.url but referencing import.meta directly might be still broken?

[XiSenao]

Yeah, perhaps we need to do additional processing for import.meta.

[vishnukolla]

may be inconsistency in variable names . give it a try though and even import.meta.url needs to be processed furthur