Skip to content

Add link validation CI and fix broken doc URL - #29

Closed
Nikola Metulev (nmetulev) with Copilot wants to merge 2 commits into
mainfrom
copilot/fix-404-doc-links
Closed

Nikola Metulev (nmetulev) with Copilot wants to merge 2 commits into
mainfrom
copilot/fix-404-doc-links

Conversation

Copilot AI commented Mar 13, 2026 •

Copy link
Copy Markdown
Contributor

AI-generated skill files occasionally reference dead Microsoft Learn URLs. Adds automated link checking to catch these on PRs and fixes one confirmed broken URL.

GitHub Actions workflow (.github/workflows/link-check.yml)

  • Runs lycheeverse/lychee-action@v2 against all **/*.md files
  • Triggers on PR to main, push to main, weekly schedule (Mondays 08:00 UTC), and workflow_dispatch
  • Fails the check if any broken links are found; weekly run catches link rot over time
  • Uses GITHUB_TOKEN to avoid GitHub API rate limiting; scoped to contents: read

Lychee configuration (.lychee.toml)

  • Accepts HTTP 429 to suppress false positives from rate-limited endpoints
  • Excludes non-navigable URLs: XML namespace URIs (schemas.microsoft.com), example.com placeholders, app.local, and DigiCert timestamp server URLs that appear in code samples

Broken link fix

  • data-persistence/SKILL.md: corrected stale files-and-folders/file-access-permissions path to the current develop/files/file-access-permissions (confirmed against file-handling/SKILL.md which already used the correct path)

Warning

Firewall rules blocked me from connecting to one or more addresses (expand for details)

I tried to connect to the following addresses, but was blocked by firewall rules:

  • learn.microsoft.com
    • Triggering command: /usr/bin/curl curl -s -o /dev/null -w %{http_code} REDACTED /frameworks/SKILL.md /setup/SKILL.md /signing/SKILL.md /troubleshoot/SKILL.md (dns block)
    • Triggering command: /home/REDACTED/work/_temp/ghcca-node/node/bin/node node /home/REDACTED/work/_temp/ghcca-node/node/bin/markdown-link-check .github/plugin/skills/winui3/data-persistence/SKILL.md --config /dev/stdin (dns block)

If you need me to access, download, or install something from one of these locations, you can either:

Original prompt

This section details on the original issue you should resolve

<issue_title>Some skills have 404 doc links</issue_title>
<issue_description>AI generated skills at times refer to dead documentation links on Microsoft Learn. It should check that the link actually exists and correct them.</issue_description>

<agent_instructions>Can you do a pass on all the links and correct any that are wrong, but also create a github action to validate all links on a pr before merging</agent_instructions>

Comments on the Issue (you are Copilot in this section)

Nikola Metulev (@nmetulev) Let's try to solve this with a github action that validates all links and cross linking

💬 Send tasks to Copilot coding agent from Slack and Teams to turn conversations into code. Copilot posts an update in your thread when it's finished.

Co-authored-by: nmetulev <711864+nmetulev@users.noreply.github.com>
Copilot AI changed the title [WIP] Fix 404 doc links for AI generated skills Add link validation CI and fix broken doc URL Mar 13, 2026
@nmetulev
Nikola Metulev (nmetulev) deleted the copilot/fix-404-doc-links branch May 11, 2026 22:45
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.

Some skills have 404 doc links

2 participants