Skip to content

fix(repo): run turbo via yarn exec for root scripts#911

Merged
grypez merged 1 commit intomainfrom
grypez/dock-stack-b-turbo
Apr 1, 2026
Merged

fix(repo): run turbo via yarn exec for root scripts#911
grypez merged 1 commit intomainfrom
grypez/dock-stack-b-turbo

Conversation

@grypez
Copy link
Copy Markdown
Contributor

@grypez grypez commented Apr 1, 2026

Root build and test scripts invoked turbo as a bare binary, which only works when node_modules/.bin is on the shell PATH.

Cursor's agent sandbox (and similar restricted runners) often use a minimal PATH without that directory, producing "command not found: turbo" even though turbo is installed.

Use yarn exec turbo so Yarn resolves and runs the workspace dependency without relying on PATH.


Note

Low Risk
Low risk: only adjusts root package.json scripts to invoke Turbo via Yarn, affecting how builds/tests are launched but not application/runtime logic.

Overview
Switches root scripts to run Turbo via Yarn resolution instead of relying on the turbo binary being on PATH.

Updates build, test:dev, and test:dev:quiet to use yarn exec turbo run ..., improving compatibility in environments with restricted PATH settings (e.g., CI/sandboxes).

Written by Cursor Bugbot for commit c21c392. This will update automatically on new commits. Configure here.

@grypez grypez changed the base branch from grypez/dock-stack-a-klms to main April 1, 2026 17:00
Root build and test scripts invoked `turbo` as a bare binary, which only works when `node_modules/.bin` is on the shell PATH.

Cursor's agent sandbox (and similar restricted runners) often use a minimal PATH without that directory, producing "command not found: turbo" even though turbo is installed.

Use `yarn exec turbo` so Yarn resolves and runs the workspace dependency without relying on PATH.

Made-with: Cursor
@grypez grypez force-pushed the grypez/dock-stack-b-turbo branch from c77a70b to c21c392 Compare April 1, 2026 17:03
@grypez grypez requested a review from rekmarks April 1, 2026 17:05
@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Apr 1, 2026

Coverage Report

Status Category Percentage Covered / Total
🔵 Lines 78.29%
⬆️ +0.16%
8487 / 10840
🔵 Statements 78.11%
⬆️ +0.16%
8623 / 11039
🔵 Functions 75.85%
⬆️ +0.31%
1979 / 2609
🔵 Branches 76.07%
⬇️ -0.55%
3641 / 4786
File CoverageNo changed files found.
Generated in workflow #4156 for commit c21c392 by the Vitest Coverage Report Action

@grypez grypez marked this pull request as ready for review April 1, 2026 17:19
@grypez grypez requested a review from a team as a code owner April 1, 2026 17:19
Copy link
Copy Markdown
Member

@rekmarks rekmarks left a comment

Choose a reason for hiding this comment

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

Sure why not!

@grypez grypez added this pull request to the merge queue Apr 1, 2026
Merged via the queue into main with commit 30b0e77 Apr 1, 2026
53 of 56 checks passed
@grypez grypez deleted the grypez/dock-stack-b-turbo branch April 1, 2026 17:53
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.

2 participants