docs(ams): port docs/miner-goal-spec.md to a website docs page (docs.ams-goal-spec.tsx) - #6397
Conversation
|
Superagent didn't find any vulnerabilities or security issues in this PR. |
…ams-goal-spec.tsx) Add content/docs/ams-goal-spec.mdx covering every MinerGoalSpec field (.loopover-miner.yml) -- minerEnabled, wantedPaths/blockedPaths, preferredLabels/blockedLabels, maxConcurrentClaims, issueDiscoveryPolicy, feasibilityGate, selfPlagiarism, killSwitch, and execution -- and its relationship to .loopover.yml, rendered via the existing DocsPage/Callout/ FeatureRow primitives. Add the page to docs-nav.tsx's "AMS: deployment" subgroup and to docs.index.tsx's Maintainers audience card, and repoint ams-deployment.mdx's and ams-config-precedence.mdx's GitHub-blob miner-goal-spec.md cross-references to the new in-app route. packages/loopover-miner/docs/miner-goal-spec.md stays as the canonical source (ships inside the published @loopover/miner package; its content is directly asserted by miner-goal-spec-doc.test.ts) with a short pointer added to the new website page.
d307e94 to
6076b12
Compare
|
Tip ✅ LoopOver review result - approve/merge recommendedReview updated: 2026-07-16 06:44:58 UTC
Review summary Nits — 4 non-blocking
Decision drivers
Context & advisory signals — never blocks the verdict
Linked issue satisfactionPartially addressed Review context
Contributor next steps
Signal definitions
🧪 Chat with LoopOverAsk LoopOver 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://loopover.ai/docs/loopover-commands 🧪 Experimental — new and may change. Visual previewClick any thumbnail to open the full-size screenshot. Before = production · After = this PR's preview deploy. 🟩 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
content/docs/ams-goal-spec.mdxportingpackages/loopover-miner/docs/miner-goal-spec.mdto the website: everyMinerGoalSpecfield (.loopover-miner.yml) —minerEnabled,wantedPaths/blockedPaths,preferredLabels/blockedLabels,maxConcurrentClaims,issueDiscoveryPolicy,feasibilityGate,selfPlagiarism,killSwitch, andexecution— and its relationship to.loopover.yml, rendered via the existingDocsPage/Callout/FeatureRowui-kit primitives, following the pattern set in docs(ams): port DEPLOYMENT.md to a website docs page (docs.ams-deployment.tsx) #6022-docs(ams): port docs/discovery-plane-operator-guide.md to a website docs page (docs.ams-discovery-plane.tsx) #6029.apps/loopover-ui/src/routes/docs.ams-goal-spec.tsx, a thin loader +docsClientLoaderroute matching every other migrated docs page.docs-nav.tsx's "AMS: deployment" subgroup and todocs.index.tsx's Maintainers audience card, and repointsams-deployment.mdx's andams-config-precedence.mdx's GitHub-blobminer-goal-spec.mdcross-references to the new in-app route.packages/loopover-miner/docs/miner-goal-spec.mdstays as the canonical source: it ships inside the published@loopover/minernpm package and its content is directly asserted byminer-goal-spec-doc.test.ts. A short pointer to the new website page was added at the top.Scope
type(scope): short summaryConventional Commit format.CONTRIBUTING.mdand does not reintroduce GitHub Pages, VitePress,site/, orCNAME.Validation
git diff --checknpm run actionlintnpm run typechecknpm run test:coveragelocally — UI-only page addition underapps/loopover-ui/**(outsidecoverage.include);codecov/patchdoes not applynpm 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=moderatenpm run test:cirun locally from a cleannpm ciinstall, green: 908 test files / 17,352 tests passedminer-goal-spec-doc.test.ts— passes unchangedSafety
Notes