posit-dev / posit-dev/rsconnect-python
`--package-installer` silently ignored in two commands and unusable as documented on click >= 8.2
Dieses Issue hat noch niemand übernommen.
- Vorherrschende Sprache
- Python
- Sterne
- 37
- Forks
- 28
- Ø Merge
- 1 T. 3 Std.
- Gemergte PRs (30 T.)
- 7
Beschreibung
--package-installer was added in #708 (c1450c57). It does not work as documented in two independent ways. Neither is visible to the current test suite.
1. write-manifest <framework> and deploy quarto accept the option and ignore it
Two code paths take package_installer and never forward it to Environment.create_python_environment:
_write_framework_manifestinrsconnect/main.py, shared bywrite-manifest streamlit|dash|bokeh|panel|shiny|api|fastapi. The call passesrequirements_file,override_python_version, andpython, but notpackage_manager.deploy_quartoinrsconnect/main.py. Same omission in itscreate_python_environmentcall.
The other commands added in #708 (deploy notebook|voila|<framework>, write-manifest notebook|voila|quarto) do pass package_manager=package_installer. git log -S "package_manager=package_installer" -- rsconnect/main.py returns only #708, so these two sites were never wired rather than wired and later lost.
Repro on click 8.1.8 (the version in uv.lock; see issue 2 for why the version matters):
$ rsconnect write-manifest shiny --overwrite --package-installer uv ./app
Checking arguments... [OK]
Inspecting Python environment... [OK]
Inspecting Shiny for Python app... [OK]
Creating manifest.json... [OK]
$ python -c "import json; print(json.load(open('app/manifest.json'))['python']['package_manager'])"
{'name': 'pip', 'version': '26.2.1', 'package_file': 'requirements.txt'}
Expected {'name': 'uv', ...}. The command exits 0 with no warning.
Consequence: deploy <framework> honors the option and write-manifest <framework> does not, so write-manifest followed by deploy manifest installs differently from the equivalent direct deploy.
2. The documented uv spelling is rejected on click >= 8.2
The option is declared as type=click.Choice(PackageInstaller) over a StrEnum. click 8.2.0 changed Choice to match enum members by name instead of by value, so the accepted spelling flips with the click version:
| click | --package-installer uv |
--package-installer UV |
|---|---|---|
| 8.1.8 | accepted | rejected |
| 8.2.1 | rejected | accepted |
| 8.3.1 | rejected | accepted |
On click 8.3.1:
$ rsconnect write-manifest shiny --overwrite --package-installer uv ./app
Error: Invalid value for '--package-installer': 'uv' is not one of 'PIP', 'UV'.
The changelog documents --package-installer=uv|pip. pyproject.toml declares click>=8.0.0 with no upper bound, so a fresh install today resolves a click where the documented invocation fails. On those versions --help also advertises the choices as PIP|UV, disagreeing with the changelog. No single spelling works across the declared click range.
This works in the repository checkout only because uv.lock pins click 8.1.8.
Why neither shows up in CI
grep -rn "package_installer" tests/ returns nothing. #708's tests cover the bundle writer at the unit level and never invoke the CLI option. Issue 1 is invisible to those tests, and issue 2 cannot appear while the lock holds click 8.1.8.
Suggested fixes
- Pass
package_manager=package_installerin thecreate_python_environmentcalls in_write_framework_manifestanddeploy_quarto. - Declare the option as
click.Choice(["pip", "uv"])and convert toPackageInstallerafterward, so the lowercase spelling works on every supported click. Bounding click would also work, at the cost of what users can install alongside. - Add a CLI-level test that asserts
manifest["python"]["package_manager"]["name"]forwrite-manifestanddeployacross the app-mode families.
Environment
- rsconnect-python 1.31.2.dev1 (
main,3d3f5841) - click 8.1.8 (from
uv.lock) and click 8.3.1 (fresh resolve) - Python 3.12, macOS
Beitragsleitfaden
Erste Schritte
- Lies das ganze Issue und danach den Beitragsleitfaden des Projekts.
- Schreib ins Issue, dass du es übernimmst — das erspart doppelte Arbeit.
- Forke das Repository und arbeite in einem Branch.
- Öffne einen Pull Request, der die Issue-Nummer nennt.
Rechercherichtung
Lesen Sie _write_framework_manifest und deploy_quarto in rsconnect/main.py, und prüfen Sie anschließend die Deklaration der Option package_installer sowie die vorhandenen CLI-Tests; pyproject.toml und uv.lock zeigen die unterstützten click-Versionen. Fügen Sie Tests für write-manifest und deploy hinzu, die überprüfen, dass der ausgewählte Installer das Manifest erreicht und dass uv in Kleinbuchstaben über den gesamten deklarierten click-Versionsbereich akzeptiert wird.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Bewertung
- Tech-Stack
- python
- Bereich
- cli, testing-qa
- Issue-Typ
- Bug
- Schwierigkeit
- 3/5
- Geschätzter Aufwand
- 1-2 Tage
- Aktivitätsstatus
- Aktiv
- Klarheit
- Klar beschrieben
- Anfängerfreundlichkeit
- 78/100