Skip to content

fix(bridge): point Windows manual download back at the plain zip - #1272

Merged
snekxs merged 1 commit into
mainfrom
fix/bridge-windows-link-revert
Oct 29, 2026
Merged

snekxs merged 1 commit into
mainfrom
fix/bridge-windows-link-revert

Conversation

@snekxs

@snekxs snekxs commented Oct 29, 2026 •

Copy link
Copy Markdown
Member

Bridge v1.0.0-beta.21 was pulled from GitHub, so the latest release is v1.0.0-beta.20 again and openmouse-bridge-windows-x64-full.zip no longer exists (the link returns 404). On beta.20, openmouse-bridge-windows-x64.zip is the complete Windows package.

Reverts the URL in src/features/bridge/manual-update.ts and its test. We will point back at -full when a release with that asset is published again.

Requested by the Bridge session. Left unmerged for the maintainer.

Tests: node --test src/features/bridge/manual-update.test.ts passes.

Note

Fix Windows manual bridge download to use standard zip asset

Changes the Windows case in manualInstallFor in manual-update.ts to return the standard ZIP asset URL instead of the full ZIP asset. macOS, Linux, and default cases are unchanged. Test expectations in manual-update.test.ts are updated to match.

Macroscope summarized 56b076b.

Summary by CodeRabbit

  • Bug Fixes
    • Updated Windows manual bridge installation to download the standard x64 ZIP release asset instead of the full-package ZIP. The download now matches the release asset expected for this installation option, helping Windows users get the intended bridge package when installing manually.

Bridge v1.0.0-beta.21 was pulled, so the latest release is beta.20 again
and openmouse-bridge-windows-x64-full.zip no longer exists (404). On
beta.20 the plain zip is the complete Windows package.
@github-actions github-actions Bot added type: bug PR title category: bug area: devices Device integration and HID support labels Oct 29, 2026
@coderabbitai

coderabbitai Bot commented Oct 29, 2026 •

Copy link
Copy Markdown
Contributor

Review in Change Stack →

Caution

Review failed

The pull request is closed.

Walkthrough

The Windows manual-install URL now points to the .zip release asset instead of the -full.zip asset. The matching test expectation uses the same asset name.

Changes: Windows manual install

Layer / File(s) Summary
Windows download asset
src/features/bridge/manual-update.ts, src/features/bridge/manual-update.test.ts
The Windows manual-install URL and its test expectation now use openmouse-bridge-windows-x64.zip instead of openmouse-bridge-windows-x64-full.zip.

Priority: Not assessed

Estimated code review effort: 1 (Trivial) | ~5 minutes

Change: Bug fix

Possibly related PRs

Suggested reviewers: macroscopeapp

Merge Risk: Minimal · up to 56b07

The Windows manual download now targets the reported available release archive. No actionable merge-blocking risk is identified; merge after normal checks.

Finishing Touches
📝 Generate docstrings
  • Commit to this branch
  • Create a new PR
🧪 Generate unit tests (beta)
  • Commit to this branch
  • Create a new PR
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@cloudflare-workers-and-pages

Copy link
Copy Markdown

Deploying openmouse with  Cloudflare Pages  Cloudflare Pages

Latest commit: 56b076b
Status: ✅  Deploy successful!
Preview URL: https://61cce08e.openmouse.pages.dev
Branch Preview URL: https://fix-bridge-windows-link-reve.openmouse.pages.dev

View logs

@macroscopeapp

macroscopeapp Bot commented Oct 29, 2026

Copy link
Copy Markdown

Approvability

Verdict: Approved at 56b076b

Macroscope's review found this PR approvable — The Windows manual-update link now points to the plain ZIP published on the official Bridge release, with the test expectation updated accordingly. The change is limited to a user-initiated download link; automatic updates and other platforms are unchanged.

Notes:

  • Your repo's Minimum Blocking Severity is Off, so correctness findings do not automatically block approval.

You can add or adjust custom eligibility rules. Learn more.

@snekxs
snekxs merged commit 59d1055 into main Oct 29, 2026
10 of 11 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area: devices Device integration and HID support type: bug PR title category: bug

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant