-
Notifications
You must be signed in to change notification settings - Fork 0
[Fleet incident] Resolve independent-review bootstrap deadlock for sole-collaborator repositories #948
Copy link
Copy link
Open
Labels
area: apiAPI, protocol, event, or external contractAPI, protocol, event, or external contractarea: authAuthentication, authorization, identity, or tenant isolationAuthentication, authorization, identity, or tenant isolationarea: ci-cdCI, GitHub Actions, checks, release, or supply chainCI, GitHub Actions, checks, release, or supply chainarea: securitySecurity boundary, hardening, or vulnerability preventionSecurity boundary, hardening, or vulnerability preventionbugSomething isn't workingSomething isn't workingpriority: mediumNormal-priority or P2 workNormal-priority or P2 workstatus: blockedBlocked by conflict, dependency, or required prerequisiteBlocked by conflict, dependency, or required prerequisitetype: bugDefect or incorrect behaviorDefect or incorrect behaviortype: featureNew or expanded product capabilityNew or expanded product capability
Description
Activity
Metadata
Metadata
Assignees
Labels
area: apiAPI, protocol, event, or external contractAPI, protocol, event, or external contractarea: authAuthentication, authorization, identity, or tenant isolationAuthentication, authorization, identity, or tenant isolationarea: ci-cdCI, GitHub Actions, checks, release, or supply chainCI, GitHub Actions, checks, release, or supply chainarea: securitySecurity boundary, hardening, or vulnerability preventionSecurity boundary, hardening, or vulnerability preventionbugSomething isn't workingSomething isn't workingpriority: mediumNormal-priority or P2 workNormal-priority or P2 workstatus: blockedBlocked by conflict, dependency, or required prerequisiteBlocked by conflict, dependency, or required prerequisitetype: bugDefect or incorrect behaviorDefect or incorrect behaviortype: featureNew or expanded product capabilityNew or expanded product capability
Type
Projects
- StatusShow more project fieldsTodo
Live policy deadlock
Fresh protected-branch rule evidence for
ContextualWisdomLab/.github/mainrequires:The live collaborators API returns exactly one human collaborator,
seonghobae, who is also the author of the current central repair pull requests. Self-approval is not a qualifying independent review.PR #835 at exact head
6a1acb1559755c27d7267e39fd6275bb9db74dc9has every observed exact-head quality/security/supply-chain workflow successful, all review threads resolved, auto-merge enabled, and one formal OpenCodeAPPROVEDreview. GitHub still reportsmergeable_state=blocked, which is consistent with the unsatisfied second approval.PR #834 repairs the Noema OIDC response-envelope consumer that is intended to supply a second independent review identity, but #834 is governed by the same two-approval rule and currently has no qualifying approval. Therefore the broken second-review path cannot bootstrap its own repair.
The same condition appears in sole-collaborator product repositories such as EmbedRelay: exact-head code/security evidence can be green, but the standard reviewer-request API cannot request a non-collaborator App identity, and the central mention/OIDC route remains the only scalable independent-review path.
Root cause
The repository ruleset assumes at least two simultaneously operational independent reviewer identities, while organization access and reviewer control-plane deployment currently provide only one functioning formal reviewer identity (
opencode-agent) in addition to the author. The second reviewer (cwl-noema-review) is itself impaired by the unmerged OIDC envelope and mention-concurrency repairs.This is not a code-quality failure in #834/#835 and not a reason to reduce the required approval count. It is an identity/bootstrap architecture gap between branch policy and installed reviewer capacity.
Feasible remedies
Revalidate live organization policy and use one of these normal-policy options:
Independent human bootstrap reviewer — immediate and least architectural change
Second review-only GitHub App identity — durable option
APPROVEsubmission;Organization team reviewer capacity
Rejected non-remedies
required_approving_review_countfrom 2;COMMENTED, status checks, mention acknowledgements, or resolved threads as approval;contents: write, merge, release, or deployment authority to a reviewer merely to satisfy the count;COPILOT_GITHUB_TOKEN;Acceptance criteria
This issue is part of the organization-wide Hourly Product Development fleet incident. The central
.githubowner loop retains all implementation and merge authority. The fleet audit remains read-only and reports this because an external independent identity or permission grant is genuinely required to break the bootstrap deadlock.