Add link checker for Gatsby Docs

[muescha]

Summary

There is a plugin for a remark Link check: https://www.gatsbyjs.org/packages/gatsby-remark-check-links/

Can this also work on the Gatsby docs/Blog?

Or as a PR check?

new edit:

Ideas

• check local links

  • add a linter to check before it is submitted
    • check if it has a local domain name (A lint rule is WIP)
    • check if local domains begin with a ., .. or /
  • allow local domain and remove local domain on build time with a transformer

• check local anchors

  • with a linter and analyse each linked file?
  • on gatsby develop/gatsby build via a graphql query to query all headers?

• check internal links for trailing slashes (because breadcrumbs and selected sidebar is depend on it)

  • check with linter

• check external links

  • HEAD request
  • with blc (broken link checker)

• check internal link files (like doc-links.yml, redirects.yml)

  • with a link checker script (like site showcase validator)

possible tools

• https://gatsbyjs.org/packages/gatsby-remark-check-links
• https://github.com/davidtheclark/remark-lint-no-dead-urls/
• https://npmjs.com/package/mdx-link-checker
  • no repo: but source @ unpkg:
    • https://unpkg.com/mdx-link-checker
    • hightlighted: https://unpkg.com/browse/[email protected]/index.js)
• https://github.com/remarkjs/remark-validate-links
  • Test: it finds some links, but just now it only checks the:
    • relative links (like [abc](docs/wronglink) but they have to be [abc](docs/rightlink.md)
    • but not find absolute paths like:
      • [abc](/docs/wronglink)
      • [abc](https://www.gatsbyjs.org/docs/wronglink)
  • maybe i have to tweak settings...
• https://github.com/zslabs/remark-relative-links -> transformer to remove domains, but can only used if we use remark instead off prettier

[github-actions]

Hiya!

This issue has gone quiet. Spooky quiet. 👻

We get a lot of issues, so we currently close issues after 30 days of inactivity. It’s been at least 20 days since the last update here.
If we missed this issue or if you want to keep it open, please reply here. You can also add the label "not stale" to keep this issue open!
As a friendly reminder: the best way to see this issue, or any other, fixed is to open a Pull Request. Check out gatsby.dev/contribute for more information about opening PRs, triaging issues, and contributing!

Thanks for being a part of the Gatsby community! 💪💜

[github-actions]

Hiya!

This issue has gone quiet. Spooky quiet. 👻

We get a lot of issues, so we currently close issues after 30 days of inactivity. It’s been at least 20 days since the last update here.
If we missed this issue or if you want to keep it open, please reply here. You can also add the label "not stale" to keep this issue open!
As a friendly reminder: the best way to see this issue, or any other, fixed is to open a Pull Request. Check out gatsby.dev/contribute for more information about opening PRs, triaging issues, and contributing!

Thanks for being a part of the Gatsby community! 💪💜

[muescha]

any opinion about this?

as seen in the linked issues there some places where 404 exists without any notification when a PR is done...

[muescha]

closing #18908 as duplicate of this Issue.
copy some comments from duplicate

Summary

It would be nice to have some link check on the gatsbyjs.org site maybe as validator while an PR or better also on gatsby develop to find old broken links

Caveeats

I know we can not find and check all links because some are generated from sourced data.

Motivation

During Hacktoberfest I see also some broken links reported. Example #18905

It would be nice to avoid broken links from beginning.

note

I don't mean the broken links reported for viewing files on github

Originally posted by @muescha in #18908 (comment)

Maybe putting something like Hyperlink into a GitHub Action could be a good solution to this so it would run similar to how the site showcase validator works right now.

Originally posted by @lannonbr in #18908 (comment)

[github-actions]

Hiya!

This issue has gone quiet. Spooky quiet. 👻

We get a lot of issues, so we currently close issues after 30 days of inactivity. It’s been at least 20 days since the last update here.
If we missed this issue or if you want to keep it open, please reply here. You can also add the label "not stale" to keep this issue open!
As a friendly reminder: the best way to see this issue, or any other, fixed is to open a Pull Request. Check out gatsby.dev/contribute for more information about opening PRs, triaging issues, and contributing!

Thanks for being a part of the Gatsby community! 💪💜

[github-actions]

Hiya!

This issue has gone quiet. Spooky quiet. 👻

We get a lot of issues, so we currently close issues after 30 days of inactivity. It’s been at least 20 days since the last update here.
If we missed this issue or if you want to keep it open, please reply here. You can also add the label "not stale" to keep this issue open!
As a friendly reminder: the best way to see this issue, or any other, fixed is to open a Pull Request. Check out gatsby.dev/contribute for more information about opening PRs, triaging issues, and contributing!

Thanks for being a part of the Gatsby community! 💪💜

[muescha]

Not stale

[karlhorky]

FYI for root-relative links with remark-validate-links I have opened an issue for that here: remarkjs/remark-validate-links#57

[karlhorky]

I ended up taking https://npmjs.com/package/mdx-link-checker and modifying it:

1. now handles href, to, src props
2. matches name as well as id for anchor checking
3. adds index.mdx if linking to a directory
4. check root relative links by passing in a base directory
5. now has an ignore glob

I will open source this in the coming days / weeks.

[karlhorky]

@muescha ok if you want to try it out, here it is:

npm: https://www.npmjs.com/package/mdx-local-link-checker
Source: https://github.com/upleveled/mdx-local-link-checker

[github-actions]

Hiya!

This issue has gone quiet. Spooky quiet. 👻

We get a lot of issues, so we currently close issues after 60 days of inactivity. It’s been at least 20 days since the last update here.
If we missed this issue or if you want to keep it open, please reply here.
As a friendly reminder: the best way to see this issue, or any other, fixed is to open a Pull Request. Check out gatsby.dev/contribute for more information about opening PRs, triaging issues, and contributing!

Thanks for being a part of the Gatsby community! 💪💜

[karlhorky]

Not stale

[github-actions]

Hiya!

This issue has gone quiet. Spooky quiet. 👻

We get a lot of issues, so we currently close issues after 60 days of inactivity. It’s been at least 20 days since the last update here.
If we missed this issue or if you want to keep it open, please reply here.
As a friendly reminder: the best way to see this issue, or any other, fixed is to open a Pull Request. Check out gatsby.dev/contribute for more information about opening PRs, triaging issues, and contributing!

Thanks for being a part of the Gatsby community! 💪💜

[karlhorky]

Not stale

[github-actions]

Hiya!

This issue has gone quiet. Spooky quiet. 👻

We get a lot of issues, so we currently close issues after 60 days of inactivity. It’s been at least 20 days since the last update here.
If we missed this issue or if you want to keep it open, please reply here.
As a friendly reminder: the best way to see this issue, or any other, fixed is to open a Pull Request. Check out gatsby.dev/contribute for more information about opening PRs, triaging issues, and contributing!

Thanks for being a part of the Gatsby community! 💪💜

[muescha]

Not stale