Skip to content

Extract runtime key safety service route#425

Merged
cbusillo merged 1 commit intomainfrom
refactor/runtime-key-safety-service-route
May 7, 2026
Merged

Extract runtime key safety service route#425
cbusillo merged 1 commit intomainfrom
refactor/runtime-key-safety-service-route

Conversation

@cbusillo
Copy link
Copy Markdown
Owner

@cbusillo cbusillo commented May 7, 2026

Summary

  • move runtime-key-safety policy apply request validation and route response handling into control_plane/runtime_key_safety_http.py
  • keep service.py responsible for idempotency/app wiring while the runtime-key-safety route details live with the runtime-key-safety boundary
  • reduce control_plane/service.py to 7,589 lines in this slice

Refs #307

Validation

  • uv run python -m unittest tests.test_service.LaunchplaneServiceTests.test_runtime_key_safety_policy_endpoint_reconciles_rules tests.test_service.LaunchplaneServiceTests.test_runtime_key_safety_policy_endpoint_rejects_without_permission tests.test_runtime_key_safety
  • uv run --extra dev ruff check --diff control_plane/service.py control_plane/runtime_key_safety_http.py
  • uv run --extra dev ruff check control_plane/service.py control_plane/runtime_key_safety_http.py
  • uv run --extra dev mypy control_plane/service.py control_plane/runtime_key_safety_http.py

@cbusillo cbusillo merged commit 60e9c66 into main May 7, 2026
12 checks passed
@cbusillo cbusillo deleted the refactor/runtime-key-safety-service-route branch May 7, 2026 21:08
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