feat(miner): add cross-repo evaluation harness (#4788) - #5790
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 #5790 +/- ##
==========================================
+ Coverage 95.06% 95.08% +0.01%
==========================================
Files 580 581 +1
Lines 46156 46310 +154
Branches 14804 14804
==========================================
+ Hits 43880 44035 +155
+ Misses 1516 1515 -1
Partials 760 760
Flags with carried forward coverage won't be shown. Click here to find out more.
🚀 New features to boost your workflow:
|
|
Tip 🟩🟩🟩🟩🟩🟩🟩🟩🟩🟩🟩🟩 ✅ LoopOver review result - approve/merge recommendedReview updated: 2026-07-14 16:41:55 UTC
✅ Suggested Action - Approve/Merge
Review summary Nits — 5 non-blocking
Review context
Contributor next steps
Signal definitions
[BETA] Chat with GittensoryAsk Gittensory 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://gittensory.aethereal.dev/docs/gittensory-commands 🟩 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.
|
Summary
detectRepoStack,resolveMinerGoalSpec, andbuildCodingTaskSpecon each benchmark repo without running the coding agent or calling forge APIs..loopover-miner.yml.Change
packages/loopover-miner/lib/cross-repo-evaluation.js(+.d.ts) — manifest parser, per-repo readiness evaluator, assumption scanner, report formatter, and summary reducer.packages/loopover-miner/benchmarks/cross-repo/manifest.json— six diverse public repos (Node, Python, Rust, Go) with optionalrequireTestCommandflags.packages/loopover-miner/scripts/cross-repo-evaluation.mjs— CLI with--manifest,--repo,--json, and--require-majority.packages/loopover-miner/docs/cross-repo-evaluation.md— operator docs for cloning benchmark repos and reading the report.test/unit/miner-cross-repo-evaluation.test.ts— 29 unit tests covering manifest parsing, taxonomy, evaluator branches, and the script driver.packages/loopover-miner/package.json—cross-repo-evalscript +node --checkfor the new lib module.package.json—cross-repo-eval:minerroot script.packages/loopover-miner/README.md— link to the harness docs.Closes #4788
Scope
type(scope): short summaryConventional Commit format.CONTRIBUTING.mdand does not touchsite/,CNAME, or**/lovable/**.Validation
git diff --checknpm run test:cinpm audit --audit-level=moderatenpm run test -- test/unit/miner-cross-repo-evaluation.test.ts— 29 passingnpm run build --workspace @loopover/miner—node --checkincludeslib/cross-repo-evaluation.jsIf any required check was skipped, explain why:
npm run test:ci+npm audit --audit-level=moderategate locally before push (required for merge).Safety
UI Evidence
No visible UI change. This PR adds a miner evaluation harness, CLI, benchmark manifest, tests, and docs only. No screenshots required.
Notes
LOOPOVER_MINER_REPO_CLONE_DIRfirst (documented indocs/cross-repo-evaluation.md). CI validates the pure evaluator via fixture paths in unit tests.main.