Skip to content

Migrate recent operations read to FastAPI#1386

Merged
cbusillo merged 1 commit into
mainfrom
v2-fastapi-recent-operations-read
Jun 17, 2026
Merged

Migrate recent operations read to FastAPI#1386
cbusillo merged 1 commit into
mainfrom
v2-fastapi-recent-operations-read

Conversation

@cbusillo

Copy link
Copy Markdown
Owner

Summary

  • migrate GET /v1/contexts/{context}/operations/recent to a native FastAPI route
  • delete the legacy WSGI operations.read matcher/handler path
  • add typed Pydantic/OpenAPI coverage plus auth-before-store and fallback-retirement tests

Validation

  • uv run python -m unittest tests.test_http_app.FastApiRecentOperationsReadTests tests.test_service.LaunchplaneServiceTests.test_single_record_read_routes_are_retired_from_legacy_wsgi_app
  • uv run python -m unittest tests.test_http_app tests.test_service
  • uv run python -m unittest
  • uv run --extra dev ruff check --diff .
  • uv run --extra dev ruff check .
  • uv run --extra dev ruff check control_plane/http_app.py control_plane/service.py tests/test_http_app.py tests/test_service.py
  • uv run --extra dev ruff format --check control_plane/http_app.py control_plane/service.py tests/test_http_app.py tests/test_service.py
  • uv run --extra dev mypy control_plane tests
  • uv run /Users/cbusillo/Developer/codex-skills/jetbrains-inspection/scripts/jb-inspect.py closeout --repo "$PWD" --scope changed_files

Review

  • Scout agents confirmed the route can move alone and should pre-authorize operations.read before store access.
  • Final review agents: 3/3 reported no blocking findings and merge-ready.

Notes

  • Full ruff format --check . still reports existing repo-wide formatting drift outside touched files; touched files are formatted.

Refs #1322

@cbusillo cbusillo merged commit 9fa0028 into main Jun 17, 2026
12 checks passed
@cbusillo cbusillo deleted the v2-fastapi-recent-operations-read branch June 17, 2026 17:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant