Rebrand and re-point the stats observability panel - #5
Draft
Lauri "datafox" Heino (melodicore) wants to merge 2 commits into
Draft
Rebrand and re-point the stats observability panel#5Lauri "datafox" Heino (melodicore) wants to merge 2 commits into
Lauri "datafox" Heino (melodicore) wants to merge 2 commits into
Conversation
Re-skins packages/stats's client to source its palette, radius scale, and type pairing (JetBrains Mono for headings/nav/labels/data, system-sans for body) from elide-dev/design-system instead of OMP's OKLCH pink/cyan tokens. Full caps labels (metric labels, table headers) keep the monospace face; title-case section headers, sidebar nav, and controls read as sans. Wordmark, chart palette, and copy across the panel updated to match. No architecture change — still fully local (aura stats -> local sync -> local server); this is purely the visual skin, step 1 of hosting this panel for Elide.
Step 4: builds a real Aura account login (`aura account login|logout|status`) on top of cloud/auth.ts's AuraAuthClient/TokenManager, which existed fully-built but had zero CLI entry point anywhere in the codebase. While wiring it against elide-cloud's real device-flow contract, found and fixed a genuine claim-name mismatch in token-manager.ts's verifyAuraToken — it read org_id/account_id/realm_id/scopes (array), but the real issuer sends org/act/realm/scope (one space-delimited string). That bug would have made every real login/refresh fail verification; fixed here along with the shared test fixture and ~20 affected test cases (429 cloud tests passing). stats-cli.ts: after the existing local sync, tryOpenHostedPanel() checks the new cloud.stats.enabled setting (off by default). When on and signed in, it pushes recent records to the Aura ingest endpoint and opens the hosted panel instead of starting a local server; any failure (switch off, not configured, not signed in, push failure) falls back to the unchanged local flow — none are hard errors. deployment.ts: adds the `stats` AuraServiceSurface (stats-ingest.$DOMAIN) and resolveStatsPanelUrl (stats.$DOMAIN), both matching the elide-cloud workers built for this. packages/stats/index.ts now exports the pre-existing getRecentRequests so the CLI can read the batch to push. omp's own `stats` command is untouched; every change here is a tracked fork divergence (docs/aura/FORK.md).
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
statsdashboard's client to Elide's design system (palette, type, wordmark) instead of OMP's branding — visual only, no architecture change.aura account login|logout|statusfor signing in to an Aura account, and fixes a JWT-claim-name bug in the existing (previously unused) token verifier found while wiring it up.aura statscan now optionally push recent local usage data to a hosted panel and open that instead of the local server, gated behind a newcloud.stats.enabledsetting (off by default) and requiring sign-in. With the setting off, or when anything about the hosted path isn't available, behavior is unchanged from today.omp's ownstatscommand and behavior are untouched; every change here is a tracked fork divergence (docs/aura/FORK.md).Test plan
bun test— full coding-agent + stats suites passbun run check:typescleanaura account login/logout/statusagainst missing/invalid configaura statsfalls back to the local dashboard in every case where the hosted path doesn't apply (setting off, not signed in, unreachable)