Skip to content

fix(ci): simplify Bun setup for Windows ARM64#965

Merged
cjpais merged 1 commit intocjpais:mainfrom
oddrationale:fix/simplify-bun-setup-windows-arm64
Mar 6, 2026
Merged

fix(ci): simplify Bun setup for Windows ARM64#965
cjpais merged 1 commit intocjpais:mainfrom
oddrationale:fix/simplify-bun-setup-windows-arm64

Conversation

@oddrationale
Copy link
Copy Markdown
Contributor

@oddrationale oddrationale commented Mar 5, 2026

Before Submitting This PR

Please confirm you have done the following:

If this is a feature or change that was previously closed/rejected:

  • I have explained in the description below why this should be reconsidered
  • I have gathered community feedback (link to discussion below)

Human Written Description

Bun now releases aarch64 native builds for Windows. This removes the previous work-around I implemented when originally creating the aaarch64/arm64 Windows build. We can now have a single, unified bun setup step.

Related Issues/Discussions

Discussion:

Community Feedback

N/A (CI/workflow cleanup based on upstream action behavior)

Testing

Screenshots/Videos (if applicable)

N/A

AI Assistance

  • No AI was used in this PR
  • AI was used (please describe below)

If AI was used:

  • Tools used: Codex (gpt-5.3-codex) and Claude Code (Opus 4.6)
  • How extensively: Perform code review and draft PR.

oven-sh/setup-bun@v2 (v2.1.1+) now automatically falls back ARM64 to
x64 with AVX2 disabled on Windows, making the manual baseline download
workaround unnecessary.

Ref: oven-sh/setup-bun#131
Ref: oven-sh/setup-bun#130

Co-Authored-By: Claude Opus 4.6 <[email protected]>
@oddrationale oddrationale marked this pull request as ready for review March 5, 2026 04:42
@github-actions
Copy link
Copy Markdown

github-actions bot commented Mar 6, 2026

🧪 Test Build Ready

Build artifacts for PR #965 are available for testing.

Download artifacts from workflow run

Artifacts expire after 30 days.

@cjpais
Copy link
Copy Markdown
Owner

cjpais commented Mar 6, 2026

Thanks!

@cjpais cjpais merged commit fc05e4a into cjpais:main Mar 6, 2026
4 checks passed
@oddrationale oddrationale deleted the fix/simplify-bun-setup-windows-arm64 branch March 6, 2026 14:47
thukabjj added a commit to thukabjj/Handy that referenced this pull request Mar 7, 2026
Upstream changes:
- Language-aware filler word removal (cjpais#971)
- Portable mode NSIS installer (cjpais#807)
- Italian translation fix (cjpais#973)
- Bun setup for Windows ARM64 (cjpais#965)
- tauri-plugin-dialog upgraded to 2.6
- Linux install docs and AppImage troubleshooting (cjpais#951)

Conflict resolution:
- package.json: take upstream dialog 2.6
- Cargo.toml: take upstream dialog 2.6, keep fork deps (symphonia, keyring)
- text.rs: take upstream's language-aware filter_transcription_output, keep fork test
- settings/mod.rs: keep fork fields + add upstream custom_filler_words
- bun.lock: take upstream version
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.

2 participants