Skip to content

fix(docs): exclude npmjs.com from lychee link checker#734

Merged
yacosta738 merged 3 commits into
mainfrom
copilot/fix-broken-links-report
May 1, 2026
Merged

fix(docs): exclude npmjs.com from lychee link checker#734
yacosta738 merged 3 commits into
mainfrom
copilot/fix-broken-links-report

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented May 1, 2026

Related Issues


Summary

npmjs.com returns 403 Forbidden to bots/crawlers, causing the Lychee link checker to flag https://www.npmjs.com/package/@dallay/corvus (referenced in README.md) as a broken link. The URL is valid — npm simply blocks automated HTTP clients.

Fix: Add npmjs.com to the exclude list in lychee.toml, consistent with existing bot-blocking exclusions (mend.io, surrealdb.com):

# npmjs.com returns 403 Forbidden to bots/crawlers
'https://www\.npmjs\.com/.*',

Tested Information

Verified the TOML parses cleanly and the regex pattern matches the failing URL (https://www.npmjs.com/package/@dallay/corvus).


Documentation Impact

  • No docs update required because: this change only adjusts the link-checker exclusion config; no user-facing docs, APIs, or behavior changed.
  • I verified the documentation matches the current behavior.

Breaking Changes


Checklist

  • I have checked that there isn't already a PR solving the same problem.
  • I have read the Contributing Guidelines.
  • I ensured my code follows the project's style guidelines.
  • I have added or updated tests that prove my fix is effective or that my feature works.
  • I have updated the documentation, or I explained above why no documentation update is needed.
  • I verified the documentation matches the current behavior.
  • I have documented any breaking changes in the Breaking Changes section.
  • I have linked the related issue (if any).

Copilot AI linked an issue May 1, 2026 that may be closed by this pull request
@cloudflare-workers-and-pages
Copy link
Copy Markdown

cloudflare-workers-and-pages Bot commented May 1, 2026

Deploying corvus with  Cloudflare Pages  Cloudflare Pages

Latest commit: b0f0b42
Status: ✅  Deploy successful!
Preview URL: https://b61d3c9f.corvus-42x.pages.dev
Branch Preview URL: https://copilot-fix-broken-links-rep.corvus-42x.pages.dev

View logs

Copilot AI changed the title [WIP] Fix broken links detected by link checker fix(docs): exclude npmjs.com from lychee link checker May 1, 2026
Copilot AI requested a review from yacosta738 May 1, 2026 07:34
@yacosta738 yacosta738 marked this pull request as ready for review May 1, 2026 07:41
@github-actions github-actions Bot added the size/xs Denotes an extra small change size label May 1, 2026
@yacosta738 yacosta738 enabled auto-merge May 1, 2026 07:45
@sonarqubecloud
Copy link
Copy Markdown

sonarqubecloud Bot commented May 1, 2026

@yacosta738 yacosta738 disabled auto-merge May 1, 2026 09:16
@yacosta738 yacosta738 merged commit a5d8d61 into main May 1, 2026
16 of 18 checks passed
@yacosta738 yacosta738 deleted the copilot/fix-broken-links-report branch May 1, 2026 09:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size/xs Denotes an extra small change size

Projects

None yet

Development

Successfully merging this pull request may close these issues.

docs: broken links report

2 participants