[Fix] Bulldozer Studio and SpaceTime DB port conflict#1346
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (2)
📝 WalkthroughWalkthroughBulldozer Studio's port configuration is updated from 8139 to 8140 across both the backend server startup script and the dev launchpad UI reference, ensuring consistent port mapping for local development. Changes
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~3 minutes Poem
🚥 Pre-merge checks | ✅ 3✅ Passed checks (3 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches📝 Generate docstrings
🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
There was a problem hiding this comment.
Pull request overview
Resolves a local development port conflict where Bulldozer Studio and SpacetimeDB were both exposed on the same default port, by moving Bulldozer Studio to a new port so both services are accessible concurrently.
Changes:
- Update the dev-launchpad “Bulldozer Studio” service link to use port suffix
40(i.e.,8140with the default prefix). - Update the backend Bulldozer Studio runner to listen on
8140(with the defaultNEXT_PUBLIC_STACK_PORT_PREFIX).
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.
| File | Description |
|---|---|
| apps/dev-launchpad/public/index.html | Updates the Bulldozer Studio entry to point to port suffix 40 so the launchpad links to the new port. |
| apps/backend/scripts/run-bulldozer-studio.ts | Changes STUDIO_PORT to end in 40, moving the studio server off the conflicting 8139 port. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Greptile SummaryThis PR resolves a port conflict where Bulldozer Studio and SpacetimeDB were both assigned port Confidence Score: 5/5Safe to merge — minimal, targeted fix with no logic changes. The change is a simple two-file port number update that correctly resolves the conflict. Both the server and the UI are updated consistently, devcontainer already exposes both ports, and there are no other references to the old port tied to Bulldozer Studio in the codebase. No files require special attention. Important Files Changed
Flowchart%%{init: {'theme': 'neutral'}}%%
flowchart TD
subgraph Before
BS1["Bulldozer Studio\nport 8139 ❌"]
ST1["SpacetimeDB\nport 8139 ❌"]
BS1 -- "conflict" --- ST1
end
subgraph After
BS2["Bulldozer Studio\nport 8140 ✅"]
ST2["SpacetimeDB\nport 8139 ✅"]
end
Reviews (1): Last reviewed commit: "fix: port conflict b/w bs and spacetime" | Re-trigger Greptile |
Context
Bulldozer Studio is what we use for debugging and inspecting bulldozer tables. Due to a merge conflict, both bulldozer studio and spacetime db were made accessible via the same port, 8139 by default.
Summary of Changes
Moving the former to 8140. Now they are both visible.
Summary by CodeRabbit