Skip to content

docs(skills): record the fresh-worktree build:wp and boundary-gate traps - #43

Merged
DavidBabinec merged 1 commit into
mainfrom
docs/run-and-test-worktree-traps
Oct 27, 2026
Merged

DavidBabinec merged 1 commit into
mainfrom
docs/run-and-test-worktree-traps

Conversation

@DavidBabinec

Copy link
Copy Markdown
Contributor

The run-and-test skill claimed bun install is the only setup a worktree needs. In a fresh worktree two gate steps fail before anything is wrong: the first build:wp exits 1 while vite.config.js creates the missing packages/wp/.env, and check:open-source crashes with ENOENT on packages/figma/dist or packages/wp/dist until both bundles exist. The skill now documents both, and the gate section says check:open-source must follow the two builds.

I left the script alone rather than making it skip a missing dist. A skip would print "passed for all" without reading the bundles, which is the false green the gate exists to prevent.

Verification

Reproduced in a fresh worktree after bun install --frozen-lockfile:

check:open-source   # exit 1, ENOENT scandir packages/figma/dist
build:wp            # exit 1, "Missing: APP_ENV", .env created
build:wp            # exit 0
build:figma         # exit 0
check:open-source   # exit 0, passed for all

The run-and-test skill said bun install was the only setup a worktree
needs. In a fresh worktree the first build:wp exits 1 while it creates
packages/wp/.env, and check:open-source crashes on ENOENT until the wp
and figma bundles are built. The gate order now says why it matters.
@DavidBabinec
DavidBabinec merged commit c449c68 into main Oct 27, 2026
8 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