Skip to content

Move merge train reads to FastAPI#1400

Merged
cbusillo merged 1 commit into
mainfrom
v2-fastapi-merge-train-reads
Jun 18, 2026
Merged

Move merge train reads to FastAPI#1400
cbusillo merged 1 commit into
mainfrom
v2-fastapi-merge-train-reads

Conversation

@cbusillo

Copy link
Copy Markdown
Owner

Summary

  • move merge-train admission, controller-status, and policy-target reads to native FastAPI routes
  • retire the matching legacy WSGI read branches and keep direct WSGI calls fail-closed
  • update FastAPI route coverage, WSGI retirement coverage, and v2 compatibility docs

Validation

  • uv run python -m unittest tests.test_http_app.FastApiMergeTrainReadTests tests.test_service.LaunchplaneServiceTests.test_merge_train_reads_are_retired_from_legacy_wsgi_app
  • uv run --extra dev ruff check .
  • uv run --extra dev mypy control_plane tests
  • uv run python -m unittest
  • uv run /Users/cbusillo/Developer/codex-skills/jetbrains-inspection/scripts/jb-inspect.py closeout --repo "/Users/cbusillo/Developer/launchplane" --scope changed_files

Review

  • Code review agents: green (2 useful green reviews; 1 empty/no-op response)

@cbusillo cbusillo merged commit d2bf9ea into main Jun 18, 2026
12 checks passed
@cbusillo cbusillo deleted the v2-fastapi-merge-train-reads branch June 18, 2026 07:51
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