Skip to content

fix(ci): check links on live pages via sitemap instead of empty dist/ - #589

Merged
mohanadft merged 1 commit into
mainfrom
fix/link-check-live-sitemap
Oct 13, 2026
Merged

mohanadft merged 1 commit into
mainfrom
fix/link-check-live-sitemap

Conversation

@mohanadft

@mohanadft mohanadft commented Oct 13, 2026 •

Copy link
Copy Markdown
Collaborator

Summary

  • The weekly link check has failed every run since at least 2026-08-24: ./dist/**/*.html matches nothing because the site is SSR (output: "server"), so lychee exits 1 with "No links were found".
  • Replace the build + dist glob with a step that reads URLs from the production sitemap and passes them to lychee as URL inputs, so each live page is fetched and its links checked.
  • Accept 403/429 and exclude LinkedIn/X/Twitter/Instagram, which block bots.

Test plan

  • Trigger "Weekly Link Check" via workflow_dispatch on this branch
  • Confirm it checks links (Total > 0) and review any genuine broken links it reports
  • Tune exclusions if bot-blocking noise appears

The site is SSR (output: server), so dist/ has no HTML and lychee failed
every week with 'No links were found'. Read URLs from the production
sitemap and let lychee fetch and check each live page.
@cloudflare-workers-and-pages

Copy link
Copy Markdown

Deploying website with  Cloudflare Pages  Cloudflare Pages

Latest commit: cc5074b
Status: ✅  Deploy successful!
Preview URL: https://8f038018.website-aun.pages.dev
Branch Preview URL: https://fix-link-check-live-sitemap.website-aun.pages.dev

View logs

@mohanadft
mohanadft merged commit b8f74c6 into main Oct 13, 2026
6 checks passed
@mohanadft
mohanadft deleted the fix/link-check-live-sitemap branch October 13, 2026 08:51
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