Skip to content

ci: enable Lychee external link checks - #785

Merged
brendanjryan merged 1 commit into
mainfrom
brendanjryan/fix-lychee-links
Jul 31, 2026
Merged

ci: enable Lychee external link checks#785
brendanjryan merged 1 commit into
mainfrom
brendanjryan/fix-lychee-links

Conversation

@brendanjryan

Copy link
Copy Markdown
Contributor

Motivation

The External Links workflow passes without scanning docs because Lychee does not expand the brace glob. Enabling the source globs directly also makes Vocs-local routes look like filesystem failures.

Summary

  • scan Markdown and MDX sources with explicit Lychee globs
  • preprocess local Vocs destinations while preserving external links and source line numbers
  • repair moved external documentation and source links
  • document narrowly scoped exclusions for verified API and app endpoints

Key design considerations

  • keep internal route and fragment validation owned by the checks introduced in ci: validate internal documentation anchors #784
  • exclude only endpoints that are verified valid but do not serve checker-friendly GET responses
  • pin historical TIP source links to an immutable commit

@vercel

vercel Bot commented Jul 31, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
tempo-docs Ready Ready Preview Jul 31, 2026 4:24pm

Request Review

@brendanjryan
brendanjryan force-pushed the brendanjryan/fix-lychee-links branch from 76956f6 to 2079438 Compare July 31, 2026 16:21
@brendanjryan
brendanjryan marked this pull request as ready for review July 31, 2026 16:28
@brendanjryan
brendanjryan merged commit b9c9d51 into main Jul 31, 2026
11 checks passed
@brendanjryan
brendanjryan deleted the brendanjryan/fix-lychee-links branch July 31, 2026 16:30

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 2079438544

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread scripts/filter-external-links.ts
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