docs(ams): port docs/discovery-plane-operator-guide.md to a website docs page (docs.ams-discovery-plane.tsx) - #6391
Conversation
|
Superagent didn't find any vulnerabilities or security issues in this PR. |
…ocs page (docs.ams-discovery-plane.tsx) Add content/docs/ams-discovery-plane.mdx covering the optional hosted discovery-index plane's opt-in-by-default posture (opposite of Orb's opt-out-only export), the shipped request/response contract vs. the still-provisional opt-in env vars, and the never-included/never-retained data boundaries -- rendered via the existing DocsPage/Callout/CodeBlock/ 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-operations-runbook.mdx's GitHub-blob discovery-plane-operator-guide.md cross-references to the new in-app route. packages/loopover-miner/docs/discovery-plane-operator-guide.md stays as the canonical source (ships inside the published @loopover/miner package) with a short pointer added to the new website page.
c62bb7e to
4762efd
Compare
|
Tip ✅ LoopOver review result - approve/merge recommendedReview updated: 2026-07-16 06:18:05 UTC
Review summary Nits — 6 non-blocking
Decision drivers
Context & advisory signals — never blocks the verdict
Linked issue satisfactionNot yet 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-discovery-plane.mdxportingpackages/loopover-miner/docs/discovery-plane-operator-guide.mdto the website: the optional hosted discovery-index plane's opt-in-by-default posture (opposite of Orb's opt-out-only export), the shipped request/response contract vs. the still-provisional opt-in env vars, and the never-included/never-retained data boundaries — rendered via the existingDocsPage/Callout/CodeBlock/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/env-reference.md to a generated website docs page (docs.ams-env-reference.tsx) #6028.apps/loopover-ui/src/routes/docs.ams-discovery-plane.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-operations-runbook.mdx's GitHub-blobdiscovery-plane-operator-guide.mdcross-references to the new in-app route.packages/loopover-miner/docs/discovery-plane-operator-guide.mdstays as the canonical source (ships inside the published@loopover/minernpm package) with a short pointer added to the new website page.warn-variantCalloutat the top of the page.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, rebased on top of the maintainer's CI cache-hit fix (fix(ci): regenerate fumadocs collections unconditionally in validate-code #6375), green: 906 test files / 17,323 tests passedSafety
Notes