docs(skills): record the fresh-worktree build:wp and boundary-gate traps - #43
Open
DavidBabinec wants to merge 1 commit into
Open
DavidBabinec wants to merge 1 commit into
DavidBabinec wants to merge 1 commit into
Conversation
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
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
The run-and-test skill claimed
bun installis the only setup a worktree needs. In a fresh worktree two gate steps fail before anything is wrong: the firstbuild:wpexits 1 whilevite.config.jscreates the missingpackages/wp/.env, andcheck:open-sourcecrashes withENOENTonpackages/figma/distorpackages/wp/distuntil both bundles exist. The skill now documents both, and the gate section sayscheck:open-sourcemust 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: