Skip to content

feat(pr): add admin merge bypass#130

Merged
ovdmar merged 7 commits into
mainfrom
fb-pr-merge-admin-bypass
Jun 8, 2026
Merged

feat(pr): add admin merge bypass#130
ovdmar merged 7 commits into
mainfrom
fb-pr-merge-admin-bypass

Conversation

@ovdmar

@ovdmar ovdmar commented Jun 7, 2026

Copy link
Copy Markdown
Owner

Summary

  • Add an explicit admin-bypass option to the PR merge contract, daemon route, hooks payload, and GitHub provider merge command.
  • Add an unchecked-by-default Admin bypass control in the PR merge menu that resets after attempts, menu/context changes, and eligibility changes.
  • Update the PR spec and coverage for normal/admin merge payloads, hook templates, route validation, and UI reset behavior.

Plan

.agents/plans/pr-merge-admin-bypass.md

Test plan

  • make check
  • pnpm e2e:isolated e2e/pr-display.spec.ts
  • pnpm vitest run apps/web/src/pr-card-actions.test.ts apps/daemon/src/pr-routes.test.ts packages/contracts/src/index.test.ts packages/providers/src/pr-merge.test.ts packages/operations/src/hooks-runner.test.ts

Generated with Codex

@ovdmar ovdmar merged commit 4aeff94 into main Jun 8, 2026
8 checks passed
@ovdmar ovdmar deleted the fb-pr-merge-admin-bypass branch June 8, 2026 07:40
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