mirror of
https://github.com/Donkie/Spoolman.git
synced 2026-10-22 10:02:17 -05:00
Donkie
e509378392
The declared floor was ">=3.10", but nothing enforced it: the Docker image has pinned python:3.14-slim-bookworm for amd64/arm64/armv7 for some time, and scripts/install.sh runs "uv sync" without UV_PYTHON_DOWNLOADS=0, so uv fetches a managed interpreter when the system one is too old. Every dependency declares a floor of <=3.10 with no upper cap. Raising it unlocks enum.StrEnum, datetime.UTC and typing.Self for the authentication work that follows. Consequences handled here: - uv.lock regenerated. Four pure-backport packages drop out (async-timeout, backports-asyncio-runner, exceptiongroup, tomli). - ruff target-version follows to py314, which switches on PEP 649 deferred annotations and with them a batch of previously-dormant rules. The safe automatic fixes are applied: datetime.timezone.utc -> datetime.UTC (UP017) and redundant quotes dropped from annotations (UP037). - TC001/TC002/TC003 are ignored rather than applied. They want type-only imports moved into "if TYPE_CHECKING:" blocks, which is wrong here: FastAPI resolves Annotated[AsyncSession, Depends(...)] and websocket parameters at runtime, Pydantic resolves field types at runtime, and SQLAlchemy resolves Mapped[...] at mapper-configuration time. A TYPE_CHECKING-only import is absent from the runtime namespace, so applying them breaks dependency injection. Ruff classifies all 50 as unsafe fixes for that reason. Verified: ruff check/format clean, uv lock --check clean, alembic head resolves, SQLAlchemy mappers configure with all relationships intact and lazy="selectin" preserved on the extra-field collections.