fix(miner): treat null listClaims filters as unscoped#2978
Conversation
listClaims no longer throws on null repo or status filters; null and omitted filters both return all matching rows. Co-authored-by: Cursor <cursoragent@cursor.com>
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #2978 +/- ##
=======================================
Coverage 96.06% 96.06%
=======================================
Files 259 259
Lines 28663 28663
Branches 10431 10431
=======================================
Hits 27536 27536
Misses 490 490
Partials 637 637 🚀 New features to boost your workflow:
|
|
Warning 🟨🟨🟨🟨🟨🟨🟨🟨🟨🟨🟨🟨 ⏸️ Gittensory review result - manual review recommendedReview updated: 2026-07-04 08:06:25 UTC
⏸️ Suggested Action - Manual Review Review summary Nits — 3 non-blocking
Review context
Contributor next steps
Signal definitions
🟩 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 Gittensory, a quiet PR intelligence layer for OSS maintainers.
|
Summary
repoFullNameandstatusfilters inlistClaimsas unscoped, matching governor-ledger and event-ledger read semantics.{ repoFullName: null }from throwing and{ status: null }from being rejected asinvalid_status.Scope
type(scope): short summaryConventional Commit format.Validation
npm run test:cilocally — CI will run the full gate on push.Safety
UI Evidence
Not applicable — miner library only.
Notes
Follow-up consistency fix after #2955 null read-filter normalization in the event ledger.