Skip to content

feat(review): make the owner-PR close-exemption per-repo configurable (closeOwnerAuthors) - #1556

Merged
JSONbored merged 1 commit into
mainfrom
chore/reconcile-close-owner
Jun 26, 2026
Merged

feat(review): make the owner-PR close-exemption per-repo configurable (closeOwnerAuthors)#1556
JSONbored merged 1 commit into
mainfrom
chore/reconcile-close-owner

Conversation

@JSONbored

Copy link
Copy Markdown
Owner

Reconciles d6a4846b onto main — final batch piece. Per-repo closeOwnerAuthors toggle (default false = owner PRs exempt from auto-close, byte-identical to today). Migration renumbered 0073→0076; wired the field into the ai-review route response (main's response evolved past d6a4846b's base). Full suite 4735 + 100% patch coverage.

… (closeOwnerAuthors)

Reconciles d6a4846b onto main. Per-repo `closeOwnerAuthors` (default false = owner/maintainer PRs
exempt from auto-close, merge-or-manual-hold only; true = closeable like a contributor's, still gated
by the close autonomy class + adverse signals). Migration renumbered 0073→0076. Also wires the field
into the maintainer ai-review route response (main's response had evolved past d6a4846b's base).
@dosubot dosubot Bot added the size:M label Jun 26, 2026
@superagent-security

Copy link
Copy Markdown
Contributor

Superagent didn't find any vulnerabilities or security issues in this PR.

@codecov

codecov Bot commented Jun 26, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 95.42%. Comparing base (2909063) to head (5bad965).
✅ All tests successful. No failed tests found.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #1556   +/-   ##
=======================================
  Coverage   95.42%   95.42%           
=======================================
  Files         202      202           
  Lines       21672    21674    +2     
  Branches     7831     7833    +2     
=======================================
+ Hits        20681    20683    +2     
  Misses        416      416           
  Partials      575      575           
Files with missing lines Coverage Δ
src/api/routes.ts 94.62% <ø> (ø)
src/db/repositories.ts 96.13% <100.00%> (+<0.01%) ⬆️
src/db/schema.ts 68.32% <ø> (ø)
src/queue/processors.ts 87.61% <ø> (ø)
src/settings/agent-actions.ts 92.79% <100.00%> (+0.06%) ⬆️
src/signals/focus-manifest.ts 99.12% <100.00%> (ø)
🚀 New features to boost your workflow:
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@JSONbored
JSONbored merged commit 1c6ce71 into main Jun 26, 2026
19 checks passed
@JSONbored
JSONbored deleted the chore/reconcile-close-owner branch June 26, 2026 22:59
JSONbored added a commit that referenced this pull request Jun 27, 2026
JSONbored added a commit that referenced this pull request Jun 27, 2026
#1393)

* fix(orb): bind enrollment admin checks to account ids

* fix(migrations): renumber orb account_id migration to 0077 (resolve 0075/0076 collision with #1555/#1556)
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