Repository navigation
Maple Review Bot / Maple / review
succeeded
Oct 31, 2026 in 58s
Confidence 10/10 · No issues found
🟢 Confidence 10/10 · safe to merge. Follow-up hunks are small and pinned by scene/tool tests; the only contract change, a required pageDraft, is set by all four incident tools.
no findings · tests covered · risk medium
Follow-up to the status-pages rollout: the MCP incident tools now say when the page is a draft, a failed bulk component add closes its panel with an error toast, and a draft page's address renders as text with no Open link. Safe to merge.
StatusPageIncidentMutationOutputgains a requiredpageDraft, andrenderIncidentwarns the incident stays private- The editor drops
addFailure: a failed add closes the add panel and toasts, so a retry cannot duplicate view-livesuppresses the public link and Open button for a draft page
Before merge
- Manual · Set
"statuspages": truein thePrevieworg's Clerk public metadata on the development instance before testing a preview of this stack
Production impact
Production traffic of the changed files (last 7 days)
| File | Calls/day | Busiest operations |
|---|---|---|
apps/api/src/routes/v2/status-pages.http.ts |
10 | preview 10/day, 26.4% err |
packages/domain/src/http/v2/status-pages.ts |
10 | preview 10/day, 26.4% err |
What was checked
StatusPagePublisherskips a draft, so a draft's address really does not resolve (StatusPagePublisher.ts:106)- All four incident tools build
pageDraftfrom the resolved page (status-pages.ts:527,582,623,670) - MCP audit actions match the REST routes' names (
status-pages.http.ts:610,681)
3a56bcc · Reply "won't fix" to dismiss a finding, or mention @maple-review-bot to ask about one.
Loading