Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Bug] Version 0.13.0 breaks dependencies optimizer #35

Closed
fabiiomiiguel opened this issue Mar 20, 2023 · 14 comments · Fixed by #36
Closed

[Bug] Version 0.13.0 breaks dependencies optimizer #35

fabiiomiiguel opened this issue Mar 20, 2023 · 14 comments · Fixed by #36

Comments

@fabiiomiiguel
Copy link

fabiiomiiguel commented Mar 20, 2023

Hello, the last version 0.13.0 breaks my project and gives this error:

npm run dev
> electron-vue-vite@0.0.1 dev
> vite


  VITE v4.2.0  ready in 308 ms

  ➜  Local:   http://localhost:5173/
  ➜  Network: use --host to expose
  ➜  press h to show help
vite v4.2.0 building for development...

watching for file changes...
vite v4.2.0 building for development...

watching for file changes...
Can not resolve path: /Users/fabio/Projects/dev/electron/node_modules/set-interval-async/dynamic

build started...

build started... (x2)
Can not resolve path: /Users/fabio/Projects/dev/electron/node_modules/virtual-module:/Users/fabio/Projects/dev/electron/src/App.vue?id=0
✘ [ERROR] Could not resolve "virtual-module:/Users/fabio/Projects/dev/electron/src/App.vue?id=0"

✓ 1 modules transformed.
✓ 1 modules transformed. (x2)
Error:   Failed to scan for dependencies from entries:
  /Users/fabio/Projects/dev/electron/index.html
/Users/fabio/Projects/dev/electron/worker.html

  ✘ [ERROR] Build failed with 1 error:
error: Could not resolve "virtual-module:/Users/fabio/Projects/dev/electron/src/App.vue?id=0" [plugin vite-plugin-target:optimizer:esbuild]

    html:/Users/fabio/Projects/dev/electron/src/App.vue:1:14:
      1 │ export * from "virtual-module:/Users/fabio/Projects/dev/electron/sr...
        ╵               ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

  This error came from the "onLoad" callback registered here:

    node_modules/vite-plugin-electron-renderer/index.js:284:12:
      284 │       build.onLoad({
          ╵             ~~~~~~

    at setup (/Users/fabio/Projects/dev/electron/node_modules/vite-plugin-electron-renderer/index.js:284:13)
    at handlePlugins (/Users/fabio/Projects/dev/electron/node_modules/esbuild/lib/main.js:1279:21)
    at buildOrContextImpl (/Users/fabio/Projects/dev/electron/node_modules/esbuild/lib/main.js:968:5)
    at Object.buildOrContext (/Users/fabio/Projects/dev/electron/node_modules/esbuild/lib/main.js:776:5)
    at /Users/fabio/Projects/dev/electron/node_modules/esbuild/lib/main.js:2172:68
    at new Promise (<anonymous>)
    at Object.context (/Users/fabio/Projects/dev/electron/node_modules/esbuild/lib/main.js:2172:27)
    at Object.context (/Users/fabio/Projects/dev/electron/node_modules/esbuild/lib/main.js:2012:58)
    at prepareEsbuildScanner (file:///Users/fabio/Projects/dev/electron/node_modules/vite/dist/node/chunks/dep-c167897e.js:43542:26)


    at failureErrorWithLog (/Users/fabio/Projects/dev/electron/node_modules/esbuild/lib/main.js:1636:15)
    at /Users/fabio/Projects/dev/electron/node_modules/esbuild/lib/main.js:1048:25
    at runOnEndCallbacks (/Users/fabio/Projects/dev/electron/node_modules/esbuild/lib/main.js:1471:45)
    at buildResponseToResult (/Users/fabio/Projects/dev/electron/node_modules/esbuild/lib/main.js:1046:7)
    at /Users/fabio/Projects/dev/electron/node_modules/esbuild/lib/main.js:1058:9
    at new Promise (<anonymous>)
    at requestCallbacks.on-end (/Users/fabio/Projects/dev/electron/node_modules/esbuild/lib/main.js:1057:54)
    at handleRequest (/Users/fabio/Projects/dev/electron/node_modules/esbuild/lib/main.js:723:19)
    at handleIncomingPacket (/Users/fabio/Projects/dev/electron/node_modules/esbuild/lib/main.js:745:7)
    at Socket.readFromStdout (/Users/fabio/Projects/dev/electron/node_modules/esbuild/lib/main.js:673:7)
dist-electron/preload/index.js  7.49 kB │ gzip: 3.26 kB │ map: 3.93 kB
built in 88ms.
dist-electron/main/index.js  2.81 kB │ gzip: 1.12 kB │ map: 5.96 kB
built in 88ms.
Can not resolve path: /Users/fabio/Projects/dev/electron/node_modules/@oruga-ui_oruga-next
Can not resolve path: /Users/fabio/Projects/dev/electron/node_modules/set-interval-async_dynamic
✘ [ERROR] Could not resolve "set-interval-async_dynamic"

✘ [ERROR] Could not resolve "@oruga-ui_oruga-next"

✘ [ERROR] Build failed with 1 error:
error: Could not resolve "set-interval-async_dynamic" [plugin vite-plugin-target:optimizer:esbuild]

  This error came from the "onLoad" callback registered here:

    node_modules/vite-plugin-electron-renderer/index.js:284:12:
      284 │       build.onLoad({
          ╵             ~~~~~~

    at setup (/Users/fabio/Projects/dev/electron/node_modules/vite-plugin-electron-renderer/index.js:284:13)
    at handlePlugins (/Users/fabio/Projects/dev/electron/node_modules/esbuild/lib/main.js:1279:21)
    at buildOrContextImpl (/Users/fabio/Projects/dev/electron/node_modules/esbuild/lib/main.js:968:5)
    at Object.buildOrContext (/Users/fabio/Projects/dev/electron/node_modules/esbuild/lib/main.js:776:5)
    at /Users/fabio/Projects/dev/electron/node_modules/esbuild/lib/main.js:2172:68
    at new Promise (<anonymous>)
    at Object.context (/Users/fabio/Projects/dev/electron/node_modules/esbuild/lib/main.js:2172:27)
    at Object.context (/Users/fabio/Projects/dev/electron/node_modules/esbuild/lib/main.js:2012:58)
    at prepareEsbuildOptimizerRun (file:///Users/fabio/Projects/dev/electron/node_modules/vite/dist/node/chunks/dep-c167897e.js:44825:35)

✘ [ERROR] Build failed with 1 error:
error: Could not resolve "@oruga-ui_oruga-next" [plugin vite-plugin-target:optimizer:esbuild]

  This error came from the "onLoad" callback registered here:

    node_modules/vite-plugin-electron-renderer/index.js:284:12:
      284 │       build.onLoad({
          ╵             ~~~~~~

    at setup (/Users/fabio/Projects/dev/electron/node_modules/vite-plugin-electron-renderer/index.js:284:13)
    at handlePlugins (/Users/fabio/Projects/dev/electron/node_modules/esbuild/lib/main.js:1279:21)
    at buildOrContextImpl (/Users/fabio/Projects/dev/electron/node_modules/esbuild/lib/main.js:968:5)
    at Object.buildOrContext (/Users/fabio/Projects/dev/electron/node_modules/esbuild/lib/main.js:776:5)
    at /Users/fabio/Projects/dev/electron/node_modules/esbuild/lib/main.js:2172:68
    at new Promise (<anonymous>)
    at Object.context (/Users/fabio/Projects/dev/electron/node_modules/esbuild/lib/main.js:2172:27)
    at Object.context (/Users/fabio/Projects/dev/electron/node_modules/esbuild/lib/main.js:2012:58)
    at prepareEsbuildOptimizerRun (file:///Users/fabio/Projects/dev/electron/node_modules/vite/dist/node/chunks/dep-c167897e.js:44825:35)

11:33:29 AM [vite] error while updating dependencies:
Error: Build failed with 2 errors:
error: Build failed with 1 error:
error: Could not resolve "set-interval-async_dynamic"
error: Build failed with 1 error:
error: Could not resolve "@oruga-ui_oruga-next"
    at failureErrorWithLog (/Users/fabio/Projects/dev/electron/node_modules/esbuild/lib/main.js:1636:15)
    at /Users/fabio/Projects/dev/electron/node_modules/esbuild/lib/main.js:1048:25
    at runOnEndCallbacks (/Users/fabio/Projects/dev/electron/node_modules/esbuild/lib/main.js:1471:45)
    at buildResponseToResult (/Users/fabio/Projects/dev/electron/node_modules/esbuild/lib/main.js:1046:7)
    at /Users/fabio/Projects/dev/electron/node_modules/esbuild/lib/main.js:1058:9
    at new Promise (<anonymous>)
    at requestCallbacks.on-end (/Users/fabio/Projects/dev/electron/node_modules/esbuild/lib/main.js:1057:54)
    at handleRequest (/Users/fabio/Projects/dev/electron/node_modules/esbuild/lib/main.js:723:19)
    at handleIncomingPacket (/Users/fabio/Projects/dev/electron/node_modules/esbuild/lib/main.js:745:7)
    at Socket.readFromStdout (/Users/fabio/Projects/dev/electron/node_modules/esbuild/lib/main.js:673:7)

With the previous version 0.12.1 everything works fine.

@caoxiemeihao
Copy link
Member

Oh! can you provide a minimal reproduction repo?

@fabiiomiiguel
Copy link
Author

Yes of course. There it is a minimal version with the bug. If you downgrade to 0.12.1 this will work. I tested myself.

https://github.com/fabiiomiiguel/-vite-plugin-electron-renderer

@aphex
Copy link

aphex commented Mar 20, 2023

I am having a similar issue on windows the optimizer is breaking my build as well.

It's trying to find node_modules in the wrong path. for example the error is
Can not resolve path: F:\path\to\monorepo\project\packages\foo\node_modules\zod

However zod is not going to be there as I am using pnpm and another package within the workspace is handling zod.

I am also getting all kinds of errors trying to optimize .html files. For reference the id here

const userType = modules[id]
is a file path ending in .html. It seemed to even be finding .html files from the release folder in the root of this pacakge. I am not sure why it was even looking in there. On windows it also seems to be doubling up paths I am seeing this error as well
Can not resolve path: F:\path\to\monorepo\project\packages\foo\node_modules\F:\path\to\monorepo\project\packages\foo\index.html

Adding a return here https://github.com/electron-vite/vite-plugin-electron-renderer/blob/main/src/optimizer.ts#L81 gets me back to building.

Or simply commenting out the whole optimizer step here

optimizer(optimizerOpts, nodeIntegration),
will get it back to running.

For now I think I just need to roll back to 0.12.1 I don't see any way to opt out of this optimizer stuff.

@caoxiemeihao caoxiemeihao mentioned this issue Mar 21, 2023
@caoxiemeihao caoxiemeihao linked a pull request Mar 21, 2023 that will close this issue
caoxiemeihao added a commit that referenced this issue Mar 21, 2023
* feat: better build types

* refactor!: improve Pre-Bundling #35

* log: v0.13.1

* v0.13.1
@caoxiemeihao
Copy link
Member

Hey! everyone, v0.13.1 is out! Maybe this verion is what we would ideally like to see!

@fabiiomiiguel
Copy link
Author

Thank you, this version fixed everything from my side 🙂

@fabiiomiiguel
Copy link
Author

fabiiomiiguel commented Mar 21, 2023

So this version creates a new bug.

npm run dev
runtime-core.esm-bundler.js:3002 Uncaught TypeError: Cannot read properties of null (reading 'isCE')
    at renderSlot (runtime-core.esm-bundler.js:3002:34)
    at Button-c75f7992.mjs:190:9
    at normalizeChildren (runtime-core.esm-bundler.js:7025:42)
    at createBaseVNode (runtime-core.esm-bundler.js:6773:9)
    at _createVNode (runtime-core.esm-bundler.js:6875:12)
    at createVNodeWithArgsTransform (runtime-core.esm-bundler.js:6730:12)
    at createBlock (runtime-core.esm-bundler.js:6701:23)
    at Proxy.render5 (Button-c75f7992.mjs:171:23)
    at renderComponentRoot (/Users/fabio/Projects/dev/electron/node_modules/@vue/runtime-core/dist/runtime-core.cjs.js:906:44)
    at ReactiveEffect.componentUpdateFn [as fn] (/Users/fabio/Projects/dev/electron/node_modules/@vue/runtime-core/dist/runtime-core.cjs.js:5587:57)

I tried some solutions from here vuejs/core#4344 but didn't work.

I updated this https://github.com/fabiiomiiguel/-vite-plugin-electron-renderer so if you want, you can give a look at the issue.

Thank you in advance 😁

@caoxiemeihao
Copy link
Member

v0.13.2 is out!

@fabiiomiiguel
Copy link
Author

fabiiomiiguel commented Mar 21, 2023

Wow so fast! Sadly that brings the old error again 😔

npm run dev
✘ [ERROR] Cannot find module '@oruga-ui_oruga-next'
Require stack:
- /Users/fabio/Projects/-vite-plugin-electron-renderer/node_modules/vite-plugin-electron-renderer/index.js [plugin vite-plugin-target:optimizer:esbuild]

  This error came from the "onLoad" callback registered here:

    node_modules/vite-plugin-electron-renderer/index.js:259:12:
      259 │       build.onLoad({
          ╵             ~~~~~~

    at setup (/Users/fabio/Projects/-vite-plugin-electron-renderer/node_modules/vite-plugin-electron-renderer/index.js:259:13)
    at handlePlugins (/Users/fabio/Projects/-vite-plugin-electron-renderer/node_modules/esbuild/lib/main.js:1279:21)
    at buildOrContextImpl (/Users/fabio/Projects/-vite-plugin-electron-renderer/node_modules/esbuild/lib/main.js:968:5)
    at Object.buildOrContext (/Users/fabio/Projects/-vite-plugin-electron-renderer/node_modules/esbuild/lib/main.js:776:5)
    at /Users/fabio/Projects/-vite-plugin-electron-renderer/node_modules/esbuild/lib/main.js:2172:68
    at new Promise (<anonymous>)
    at Object.context (/Users/fabio/Projects/-vite-plugin-electron-renderer/node_modules/esbuild/lib/main.js:2172:27)
    at Object.context (/Users/fabio/Projects/-vite-plugin-electron-renderer/node_modules/esbuild/lib/main.js:2012:58)
    at prepareEsbuildOptimizerRun (file:///Users/fabio/Projects/-vite-plugin-electron-renderer/node_modules/vite/dist/node/chunks/dep-79892de8.js:44840:35)

12:32:21 PM [vite] error while updating dependencies:
Error: Build failed with 1 error:
error: Cannot find module '@oruga-ui_oruga-next'
Require stack:
- /Users/fabio/Projects/-vite-plugin-electron-renderer/node_modules/vite-plugin-electron-renderer/index.js
    at failureErrorWithLog (/Users/fabio/Projects/-vite-plugin-electron-renderer/node_modules/esbuild/lib/main.js:1636:15)
    at /Users/fabio/Projects/-vite-plugin-electron-renderer/node_modules/esbuild/lib/main.js:1048:25
    at runOnEndCallbacks (/Users/fabio/Projects/-vite-plugin-electron-renderer/node_modules/esbuild/lib/main.js:1471:45)
    at buildResponseToResult (/Users/fabio/Projects/-vite-plugin-electron-renderer/node_modules/esbuild/lib/main.js:1046:7)
    at /Users/fabio/Projects/-vite-plugin-electron-renderer/node_modules/esbuild/lib/main.js:1058:9
    at new Promise (<anonymous>)
    at requestCallbacks.on-end (/Users/fabio/Projects/-vite-plugin-electron-renderer/node_modules/esbuild/lib/main.js:1057:54)
    at handleRequest (/Users/fabio/Projects/-vite-plugin-electron-renderer/node_modules/esbuild/lib/main.js:723:19)
    at handleIncomingPacket (/Users/fabio/Projects/-vite-plugin-electron-renderer/node_modules/esbuild/lib/main.js:745:7)
    at Socket.readFromStdout (/Users/fabio/Projects/-vite-plugin-electron-renderer/node_modules/esbuild/lib/main.js:673:7)

@caoxiemeihao
Copy link
Member

caoxiemeihao commented Mar 21, 2023

Wow I has been tested v0.13.2 on my local is well 😅

@fabiiomiiguel
Copy link
Author

fabiiomiiguel commented Mar 21, 2023

I deleted the node_modules and package-lock.json, but I still have the issue in this repo https://github.com/fabiiomiiguel/-vite-plugin-electron-renderer as well on my project. 😔

@caoxiemeihao
Copy link
Member

v0.13.3 is out! 🔥

@fabiiomiiguel
Copy link
Author

fabiiomiiguel commented Mar 21, 2023

What a legend! I can confirm now everything works! Thanks and sorry to bother you so much.

@caoxiemeihao
Copy link
Member

Maybe I'm more careless. 👻

@aphex
Copy link

aphex commented Mar 21, 2023

@caoxiemeihao heck ya! working here as well 🙏

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants