docs: note that pnpm cli serves the last build, not the working tree - #164
Merged
Conversation
`dist/` is a build artifact and nothing rebuilds it on demand, so `pnpm cli agent <topic>` can serve recipes from an arbitrarily old commit — the recipe text is embedded into the bundle at build time rather than fetched. Measured: a `dist/` built 26 commits back served topic v2 while HEAD served v3, and the missing revision was the one documenting that `language:` takes ast-grep's own spelling. Also records how the skill's pinned nightly relates to `pnpm cli`, since the two disagree exactly when `dist/` is behind HEAD and neither is automatically right. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_017cEN93Acyp4zBwP3oDnyy1
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.
What
Three paragraphs under Local Development in
CLAUDE.md, recording thatpnpm clirunspackages/cli/dist/index.js— the last build — and not the working tree.Why
agent <topic>recipes are embedded into the bundle at build time rather than fetched over the network, so a staledist/serves stale instructions with nothing to signal it.Measured while onboarding this repo with Taskless: a
dist/built 26 commits earlier servedcreate-sg-ruletopic v2 while HEAD served v3. The revision it missed was the one documenting thatlanguage:takes ast-grep's own spelling, so four new rules were authored with an off-list lowercasetypescript. That spelling happened to reach the right parser — a name ast-grep does not recognize at all aborts config parsing and takes every other rule's report down with it, silently.The note also records how the skill's pinned nightly relates to
pnpm cli. The two disagree exactly whendist/is behind HEAD, and neither is automatically right: the pin is a real build of some commit,pnpm cliis this tree only once rebuilt.Note on the diff
Prettier normalized four pre-existing
*emphasis*markers to_emphasis_and one blank line elsewhere in the file.lint-stagedapplies this on any commit that stagesCLAUDE.md, so it is unavoidable rather than incidental scope.No changeset: a
CLAUDE.mdedit ships no release note.