docs(ams): port docs/observability.md to a website docs page (docs.ams-observability.tsx) - #6331
Conversation
…s-observability.tsx) Add content/docs/ams-observability.mdx covering the AMS Grafana datasources (attempt-log + prediction-ledger reporting exports), dashboard loading, opt-in Prometheus textfile metrics, and the opt-in anonymized central telemetry export -- 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. Repoint the shared AmsObservabilityCallout component (embedded in self-hosting-operations, miner-quickstart, and miner-workflow) from a GitHub-blob link at the raw markdown to an in-app Link at the new website page, per the epic's "update any internal cross-references" instruction -- mock Link in its test the same way site-header.test.tsx already does. packages/loopover-miner/docs/observability.md stays as the canonical source (ships inside the published @loopover/miner package) with a short pointer added to the new website page.
|
Superagent didn't find any vulnerabilities or security issues in this PR. |
|
Tip ✅ LoopOver review result - approve/merge recommendedReview updated: 2026-07-16 02:18:59 UTC
Review summary Nits — 5 non-blocking
Decision drivers
Context & advisory signals — never blocks the verdict
Linked issue satisfactionAddressed 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://gittensory.aethereal.dev/docs/loopover-commands 🧪 Experimental — new and may change. 🟩 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-observability.mdxportingpackages/loopover-miner/docs/observability.mdto the website: pointing Grafana at the redacted AMS attempt-log/prediction-ledger reporting exports, loading a dashboard, opt-in Prometheus textfile metrics, and the opt-in anonymized central telemetry export — 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/operations-runbook.md to a website docs page (docs.ams-operations-runbook.tsx) #6023.apps/loopover-ui/src/routes/docs.ams-observability.tsx, a thin loader +docsClientLoaderroute matching every other migrated docs page.docs-nav.tsx's "AMS: deployment" subgroup (alongside the deployment guide and operations runbook) and todocs.index.tsx's Maintainers audience card.AmsObservabilityCalloutcomponent (embedded inself-hosting-operations,miner-quickstart, andminer-workflow) from a GitHub-blob link at the raw markdown to an in-appLinkat the new website page, per the epic's "update any internal cross-references" instruction. Its test mocksLinkthe same waysite-header.test.tsxalready does.packages/loopover-miner/docs/observability.mdstays as the canonical source: it ships inside the published@loopover/minernpm package and its content is directly referenced by other tests. A short pointer to the new website page was added at the top.Scope
type(scope): short summaryConventional Commit format.apps/loopover-ui/**, plus a documentation pointer inpackages/loopover-miner/docs/observability.md.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: 894 test files / 17,168 tests passeddocs.ams-observability-callout.test.tsxre-run in isolation — all 5 tests pass with the newLink-based targetSafety
Notes
hrefswap within an already-shipped component.