Repository navigation
Configure Superset for parallel workspaces - #30
Conversation
Wire up .superset/config.json (setup/run/teardown) so a new workspace installs deps automatically, and add a shared port allocator so several worktrees can run `npm run dev` at once without colliding on the hardcoded :8080. Serialize the Docker-backed Supabase DB gate behind a lock instead, since it has no per-project port config. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_0163LcrphWJjYjTXM33mQuH6
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: ASSERTIVE Plan: Free Run ID: 📒 Files selected for processing (9)
Included review availability: Your plan provides up to 1 included review per hour; 0 remain after this review. 📝 WalkthroughWalkthroughAdded tooling for concurrent Superset workspaces. The tooling allocates ports, coordinates shared locks, manages Supabase lifecycle operations, configures Vite, and documents workspace setup. ChangesSuperset workspace tooling
Estimated code review effort: 4 (Complex) | ~45 minutes Sequence Diagram(s)sequenceDiagram
participant Workspace as Superset workspace
participant DevScript as .superset/dev.sh
participant PortAllocator as port-alloc.cjs
participant Vite as Vite dev server
Workspace->>DevScript: start development
DevScript->>PortAllocator: allocate workspace port
PortAllocator-->>DevScript: return port
DevScript->>Vite: run with PORT
Vite-->>Workspace: serve on allocated localhost port
Note 🎁 Summarized by CodeRabbit FreeYour organization is on the Free plan. CodeRabbit will generate a high-level summary and a walkthrough for each pull request. For a comprehensive line-by-line review, please upgrade your subscription to CodeRabbit Essentials by visiting https://app.coderabbit.ai/settings/billing. Comment |
Summary
.superset/config.json(setup/run/teardown) so a new Superset workspace installs deps automatically and starts the dev server through.superset/dev.sh..superset/lib/port-alloc.cjs+.superset/lib/lock.sh) so parallel worktrees each get a unique port instead of colliding on the hardcoded:8080invite.config.ts(now readsprocess.env.PORTfirst)..superset/db-gate.shto serialize the Docker-backed Supabase DB gate (supabase db start/npm run test:db) across workspaces, sincesupabase/config.tomlhas no per-project port config and two workspaces running it at once collide in Docker.AGENTS.mdand gitignore personal.superset/config.local.jsonoverrides.Test plan
bash -non all new shell scripts,node --checkon the port allocator, JSON validatednpm ci+./.superset/dev.shend-to-end: allocated port 8080,curlreturned HTTP 200,./.superset/teardown.shreleased the allocationnpm run typecheckandnpm run lintpass with thevite.config.tschange🤖 Generated with Claude Code
https://claude.ai/code/session_0163LcrphWJjYjTXM33mQuH6
Summary by CodeRabbit