Skip to content

fix: repair links identified in weekly checker reports - #392

Merged
rjan90 merged 1 commit into
mainfrom
phi/fix-weekly-link-report
Oct 26, 2026
Merged

rjan90 merged 1 commit into
mainfrom
phi/fix-weekly-link-report

Conversation

@rjan90

@rjan90 rjan90 commented Oct 26, 2026

Copy link
Copy Markdown
Contributor

📝 Description

Repair the actionable links identified in eight weekly reports from 6 September–25 October 2026. The latest report (#376) includes a deleted FWSS Solidity source file, an incomplete CID example extracted as a URL, recurring Geo Web TLS failures, and recurring Cha-Ching timeouts.

Type: Bug fix

🛠️ Key Changes

  • Point the FWSS source link to service_contracts/src/modules, following the v1.5.0 move from a single Solidity contract to modules.
  • Replace Geo Web's failing website destination with its working official GitHub organization.
  • Exclude only the incomplete https://cid.contact/routing/v1/providers/ prefix. The anchored pattern leaves populated provider-lookup URLs checked.
  • Update both ERC-8004 tutorial links, the PDP provider documentation link, and the FAQ blog link to their current canonical Filecoin paths.

The CID prefix failed in all eight reviewed reports; Geo Web failed TLS in the latest three. Cha-Ching timed out in the latest five but returned HTTP 200 during verification, so it remains checked without an exclusion.

Reporter improvements are tracked separately in #391. This PR does not automatically close #376 because a complete weekly scan has not been rerun.

🧪 How to Test

  • Passed Biome checks for all three changed TypeScript/TSX files.
  • Passed git diff --check.
  • Verified the new CID exclusion against seven positive/negative cases, including populated provider URLs and lookalike domains.
  • All five unique replacement URLs returned HTTP 200 using the workflow's LinkChecker user agent.
  • Confirmed the incomplete CID prefix returns 404 while a complete lookup returns HTTP 200 and provider data.
  • Reproduced Geo Web's TLS failure and verified Cha-Ching currently returns HTTP 200.

The full scheduled Lychee scan was not run locally; validation targeted the changed destinations and reported failures.

🔖 Resources

@vercel

vercel Bot commented Oct 26, 2026 •

Copy link
Copy Markdown

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

Project Deployment Actions Updated
filecoin-cloud Ready Ready Preview Oct 26, 2026 9:03am UTC

Request Review

@FilOzzy FilOzzy added this to FOC Oct 26, 2026
@github-project-automation github-project-automation Bot moved this to 📌 Triage in FOC Oct 26, 2026
@rjan90
rjan90 marked this pull request as ready for review October 26, 2026 09:05
@rjan90
rjan90 requested a review from TippyFlitsUK as a code owner October 26, 2026 09:05
@rjan90 rjan90 self-assigned this Oct 26, 2026
@rjan90 rjan90 moved this from 📌 Triage to 🔎 Awaiting review in FOC Oct 26, 2026
@github-project-automation github-project-automation Bot moved this from 🔎 Awaiting review to ✔️ Approved by reviewer in FOC Oct 26, 2026
@rjan90
rjan90 merged commit 8c583c9 into main Oct 26, 2026
7 checks passed
@rjan90
rjan90 deleted the phi/fix-weekly-link-report branch October 26, 2026 13:48
@github-project-automation github-project-automation Bot moved this from ✔️ Approved by reviewer to 🎉 Done in FOC Oct 26, 2026
@github-actions github-actions Bot added size/M and removed size/M labels Oct 26, 2026

This branch was successfully deployed

1 active deployment
Preview – filecoin-cloud — faef16a8 Deployed Oct 26, 2026 by vercel[bot]
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

Status: 🎉 Done

Development

Successfully merging this pull request may close these issues.

Link Checker Report

2 participants