Add HTML link checking, move files to match SUMMARY nesting, fix all broken links#1091
Merged
Conversation
louis-pre
changed the title
louis-pre
changed the title
- Add HTML <a href> link checking to validate-links (both relative and absolute docs.seam.co URLs) - Stop absolute URL pattern at quote characters for HTML compatibility - Extend validate-paths to check that nested SUMMARY.md entries have file paths under their parent's directory, matching how GitBook builds published URLs from the tree structure Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
GitBook derives published URLs from SUMMARY.md tree nesting. Files nested under a parent must live in the parent's directory. Moved 10 brand-guide get-started pages into their parent directories and 2 guides files. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
louis-pre
force-pushed
the
fix-ecobee-landing-page
branch
from
f710a0c to
789e927
Compare
louis-pre
changed the title
Fix broken links found by the new HTML link checker: - Fix asset paths in brand-guides (wrong ../ depth after section move) - Fix cross-section asset refs pointing to guides/.gitbook/assets/ - Fix brand-guides/README.md card links to use correct local paths - Fix guides/README.md card links with correct absolute URLs - Fix cross-section API and capability guide links - Fix internal links broken by file moves Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
louis-pre
force-pushed
the
fix-ecobee-landing-page
branch
from
da91316 to
fd012bb
Compare
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
2 participants
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
1. Extend validators (
bb9dc91a)validate-links: Check<a href>tags (relative + absolutedocs.seam.coURLs)validate-paths: Check that nested SUMMARY.md entries have file paths under their parent's directory, matching how GitBook builds published URLs2. Move files to match SUMMARY.md nesting (
10b51ccf)get-started-with-ecobee-thermostats.md→ecobee-thermostats/)3. Fix all broken HTML links (
789e9277)../depth after section move)guides/.gitbook/assets/Test plan
npm run validate-linkspassesnpm run validate-pathspassesnpx tsc --noEmitpasses🤖 Generated with Claude Code