Skip to content

fix(desktop): wire icon.ico into bundle config for Windows builds#19

Merged
GuilhermeVozniak merged 1 commit into
mainfrom
ci/release-windows-ico
May 15, 2026
Merged

fix(desktop): wire icon.ico into bundle config for Windows builds#19
GuilhermeVozniak merged 1 commit into
mainfrom
ci/release-windows-ico

Conversation

@GuilhermeVozniak
Copy link
Copy Markdown
Collaborator

Summary

The Windows job in the `v0.1.1-test` dry-run failed at the bundling step:

```
failed to bundle project `Couldn't find a .ico icon`
```

`packages/desktop/src-tauri/icons/icon.ico` already exists on disk — the bundle config just listed `icons/icon.png` and nothing else, so the Windows bundler had no `.ico` path to consume.

Changes

`tauri.conf.json` bundle.icon array now follows Tauri's per-platform convention:

```json
"icon": [
"icons/32x32.png",
"icons/128x128.png",
"icons/128x128@2x.png",
"icons/icon.icns",
"icons/icon.ico"
]
```

Tauri picks the right one per target: sized PNGs for Linux deb/AppImage, `.icns` for macOS, `.ico` for Windows.

Test plan

  • After merge, dry-run via Actions → Release → Run workflow → tag `v0.1.1-test-2`.
  • Confirm all three matrix jobs (macOS / Linux / Windows) finish green.
  • If green, cut v0.1.1 for real.

🧙 Built with WOZCODE

The Windows job in the v0.1.1-test dry-run failed at the bundling step:

  failed to bundle project `Couldn't find a .ico icon`

`packages/desktop/src-tauri/icons/icon.ico` already exists on disk —
the bundle config just listed `icons/icon.png` and nothing else, so
the Windows bundler had no .ico path to consume and the macOS bundler
fell back to converting the PNG to .icns at build time.

Now the icon array follows Tauri's per-platform convention: sized PNGs
for Linux deb/AppImage, .icns for macOS, .ico for Windows. Tauri picks
the right one per target.

Co-Authored-By: WOZCODE <contact@withwoz.com>
@GuilhermeVozniak GuilhermeVozniak merged commit 7fa013d into main May 15, 2026
8 checks passed
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 this pull request may close these issues.

1 participant