Skip to content

Add shared/global contributor blacklist (#1425)#1531

Open
tmimmanuel wants to merge 2 commits into
JSONbored:mainfrom
tmimmanuel:fix-1425-shared-contributor-blacklist
Open

Add shared/global contributor blacklist (#1425)#1531
tmimmanuel wants to merge 2 commits into
JSONbored:mainfrom
tmimmanuel:fix-1425-shared-contributor-blacklist

Conversation

@tmimmanuel

Copy link
Copy Markdown

Summary

  • Added shared/global contributor blacklist persistence via new migration-backed singleton table global_contributor_blacklist and DB read/write APIs.
  • Wired global blacklist into repository settings resolution so effective settings are .gittensory.yml/DB + shared/global list.
  • Added focused unit coverage for DB round-trip, resolution merge behavior, and a self-tune path regression guard.

Validation

  • npm run db:migrations:check
  • npm run typecheck
  • npm test -- test/unit/contributor-blacklist.test.ts test/unit/focus-manifest.test.ts test/unit/selftune-readback.test.ts
  • npm run test:unit

Closes #1425

@tmimmanuel tmimmanuel requested a review from JSONbored as a code owner June 26, 2026 15:30
@dosubot dosubot Bot added the size:M This PR changes 30-99 lines, ignoring generated files. label Jun 26, 2026
@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.47%. Comparing base (7c7a37e) to head (aac6d5a).
✅ All tests successful. No failed tests found.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #1531   +/-   ##
=======================================
  Coverage   95.47%   95.47%           
=======================================
  Files         196      196           
  Lines       21197    21208   +11     
  Branches     7662     7666    +4     
=======================================
+ Hits        20237    20248   +11     
  Misses        383      383           
  Partials      577      577           
Files with missing lines Coverage Δ
src/db/repositories.ts 96.14% <100.00%> (+0.02%) ⬆️
src/settings/repository-settings.ts 100.00% <100.00%> (ø)
src/signals/focus-manifest.ts 98.14% <100.00%> (+<0.01%) ⬆️
🚀 New features to boost your workflow:
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@superagent-security

Copy link
Copy Markdown

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size:M This PR changes 30-99 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

feat(anti-abuse): configurable contributor blacklist — auto-label + auto-close banned logins

1 participant