Skip to content

Serve operator UI at root route#59

Merged
cbusillo merged 1 commit into
mainfrom
ui/root-operator-route
Apr 29, 2026
Merged

Serve operator UI at root route#59
cbusillo merged 1 commit into
mainfrom
ui/root-operator-route

Conversation

@cbusillo
Copy link
Copy Markdown
Owner

Summary

  • serve the operator UI app shell from /
  • keep /ui and /ui/* as compatibility routes for the existing UI path and client routes
  • preserve /v1/... as the stable API namespace
  • document the root UI route and add focused service coverage

Verification

  • uv run python -m unittest tests.test_service.LaunchplaneServiceTests.test_root_route_serves_ui_shell_without_authentication tests.test_service.LaunchplaneServiceTests.test_ui_route_serves_static_shell_without_authentication tests.test_service.LaunchplaneServiceTests.test_ui_route_falls_back_to_shell_for_nested_paths tests.test_service.LaunchplaneServiceTests.test_ui_asset_route_rejects_parent_directory_segments passed
  • uv run python -m unittest passed, 422 tests
  • npx pnpm@10.10.0 --dir frontend validate passed
    • local Node emits the expected engine warning because this machine is on Node v25; CI uses Node 22
  • git diff --check passed

@cbusillo cbusillo merged commit e011908 into main Apr 29, 2026
9 checks passed
@cbusillo cbusillo deleted the ui/root-operator-route branch April 29, 2026 14:37
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