Fix link checker: check combined Nuxt+11ty output#5057
Open
Yndira-E wants to merge 1 commit into
Open
Conversation
Runs untitaker/hyperlink against nuxt/.output/public/ which contains the full combined output of both 11ty and Nuxt pages, replacing the previous approach that only checked the 11ty _site/ output. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
✅ Deploy Preview for flowforge-website ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
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.
Summary
untitaker/hyperlinkto the website CI (test.yml) pointing tonuxt/.output/public/, which contains the full combined output of both 11ty and Nuxt pagesuntitaker/hyperlinkwas removed in nuxt: privacy migration #4984 and replaced withnuxt-link-checker, but investigation showednuxt-link-checkeras configured only checks 2 pages (/termsand/privacy-policy) and skips links to 11ty pages entirelyContext
As part of the Nuxt migration,
privacy-policywas moved to Nuxt, so it no longer exists in the 11ty_site/output. This caused a false positive in theflowfuserepo CI (PR #7328) and led to the removal ofuntitaker/hyperlinkfrom the website CI — leaving no effective link checking in place.--sourcesonly accepts a single value, so it's set tosrc/(11ty sources). For broken links originating in Nuxt pages, the error will point to the generated HTML file innuxt/.output/public/rather than the source.vuefile — but all broken links across the full site are still detected regardless.Once the Nuxt migration is complete and all pages are served by Nuxt, we'll evaluate whether
untitaker/hyperlinkis still needed or can be replaced entirely bynuxt-link-checker.Test plan
🤖 Generated with Claude Code