翠 / green

It passes on my local. Currently I've marked as fixme. (#11) It seems the websocket isn't disconnected even after the container was stopped.

Support `file:` protocol

3

Currently file protocol (e.g. `/__open-in-editor?file=file%3A%2F%2F%2FC%3A%2Ffoo.js:13:24`) is not supported. Because using `fileURLToPath` in browser requires a polyfill, I think it would be nice if file protocol is supported.

Now Node.js recommends using `os.availableParallelism` instead of `os.cpus().length` for this purpose. https://nodejs.org/api/os.html#oscpus:~:text=os.cpus().length%20should%20not%20be%20used%20to%20calculate%20the%20amount%20of%20parallelism%20available%20to%20an%20application.%20Use%20os.availableParallelism()%20for%20this%20purpose. This PR makes the default value to use that when it's available. (Node.js 18.14.0+, 19.4.0+) https://nodejs.org/api/os.html#osavailableparallelism I also...

Improve stringify_string performance

2

This PR improves `stringify_string` performance. ### Description The ECMAScript spec specifies that `JSON.stringify` will escape `"`, `\\`, `\n`, `\r`, `\t`, `\b`, `\f`, and chars less than space. https://tc39.es/ecma262/multipage/structured-data.html#sec-quotejsonstring This is...

fix(css): page reload was not happening with .css?raw

10

### Description fixes #16396

chore: bump rollup to 4.17.0

3

### Description

Importing a script file containing `import` with `?url` does not work with build

6

### Describe the bug When the following conditions are met, it works with `vite dev` but it does not with `vite preview`. - a script (`foo.js`) contains a `import` -...

fix(hmr): watch files in workspace root

2

### Description fixes #16399 refs #15712 We need to make these values consistent. Otherwise, the cache could be different from the actual file system. https://github.com/vitejs/vite/blob/6a127d67ba953004ab10c21b50429050c7eadf11/packages/vite/src/node/server/index.ts#L479 https://github.com/vitejs/vite/blob/6a127d67ba953004ab10c21b50429050c7eadf11/packages/vite/src/node/fsUtils.ts#L134 Alternative fix for this...

test: don't skip unicode related tests anymore

1

### Description This PR reverts #15096 as the bug (https://github.com/nodejs/node/issues/48673) was fixed in node 20.12.0.