Skip to content

Make README a quick start and split detailed documentation - #307

Merged
bcomnes merged 29 commits into
masterfrom
bret/readme-entry
Sep 10, 2026
Merged

Make README a quick start and split detailed documentation#307
bcomnes merged 29 commits into
masterfrom
bret/readme-entry

Conversation

@bcomnes

@bcomnes bcomnes commented Sep 6, 2026

Copy link
Copy Markdown
Owner

Summary

Closes #299.

This documentation change is independent of the watch stack and based on master.

  • Replace the 3,000-line README with an introduction, minimal successful build, core source-to-output model, and documentation links.
  • Move detailed reference material into twelve topic pages plus a documentation index under docs/.
  • Add a shared docs layout using parentLayout with path-based breadcrumbs above and below documentation pages, following the breadcrum.net convention.
  • Preserve moved README bookmarks with a collapsed compatibility index, documentation-site redirects, and anchors usable by both GitHub and the generated site.
  • Update migration/reference links, point excluded example/source links to GitHub, and fix introductory/editorial wording.
  • Preserve the merged parentLayout, dataDeps, separate data parameter, and migration documentation throughout the split.
  • Use one sentence per line for moved repository prose.

Validation

  • Full npm test passed outside the restricted runner.
  • Five breadcrumb unit tests cover the docs index, topic pages, loose migration pages, nested paths, relative links under a deployment prefix, and HTML escaping.
  • npm run build:domstack passed with all new documentation pages included.
  • An ad hoc generated-site check passed all 425 local links/anchors across 23 pages and verified paired breadcrumbs on all 15 documentation pages.
  • Chromium checks passed for legacy README bookmarks and hash-change navigation, breadcrumb navigation, and mobile/desktop table-of-contents transitions.
  • The README's minimal source fixture builds successfully to public/index.html with the documented content and default layout.

Notes

The example reference uses docs/example-projects/ because the documentation build intentionally excludes directories named examples. The existing migration guides remain separate documents with updated moved-reference links and the shared breadcrumb layout.

@coveralls

coveralls commented Sep 6, 2026

Copy link
Copy Markdown

Coverage Report for CI Build 34392019473

Coverage decreased (-2.3%) to 94.12%

Details

  • Coverage decreased (-2.3%) from the base build.
  • Patch coverage: 6 uncovered changes across 2 files (349 of 355 lines covered, 98.31%).
  • 1 coverage regression across 1 file.

Uncovered Changes

File Changed Covered %
lib/build-esbuild/index.js 38 34 89.47%
bin.js 28 26 92.86%
Total (6 files) 355 349 98.31%

Coverage Regressions

1 previously-covered line in 1 file lost coverage.

File Lines Losing Coverage Coverage
lib/build-esbuild/index.js 1 92.67%

Coverage Stats

Coverage Status
Relevant Lines: 9027
Covered Lines: 8656
Line Coverage: 95.89%
Relevant Branches: 2350
Covered Branches: 2052
Branch Coverage: 87.32%
Branches in Coverage %: Yes
Coverage Strength: 292.11 hits per line

💛 - Coveralls

@bcomnes

bcomnes commented Sep 6, 2026

Copy link
Copy Markdown
Owner Author

For the sub-pages, can you adopt a docs breadcrumb convention similar to the docs in https://github.com/hifiwi-fi/breadcrum.net/tree/master/packages/web/client/layouts/docs

@bcomnes

bcomnes commented Sep 6, 2026

Copy link
Copy Markdown
Owner Author

Implemented in 04ddfa0, following the shared-layout convention from breadcrum.net.

All topic pages, the docs index, and both migration guides now select layout: docs and receive path-derived breadcrumbs above and below their content.
The breadcrumbs use slash separators, retain a Home link, and mark the current page with aria-current="page".
Relative parent links work for both directory pages and standalone migration guides, including deployments under a URL prefix.
The existing responsive table of contents is preserved, with no new dependencies.

Validation passed: full npm test, five breadcrumb unit tests, the documentation build, all 414 local links/anchors across 23 generated pages, and Chromium checks for breadcrumb navigation, mobile/desktop layout transitions, and legacy bookmarks.

@bcomnes
bcomnes marked this pull request as ready for review September 8, 2026 05:06
@bcomnes

bcomnes commented Sep 8, 2026

Copy link
Copy Markdown
Owner Author

This needs to be rebased and all the changes factored in.

@bcomnes
bcomnes force-pushed the bret/readme-entry branch 2 times, most recently from b23e65d to 11b018a Compare September 9, 2026 02:30
@socket-security

socket-security Bot commented Sep 9, 2026

Copy link
Copy Markdown

Review the following changes in direct dependencies. Learn more about Socket for GitHub.

Diff Package Supply Chain
Security
Vulnerability Quality Maintenance License
Addednpm/​fragtml@​0.0.107610010090100
Updatednpm/​@​domstack/​sync@​0.0.6 ⏵ 0.0.878 +1100100 +191 +3100
Addednpm/​mermaid@​11.17.29810010096100

View full report

@socket-security

socket-security Bot commented Sep 9, 2026

Copy link
Copy Markdown

Warning

Review the following alerts detected in dependencies.

According to your organization's Security Policy, it is recommended to resolve "Warn" alerts. Learn more about Socket for GitHub.

Action Severity Alert  (click "▶" to expand/collapse)
Warn High
Obfuscated code: npm robust-predicates is 90.0% likely obfuscated

Confidence: 0.90

Location: Package overview

From: package.jsonnpm/mermaid@11.17.2npm/robust-predicates@3.0.3

ℹ Read more on: This package | This alert | What is obfuscated code?

Next steps: Take a moment to review the security alert above. Review the linked package source code to understand the potential risk. Ensure the package is not malicious before proceeding. If you're unsure how to proceed, reach out to your security team or ask the Socket team for help at support@socket.dev.

Suggestion: Packages should not obfuscate their code. Consider not using packages with obfuscated code.

Mark the package as acceptable risk. To ignore this alert only in this pull request, reply with the comment @SocketSecurity ignore npm/robust-predicates@3.0.3. You can also ignore all packages with @SocketSecurity ignore-all. To ignore an alert for all future pull requests, use Socket's Dashboard to change the triage state of this alert.

View full report

Keep development directories, internal plans, agent instructions, and test artifacts out of source discovery and static copying. Preserve resources linked from the README and reuse the actual build-command exclusions in website browser tests.
Await the first watch build rather than rebuilding before watch startup. Pass the configured logger to browser and service-worker contexts, preserving structured diagnostics and respecting log levels.

Summarize initial copying, move the build inventory and copy details to debug, and expose --verbose in the CLI. Cover startup build counts, rebuild failure/recovery, silent loggers, and verbose CLI output.
Comment thread docs/about/README.md
Comment thread docs/api/README.md
Comment thread docs/api/README.md Outdated
Comment thread docs/assets/README.md
Comment thread docs/cookbook/README.md
Comment thread docs/cookbook/README.md Outdated
Comment thread docs/cookbook/README.md Outdated
Comment thread docs/data/README.md Outdated
@bcomnes
bcomnes merged commit 6e21ebe into master Sep 10, 2026
2 of 6 checks passed
@bcomnes
bcomnes deleted the bret/readme-entry branch September 10, 2026 02:19
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.

Make the README a concise project entry point

2 participants