Skip to content

feat(landing): remote server monitoring guide - #1582

Open
JeremyFunk wants to merge 6 commits into
mainfrom
feat/guide-remote-server-monitoring
Open

JeremyFunk wants to merge 6 commits into
mainfrom
feat/guide-remote-server-monitoring

feat(landing): remote server monitoring intro, key takeaways and tool…

cfc562a
Select commit
Loading
Failed to load commit list.
Maple Review Bot / Maple / review completed Oct 31, 2026 in 1m 1s

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.ts gains a sql language and stops coloring numbers after . or a word character
  • Guide covers Collector configs, http_check/ssh_check and Raw SQL heartbeat rules
  • Guides index and llms.txt list the new page

Before merge

  • Manual · Merge #1581 first, so /guides/windows-server-monitoring resolves

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

Fixed since the last review

  • ✅ F2 · Three guide screenshots are referenced but missing from public/

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}.webp files are at this head
  • SQL examples match metrics_gauge columns (datasources.ts:1520) and the $__ macros in docs/reference/sql.md
  • Host statuses and alert defaults in the copy match format.ts:32 and alert-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

See this annotation in the file changed.

@maple-review-bot 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.