fix: Add refresh option to update check for doctor command #213
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
The doctor command was showing stale update information because it read from the 4-hour cached version info. When a new release was published after the last check, users running doctor would not see the update until the next periodic check. Changes: - Add CheckNow() method to update checker for immediate checks - Add RefreshVersionInfo() method to Runtime and Server - Add refresh query parameter to /api/v1/info endpoint - Update doctor command to always refresh version info - Add unit tests for update checker When running `mcpproxy doctor`, it now performs a fresh check against GitHub to ensure users see the latest available version. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude Opus 4.5 <[email protected]>
Deploying mcpproxy-docs with
|
| Latest commit: |
9652dee
|
| Status: | ✅ Deploy successful! |
| Preview URL: | https://ea5e4a36.mcpproxy-docs.pages.dev |
| Branch Preview URL: | https://fix-doctor-update-check-refr.mcpproxy-docs.pages.dev |
🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude Opus 4.5 <[email protected]>
📦 Build ArtifactsWorkflow Run: View Run Available Artifacts
How to DownloadOption 1: GitHub Web UI (easiest)
Option 2: GitHub CLI gh run download 20398241224 --repo smart-mcp-proxy/mcpproxy-go
|
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Summary
mcpproxy doctorshowing stale update information when a new release was published after the last periodic check?refresh=truequery parameter to/api/v1/infoendpoint to force immediate update checkProblem
When running
mcpproxy doctor, users were seeing cached update information from the 4-hour periodic check. If a new version was released after the last check, users would not be notified until the next periodic check ran.Example: v0.11.3 was released, but doctor showed v0.11.2 as latest because the last check happened before the release.
Solution
CheckNow()method to update checker for on-demand checksRefreshVersionInfo()method through the stack (Runtime, Server, Controller)/api/v1/infoendpoint to acceptrefresh=truequery parameterTest plan
CheckNow()method in update checker🤖 Generated with Claude Code