docs: Pages site accuracy fixes + tools-table drift guard#59
Merged
Conversation
- Correct the window-capture note: matching is case-insensitive exact first, then substring, topmost match wins (was: "exact title" only). - Add heading ids so deep links like #tools work. - Inline SVG favicon (was a 404 console error on every load). - Note the 0.9.0 overwrite-refusal behavior above the tools table. - Open Graph / Twitter meta tags for link unfurls. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
The table is hand-mirrored with no check; this fails CI when a tool is added/removed without updating docs/index.html, or when the site lists a tool that is not registered. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
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.
Summary
From a Playwright review of the live GitHub Pages site:
src/utils/windows.tsandmcp-tools.json) is case-insensitive exact first, then substring, topmost match wins.ids on all<h2>s so deep links like#toolswork (previously dead).outputpath withoutoverwrite: true.src/__tests__/docs.test.ts): asserts the site's tools table andmcp-tools.jsonlist exactly the same tool names, both directions. The table is hand-mirrored and nothing else checks it.No version bump: docs and tests are not part of the published package (precedent: #49), and release.yml skips the already-tagged v0.9.0. Merging auto-triggers the Pages deploy (
pages.ymlfires ondocs/**).Testing
reframerow makes the drift test fail with "tools missing from docs/index.html: reframe"; restored, it passes.🤖 Generated with Claude Code