learntocloud Validate Changes
From the repository root run the authoritative quality gate:
uv run poe check
It runs static checks, package installation smoke testing, and all test suites.
New files must be staged before prek --all-files can inspect them; stage only
files belonging to the current task.
Before changing a function signature or removing async, trace its callers,
dependency declarations, callback contracts, and test mocks. Ruff's ARG001
(unused function arguments) and preview RUF029 (unneeded async) are enforced
across all three workspace packages, including tests. The file-specific
exceptions and their callable contracts are documented in
contributing.
Review new code in those files manually for the exempted rules.
Remove genuinely unused parameters and unnecessary async, updating callers
and mocks together. Preserve authentication dependencies, required callback
parameters, async interfaces, and fixtures that execute for their side effects.
Do not add dummy awaits, rename parameters just to hide findings, or suppress
rules to pass the gate. If an interface requires an apparent violation, explain
the contract and get agreement on an appropriate rule scope before changing it.
After Python application changes, start a fresh API process on
127.0.0.1:8000 and request:
/health(200)/ready(200 once dependencies are ready)/openapi.json(200 and valid JSON)
Track the exact process ID and always terminate that process during cleanup. Do not kill unrelated listeners. Report startup logs if any endpoint fails.
For authentication or route-policy changes, preserve the real-route coverage in
api/tests/routes/test_auth_http.py: API/HTMX 401s, page 303s, redirect methods,
persisted-session success, and repeatable logout. Preserve the real PostgreSQL
lifecycle coverage in api/tests/routes/test_session_lifecycle.py, including
copied-cookie replay, independent browsers, expiry, and deletion/recreation.
Do not bypass authentication
with dependency overrides when testing authentication itself. The same file
checks exported request spans; api/tests/core/test_middleware.py covers nested
router templates. These tests already run in uv run poe check.
See Authentication and sessions for the response and session-lifecycle contract.
For workflow changes that add a Python command, also run that exact command with only the environment variables supplied by the workflow. Do not assume the local WSL environment is available in CI.
Fix root causes and rerun the failed gate. Do not suppress lint or type errors.