docs: fix demo instructions in install.md, resolve stale-text review findings#26
Merged
Conversation
…findings - install.md "Running the demo locally" told readers to run npm run dev, which serves the src/ dev pages, not the demo. Correct flow: build:component:legacy + npm run demo, plus a short note separating the three local servers (demo / dev / docs:dev). - build-output.md: note that dist/ is gitignored but is what npm ships (files whitelist + prepublishOnly rebuild). - claude_codex_workflow_context.md: dated historical banner (references the merged webcomponent branch and a nonexistent npm run build script). - Delete .github/review-prompt.md: one-shot review prompt frozen at an old webcomponent HEAD, no operational value, unreferenced; recoverable from git history. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
fan-droide
added a commit
that referenced
this pull request
Jun 12, 2026
docs: AGENTS.md fixes stranded by PR #26 merge race
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
1 participant
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.
Resolves the live findings from the earlier Codex text-cleanup review (the AGENTS.md branch finding was already fixed in #25).
Changes
npm run dev, which serves the internalsrc/dev pages, not the demo. Corrected tonpm run build:component:legacy+npm run demo(the demo loads the built legacy IIFE bundle), with a short note separating the three local servers (demo / dev / docs:dev).dist/is gitignored but is exactly what npm ships (fileswhitelist +prepublishOnlyrebuild), to avoid confusion betweengit statusandnpm pack --dry-run.webcomponentbranch and a nonexistentnpm run buildscript; current workflow lives in AGENTS.md / SESSION_HANDOFF.md.webcomponentHEAD, unreferenced anywhere, no operational value; recoverable from git history.Validation
npm run docs:buildpasses (validates internal links).grepconfirms no remaining references toreview-prompt.mdor stalenpm run buildoutside the banner explanation.🤖 Generated with Claude Code