Skip to content

feat(landing): log parsing guide - #1578

Open
JeremyFunk wants to merge 4 commits into
mainfrom
feat/guide-log-parsing
Open

JeremyFunk wants to merge 4 commits into
mainfrom
feat/guide-log-parsing

feat(landing): add the log parsing guide screenshots

a8109ad
Select commit
Loading
Failed to load commit list.
Maple Review Bot / Maple / review succeeded Oct 20, 2026 in 29s

Confidence 10/10 · No issues found

🟢 Confidence 10/10 · safe to merge
Static content page; the screenshots that broke F1 are committed with matching intrinsic sizes, and only the index card's read time is still out of step.
1 note · tests not needed · risk low

Commits the two guide screenshots that F1 reported missing, alongside the log parsing guide page. Mergeable; the only thing left is the read-time mismatch F2 between the guides index and the page header.

  • log-parsing.astro references both figures, now backed by committed webp files
  • Page still declares 10 min read while guides/index.astro:74 lists 14 min

Still open from earlier reviews

Fixed since the last review

  • ✅ F1 · Two guide screenshots are referenced but not committed

Production impact

Production traffic of the changed files (last 7 days)
File Calls/day Busiest operations
apps/landing/src/pages/guides/log-parsing.astro 99k maple 98k/day, 90.4% err
GET 925/day, 2.6% err
GET 133/day, 0.1% err
What was checked
  • Both webp files exist at the head and their headers report 1342x1454 and 2362x1830, matching the width/height in log-parsing.astro:653,658
  • mine_log_patterns named at line 662 exists (apps/ai/src/mcp/tools/mine-log-patterns.ts:18)
  • data-zoomable on both figures is handled by MediaLightbox.astro:221

a8109ad · Updated on every push. Reply "won't fix" to dismiss a finding, or mention @maple-review-bot to ask about one.