fix(rules): match label multipliers with scoring parity in advisory#2526
Conversation
Advisory label_context_found used exact case-sensitive keys, missing labels that scoring already matches via labelMatchesPattern glob semantics. 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 #2526 +/- ##
==========================================
+ Coverage 95.95% 95.97% +0.02%
==========================================
Files 226 226
Lines 25387 25533 +146
Branches 9234 9293 +59
==========================================
+ Hits 24359 24505 +146
Misses 417 417
Partials 611 611
🚀 New features to boost your workflow:
|
|
Warning 🟨🟨🟨🟨🟨🟨🟨🟨🟨🟨🟨🟨 ⏸️ Gittensory review result - manual review recommendedReview updated: 2026-07-02 08:22:14 UTC
⏸️ Suggested Action - Manual Review
Review summary Nits — 4 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
buildPullRequestAdvisorysurfacedlabel_context_foundvia exactrepoMultipliers[label]lookup.labelMatchesPattern(case-insensitive fnmatch glob semantics), so advisory missed labels likeFeaturewhen config hadfeature, ortype:bug-fixwhen config hadtype:*.labelMatchesPatternhelper so predicted-gate and live gate advisories agree with scoring.Scope
type(scope): short summaryConventional Commit format.CONTRIBUTING.mdand does not reintroduce GitHub Pages, VitePress,site/, orCNAME.Validation
git diff --checknpx vitest run test/unit/rules.test.ts -t "label multipliers"— passing (case + glob regressions)Safety
Notes
scoring/preview.ts.Made with Cursor