Repository navigation
feat(mcp): SLO tools - #2297
Merged
Merged
feat(mcp): SLO tools#2297
Maple Review Bot / Maple / review
succeeded
Nov 6, 2026 in 13s
Confidence 10/10 · No issues found
🟢 Confidence 10/10 · safe to merge. The delta file has no diff versus base and the rest was reviewed with no open findings.
no findings · tests not needed · risk low
No reviewable change remains at this head: the only file listed as updated since the last review, AlertsService.slo.test.ts, is identical to the base (git diff against 2f4e shows no changes), and every other file was reviewed at f955bfe with no finding left open.
Production impact
Open errors in the changed files (2)
| Issue | Service | Occurrences | File |
|---|---|---|---|
| @maple/mcp/errors/McpQueryError (5 issues) | maple-chat | 33 | apps/ai/src/mcp/tools/registry.ts |
| @maple/mcp/errors/McpQueryError (5 issues) | maple-ai | 15 | apps/ai/src/mcp/tools/registry.ts |
After this merges, Maple checks whether they stop.
Production traffic of the changed files (last 7 days)
| File | Calls/day | Busiest operations |
|---|---|---|
apps/ai/src/mcp/tool-descriptors.generated.ts |
3.8k | POST 2.2k/day, 0.0% errPOST 750/day, 0.4% errGET 440/day, 0.0% err |
Telemetry this change adds and removes (6)
- ➕ span name
McpTool.listSlos·apps/ai/src/mcp/tools/slos.ts:165 - ➕ attribute
maple.slo.status_error·apps/ai/src/mcp/tools/slos.ts:178 - ➕ span name
McpTool.inspectSlo·apps/ai/src/mcp/tools/slos.ts:286 - ➕ span name
McpTool.createSlo·apps/ai/src/mcp/tools/slos.ts:376 - ➕ span name
McpTool.updateSlo·apps/ai/src/mcp/tools/slos.ts:421 - ➕ span name
McpTool.deleteSlo·apps/ai/src/mcp/tools/slos.ts:468
What was checked
- git diff base...head for
packages/backend/src/services/alerts/AlertsService.slo.test.tsreturns nothing
014926e · Reply "won't fix" to dismiss a finding, or mention @maple-review-bot to ask about one.
Loading