Skip to content

chore: add conductor.json for workspace setup#249

Merged
kianbazza merged 1 commit intocanaryfrom
kianbazza/conductor-setup-2
Mar 28, 2026
Merged

chore: add conductor.json for workspace setup#249
kianbazza merged 1 commit intocanaryfrom
kianbazza/conductor-setup-2

Conversation

@kianbazza
Copy link
Copy Markdown
Collaborator

@kianbazza kianbazza commented Feb 4, 2026

Configures Conductor to automatically symlink .env.local and install
dependencies when creating new workspaces.

@vercel
Copy link
Copy Markdown

vercel bot commented Feb 4, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
ui-canary Ready Ready Preview, Comment Mar 28, 2026 9:06pm

Request Review

@kianbazza kianbazza mentioned this pull request Feb 4, 2026
Copy link
Copy Markdown
Collaborator Author

kianbazza commented Feb 4, 2026

Copy link
Copy Markdown

@cursor cursor bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Bugbot Autofix is OFF. To automatically fix reported issues with Cloud Agents, enable Autofix in the Cursor dashboard.

This PR is being reviewed by Cursor Bugbot

Details

You are on the Bugbot Free tier. On this plan, Bugbot will review limited PRs each billing cycle.

To receive Bugbot reviews on all of your PRs, visit the Cursor dashboard to activate Pro and start your 14-day free trial.

Comment thread conductor.json
@@ -0,0 +1,6 @@
{
"scripts": {
"setup": "ln -sf ~/repos/bazzalabs/ui/apps/web/.env.local apps/web/.env.local && bun install",
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hardcoded user-specific path breaks team portability

Medium Severity

The setup script uses a hardcoded absolute path ~/repos/bazzalabs/ui/apps/web/.env.local which assumes a specific directory structure on the developer's machine. This path will fail for any teammate whose repository is cloned to a different location (e.g., ~/code/, ~/projects/, or any other directory). Since conductor.json is committed to the repository and intended for team-wide workspace setup, this makes the configuration non-portable and will cause the symlink command to fail for most users.

Fix in Cursor Fix in Web

@kianbazza kianbazza changed the base branch from kianbazza/font-fallback-geist to graphite-base/249 February 5, 2026 12:43
@kianbazza kianbazza mentioned this pull request Feb 7, 2026
@kianbazza kianbazza changed the base branch from graphite-base/249 to kianbazza/font-fallback-geist February 18, 2026 23:10
@kianbazza kianbazza changed the base branch from kianbazza/font-fallback-geist to graphite-base/249 February 19, 2026 00:15
Copy link
Copy Markdown
Collaborator Author

kianbazza commented Mar 28, 2026

Merge activity

  • Mar 28, 9:03 PM UTC: A user started a stack merge that includes this pull request via Graphite.
  • Mar 28, 9:04 PM UTC: Graphite rebased this pull request as part of a merge.
  • Mar 28, 9:05 PM UTC: @kianbazza merged this pull request with Graphite.

@kianbazza kianbazza changed the base branch from graphite-base/249 to canary March 28, 2026 21:03
Configures Conductor to automatically symlink .env.local and install
dependencies when creating new workspaces.
@kianbazza kianbazza force-pushed the kianbazza/conductor-setup-2 branch from 137ccd2 to 609228b Compare March 28, 2026 21:04
@kianbazza kianbazza merged commit 4a1db6d into canary Mar 28, 2026
3 of 4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant