feat(engine): add blockedLabels to miner goal spec and lane fit#2729
Conversation
Let maintainers declare issue labels miners must skip; lane-fit returns zero when a candidate carries a blocked label, mirroring blockedPaths for discovery metadata. 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 #2729 +/- ##
==========================================
+ Coverage 96.03% 96.06% +0.03%
==========================================
Files 244 244
Lines 27255 27303 +48
Branches 9902 9919 +17
==========================================
+ Hits 26173 26228 +55
+ Misses 457 445 -12
- Partials 625 630 +5
🚀 New features to boost your workflow:
|
|
Warning 🟨🟨🟨🟨🟨🟨🟨🟨🟨🟨🟨🟨 ⏸️ Gittensory review result - manual review recommendedReview updated: 2026-07-03 08:50:58 UTC
⏸️ Suggested Action - Manual Review Review summary Nits — 5 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.
|
Add root vitest coverage for computeMinerGoalLaneFit blocked-label exclusion so codecov/patch hits the changed lines in miner-goal-lane-fit.ts. Co-authored-by: Cursor <cursoragent@cursor.com>
Summary
blockedLabelstoMinerGoalSpec(.gittensory-miner.yml) as the label analogue ofblockedPathsblockedLabelsin both engine parsers with the same tolerant string-list normalization aspreferredLabels0when an issue carries a blocked label (case-insensitive), so discovery ranking skips those candidatesScope
type(scope): short summaryConventional Commit format, for examplefix(api): restore profile access checks.CONTRIBUTING.mdand does not reintroduce GitHub Pages, VitePress,site/, orCNAME.Validation
git diff --checknpm run actionlintnpm run typechecknpm run test:coveragelocally;codecov/patchrequires ≥99% coverage of the lines AND branches you changed (aim for 100% on your diff so CI variance does not fail near the threshold). Global coverage is a non-blocking trend with a loose 90% backstop, not the gate.npm run test:workersnpm run build:mcpnpm run test:mcp-packnpm run ui:openapi:checknpm run ui:lintnpm run ui:typechecknpm run ui:buildnpm audit --audit-level=moderateIf any required check was skipped, explain why:
npm test --workspace @jsonbored/gittensory-engine) andtest/unit/miner-goal-spec-parser.test.tspassed locally. Fullnpm run test:cinot re-run for this engine-only diff.Safety
UI Evidencesection below with JPG/JPEG or PNG screenshots arranged as organized, captioned, clickable thumbnails. SVG screenshots are not used as review evidence. Review-only screenshots or recordings are not committed to the repository.UI Evidence
N/A — engine/miner config only; no visible UI changes.
Notes
feat/mcp-issue-slop-cli— no overlapping files, so both PRs can merge in either order without conflict.