Skip to content

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

Open
DavidBabinec wants to merge 1 commit into
mainfrom
docs/run-and-test-worktree-traps
Open

DavidBabinec wants to merge 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.

This branch has not been deployed

No deployments
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