Skip to content

feat(settings): add advisory reviewer routing - #2

Open
statxc wants to merge 1 commit into
split/830-pr2-reviewer-routingfrom
split/830-pr3-reviewer-routing-advisory
Open

feat(settings): add advisory reviewer routing#2
statxc wants to merge 1 commit into
split/830-pr2-reviewer-routingfrom
split/830-pr3-reviewer-routing-advisory

Conversation

@statxc

@statxc statxc commented Jun 24, 2026

Copy link
Copy Markdown
Owner

Summary

  • add reviewer routing settings, schema, migration, and OpenAPI plumbing for off|advisory
  • load repository CODEOWNERS and render suggested reviewers in the PR panel
  • keep this PR advisory-only with no live reviewer-request write path

Stack

Why

Part 3 of the JSONbored#830 stack. This exposes the advisory routing surface without mixing it with the higher-risk auto-request behavior.

Validation

  • npm run typecheck
  • npm run db:migrations:check
  • npm run ui:openapi:check
  • npx vitest run test/unit/codeowners.test.ts test/unit/reviewer-routing.test.ts test/unit/focus-manifest.test.ts test/unit/settings-preview.test.ts test/unit/unified-comment-parity.test.ts test/unit/openapi.test.ts test/integration/api.test.ts
  • npx vitest run test/unit/maintainer-activation.test.ts
  • npx vitest run test/unit/signals.test.ts test/unit/signals-v2.test.ts test/unit/signals-coverage.test.ts
  • npx vitest run test/unit/policy-sanitizer.test.ts test/unit/registration-readiness.test.ts test/unit/repo-policy-readiness.test.ts test/unit/repository-settings-enforcement.test.ts test/unit/self-dogfood-registration-pack.test.ts

@statxc
statxc force-pushed the split/830-pr3-reviewer-routing-advisory branch from 0af01a4 to 8976660 Compare June 24, 2026 23:32
@statxc
statxc force-pushed the split/830-pr2-reviewer-routing branch from 777877a to 7410018 Compare June 24, 2026 23:32
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