Repository navigation
feat(landing): remote server monitoring guide - #1582
JeremyFunk wants to merge 6 commits into
Confidence 6/10 · 1 issue to address
🟡 Confidence 6/10 · needs attention. Held at 6 because a correctness warning is open.
1 warning −1 · 1 note · tests not needed · risk low
Adds a remote server monitoring guide (agent vs agentless, Collector setup, heartbeat and threshold alert SQL), a sql highlighter, and its index/llms.txt entries. Safe to merge once the Windows guide it links to exists.
guide-highlight.tsgains asqllanguage and stops coloring numbers after.or a word character- Guide covers Collector configs,
http_check/ssh_checkand Raw SQL heartbeat rules - Guides index and
llms.txtlist the new page
Before merge
- Manual · Merge #1581 first, so
/guides/windows-server-monitoringresolves
Findings
🔵 Note · F3 · Tools caption claims prices verified 11 days after the page's last update
correctness · apps/landing/src/pages/guides/remote-server-monitoring.astro:565
The page is dated updated October 20, 2026 at line 385, but the tools table caption says the prices were "last verified October 31, 2026", a date the page cannot carry. Sibling pricing guides derive this from a verified constant rendered in a <time> element (sentry-pricing.astro:20), so the reader sees a verification date later than the update.
Add `const verified = "2026-10-20"` next to `updatedLabel` and render it in the caption as `{verified}`, keeping the page's dates in one place.
Still open from earlier reviews
- 🟠 Warning · F1 · Link to
/guides/windows-server-monitoring404s; that page is not in the repo ·apps/landing/src/pages/guides/remote-server-monitoring.astro:515
Fixed since the last review
- ✅
F2 · Three guide screenshots are referenced but missing frompublic/
Production impact
Production traffic of the changed files (last 7 days)
| File | Calls/day | Busiest operations |
|---|---|---|
apps/landing/src/pages/guides/remote-server-monitoring.astro |
58k | maple 58k/day, 83.0% err |
What was checked
- The three
public/screenshots/guides/remote-server-monitoring-0{1,2,3}.webpfiles are at this head - SQL examples match
metrics_gaugecolumns (datasources.ts:1520) and the$__macros indocs/reference/sql.md - Host statuses and alert defaults in the copy match
format.ts:32andalert-rules.md:110
cfc562a · Reply "won't fix" to dismiss a finding, or mention @maple-review-bot to ask about one.
Annotations
Check notice on line 565 in apps/landing/src/pages/guides/remote-server-monitoring.astro
maple-review-bot / Maple / review
correctness: Tools caption claims prices verified 11 days after the page's last update
The page is dated `updated` October 20, 2026 at line 385, but the tools table caption says the prices were "last verified October 31, 2026", a date the page cannot carry. Sibling pricing guides derive this from a `verified` constant rendered in a `<time>` element (`sentry-pricing.astro:20`), so the reader sees a verification date later than the update.