Update serializeDefine implementation from upstream vite

[vasiliicuhar]

This PR updates serializeDefine to latest version from vite. It fixes compilation errors when define contains string values.

[vite:define] Transform failed with 1 error:
error: Invalid define value (must be an entity name or valid JSON syntax)

For some unknown reasons it's not working only in combination with @vitejs/plugin-react, however, the issue is with (${val}) syntax, that esbuild doesn't accept.

Environment

node@18.18.2
vite@^5.2.12
vite-plugin-web-extension@4.1.3
@vitejs/plugin-react@^4.3.0

Sample config

export default defineConfig({
  define: {
    "process.env.REACT_APP_BASEURL": '""'
  },
  plugins: [
    react(),
    webExtension({
      manifest: generateManifest,
    }),
  ]
})

[aklinker1]

Interesting, must have broken on some new version of vite? Last I checked, you still had to define string constants like so '"some text"', and JSON.stringify did that for you...

That and skipping the final comma seen to be the only changes?

[aklinker1]

Oh wait, I see. This is for define.__DEFINES__ not define. That makes sense. Thanks for the PR, I'll get this deployed!