Conversation
Add authors, license, readme, classifiers, and project URLs to cli/pyproject.toml. Update CLI README with install instructions and a proper description for the PyPI page.
Paragon SummaryThis pull request review identified 3 issues across 1 category in 2 files. The review analyzed code changes, potential bugs, security vulnerabilities, performance issues, and code quality concerns using automated analysis tools. This PR prepares the CLI package for PyPI publication by adding required metadata fields (authors, license, classifiers, project URLs) to Key changes:
Confidence score: 5/5
2 files reviewed, 3 comments Severity breakdown: Low: 3 Tip: |
![paragon-review[bot]](https://avatars.githubusercontent.com/in/1544258?s=60&v=4){kind=small}
| ## Install | ||
|
|
||
| (install instructions coming soon, for now follow Build Locally instructions below) | ||
| ```bash |
There was a problem hiding this comment.
Bug: Install docs reference PyPI before the package is confirmed published
Install docs reference PyPI before the package is confirmed published. Users following the README after merge will hit package-not-found errors. Gate the install line with a note until the package is live.
View Details
Location: cli/README.md (lines 11)
Analysis
Install docs reference PyPI before the package is confirmed published
| What fails | The PR test plan only verifies 'uv build succeeds' and 'uv publish --dry-run' with no actual publish step, yet the README documents the install command as if the package is already on PyPI. |
| Result | uv tool install transformerlab-cli fails with package-not-found error |
| Expected | Install instructions should only be documented after the package is verified live on PyPI, or include a note that it is available after the first release. |
| Impact | Any user following the README immediately after merge will hit a hard failure on the install step. |
How to reproduce
1. Merge this PR
2. Follow README install instructions: `uv tool install transformerlab-cli`
3. Observe failure if package is not yet published to PyPIPatch Details
-uv tool install transformerlab-cli
+uv tool install transformerlab-cli # Available after first PyPI releaseAI Fix Prompt
Fix this issue: Install docs reference PyPI before the package is confirmed published. Users following the README after merge will hit package-not-found errors. Gate the install line with a note until the package is live.
Location: cli/README.md (lines 11)
Problem: The PR test plan only verifies 'uv build succeeds' and 'uv publish --dry-run' with no actual publish step, yet the README documents the install command as if the package is already on PyPI.
Current behavior: uv tool install transformerlab-cli fails with package-not-found error
Expected: Install instructions should only be documented after the package is verified live on PyPI, or include a note that it is available after the first release.
Steps to reproduce: 1. Merge this PR
2. Follow README install instructions: `uv tool install transformerlab-cli`
3. Observe failure if package is not yet published to PyPI
Provide a code fix.
Tip: Reply with @paragon-run to automatically fix this issue
Add missing 'Python :: 3 :: Only' parent classifier for PyPI taxonomy completeness. Replace library classifier with 'Environment :: Console' to correctly categorize the package as a CLI tool.
2 participants
Summary
cli/pyproject.toml: authors, license (Apache-2.0), readme, classifiers, and[project.urls]with homepage and repository links.cli/README.mdwith install instructions (uv tool install transformerlab-cli) and a description suitable for the PyPI page.Test plan
uv buildsucceeds incli/directoryuv publish --dry-runortwine check