refactor: remove dashboard, flatten to single-package project#33
Merged
Conversation
Scope reset: the dashboard server + UI subsystem was overengineered
for the actual use case (CI consumption + ad-hoc developer queries).
This rips it out and returns the project to a single src/ at root,
keeping just the analytics columns in cache.db so CI can still query
run history via 'vzn stats' or 'sqlite3 .vzn/cache/cache.db'.
Deleted:
- apps/ (entire apps/dashboard/ Solid app)
- src/dashboard.ts + src/dashboard.test.ts
- packages/run/ (whole directory; src/ promoted to root)
- 'dashboard' subcommand from cli.ts + help text
- docs/modules/dashboard.md
- docs/design/dashboard.md
- 'Build dashboard UI' step from CI
- workspaces glob, build:dashboard script from package.json
Kept (analytics for CI, no UI):
- runs table + v11 analytics columns (run_id, cpu_ms, peak_rss_bytes,
wallclock_start/end_ns, cache_hit, bytes_uploaded/downloaded)
- ulid.ts + hrtime span stamping in orchestrator
- Bun.spawn resourceUsage capture in runner.ts/sandbox.ts
- 'vzn stats' command
Kept (untouched):
- sandbox, layered-cache, remote-cache, cache-archive
Layout: packages/run/src/ -> src/, packages/run/{package.json,
tsconfig.json} -> root (merged into root package.json which now is
@vzn/run). bun.lock regenerated; dep tree 304 -> 19 packages.
CI test glob: packages/*/src/ apps/*/src/ -> src/.
248 tests pass / 0 fail. Lint + format clean.
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.
Scope reset per owner direction. The dashboard server + Solid UI
subsystem (PRs #19 → #29, the entire dashboard track) was overengineered
for the actual need — CI consumption + ad-hoc developer queries
already work fine against
cache.dbwithvzn statsorsqlite3. This PR rips it all out and returns the project to asingle
src/at root.Diff stats
Dep tree: 304 packages → 19. Test count: 286 → 248 (the dashboard
tests went with the code; everything else still passes).
What's deleted
apps/(the entireapps/dashboard/Solid + Vite + UnoCSS app)src/dashboard.ts+src/dashboard.test.ts(the JSON API server)packages/run/(the directory;src/promoted to root)dashboardsubcommand fromcli.ts+ its help textparseDashboardArgs,dashboardCmd,DashboardUiMissingError,resolveUiDir,createDashboardServer, the full set of API typesdocs/modules/dashboard.mddocs/design/dashboard.md.github/workflows/ci.ymlworkspacesglob,build:dashboardscript frompackage.jsonWhat stays (analytics for CI, no UI)
runstable + v11 analytics columns (run_id,cpu_ms,peak_rss_bytes,wallclock_start/end_ns,cache_hit,bytes_uploaded/downloaded)src/ulid.ts+ hrtime span stamping in orchestratorBun.spawnresourceUsage()capture inrunner.ts/sandbox.tsvzn statscommand — surfaces the same data as the dashboardOverview page used to
.vzn/cache/cache.dbdirectly withsqlite3for any custom metric. The cache file IS the API.What stays untouched
bwrap/sandbox-exec)/v8/artifacts/wire)defineWorkspace)vzn runLayout
Test plan
bun installclean — 19 packagesbun run formatcleanbun run lintcleanbun test src/— 248 pass / 0 failbun src/bin.ts --version/--help— no dashboard refs inhelp text
Generated by Claude Code