Skip to content

feat(landing): Pino logger guide - #1573

Merged
JeremyFunk merged 41 commits into
mainfrom
feat/guide-pino-logger
Oct 31, 2026
Merged

JeremyFunk merged 41 commits into
mainfrom
feat/guide-pino-logger

Merge remote-tracking branch 'origin/main' into feat/guide-pino-logger

c72b723
Select commit
Loading
Failed to load commit list.
Maple Review Bot / Maple / review succeeded Oct 31, 2026 in 15s

Confidence 8/10 · No issues found

🟢 Confidence 8/10 · likely safe to merge. Held at 8 because earlier reviews of this pull request found 1 defect.
2 notes · tests not needed · risk low · 1 defect fixed in earlier rounds

Adds the Pino logger guide at /guides/pino-logger, its three screenshots, an index entry, llms.txt coverage and an InstallTabs component for package-manager commands. Content-only; the two open findings still stand at this head.

  • pino-logger.astro adds the guide with Node, Express and Fastify snippets
  • InstallTabs.astro renders npm/pnpm/bun switchers using guide-highlight
  • guide-highlight gains javascript, json and shell; guides index and llms.txt list the page

Before merge

Still open from earlier reviews

Production impact

Production traffic of the changed files (last 7 days)
File Calls/day Busiest operations
apps/landing/src/pages/guides/pino-logger.astro 59k maple 59k/day, 83.1% err
Ingest cost of new log lines
  • apps/landing/src/pages/guides/pino-logger.astro:61: If this logs once per call, about 473 MB/month: the file's operations run 59k times a day.
  • apps/landing/src/pages/guides/pino-logger.astro:62: If this logs once per call, about 473 MB/month: the file's operations run 59k times a day.
  • apps/landing/src/pages/guides/pino-logger.astro:63: If this logs once per call, about 473 MB/month: the file's operations run 59k times a day.
  • apps/landing/src/pages/guides/pino-logger.astro:64: If this logs once per call, about 473 MB/month: the file's operations run 59k times a day.
  • apps/landing/src/pages/guides/pino-logger.astro:84: If this logs once per call, about 473 MB/month: the file's operations run 59k times a day.
  • apps/landing/src/pages/guides/pino-logger.astro:87: If this logs once per call, about 473 MB/month: the file's operations run 59k times a day.
  • apps/landing/src/pages/guides/pino-logger.astro:90: If this logs once per call, about 473 MB/month: the file's operations run 59k times a day.
  • apps/landing/src/pages/guides/pino-logger.astro:98: If this logs once per call, about 473 MB/month: the file's operations run 59k times a day.
  • apps/landing/src/pages/guides/pino-logger.astro:115: If this logs once per call, about 473 MB/month: the file's operations run 59k times a day.
  • apps/landing/src/pages/guides/pino-logger.astro:116: If this logs once per call, about 473 MB/month: the file's operations run 59k times a day.
What was checked
  • Open findings re-read at the head: InstallTabs.astro:15-18 and sentry-pricing.astro:324 against errorTiers at :62-64
  • The three webp figures the earlier review reported missing are now added under public/screenshots/guides/

c72b723 · Reply "won't fix" to dismiss a finding, or mention @maple-review-bot to ask about one.