Repository navigation
Conversation
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes using default effort and found 1 potential issue.
Bugbot Autofix prepared a fix for the issue found in the latest run.
- ✅ Fixed: Unquoted benchmark labels split on spaces
- Added quotes around $1 in the measure() function's local assignment to prevent word-splitting of benchmark labels containing spaces.
Or push these changes by commenting:
@cursor push 0e87334825
Preview (0e87334825)
diff --git a/scripts/bench-tasks.sh b/scripts/bench-tasks.sh
--- a/scripts/bench-tasks.sh
+++ b/scripts/bench-tasks.sh
@@ -52,7 +52,7 @@
}
measure() {
- local label=$1; shift
+ local label="$1"; shift
for _ in $(seq "$RUNS"); do
local start; start=$(now)You can send follow-ups to the cloud agent here.
Reviewed by Cursor Bugbot for commit f505b1d. Configure here.
| } | ||
|
|
||
| measure() { | ||
| local label=$1; shift |
There was a problem hiding this comment.
Unquoted benchmark labels split on spaces
Medium Severity
local label=$1 leaves $1 unquoted, so every measure label is word-split. All four labels contain spaces, so local treats extra words as identifiers (and can error under set -e), and printed timings never show the intended names.
Reviewed by Cursor Bugbot for commit f505b1d. Configure here.
There was a problem hiding this comment.
Not an issue: bash doesn't word-split the right-hand side of assignments, including local label=$1 (checked with labels containing spaces under set -e).
- tasks.md covers `^name`, `@workspaces` defaults, workspace selection, failure handling, the CI standalone default and long-lived tasks with `^build`. - A new "Migrating from Turborepo" page maps turbo.json and the turbo CLI to taskfiles and `yarn tasks run`. - scripts/bench-tasks.sh compares `workspaces foreach` and `tasks run` on a generated project.
9e93395 to
d485ee4
Compare
f505b1d to
b472b94
Compare



Docs for the task orchestration stack:
tasks.mdcovers^name,@workspacesdefaults, workspace selection, failure handling, the CI standalone default, and long-lived tasks with^build.turbo.jsonand turbo commands to taskfiles andyarn tasks run.scripts/bench-tasks.shcomparesworkspaces foreachandtasks runon a generated project.Stacked on #379.
Note
Low Risk
Documentation and an optional dev benchmark script only; no runtime or product behavior changes in this diff.
Overview
Documents multi-workspace Yarn tasks orchestration and adds a Turborepo migration guide, plus a local benchmark script.
tasks.mdgains sections on^buildshorthand, root@workspacesdefaults (script delegation, ordering-only no-ops, overrides),yarn tasks runworkspace selection (-A,--from,--affected, dependency/dependent expansion, concurrency,--continue,--errors-only, CI--standalone), and@long-liveddev servers.A new
getting-started/migrating/turborepopage mapsturbo.jsonpipelines to root taskfiles (includingchangesetGlobalFiles), notes that task caching is not implemented yet, and tables common turbo CLI flags toyarn tasks run/yarn workspaces list --since.scripts/bench-tasks.shgenerates a random-DAG monorepo and timesworkspaces foreachvs severaltasks run --standalone -A buildmodes (script-backed, inline body, scheduling-only).Reviewed by Cursor Bugbot for commit b472b94. Bugbot is set up for automated code reviews on this repo. Configure here.