Repository navigation
Conversation
#394 added content/{de,en}/getting-started/live-demo.md and its nav entry, taking the documentation site from 43 to 44 pages per language. README named the old figure in both places it cites the site. The same PR made the browser demo a required input of assemble-site.mjs, but CONTRIBUTING still described the assembly as running over two builds and listed an output layout without demo/. Named the third build and the demo output, and recorded that every input is required.
2 tasks done
Owner
Author
|
Superseded by #403, which carried the same sync on top of the copy-editing pass in #402. Everything this PR changes is already on |
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.
What & why
Automated documentation-drift sync for pull requests that landed in the last 26 hours. One PR was
in the window: #394 (Serve the project website from real addresses and harden its hosting).
Two claims went stale against
mainand are corrected here. Both were verified against thechecked-out code, not inferred from the diff.
mainREADME.md(×2)nav.tslists 44 paths andcontent/de/content/enhold 44.mdfiles eachgetting-started/live-demo.md+ its nav entryCONTRIBUTING.mdassemble:siteruns "from the two finished builds"; output layout lists root /docs//media/;preview:site"runs both builds"scripts/assemble-site.mjsrequires three builds (dist-site,dist,dist-demo) pluspages-media, and writesdemo/as well;preview:siterunsbuild,build:siteandbuild:demoThe page count is confirmed by comparing the two commits directly:
content/enheld 43.mdfiles at
d4cf015(the merge base) and holds 44 at62341af.The
CONTRIBUTINGcorrection also records that every assembly input is required —assemble-site.mjssays in its own comment that an optional input "would let a deploy publish a site that silently
keeps the previous demo", and
CLAUDE.mdalready states this.CONTRIBUTINGwas the one surfacethat did not.
How
Text-only. No code, no config, no test changes.
Testing
Nothing executable changed, so no suite applies. The two figures are derived from the repository
rather than asserted by a guard:
grep -oE "path: '[^']+'" src/nodepilot-docs-ui/src/data/nav.ts | wc -l, matchingfind src/nodepilot-docs-ui/content/{de,en} -name '*.md' | wc -l(44 each).REQUIRED/cpSyncinsrc/nodepilot-docs-ui/scripts/assemble-site.mjsand the
preview:sitescript insrc/nodepilot-docs-ui/package.json.DocumentationCountsTestsdoes not pin the page count, so no guard pattern needed updating. Itsexisting rows (MCP tools 102, activity types 27, skins 10, 234 Vitest files, 77 E2E specs) were
re-checked against the code and all still match — no drift there.
content/de/andcontent/en/) if behavior or config changedmain; no direct commits tomainNot applied here
Findings from #394 that needed a human decision rather than an objective correction were filed
separately as a docs-drift issue, not fixed in this PR — chiefly that the demo's guided walkthrough
(
demo/ui/tour.ts, covered bye2e-demo/guided-tour.spec.ts) is absent from thelive-demodocpage in both languages and has no entry in the E2E catalogue.
Generated by Claude Code