feat(dashboards): add table-backed dashboard resources - #8305
Draft
TheodoreSpeaks wants to merge 4 commits into
Draft
TheodoreSpeaks wants to merge 4 commits into
TheodoreSpeaks wants to merge 4 commits into
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub. |
|
This branch was previously deployed
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
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
Adds dashboards as a workspace resource with their own navigation, folders, resource tabs, and Mothership tools. Dashboard YAML selects authorized table queries and lays out metrics, ECharts charts, tables, text, and tabs; aggregation runs on the server with bounded results and timeouts.
Companion: https://github.com/simstudioai/mothership/pull/531
listed/unlisted) from storage/ownership context. Files listings, Mothership file inventory, pickers, and search filter in SQL before pagination. Chat uploads and new dashboard definitions are unlisted; dashboards have their own listing API. Explicit references retain existing authorization, and copies preserve discovery.DASHBOARDS=true.Type of Change
Testing
staging.Review the shared chart renderer, table aggregate semantics, discovery lifecycle, and rollout gates. Idle chart summaries describe plotted buckets; an average of bucket percentages is not a weighted overall rate. Independent panels share time bounds, not a database snapshot.
Rollout and scope
This draft targets
staging. Migration0384_dashboard_foldersaddsdashboardto the existing folder resource enum; it creates no new table. Migration0385_file_discoveryadds the defaulted discovery column, updates search-index eligibility, and installs a temporary bridge for old upload writers. Script migration0025_backfill_file_discoveryclassifies existing non-workspace uploads in idempotent pages of 1,000 without rewriting content or ownership. There is no dashboard backfill because dashboards have not shipped. A documented follow-up removes the bridge once discovery-aware writers are fully deployed.The companion Mothership change adds dashboard CLI authoring and persists per-turn rollout availability. Apply both repositories' additive migrations and deploy the worker before enabling dashboards. The discovery follow-up needs no separate worker change because its file inventory uses the shared Sim application operations.
No merge or deployment is included. Public sharing, log queries, and a tool for inspecting the user's displayed data remain deferred. Local research artifacts and workspace data are excluded.
Checklist