Skip to content

[docs-agent] Exclude bscscan.com and tenderly.co from lychee#1428

Open
alchemy-bot wants to merge 2 commits into
mainfrom
docs-agent/lychee-exclude-bscscan-tenderly
Open

[docs-agent] Exclude bscscan.com and tenderly.co from lychee#1428
alchemy-bot wants to merge 2 commits into
mainfrom
docs-agent/lychee-exclude-bscscan-tenderly

Conversation

@alchemy-bot

Copy link
Copy Markdown
Contributor

Summary

The 2026-07-06 weekly link check flagged 3 broken links, all third-party bot-block / rate-limit failures (matching the classification the bot itself recommends handling via the exclude list):

File URL Status
content/wallets/pages/gas-manager-admin-api/gas-manager-faqs.mdx https://bscscan.com/token/0x8AC76a51cc950d9822D68b83fE1Ad97B32Cd580d 403
content/wallets/pages/bundler-api/entrypoint-revert-codes/entrypoint-v06-revert-codes.mdx https://tenderly.co/ 429
content/wallets/pages/transactions/debug-transactions/debug-with-tenderly.mdx https://tenderly.co/ 429

All three URLs point to valid, current destinations (BSC's block explorer page for the USDT token contract; Tenderly's homepage). The failures are purely crawler-side. Following the established pattern in lychee.toml (existing precedent: celoscan.io, docs.arbitrum.io, developer.offchainlabs.com, medium.*, etc.), added two new exclude patterns:

".*bscscan\\.com.*",             # Bot protection blocks crawlers with 403 (Etherscan-family explorer)
".*tenderly\\.co.*",             # Rate-limits crawlers with 429 (page loads fine in browser)

No MDX content changes needed. Next week's link check should report clean for these domains.

Linear

DOCS-133 — https://linear.app/alchemyapi/issue/DOCS-133/weekly-link-check-exclude-bscscan-and-tenderly-from-lychee

Slack

Originating thread: https://alchemyplatform.slack.com/archives/C0852FMMUMR/p1783360729767349

Requested by

Weekly Link Check bot (no human originator; no Requested-by: trailer per no-self-approval workflow's degraded-attribution path).

Weekly link check on 2026-07-06 flagged 3 broken links, all third-party bot-block / rate-limit failures:

- bscscan.com/token/0x8AC76a51cc950d9822D68b83fE1Ad97B32Cd580d (403) - referenced from content/wallets/pages/gas-manager-admin-api/gas-manager-faqs.mdx
- tenderly.co/ (429) - referenced from two MDX pages under content/wallets/pages/

Both are legitimate destinations, just crawler-blocked. Add exclude patterns matching the existing Etherscan-family and rate-limited-third-party conventions in the same file.

Refs DOCS-133
@alchemy-bot alchemy-bot requested a review from a team as a code owner July 6, 2026 18:00
@github-actions

github-actions Bot commented Jul 6, 2026

Copy link
Copy Markdown

🔗 Preview Mode

Name Status Preview Updated (UTC)
Alchemy Docs ✅ Ready 🔗 Visit Preview Jul 6, 2026, 8:05 PM

@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: effb852e1c

ℹ️ About Codex in GitHub

Codex has been enabled to automatically 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 👍.

When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".

Comment thread lychee.toml Outdated
".*celoscan\\.io.*", # Bot protection blocks crawlers with 403 (Etherscan-family explorer)
".*bitsofco\\.de.*", # Intermittent connection failures from crawlers (page loads fine in browser)
".*bscscan\\.com.*", # Bot protection blocks crawlers with 403 (Etherscan-family explorer)
".*tenderly\\.co.*", # Rate-limits crawlers with 429 (page loads fine in browser)

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P2 Badge Narrow the Tenderly exclusion to the homepage

In the scheduled link check, .github/workflows/link-check-scheduled.yml runs lychee over ./content with this config, and lychee excludes regex-matching links. This unanchored pattern matches every Tenderly subdomain, so current links like https://docs.tenderly.co/debugger and https://dashboard.tenderly.co/register are now skipped even though only https://tenderly.co/ was failing with 429. If any of those docs/dashboard URLs typo or return 404, the weekly check will stay green; please restrict the regex to the root host/path or add includes for docs/dashboard.

Useful? React with 👍 / 👎.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Good catch, applied in 1439da1. Tightened to .*//tenderly\.co/?$ so only the bare root URL (which is what returned 429) is excluded. docs.tenderly.co/debugger, docs.tenderly.co/simulator-ui, and dashboard.tenderly.co/* will keep getting checked by the weekly lychee run.

Verified with the Rust regex::is_match-equivalent search behavior against the actual URLs in content/wallets/pages/transactions/debug-transactions/debug-with-tenderly.mdx and content/wallets/pages/bundler-api/entrypoint-revert-codes/entrypoint-v06-revert-codes.mdx — only https://tenderly.co/ (and https://tenderly.co) match.

bscscan.com intentionally stays broad because it's Etherscan-family bot protection (uniform across the whole domain, matching the celoscan.io precedent), and only the token page under the root is linked in the docs.

Codex P2 review flagged that the unanchored '.*tenderly\.co.*' pattern also excludes legitimate docs.tenderly.co/dashboard.tenderly.co links used elsewhere in the docs (e.g. debug-with-tenderly.mdx links to https://docs.tenderly.co/debugger and https://docs.tenderly.co/simulator-ui). Tightening to '.*//tenderly\.co/?$' matches only the bare root URL that was returning 429 while keeping the subdomain URLs discoverable to the weekly link check.

Refs DOCS-133
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