Skip to content

feat(status-pages): dashboard tab and editor - #1932

Merged
Makisuo merged 7 commits into
status-pages/4-apifrom
status-pages/5-web
Oct 31, 2026
Merged

Makisuo merged 7 commits into
status-pages/4-apifrom
status-pages/5-web

fix(status-pages): list rows link through ui/table rowLink

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

Confidence 6/10 · 2 issues to address

🟡 Confidence 6/10 · needs attention. Held at 6 because a correctness warning is open.
2 warnings −2 · 7 notes · tests covered · risk medium · 8 defects fixed in earlier rounds

Adds the Status pages tab to the alerts hub and the /alerts/status-pages/<id> editor with components, incidents and settings tabs. Safe to merge apart from the nine findings carried over from earlier reviews, none of which this head fixes. The pre-existing McpQueryError issues surfacing through apps/web/src/page/registry.ts are untouched: the diff only registers one more lazy page there.

  • /alerts?tab=status-pages lists pages with worst status, open incidents and publish state
  • /alerts/status-pages/<id> editor with components, incidents and settings tabs
  • Incidents composer, post-update and resolve flows; maintenance windows read in the viewer's zone
  • Break-glass token issue, replace and revoke, plus settings save bar and page delete

Still open from earlier reviews

Production impact

Open errors in the changed files (2)
Issue Service Occurrences File
@maple/mcp/errors/McpQueryError (6 issues) maple-chat 1.3k apps/web/src/page/registry.ts
@maple/mcp/errors/McpQueryError (4 issues) maple-ai 349 apps/web/src/page/registry.ts

After this merges, Maple checks whether they stop.

What was checked
  • Maintenance window conversion pinned by the Europe/Berlin test: 22:00 local becomes 21:00Z
  • API returns incident updates newest-first (StatusPagesService.ts:387), so updates[0] really is the latest
  • Editor remounts per page id (editor/index.ts keys on route.pageId), so routeChanged ignoring the id is safe

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