[ToddHebebrand]

Summary

• GET /analytics/sla/:id/compliance now measures the definition's own targets (site / device / group via target_type + target_ids; org-wide otherwise; a scoped definition with no targets measures nothing, never the whole org) and returns uptimeTarget, meetsTarget and an exclusions block alongside liveUptime.
• exclude_weekends: Sat/Sun do not count toward the window length (window reaches back over the same number of weekdays).
• exclude_maintenance_windows: devices in a maintenance window right now (existing isDeviceInMaintenance) leave numerator and denominator. Point-in-time only, checked for up to 500 target devices (exclusions.maintenanceWindows = applied / skipped_too_many_devices). Historical window overlap needs the availability-interval work (Uptime KPIs from data Breeze already has: site WAN and device availability, with maintenance excluded #9273).
• history: [] returned honestly (nothing writes sla_compliance; no period-close writes added). Dead history[0].compliancePercentage reads removed from the web page.

Linked Issues

Closes #9371

Merge Danger

Door: two-way (revert the PR; no schema/migration change)

Blast radius: SLA uptime figure on the Analytics page / compliance endpoint

Type of Change

• Bug fix

Tests

• Red first: 4 new route tests (site-targeted scoping + missed target, weekend widening, maintenance exclusion, empty history/meetsTarget) failed before the change.
• apps/api vitest run src/routes/analytics.test.ts: 51 passed. apps/web vitest run src/components/analytics: 3 files / 10 tests passed.
• tsc --noEmit exit 0 in apps/api and apps/web.

🤖 Generated with Claude Code

[cloudflare-workers-and-pages]

Deploying breeze with    Cloudflare Pages

Latest commit:	f76bf89
Status:	✅  Deploy successful!
Preview URL:	https://25b30cbd.breeze-9te.pages.dev
Branch Preview URL:	https://fix-9371-sla-compliance-hone.breeze-9te.pages.dev

View logs

[ToddHebebrand]

Review run: /pr-review-toolkit:review-pr (code-reviewer, scoped to code + tests + error handling)
Findings: 3 important, 4 suggestions. Fixed in f76bf89: maintenance-lookup failure now reported as exclusions.maintenanceWindows: 'unavailable' (was an unhandled 500; concurrency lowered to 10), group-targeted definitions denied to site-narrowed callers (shared gate treats group as org aggregate), DST-safe weekend window length, tests added for lookup failure, empty targets and group denial. Deferred: SQL-side aggregation instead of fetching target device rows (3 columns/device; fine at current scale, revisit with #9273), group/device/over-cap test paths.
Tests: api analytics.test.ts + slaSiteScope.test.ts 78 passed; web analytics 3 files/10 tests passed; tsc --noEmit exit 0 in api and web (api needs a 20GB heap on this host).
Status: review-clean for blocking issues, awaiting maintainer merge.