Skip to content

Document Copilot review process and default reviewer ownership - #2

Draft
antuz123 wants to merge 4 commits into
mainfrom
codex/relay-review-owners
Draft

antuz123 wants to merge 4 commits into
mainfrom
codex/relay-review-owners

Conversation

@antuz123

@antuz123 antuz123 commented Oct 7, 2026 •

Copy link
Copy Markdown
Contributor

Add Ben, Utkarsh, and Parsh as default code owners for future PRs into main. Any one of them may provide the required developer approval. The active branch ruleset separately requires resolved review conversations, compile/test and secret-scan checks. Automatic Copilot review of ready PRs remains enabled; automatic draft and every-push reviews are disabled.

Access dependency: Ben's and Parsh's write-access invitations have been sent and are pending acceptance. Keep this PR draft until GitHub recognizes all three owners and checks pass. Existing R3 PRs have Utkarsh requested; Ben and Parsh will be requested when their invitations are accepted.

Document the agreed effort, review timing, and evidence requirements in .github/REVIEW_POLICY.md, .github/PULL_REQUEST_TEMPLATE.md, and lean root AGENTS.md; link the existing contribution guide. Routine changes use Lite. Keys, signing, recovery, backup, authentication, pairing, transactions, migrations, security-critical copy/dependencies/permissions, complex or cross-service changes, and unknown risk use Balanced. Select effort before the first request or ready transition; batch material fixes and stabilize checks before requesting another review.

Review record:

  • Repository default: explicit Balanced, verified in GitHub settings on 7 October 2026.
  • Intended effort for this PR: Balanced because the existing CODEOWNERS proposal concerns review ownership. Verify the PR's saved effort before a new request or ready transition; remembered PR choices can override defaults.
  • Actual historical Copilot review: Balanced at d86b0ef2cf25700d63a0ddd11cf281fb2b4dce41. Its substantive overview concern about Ben's access remains tracked by the pending invitation dependency. The new documentation head is not claimed as reviewed.
  • No new Copilot review requested for the documentation additions. The requester quota is exhausted; quota responses are unavailable reviews.
  • One independent approval from Ben, Utkarsh, or Parsh plus current required checks is required. Copilot does not satisfy human approval, and all three approvals are unnecessary.

Verification: documentation links, formatting, final newlines, scoped diff, and clean local/remote branch comparisons passed. No runtime files, CI gates, billing, or permissions changed. Agent verification remains proportionate before delivery; human live verification remains retrospective.

@antuz123
antuz123 requested review from cakesoft-utkarsh and a balanced review from Copilot October 7, 2026 12:25

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot review overview

🔵 Needs a closer look

The PR must remain draft until Ben’s repository access is explicitly approved and verified.

Review effort: Balanced
Findings: None

What changed in this PR

Adds Ben and Utkarsh as default code owners for future Relay PRs into main.

Changes:

  • Adds a catch-all ownership rule for both developers.
  • Documents the separately enforced one-approval requirement.
File Description
.github/​CODEOWNERS Assigns both developers as owners of all files.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

@antuz123 antuz123 changed the title Request Ben and Utkarsh on future Relay PRs Request Ben, Utkarsh, and Parsh on future Relay PRs Oct 7, 2026
@antuz123 antuz123 changed the title Request Ben, Utkarsh, and Parsh on future Relay PRs Document Copilot review process and default reviewer ownership Oct 7, 2026

Copy link
Copy Markdown
Contributor Author

Cloud review-policy dependency check — 12 October 2026

Current source head 93d56a43029e1886ea8239f65734debe48d3a190 is unchanged; exact-head hosted CI is successful. Public GitHub permission read reports Ben=read, Parsh=read, Utkarsh=admin. Read access does not establish accepted CODEOWNERS write access. Keep this unapproved PR draft while the documented access dependency remains open; verify all intended owners before ready/merge. No invitation, role, review effort/quota, permission or branch-protection setting was changed. No fresh Copilot review was requested. Documentation review does not need native Mac testing, and no source-policy approval substitutes for app/Desktop/backend or release acceptance.

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