Skip to content

Harden runner registration boundary#1237

Merged
cbusillo merged 1 commit into
mainfrom
feat/durable-runner-maintainer
Jun 8, 2026
Merged

Harden runner registration boundary#1237
cbusillo merged 1 commit into
mainfrom
feat/durable-runner-maintainer

Conversation

@cbusillo

@cbusillo cbusillo commented Jun 8, 2026

Copy link
Copy Markdown
Owner

Summary

  • Keep runner registration non-mutating while hardening the Launchplane-only boundary for the future durable maintainer.
  • Route workflow confirmation/mutate inputs through environment variables, preserve failure artifacts with a fallback result JSON, and stop allowlisting a user-provided registration root.
  • Require explicit GitHub token and Launchplane Actions execution context in the registration CLI before executor work.
  • Validate host/lane/label identifiers before any future path/systemd derivation can use them.

Non-goals

  • This does not add live runner registration, run.sh, nohup, systemd mutation, or product-repo workflow adoption.
  • cm-website remains a proving target only; no product repo files were touched.

Validation

  • uv run python -m unittest tests.test_runner_lane_registration
  • uv run --extra dev ruff format --check control_plane/contracts/runner_lane_registration.py control_plane/workflows/runner_lane_registration_executor.py control_plane/cli_runner_lanes.py tests/test_runner_lane_registration.py
  • uv run --extra dev ruff check control_plane/contracts/runner_lane_registration.py control_plane/workflows/runner_lane_registration_executor.py control_plane/cli_runner_lanes.py tests/test_runner_lane_registration.py
  • uv run --extra dev mypy control_plane/contracts/runner_lane_registration.py control_plane/workflows/runner_lane_registration_executor.py control_plane/cli_runner_lanes.py tests/test_runner_lane_registration.py
  • actionlint -config-file .github/actionlint.yaml .github/workflows/runner-lane-registration.yml
  • git diff --check
  • uv run python -m unittest tests.test_runner_lane_registration tests.test_runner_lane_inventory tests.test_service tests.test_filesystem_store tests.test_postgres_store (619 tests OK)

@cbusillo cbusillo force-pushed the feat/durable-runner-maintainer branch from d711ce9 to 400daf8 Compare June 8, 2026 20:46
@cbusillo cbusillo merged commit 01d7648 into main Jun 8, 2026
12 checks passed
@cbusillo cbusillo deleted the feat/durable-runner-maintainer branch June 8, 2026 20:49
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