docs(miner): add the repo-agnostic capability audit checklist#5359
Conversation
Audits the miner's discovery/claim/scoring code for hardcoded/gittensory-specific assumptions that must become per-tenant config, organized by file/module with file:line citations, current override status, and the target config for each. Feeds the de-hardcoding work in JSONbored#4784. Key findings: the code is already largely repo-agnostic (caller-supplied discovery queries, generic config label preferences, engine-delegated + config-overridable scoring); the real gaps are GitHub-forge hardcoding in opportunity-fanout.js, an apiBaseUrl override that isn't threaded to discover-cli, and engine label/goal-spec defaults that fall back to gittensory's conventions. Audit and document only — no code changes. Closes JSONbored#4780
|
Superagent didn't find any vulnerabilities or security issues in this PR. |
|
Tip 🟩🟩🟩🟩🟩🟩🟩🟩🟩🟩🟩🟩 ✅ Gittensory review result - approve/merge recommendedReview updated: 2026-07-12 14:58:17 UTC
✅ Suggested Action - Approve/Merge
Review summary Nits — 6 non-blocking
Linked issue satisfactionAddressed 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 Gittensory, a quiet PR intelligence layer for OSS maintainers.
|
Summary
Adds the repo-agnostic capability audit for the miner package (#4780) — a checklist, organized by file/module, of every hardcoded or implicitly gittensory-specific assumption in the miner's discovery/claim/scoring code that must become per-tenant configuration. This is the deliverable that #4784 executes directly; audit-and-document only, no code changes (per the issue's boundaries).
Doc:
packages/gittensory-miner/docs/repo-agnostic-capability-audit.md.Each finding carries a
file:linecitation, its category, current override status, and the target per-tenant config. Key conclusions:.gittensory-miner.ymlusesbug/enhancement, notgittensor:*), and ranking/feasibility are engine-delegated with config-overridable inputs. The audit explicitly lists what's already parameterized so De-hardcode discovery from gittensory's own conventions #4784 doesn't redo it.opportunity-fanout.js(API version, headers, REST paths, search-qualifier dialect), anapiBaseUrloverride that exists in the fan-out but isn't threaded todiscover-cli, and engine label/goal-spec defaults (DEFAULT_TYPE_LABELS,DEFAULT_MINER_GOAL_SPEC) that silently fall back to gittensory's conventions.Scope
type(scope): short summaryConventional Commit format.CONTRIBUTING.mdand does not reintroduce GitHub Pages, VitePress,site/, orCNAME.Closes #4780.Validation
git diff --checknpm run docs:drift-check(passes — the new doc makes no mechanically-checkable claim that goes stale)file:linecitation in the doc spot-verified against the current source (opportunity-fanout.js4/7/202/241,discover-cli.js:102, enginepr-type-label.ts:26-28)If any required check was skipped, explain why:
.md, no code). Nosrc/**/apps/**/package changes, so Codecov and the backend/UI/MCP CI jobs are path-skipped; there is no code to test.Safety
Notes
Closes #4780