-
Notifications
You must be signed in to change notification settings - Fork 2
feat: Dev to alpha #159
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
feat: Dev to alpha #159
Changes from all commits
fbc6f5f
e30ac85
a4bc31c
76dd080
a4e738e
c4c7dc4
b713e6d
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -48,10 +48,10 @@ | |
| "version:prerelease": "tsx scripts/version-manager.ts prerelease", | ||
| "version:beta": "tsx scripts/version-manager.ts minor beta", | ||
| "version:beta-patch": "tsx scripts/version-manager.ts patch beta", | ||
| "release": "npm run ffmpeg:download-all && npm run build:release && electron-builder --publish onTagOrDraft", | ||
| "release:all": "npm run ffmpeg:download-all && npm run build:release && electron-builder --publish always", | ||
| "release:never": "npm run ffmpeg:download-all && npm run build:release && electron-builder --publish never", | ||
| "release:draft": "npm run ffmpeg:download-all && npm run build:release && electron-builder --publish onTagOrDraft", | ||
| "release": "npm run build:release && electron-builder --publish onTagOrDraft", | ||
| "release:all": "npm run build:release && electron-builder --publish always", | ||
| "release:never": "npm run build:release && electron-builder --publish never", | ||
| "release:draft": "npm run build:release && electron-builder --publish onTagOrDraft", | ||
| "migrate": "tsx src/main/db/migration-cli.ts", | ||
|
Comment on lines
+51
to
55
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. 🧹 Nitpick (assertive) Release scripts simplified — ensure build commands are consistent across OS targets Good removal of FFmpeg pre-steps. For consistency and type safety, consider aligning platform builds to run the same typecheck + build sequence used by -"build:mac": "electron-vite build && electron-builder --mac --publish never",
+"build:mac": "npm run build && electron-builder --mac --publish never",
-"build:linux": "electron-vite build && electron-builder --linux --publish never",
+"build:linux": "npm run build && electron-builder --linux --publish never",Confirm CI still publishes drafts correctly with
|
||
| "migrate:up": "npm run migrate up", | ||
| "migrate:down": "npm run migrate down", | ||
|
|
@@ -71,9 +71,7 @@ | |
| "ffmpeg:download": "tsx scripts/download-ffmpeg.ts current", | ||
| "ffmpeg:download-all": "tsx scripts/download-ffmpeg.ts all", | ||
| "ffmpeg:clean": "tsx scripts/download-ffmpeg.ts clean", | ||
| "ffmpeg:test": "tsx scripts/test-ffmpeg-integration.ts", | ||
| "prebuild": "npm run ffmpeg:download", | ||
| "prebuild:release": "echo 'FFmpeg already downloaded by release script'" | ||
| "ffmpeg:test": "tsx scripts/test-ffmpeg-integration.ts" | ||
| }, | ||
| "dependencies": { | ||
| "@ant-design/icons": "^6.0.1", | ||
|
|
||
| Original file line number | Diff line number | Diff line change | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
|
|
@@ -166,7 +166,17 @@ vi.mock('../services/FFmpegService', () => ({ | |||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| getVideoInfo: vi.fn(), | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| transcodeVideo: vi.fn(), | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| cancelTranscode: vi.fn(), | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| getFFmpegPath: vi.fn() | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| getFFmpegPath: vi.fn(), | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| getDownloadService: vi.fn(() => ({ | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| checkFFmpegExists: vi.fn(), | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| getFFmpegVersion: vi.fn(), | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| downloadFFmpeg: vi.fn(), | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| getDownloadProgress: vi.fn(), | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| cancelDownload: vi.fn(), | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| removeFFmpeg: vi.fn(), | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| getAllSupportedVersions: vi.fn(), | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| cleanupTempFiles: vi.fn() | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| })) | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
|
Comment on lines
+169
to
+179
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. 🧹 Nitpick (assertive) Mock surface alignment for new FFmpeg download API The nested getDownloadService mock mirrors production API well. To reduce drift, strongly type this mock against the service return type so CI breaks if the surface changes. Example: -vi.mock('../services/FFmpegService', () => ({
- default: vi.fn().mockImplementation(() => ({
+vi.mock('../services/FFmpegService', () => {
+ type DownloadSvc = ReturnType<import('../services/FFmpegService').default['getDownloadService']>;
+ const downloadSvc: DownloadSvc = {
+ checkFFmpegExists: vi.fn(),
+ getFFmpegVersion: vi.fn(),
+ downloadFFmpeg: vi.fn(),
+ getDownloadProgress: vi.fn(),
+ cancelDownload: vi.fn(),
+ removeFFmpeg: vi.fn(),
+ getAllSupportedVersions: vi.fn(),
+ cleanupTempFiles: vi.fn()
+ };
+ return {
+ default: vi.fn().mockImplementation(() => ({
checkFFmpegExists: vi.fn(),
getFFmpegVersion: vi.fn(),
downloadFFmpeg: vi.fn(),
getVideoInfo: vi.fn(),
transcodeVideo: vi.fn(),
cancelTranscode: vi.fn(),
getFFmpegPath: vi.fn(),
- getDownloadService: vi.fn(() => ({
- checkFFmpegExists: vi.fn(),
- getFFmpegVersion: vi.fn(),
- downloadFFmpeg: vi.fn(),
- getDownloadProgress: vi.fn(),
- cancelDownload: vi.fn(),
- removeFFmpeg: vi.fn(),
- getAllSupportedVersions: vi.fn(),
- cleanupTempFiles: vi.fn()
- }))
- }))
- }))
-}))
+ getDownloadService: vi.fn(() => downloadSvc)
+ }))
+ };
+})📝 Committable suggestion
Suggested change
🤖 Prompt for AI Agents |
||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| })) | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| })) | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| Original file line number | Diff line number | Diff line change | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
|
|
@@ -474,6 +474,51 @@ export function registerIpc(mainWindow: BrowserWindow, app: Electron.App) { | |||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| ipcMain.handle(IpcChannel.Ffmpeg_GetWarmupStatus, async () => { | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| return FFmpegService.getWarmupStatus() | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| }) | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| ipcMain.handle(IpcChannel.Ffmpeg_GetInfo, async () => { | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| return ffmpegService.getFFmpegInfo() | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| }) | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| ipcMain.handle(IpcChannel.Ffmpeg_AutoDetectAndDownload, async () => { | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| return await ffmpegService.autoDetectAndDownload() | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| }) | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| // FFmpeg 下载服务 | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| const ffmpegDownloadService = ffmpegService.getDownloadService() | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| ipcMain.handle( | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| IpcChannel.FfmpegDownload_CheckExists, | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| async (_, platform?: string, arch?: string) => { | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| return ffmpegDownloadService.checkFFmpegExists(platform as any, arch as any) | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| } | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| ) | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| ipcMain.handle( | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| IpcChannel.FfmpegDownload_GetVersion, | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| async (_, platform?: string, arch?: string) => { | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| return ffmpegDownloadService.getFFmpegVersion(platform as any, arch as any) | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| } | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| ) | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| ipcMain.handle( | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| IpcChannel.FfmpegDownload_Download, | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| async (_, platform?: string, arch?: string) => { | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| return await ffmpegDownloadService.downloadFFmpeg(platform as any, arch as any) | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| } | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| ) | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| ipcMain.handle( | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| IpcChannel.FfmpegDownload_GetProgress, | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| async (_, platform?: string, arch?: string) => { | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| return ffmpegDownloadService.getDownloadProgress(platform as any, arch as any) | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| } | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| ) | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| ipcMain.handle(IpcChannel.FfmpegDownload_Cancel, async (_, platform?: string, arch?: string) => { | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| return ffmpegDownloadService.cancelDownload(platform as any, arch as any) | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| }) | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| ipcMain.handle(IpcChannel.FfmpegDownload_Remove, async (_, platform?: string, arch?: string) => { | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| return ffmpegDownloadService.removeFFmpeg(platform as any, arch as any) | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| }) | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| ipcMain.handle(IpcChannel.FfmpegDownload_GetAllVersions, async () => { | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| return ffmpegDownloadService.getAllSupportedVersions() | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| }) | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| ipcMain.handle(IpcChannel.FfmpegDownload_CleanupTemp, async () => { | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| return ffmpegDownloadService.cleanupTempFiles() | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| }) | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
|
Comment on lines
+477
to
+521
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. 🛠️ Refactor suggestion Type the platform/arch params; avoid any-casts Use Node’s platform/arch types to prevent invalid inputs and remove as any. - const ffmpegDownloadService = ffmpegService.getDownloadService()
+ const ffmpegDownloadService = ffmpegService.getDownloadService()
@@
- async (_, platform?: string, arch?: string) => {
- return ffmpegDownloadService.checkFFmpegExists(platform as any, arch as any)
+ async (_, platform?: typeof process.platform, arch?: typeof process.arch) => {
+ return ffmpegDownloadService.checkFFmpegExists(platform, arch)
}
)
@@
- async (_, platform?: string, arch?: string) => {
- return ffmpegDownloadService.getFFmpegVersion(platform as any, arch as any)
+ async (_, platform?: typeof process.platform, arch?: typeof process.arch) => {
+ return ffmpegDownloadService.getFFmpegVersion(platform, arch)
}
)
@@
- async (_, platform?: string, arch?: string) => {
- return await ffmpegDownloadService.downloadFFmpeg(platform as any, arch as any)
+ async (_, platform?: typeof process.platform, arch?: typeof process.arch) => {
+ return await ffmpegDownloadService.downloadFFmpeg(platform, arch)
}
)
@@
- async (_, platform?: string, arch?: string) => {
- return ffmpegDownloadService.getDownloadProgress(platform as any, arch as any)
+ async (_, platform?: typeof process.platform, arch?: typeof process.arch) => {
+ return ffmpegDownloadService.getDownloadProgress(platform, arch)
}
)
- ipcMain.handle(IpcChannel.FfmpegDownload_Cancel, async (_, platform?: string, arch?: string) => {
- return ffmpegDownloadService.cancelDownload(platform as any, arch as any)
+ ipcMain.handle(IpcChannel.FfmpegDownload_Cancel, async (_, platform?: typeof process.platform, arch?: typeof process.arch) => {
+ return ffmpegDownloadService.cancelDownload(platform, arch)
})
- ipcMain.handle(IpcChannel.FfmpegDownload_Remove, async (_, platform?: string, arch?: string) => {
- return ffmpegDownloadService.removeFFmpeg(platform as any, arch as any)
+ ipcMain.handle(IpcChannel.FfmpegDownload_Remove, async (_, platform?: typeof process.platform, arch?: typeof process.arch) => {
+ return ffmpegDownloadService.removeFFmpeg(platform, arch)
})📝 Committable suggestion
Suggested change
🤖 Prompt for AI Agents |
||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| // MediaParser (Remotion) | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| ipcMain.handle(IpcChannel.MediaInfo_CheckExists, async () => { | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🧹 Nitpick (assertive)
Win ARM64 target added — verify code signing and auto-update paths
Adding NSIS/portable arm64 is good, but please confirm:
Run locally on a Windows ARM64 VM/device to validate install/update flows. If not feasible, at least confirm release assets include separate arm64 .yml and artifacts and that updater feed lists the correct arch.
🤖 Prompt for AI Agents