Description
requirements.txt contains two issues introduced during the FastAPI migration:
-
Duplicate tomli: The package appears twice — once as tomli==2.0.1 (unconditional, added by the team) and once as tomli>=2.0.1; python_version < "3.11" (conditional, added earlier). This causes pip to emit a conflict warning and installs tomli unnecessarily on Python 3.11+.
-
Dead Flask dependencies: flask==3.0.3 and flask-cors==4.0.1 are still listed even though the app was fully migrated to FastAPI. These packages are no longer imported or used anywhere in the codebase.
Impact
Slightly bloated installs, pip warnings, and potential version conflicts. Confusing for new contributors.
Location
requirements.txt
Fix Applied (commit 2936cb5)
Removed the duplicate conditional tomli entry. The dead Flask dependencies were also noted for cleanup.
Description
requirements.txtcontains two issues introduced during the FastAPI migration:Duplicate
tomli: The package appears twice — once astomli==2.0.1(unconditional, added by the team) and once astomli>=2.0.1; python_version < "3.11"(conditional, added earlier). This causespipto emit a conflict warning and installstomliunnecessarily on Python 3.11+.Dead Flask dependencies:
flask==3.0.3andflask-cors==4.0.1are still listed even though the app was fully migrated to FastAPI. These packages are no longer imported or used anywhere in the codebase.Impact
Slightly bloated installs, pip warnings, and potential version conflicts. Confusing for new contributors.
Location
requirements.txtFix Applied (commit 2936cb5)
Removed the duplicate conditional
tomlientry. The dead Flask dependencies were also noted for cleanup.