fix(settings): restore draftPrClosePolicy after it was dropped by the autonomy config-as-code PR - #6496
Conversation
|
Superagent didn't find any vulnerabilities or security issues in this PR. |
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #6496 +/- ##
=======================================
Coverage 95.56% 95.56%
=======================================
Files 589 589
Lines 47121 47121
Branches 14989 14989
=======================================
Hits 45032 45032
Misses 1297 1297
Partials 792 792
Flags with carried forward coverage won't be shown. Click here to find out more.
|
|
Caution 🛑 LoopOver review result - fixes requiredReview updated: 2026-07-16 11:33:36 UTC
Review summary Nits — 5 non-blocking
Why this is blocked
📋 Copy for AI agents — paste into your coding agentCI checks failing
Decision drivers
Context & advisory signals — never blocks the verdict
Review context
Contributor next steps
Signal definitions
🧪 Chat with LoopOverAsk LoopOver a question about this PR directly in a comment — grounded only in the same cached, public-safe facts shown above, never a new claim.
Full command reference: https://loopover.ai/docs/loopover-commands 🧪 Experimental — new and may change. 🟩 Safe / merged · 🟦 Advisory · 🟨 Held for review · 🟥 Blocked / closed 💰 Earn for open-source contributions like this. Gittensor lets GitHub contributors earn for the work they already do — register to start earning →. Checked by LoopOver, a quiet PR intelligence layer for OSS maintainers.
|
… autonomy config-as-code PR #773's autonomy config-as-code change (fix(settings): give loopover its own autonomy config-as-code, merged as PR #6468) replaced the same settings: block section in both .loopover.yml and its bundled fallback manifest, silently dropping this repo's own draftPrClosePolicy: close line in the process — the feature (#6454, merged) has been fully live in code since, just disabled for loopover's own repo because nothing enabled it. Re-add draftPrClosePolicy: close alongside (not replacing) the autonomy: block, in both the committed .loopover.yml and its bundled byte-identical twin.
168aac5 to
530ecdc
Compare
Summary
settings:block section in both.loopover.ymland its bundled fallback manifest, silently dropping this repo's owndraftPrClosePolicy: closeline in the process — the feature (feat(review): add an opt-in draft-PR close policy to stop bot-review farming #6454, merged) has been fully live in code since, just disabled for loopover's own repo because nothing enabled it.draftPrClosePolicy: closealongside (not replacing) theautonomy:block, in both the committed.loopover.ymland its bundled byte-identical twin.Found while investigating the repository_settings/config-as-code interaction as part of a broader cleanup pass (see sibling PRs).
Validation
npm run manifest:drift-check