Add Conductor Bun setup script#361
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
![claude[bot]](https://avatars.githubusercontent.com/in/1236702?s=60&v=4){kind=small}
There was a problem hiding this comment.
LGTM, simple Conductor workspace config addition.
Extended reasoning...
Overview
This PR adds a single 4-line config file (.conductor/settings.toml) that defines a workspace setup script for Conductor. The script runs bun install --frozen-lockfile to install dependencies from the checked-in bun.lock.
Security risks
None. This is a developer-tooling config file used only when setting up local Conductor workspaces. It does not run in CI/CD or production, does not touch auth/crypto/permissions, and uses --frozen-lockfile which is the safer install mode (fails on lockfile mismatch rather than silently updating).
Level of scrutiny
Minimal. This is a mechanical, self-contained config file with no logic and no impact on production code paths. The referenced lockfile (bun.lock) exists in the repo, so the command is valid.
Other factors
No bugs were found by the bug hunting system. No prior reviews on this PR. Standard pattern for repo-level tool config.
There was a problem hiding this comment.
Pull request overview
Adds a Conductor repository configuration to standardize workspace setup by installing JS/TS dependencies via Bun using the checked-in lockfile.
Changes:
- Introduces
.conductor/settings.tomlwith a Conductorsetupscript. - Configures setup to run
bun install --frozen-lockfilefor reproducible dependency installs.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Adds shared Conductor repository settings for workspace setup. The setup script installs dependencies with Bun using the checked-in lockfile via
bun install --frozen-lockfile.