✅ No new issues found.

Reviewed changes — adds GetInstanceMeterUsage to the ClickHouse client, a billing query that computes deploy meter usage (CPU, memory, egress, disk) from Heimdall checkpoint data using window-function pair integration over adjustable time windows.

• Add GetInstanceMeterUsage query with leadInFrame pair integration — counter deltas for CPU/egress, gauge × interval for memory/disk, gap-drop at 2 minutes, conservative least() for resize safety.
• Add GetInstanceMeterUsageRequest / InstanceMeterUsage types with ch: struct tags for ClickHouse-go deserialization and Float64 accumulators to prevent int64 overflow on large billing windows.
• Add integration tests with real ClickHouse covering counter deltas, outage-gap dropping, restart-boundary isolation, cross-workspace aggregation, and empty-window returning zero rows.
• Add no-op implementation and Querier interface entry matching existing patterns.

  ^{｜ View workflow run ｜ Using DeepSeek Pro ｜ 𝕏}

✅ No new issues found.

Reviewed changes — the delta since the prior pullfrog review (72885e6) is entirely a dashboard settings UI refactor; the GetInstanceMeterUsage ClickHouse query is unchanged. This run reviewed the new @unkey/ui page-layout primitives, the settings-page migration onto them, and the supporting design-app docs.

• Add @unkey/ui page-layout primitives — new PageContainer (compound, width="default"|"full" propagated via a group/page data attribute), PageBody, the PageHeader* family, and a cva-based SecondaryNav* family under web/internal/ui/src/components/page-header/, re-exported from the package index.
• Migrate settings pages to the primitives — general, team, root-keys, and billing (new billing-container.tsx) drop WorkspaceNavbar/SettingsShell for the new layout; a new settings/layout.tsx renders the SecondaryNav rail.
• Remove the settings sub-nav from the global sidebar — buildSettingsLinks is deleted from leaves.ts and case "settings" in sidebar-body.tsx now returns buildWorkspaceSections; deleted settings/workspace-navbar.tsx and settings/root-keys/navigation.tsx.
• Design app additions — Preview gains a bleed prop, copy-button.ts gains an inline option, and a new patterns/layout MDX page with example sources documents the primitives.

I verified the load-bearing parts of the refactor: general/page.tsx drops the old workspace && guard but useWorkspaceNavigation() redirects on a null workspace so the access is safe; buildSettingsLinks, WorkspaceNavbar, the settings Navigation, and SettingsShell have no remaining importers in settings; and the billing page never double-wraps BillingContainer since its branches and Client are mutually exclusive.

  ^{｜ View workflow run ｜ Using Claude Opus ｜ 𝕏}

just nits really, looks good

✅ No new issues found.

Reviewed changes — the delta since the prior pullfrog review (f3f4377) addresses the two nits from chronark's approval. Both land in pkg/clickhouse and leave the query's billing semantics unchanged.

• Switch the gap constant to a time.Duration — maxSampleGapMillis int64 = 2*60*1000 becomes maxSampleGap = 2 * time.Minute, converted with .Milliseconds() at the query-param boundary; doc comments updated to drop the Millis suffix. The wire value is still 120000ms, so query behavior is identical.
• Derive the test anchor from time.Now() — the integration test's base moves from a hardcoded 2026-06-01 to time.Now().UTC().Truncate(24h).Add(-24h), keeping inserted rows inside the table's 95-day TTL so ClickHouse can't silently drop the parts before the query runs. This fixes a latent flakiness that would have surfaced once the fixed date aged past the TTL.

  ^{｜ View workflow run ｜ Using Claude Opus ｜ 𝕏}