Skip to content

Migrate link checking to lychee - #304

Merged
sobolevbel merged 4 commits into
masterfrom
migrate-link-check-to-lychee
Aug 16, 2026
Merged

Migrate link checking to lychee#304
sobolevbel merged 4 commits into
masterfrom
migrate-link-check-to-lychee

Conversation

@sobolevbel

Copy link
Copy Markdown
Owner

Replaces tcort/github-action-markdown-link-check with lycheeverse/lychee-action.

Why: the old checker runs files sequentially with no concurrency option — 6–7 min per run — and its default HTTP client trips bot protection (today it took three whack-a-mole ignore commits to get PR #303 green).

What changed:

  • .github/workflows/markdown_link_check.yml → lychee-action@v2, job name kept as markdown-link-check. Results cached via actions/cache (--max-cache-age 1d), GITHUB_TOKEN passed to avoid github.com rate limits.
  • mlc_config.jsonlychee.toml: all ignorePatterns carried over; browser-like User-Agent; accept = 200..=204 as before.
  • New excludes for sites that reject non-browser clients even with a browser UA: facebook.com (400), aplikacja.ceidg.gov.pl (403), pz.gov.pl (426), plus one www.gov.pl page that intermittently resets connections.
  • CLAUDE.md updated to mention lychee.

Local run: 1851 links (796 unique) checked in ~3.2s, 0 errors.

⚠️ Touches .github/workflows/ — must be merged locally (rebase + fast-forward), gh pr merge will fail due to missing workflow scope.

Parallel Rust checker: full run takes ~3s instead of 6-7 minutes.
Browser-like User-Agent cuts down bot-blocking false positives.
All ignorePatterns from mlc_config.json carried over to lychee.toml,
plus facebook.com, aplikacja.ceidg.gov.pl and pz.gov.pl which reject
non-browser clients (400/403/426). Results cached for a day via
actions/cache.
…ners

Also point the FAQ link at the redirect target and drop the utm junk.
…sets non-browser TLS clients

Other gov.pl subdomains (biznes, podatki, zus...) check fine and stay covered.
@sobolevbel
sobolevbel merged commit f3bde7b into master Aug 16, 2026
3 checks passed
@sobolevbel
sobolevbel deleted the migrate-link-check-to-lychee branch August 16, 2026 15:17
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