Skip to content

ci(lychee): exclude the VS Code Marketplace, it returns 503 to automated requests - #1735

Merged
sbryngelson merged 1 commit into
masterfrom
fix/lychee-vscode-marketplace
Aug 19, 2026
Merged

ci(lychee): exclude the VS Code Marketplace, it returns 503 to automated requests#1735
sbryngelson merged 1 commit into
masterfrom
fix/lychee-vscode-marketplace

Conversation

@sbryngelson

Copy link
Copy Markdown
Member

Description

The Lychee link check fails on Build & Verify with a single error:

### Errors in build/install/docs/mfc/documentation/getting-started.html

* [503] <https://marketplace.visualstudio.com/items?itemName=ms-vscode-remote.remote-wsl> (at 218:86)
      | Rejected status code: 503 Service Unavailable

That is the "Remote - WSL" extension link at docs/documentation/getting-started.md:87, the only
marketplace.visualstudio.com URL in the tree. The link is not broken for a human — the Marketplace
returns 503 to automated requests, the same bot-blocking .lychee.toml already carries exclusions
for on apps.microsoft.com/store/detail/ (403), code.visualstudio.com (403), stackoverflow.com
(403) and olcf.ornl.gov/summit (503). accept lists 502 and 504 but not 503, so it fails hard
after all 5 retries and takes the job with it.

Excluding the host is the targeted fix; adding "503" to accept would suppress genuinely dead
links everywhere else.

Type of change

  • Other (CI configuration)

Testing

  • .lychee.toml parses, and the new pattern matches the failing URL.
  • Checked it does not over-match: visualstudio.com, github.com and mflowcode.github.io links
    are still checked.
  • ./mfc.sh precheck passes all seven gates.

Checklist

  • I added or updated tests for new behavior
  • I updated documentation if user-facing behavior changed

Copilot AI lite review requested due to automatic review settings August 18, 2026 16:06

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Warning

Copilot couldn't run its full agentic review because it didn't start before the timeout. Make sure your repository has a runner available, or add a copilot-code-review.yml file specifying one with the runs-on attribute. See the docs for more details.

Updates Lychee link-check configuration to avoid CI failures caused by VS Code Marketplace returning 503 to automated requests.

Changes:

  • Add marketplace.visualstudio.com to Lychee’s exclude list to prevent repeated 503 failures during link checking.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment thread .lychee.toml
"https://apps\\.microsoft\\.com/store/detail/", # Microsoft Store detail pages return 403 to automated requests
"https://code\\.visualstudio\\.com/?$", # Root page returns 403 to automated requests
"https://stackoverflow\\.com", # Returns 403 to automated requests
"https://marketplace\\.visualstudio\\.com", # Returns 503 to automated requests
@sbryngelson
sbryngelson merged commit b281bfe into master Aug 19, 2026
34 of 35 checks passed
@sbryngelson
sbryngelson deleted the fix/lychee-vscode-marketplace branch August 19, 2026 02:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants