Repository navigation
feat(landing): Pino logger guide - #1573
Merged
Merged
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.astroadds the guide with Node, Express and Fastify snippetsInstallTabs.astrorenders npm/pnpm/bun switchers usingguide-highlightguide-highlightgainsjavascript,jsonandshell; guides index and llms.txt list the page
Before merge
- Env var · Set
LOG_LEVELin every environment, or confirm its default ·apps/landing/src/pages/guides/pino-logger.astro:82
Still open from earlier reviews
- 🔵 Note · F2 ·
InstallTabshardcodes the storage keyinstallVariantsreturns ·apps/landing/src/components/docs/InstallTabs.astro:15 - 🔵 Note · F3 ·
errorTiersfirst tier charges 3.07x Team, not "twice the Team rate" ·apps/landing/src/pages/guides/sentry-pricing.astro:324
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-18andsentry-pricing.astro:324againsterrorTiersat: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.
Loading