Skip to content

fix(update): auto-install minor bumps so 1.0.x users get updates#3342

Merged
la14-1 merged 1 commit intoOpenRouterTeam:mainfrom
AhmedTMM:fix/autoupdate-minor-bump
Apr 22, 2026
Merged

fix(update): auto-install minor bumps so 1.0.x users get updates#3342
la14-1 merged 1 commit intoOpenRouterTeam:mainfrom
AhmedTMM:fix/autoupdate-minor-bump

Conversation

@AhmedTMM
Copy link
Copy Markdown
Collaborator

Summary

  • Auto-update now covers patch AND minor bumps (same major version)
  • Only major bumps (2.0.0+) require SPAWN_AUTO_UPDATE=1 opt-in
  • The 1.0.x → 1.1.0 jump in feat(digitalocean): guided readiness before deploy #3336 blocked all users from auto-updating since only patch bumps were auto-installed — users without SPAWN_AUTO_UPDATE=1 (everyone) were stuck on 1.0.x and never received the telemetry fix

Test plan

  • bun test src/__tests__/update-check — 28 pass, 0 fail
  • Lint clean
  • Verify 1.0.x users auto-update to 1.1.2 on next run

🤖 Generated with Claude Code

The 1.0.x → 1.1.0 minor bump blocked auto-update for all users since
only patch bumps were auto-installed. Users without SPAWN_AUTO_UPDATE=1
were stuck on 1.0.x and never received the telemetry fix.

Version set to 1.0.20 so existing 1.0.x users see it as a patch bump
and auto-install it. The new update logic then allows future minor bumps
(same major) to auto-install too. Only major bumps (2.0.0+) require
SPAWN_AUTO_UPDATE=1.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@AhmedTMM AhmedTMM force-pushed the fix/autoupdate-minor-bump branch from 0774d67 to ca05fab Compare April 22, 2026 21:02
Copy link
Copy Markdown
Member

@la14-1 la14-1 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Approved. The 1.1.1 → 1.0.20 version shift is intentional as a bridge release: 1.0.x users pick up 1.0.20 as a patch-level auto-update (under the old rules), and with the new logic in this PR they'll then auto-install the next 1.1.x. Clean way to unstick the fleet.

Rest of the change:

  • isSameMinorparseSemver(...)[0] major-only check is the right simplification
  • Tests rewritten to match (minor bumps now auto-install, major bumps still gated on SPAWN_AUTO_UPDATE=1)
  • SPAWN_NO_AUTO_UPDATE=1 escape hatch preserved for pinned CI envs

Shipping.

@la14-1 la14-1 merged commit 75a22f2 into OpenRouterTeam:main Apr 22, 2026
5 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.

2 participants