Skip to content

Link hovers to real documentation pages (0.3.2) - #7

Merged
zabarov merged 2 commits into
mainfrom
fix/docs-links
Oct 30, 2026
Merged

zabarov merged 2 commits into
mainfrom
fix/docs-links

Conversation

@zabarov

@zabarov zabarov commented Oct 30, 2026

Copy link
Copy Markdown
Contributor

A reader reported that the hover link leads to a missing page: https://ui.simai.io/smart.buttons. Links were built as <baseUrl>/<registry ID>, and the site has no such pages.

  • Links now come from contracts/docs-pages.json, the reference pages recorded from the site's sitemap (npm run vendor:docs). sf-button links to /ru/smart-components/actions/buttons/.
  • sf.docs.baseUrl defaults to https://ui.simai.io; an empty value means no link.
  • The generic README link is shown only when a tag has no page.
  • Utility families with no page of their own (11 of 58) link to /ru/utilities/.
  • npm run docs:check fails the release and the weekly workflow if a recorded page is no longer published.

Verified: 52 of 52 tags have a page, and each page names its tag; all 95 link targets answer 200; unit tests 32/32.

Known limit: the reference exists only in Russian on the site for now, so the links open Russian pages.

🤖 Generated with Claude Code

zabarov and others added 2 commits October 30, 2026 12:45
Hovers linked to <baseUrl>/<registry ID>, for example /smart.buttons, a page
the site never had. Build links from the pages the site lists instead, record
those pages in contracts/docs-pages.json, default sf.docs.baseUrl to
https://ui.simai.io, and fail the release and the weekly check if a recorded
page disappears.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@zabarov
zabarov merged commit 11550ed into main Oct 30, 2026
1 check passed
@zabarov
zabarov deleted the fix/docs-links branch October 30, 2026 09:48
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.

1 participant