Removal of empty chunks doesn't update all imports

[matthewp]

Describe the bug

In Astro we are updating to use vite:css-post for bundling instead of doing it our own thanks to the new viteMetadata property.

One thing that this plugin does is when there is a CSS chunk that only contains CSS, the JS bundle gets removed and any imports of that bundle are removed.

It uses a regexp to find import statements to remove. That code is here. However the code assumes the import statement is using double quotes.

I'm not sure why, but in Astro these become double quotes. As far as I can tell this isn't something Rollup lets you control. I'm not sure why it is happening, perhaps because we bundle to ESM?

Updating this regex to work with either single or double quotes would fix the issue. I'm willing to submit a PR to fix.

Reproduction

https://github.com/withastro/astro/runs/6595302507?check_suite_focus=true

System Info

System:
    OS: macOS 12.0.1
    CPU: (10) x64 Apple M1 Pro
    Memory: 32.51 MB / 16.00 GB
    Shell: 5.8 - /bin/zsh
  Binaries:
    Node: 14.19.1 - ~/.volta/tools/image/node/14.19.1/bin/node
    Yarn: 1.22.18 - ~/.volta/bin/yarn
    npm: 6.14.16 - ~/.volta/tools/image/node/14.19.1/bin/npm
  Browsers:
    Chrome: 101.0.4951.64
    Safari: 15.1

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 https://github.com/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.

[sapphi-red]

It also happened when esbuild: false. https://stackblitz.com/edit/vitejs-vite-pcufvl?terminal=dev