Add CLI and server version checks with upgrade guidance#1677
Merged
Conversation
Replace the server-based version check with a PyPI-based one so the CLI checks its own package version independently of the server. Add file-based caching (~/.lab/.version_cache.json, 4-hour TTL) to avoid hitting PyPI on every invocation. Enrich `lab version` to show update status and support --format json output. The background banner on every command now shows `uv tool upgrade transformerlab-cli` instead of a web link.
Add a new `lab server version` command that checks the installed server version against GitHub releases and advises running `lab server update` when outdated. Supports both pretty and JSON output formats. Also fix a bug in _get_current_version() which called .read_text() on a str path instead of using open().
Paragon SummaryThis pull request review identified 4 issues across 2 categories in 7 files. The review analyzed code changes, potential bugs, security vulnerabilities, performance issues, and code quality concerns using automated analysis tools. This PR adds version-awareness to the CLI and server commands: Key changes:
Confidence score: 4/5
7 files reviewed, 4 comments Severity breakdown: Medium: 3, Low: 1 Tip: |
![paragon-review[bot]](https://avatars.githubusercontent.com/in/1544258?s=60&v=4){kind=small}
- _parse_version now handles pre-release strings like '0.30.0rc1' by
extracting only the numeric release segments via regex
- server_version uses > comparison instead of != to avoid suggesting
downgrades when the local version is newer than GitHub latest
- version.py distinguishes "up to date" from "check failed" instead of
always showing green "You are up to date" on PyPI network failure
- is_update_available fallback returns ("unknown", None) instead of
re-calling get_installed_version() which could itself raise
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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
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
lab versionnow checks the CLI package version against PyPI and shows whether an update is available, with the exact upgrade command (uv tool upgrade transformerlab-cli). Supports--format json.lab server version(new command) checks the installed server version against GitHub releases and advises runninglab server updatewhen outdated. Supports--format json.~/.lab/.version_cache.json._get_current_version()that called.read_text()on astrinstead of usingopen().Test plan
cd cli && python -m pytest tests/ -v)ruff checkandruff format --check)lab versionshows CLI version + PyPI update statuslab server versionshows server version + GitHub update statuslab statusshows background CLI version banner when outdated