docs(solidity-resources): drop dead LearnWeb3 link - #74
Conversation
https://learnweb3.io has returned 5xx on every weekly external-link run since 2026-08-03 (#52, #70, #73). It is not a transient outage: the host answers with `x-render-routing: suspend` and the body "This service has been suspended", the LearnWeb3DAO GitHub org has had no activity since Aug 2025 and still points at the dead domain, and no successor domain exists (learnweb3.xyz is an unrelated marketing page). Remove the entry rather than send readers to a suspended service; the Basic Solidity section still lists four live resources. The VS Code Marketplace link flagged in #63 was a one-off 503 on 2026-08-17; it has passed every run since and resolves to the NomicFoundation.hardhat-solidity extension, so it is left as is. Closes #52, #63, #70, #73 Co-authored-by: Cursor <cursoragent@cursor.com>
|
Preview deployment for your docs. Learn more about Mintlify Previews.
💡 Tip: Enable Automations to automatically generate PRs for you. |
PR SummaryLow Risk Overview The page still lists four beginner resources (CryptoZombies, Solidity by Example, and two Cyfrin Updraft courses). No other edits on the page; this clears broken-link checks tied to that URL. Reviewed by Cursor Bugbot for commit 50054f0. Bugbot is set up for automated code reviews on this repo. Configure here. |
There was a problem hiding this comment.
A clean, well-evidenced 2-line docs removal of a dead LearnWeb3 link from evm/solidity-resources.mdx, with sound reasoning for leaving the VS Code Marketplace link untouched. No correctness, security, or style issues; the only notes are the stale copy in the generated llms-full.txt (self-resolving) and the fact that both second-opinion review passes produced no output.
Findings: 0 blocking | 5 non-blocking | 0 posted inline
Blockers
- None at the file/PR level.
Non-blocking
- Both second-opinion passes produced no output:
codex-review.md(OpenAI Codex) andcursor-review.md(Cursor) are empty (1 byte each), so this review reflects only my own pass.REVIEW_GUIDELINES.mdis also empty, so no repo-specific standards could be applied. llms-full.txt:16921still contains the removed LearnWeb3 entry. This is correct to leave alone —AGENTS.md:51states the file is generated byscripts/generate-llms.mjsand says "do not hand-edit the generated.txtfiles." One caveat worth knowing: that script fetches each page's.mdfrom the deployed site, so the weeklyregenerate-llms.ymlrun will only drop the line after this PR is merged and deployed. If the timing matters, trigger the job viaworkflow_dispatchpost-deploy rather than waiting a week.- Agreed with the decision not to add
marketplace.visualstudio.comto thelychee.tomlexcludelist. That list is scoped to local/example hosts and domains that hard-block automated requests; a URL that is currently returning 200 doesn't belong there, and excluding it would mask genuine rot if the extension is later renamed or unpublished. - I could not independently re-verify the live HTTP status of
learnweb3.iofrom this environment (no outbound network access). The verdict rests on the evidence in the PR description — 502/503 across four consecutive weekly runs, thex-render-routing: suspendheader indicating a suspended Render deployment, and no successor domain — which is thorough and internally consistent. - Nit, pre-existing and out of scope: line 12 of
evm/solidity-resources.mdxmisspells "Solidity" as "Soldity". Not introduced by this PR, but it is two lines from the edit if you want a drive-by fix.
Resolves every open broken-external-links report: #52, #63, #70, #73. All four point at
evm/solidity-resources.mdxand between them name two URLs.What is the purpose of the change?
Delete a stale link to a service that no longer exists, and close out the tracking issues.
Describe the changes to the documentation
Removed the "Ethereum Developer Degree by LearnWeb3" entry (
https://learnweb3.io) from the Basic Solidity list. The link has failed every weekly run since 2026-08-03 (502 in #52, 503 in #70 and #73), and it is not a transient outage:HTTP 503withx-render-routing: suspendand a body of "This service has been suspended" — Render's response for a suspended deployment. Same result for/,/degrees/, and the degree page itself, with and without a browser user agent.learnweb3.xyzresponds but is an unrelated "airdrops / make money in web3" landing page, not the education platform.learnweb3.com/.devdon't resolve.Sending readers to a "Service Suspended" page is worse than not listing it. The section keeps four live resources (CryptoZombies, Solidity by Example, and two Cyfrin Updraft courses). Nothing else on the page references LearnWeb3.
Left the VS Code Marketplace link as is. #63 flagged
marketplace.visualstudio.com/items?itemName=NomicFoundation.hardhat-soliditywith a 503 on 2026-08-17. That was a one-off: it passed the 2026-08-31 and 2026-09-07 runs, returns 200 on GET (lychee's method) today, and the page body identifiesPublisherName: NomicFoundation/ExtensionName: hardhat-solidity, so it isn't a soft 404. Not adding it to the lychee exclude list either — it's being checked successfully, and excluding it would hide real rot if the extension is ever renamed or unpublished.Notes
Verification — lychee
0.24.2with the repo'slychee.toml, same args asexternal-links.yml:This morning's scheduled run reported 1035 / 599 / 1 error; the one-link difference is the removed URL.
llms-full.txtstill contains the old line, but perAGENTS.mdit is generated, not hand-edited — the weeklyregenerate-llms.ymljob will pick this up.Closes #52
Closes #63
Closes #70
Closes #73