Skip to content

Add GitHub Actions authz rule removals#1081

Merged
shiny-code-bot merged 2 commits into
mainfrom
add-github-actions-authz-removals
Jun 1, 2026
Merged

Add GitHub Actions authz rule removals#1081
shiny-code-bot merged 2 commits into
mainfrom
add-github-actions-authz-removals

Conversation

@shiny-code-bot
Copy link
Copy Markdown
Collaborator

Summary

  • add a service-backed exact GitHub Actions authz rule removal route
  • expose authz-policies remove-workflow-rule for dry-run/apply cleanup through the service
  • document and test exact-match cleanup so broad deploy grants can be removed without local DB writes

Validation

  • uv run python -m unittest tests.test_authz_grant_service tests.test_every_code_reconciliation.EveryCodeIssueReconciliationTests.test_cli_authz_remove_workflow_rule_posts_service_dry_run_request tests.test_service.LaunchplaneServiceTests.test_authz_policy_removal_endpoint_writes_db_record_and_updates_runtime tests.test_service.LaunchplaneServiceTests.test_authz_policy_removal_endpoint_dry_run_does_not_write_or_reload tests.test_service.LaunchplaneServiceTests.test_authz_policy_removal_endpoint_rejects_self_deploy_authority
  • uv run --extra dev ruff check control_plane/authz_grant_service.py control_plane/service.py control_plane/cli_policy_profiles.py tests/test_authz_grant_service.py tests/test_service.py tests/test_every_code_reconciliation.py docs/operations.md docs/service-boundary.md
  • npx --no-install markdownlint-cli2 docs/operations.md docs/service-boundary.md
  • git diff --check
  • uv run /Users/cbusillo/Developer/codex-skills/jetbrains-inspection/scripts/jb-inspect.py closeout --repo "$PWD" --scope changed_files

Refs #1049

@shiny-code-bot shiny-code-bot merged commit 0d06804 into main Jun 1, 2026
12 checks passed
@shiny-code-bot shiny-code-bot deleted the add-github-actions-authz-removals branch June 1, 2026 15:22
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.

2 participants