Skip to content

fix(versioncheck): detect mise install for update command#659

Merged
Soph merged 2 commits intoentireio:mainfrom
mvanhorn:fix/update-command-mise-detection
Mar 10, 2026
Merged

fix(versioncheck): detect mise install for update command#659
Soph merged 2 commits intoentireio:mainfrom
mvanhorn:fix/update-command-mise-detection

Conversation

@mvanhorn
Copy link
Contributor

@mvanhorn mvanhorn commented Mar 8, 2026

Summary

Closes #432

When the CLI detects a newer version is available, it shows an update command. Previously it only distinguished between Homebrew (brew upgrade entire) and the install script. Users who installed via mise were shown the generic install script command instead of mise upgrade entire.

This adds mise detection by checking if the resolved binary path contains /mise/, using the same symlink-resolution pattern already used for Homebrew detection.

Changes

  • versioncheck/versioncheck.go - Added mise path check in updateCommand() before the fallback
  • versioncheck/versioncheck_test.go - Added "mise upgrade entire" to valid update commands

Testing

  • Existing TestUpdateCommand validates the new command is in the accepted set
  • Detection logic follows the same filepath.EvalSymlinks + strings.Contains pattern as Homebrew

🤖 Generated with Claude Code

Co-Authored-By: Claude Opus 4.6 noreply@anthropic.com

When the binary is installed via mise, show `mise upgrade entire`
instead of the curl command. Detection is path-based (checking for
/mise/ in the resolved binary path), consistent with the existing
Homebrew detection.

Closes entireio#432

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@mvanhorn mvanhorn requested a review from a team as a code owner March 8, 2026 00:41
Entire-Checkpoint: 46875b53634f
@Soph Soph merged commit 4bac3a3 into entireio:main Mar 10, 2026
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

Incorrect download strategy on call to update

2 participants