fix(vite-template): fix vite template start failed after package

[caoxiemeihao]

• I have read the contribution documentation for this project.
• I agree to follow the code of conduct that this project follows, as appropriate.
• The changes are appropriately documented (if applicable).
• The changes have sufficient test coverage (if applicable).
• The testsuite passes successfully on my local machine (if applicable).

Summarize your changes:

Closes #3714

Since we removed the copy dependencies logic in vite-plugin of v7.5.0, this will cause if (require('electron-squirrel-startup')) to fail to load.
This is because Vite's built-in @rollup/plugin-commonjs cannot handle CommonJS's require() function like Webpack in many cases. We need to make some compromises in code writing and try to use ESModule's import statement to avoid this problem.

If anyone wants to know more about this issue, I wrote an article introducing the problem of Vite building CommonJS format code. 👉🏻 https://github.com/vite-plugin/vite-plugin-commonjs/blob/main/commonjs.zh-CN.md

[BlackHole1]

LGTM

[hichemfantar]

let's merge this bad boy

[BlackHole1]

PTAL @electron/forgers

[hichemfantar]

@caoxiemeihao can you explain the changes you’ve made to the type declarations code?

[hichemfantar]

hey, this pr removed the vite-forge type declarations in forge.env.d.ts which were broken because the actual declarations file wasn't present in @electron-forge/plugin-vite/forge-vite-env for some reason

i think the packaging process is not including the forge-vite-env.d.ts file for some reason

i added the file and now the reference /// <reference types="@electron-forge/plugin-vite/forge-vite-env" /> in forge.env.d.ts seems to be working correctly

[hichemfantar]

ah looks like the vite const type declarations were fixed in 3ad7485 but haven't been released yet which fixed the forge-vite-env.d.ts issue