Repository navigation
docs(bridges): remove retired Satellite, offline Galaxy and deprecated Allbridge, fix redirected links - #2374
Conversation
GigaHierz
left a comment
There was a problem hiding this comment.
head 340cf0d · local ref and API headRefOid match; merged into origin/main a12466b in a scratch checkout: clean, mint broken-links --check-redirects success, check-orphans.sh clean; both changed pages render on :3441 at 1280 and 412x915 with 0 console errors and no Satellite/Galaxy text
Verdict: REQUEST-CHANGES
Thanks for this, and for the unusually careful evidence in the description. The removals are correct and I could reproduce them. There is one entry I'd like handled before merge and one inconsistency the PR creates, both small.
Findings
-
Medium ·
tooling/bridges/bridges.mdx:90-100(Allbridge section),tooling/bridges/bridges.mdx:3(description),home/bridged-tokens/bridges.mdx:23(learn/bridged-tokens/bridges.mdx:23on main)- Defect: the PR keeps an entry it has itself shown is about to stop working.
docs.allbridge.iosays "Allbridge Classic is deprecated and will be stopped in November 2026", the link targets Classic (app.allbridge.io), and the live Core chain list has no Celo (I re-ranhttps://api.core.allbridge.io/token-info: ETH, POL, ARB, AVA, OPT, BAS, UNI, BSC, TRX, SOL, SRB, SUI, STX). - Failure scenario: within weeks a reader, or an agent reading
llms.txtwhere the description still advertises "Allbridge", is sent to a bridge that no longer works, which is the same class of problem this PR fixes for Satellite. - Fix shape: remove the Allbridge section, the word "Allbridge" in the description, and the Learn-page bullet in this PR. If you would rather leave the decision to the maintainers, a filed issue linked from the PR body is the only other acceptable resolution; a checkbox in the body is not enough. Restoring it later is cheap if Core adds Celo.
- Defect: the PR keeps an entry it has itself shown is about to stop working.
-
Medium ·
learn/bridged-tokens/bridges.mdx:19and:21(home/bridged-tokens/bridges.mdxon the PR head),learn/gas-fees.mdx:27(home/gas-fees.mdx:27on the PR head)- Defect: the title says "fix redirected links", but the same providers are still linked at their redirecting URLs on two other pages. After this PR the Bridge Providers page says
https://app.squidrouter.com/andhttps://jumper.xyz/while the Learn bridging page sayshttps://v2.app.squidrouter.com/?...andhttps://jumper.exchange/?.... - Failure scenario: one provider, two URLs across pages meant to agree; the next link audit flags the leftover redirects again and the PR title over-promises.
- Fix shape: change the host on those three lines. I checked that query strings survive both redirects (
v2.app.squidrouter.com/?chains=10%2C42220goes 307 toapp.squidrouter.com/?chains=10%2C42220;jumper.exchange/?fromChain=10&toChain=42220goes 301 tojumper.xyz/?fromChain=10&toChain=42220), so it is a host-only edit. Make it on the path as it exists on main (learn/), see merge notes.
- Defect: the title says "fix redirected links", but the same providers are still linked at their redirecting URLs on two other pages. After this PR the Bridge Providers page says
Verified good
- Satellite:
https://docs.axelar.dev/resources/satellite/is titled "Satellite (Retired)" and says "Satellite has been retired. Use Axelar's current interface at app.axelar.network for supported transfers. Do not use satellite.money - that domain is no long[er ...]".curlofsatellite.money/landerreturns 307 toforsale.godaddy.com/forsale/satellite.money. Removal is right, and not substitutingapp.axelar.network(HTTP 200, Celo support unconfirmed) is the right call. - Galaxy:
galaxy.exchange/and/swapboth return HTTP 402 withx-vercel-error: DEPLOYMENT_DISABLED. I found no vendor announcement; the evidence is the disabled deployment only. That is enough to remove a link, but I would soften "is not a bridge" in the description: public listings describe Galaxy as a cross-chain swap aggregator. Offline is the reason, not the category. - Redirected links, old and new with
curl -sIL:v2.app.squidrouter.com307 toapp.squidrouter.com200;jumper.exchange301 tojumper.xyz200 (403 with a short user agent, 200 with a full Chrome one, so a bot challenge and not a dead link);www.usenexus.org301 tousenexus.org308 tonexus.hyperlane.xyz200. All three new targets are the canonical hosts. - Portal wording:
portalbridge.commeta description "Powered by Wormhole", author "Wormhole Labs". "(built on Wormhole)" is accurate. Portal's Celo support I could not confirm (page is client-rendered); the PR says the same. - Leftovers:
git grep -i satelliteandgalaxyover the repo (includingdocs.json) return nothing. Remainingaxelarhits are Squid's description,learn/bridged-tokens/bridges.mdx:56andtooling/bridges/cross-chain-messaging.mdx(Axelar Network as a messaging protocol); none link to Satellite. - No page moved or deleted, so no redirect is owed (AGENTS section 2); no orphan produced. No
#satellite-axelaror#galaxyanchor is linked anywhere. - Remaining providers still answer: transporter.io, mach.exchange, smolrefuel.com, layerzero.network, app.allbridge.io all 200.
- Not reproducible, harmless: the body says both files keep CRLF endings; at the base commit
filereports plain UTF-8 fortooling/bridges/bridges.mdx. Probably a local autocrlf artefact; the diff is clean.
What the PR got right
- Removal of two dead entries with a rationale for each, and the explicit refusal to guess a replacement URL for Satellite.
- Correcting "formerly known as Wormhole", which was a real factual error.
- An honest residual-risk section (Allbridge, leftover links), which is what made findings 1 and 2 quick to confirm.
- Trimming "Axelar" from the description so
llms.txtstays accurate, and keeping the diff to the bridge pages.
Merge-order hazards
- Behind main by 1 commit (the Home to Learn rename, #2367).
git merge-tree --write-tree origin/main refs/remotes/pr/2374is clean: git follows the rename and applies the one-line deletion tolearn/bridged-tokens/bridges.mdx(confirmed with a real merge in a scratch checkout). Agit rebase origin/maindoes the same; the only manual step is that the finding-2 edits go inlearn/..., nothome/.... - #2368 (Build tab) has not landed:
tooling/bridges/bridges.mdxis still at that path on main. When it does, it renames this file tobuild/tools/bridges/bridges.mdx; the PR's edits sit inside a file that stays well over 50% similar, so rename detection should carry them, but whoever merges second should re-rungit merge-treefirst. - #2375 touches different files (
operate/specification/*); no conflict between the two PRs.
Checks run
mint broken-links --check-redirects: success on the merge with main (the Windows false positives described in the PR body do not reproduce). bash scripts/check-orphans.sh: no orphans. gh pr checks 2374: only GitGuardian and StepSecurity listed, both pass; the repo's mint broken-links workflow does not appear, most likely waiting for maintainer approval for an outside contributor.
340cf0d to
54a5651
Compare
|
Rebased onto main after #2367. The change that touched |
|
Previewed in a local live deployment (
Each page returns 200 and renders the new text, with no horizontal overflow on phone, no broken images and 0 console errors. |
…ected links Satellite was retired by Axelar and satellite.money now forwards to a GoDaddy for-sale page, so readers following the link land on a domain Axelar no longer controls. Remove it from the bridge providers page and the bridging overview. Remove Galaxy: galaxy.exchange returns HTTP 402 DEPLOYMENT_DISABLED and it is a DEX, not a bridge. Point Squid, Jumper and Hyperlane Nexus at the URLs their old links redirect to, and describe Portal as built on Wormhole rather than formerly known as Wormhole. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
… on Learn pages Review follow-up for celo-org#2374 (GigaHierz, findings 1 and 2). - Allbridge: remove the section from build/tools/bridges/bridges.mdx, the word from its description, and the bullet from learn/bridged-tokens/bridges.mdx. Re-checked 2026-10-06: docs.allbridge.io says "Allbridge Classic is deprecated and will be stopped in November 2026"; the link targeted Classic (app.allbridge.io); api.core.allbridge.io/token-info lists ETH,POL,ARB,AVA,OPT,BAS,UNI,BSC,TRX,SOL,SRB,SUI,STX with no Celo. - Squid and Jumper: change only the host on learn/bridged-tokens/bridges.mdx:19 and :21 and learn/gas-fees.mdx:27 (v2.app.squidrouter.com -> app.squidrouter.com, jumper.exchange -> jumper.xyz). Query strings unchanged; both final URLs return 200. Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
54a5651 to
abc13cc
Compare
|
Head
Re-checked today (2026-10-06, 17:48-17:52 UTC) before removing Allbridge:
Checks on
Push was a rebase: #2368 merged at 17:44 UTC, so the branch is now on |
GigaHierz
left a comment
There was a problem hiding this comment.
head abc13cc · local ref and API headRefOid match; PR merge-base is origin/main 7b8f97c (no merge needed), git merge-tree --write-tree clean; mint broken-links --check-redirects success, check-orphans.sh clean; both bridge pages served from the PR on :3611 (no Satellite/Galaxy/Allbridge text, new Squid/Jumper hosts in the rendered hrefs)
Verdict: REQUEST-CHANGES
Thank you for the quick turnaround, and for rebasing onto the new learn/ and build/ layout. Both substantive findings from the last round are fixed and I could reproduce your evidence for each. One small item is left: the PR description now describes the code as it was before the follow-up commit.
Status of earlier findings
- Allbridge kept although Classic stops in November 2026 and Core has no Celo: FIXED.
build/tools/bridges/bridges.mdxno longer has the## AllBridgesection, the description no longer says "Allbridge" (line 3), and the bullet is gone fromlearn/bridged-tokens/bridges.mdx. Re-checked:docs.allbridge.iostill reads "Allbridge Classic is deprecated and will be stopped in November 2026", andhttps://api.core.allbridge.io/token-infostill returns ETH, POL, ARB, AVA, OPT, BAS, UNI, BSC, TRX, SOL, SRB, SUI, STX with no Celo. Removal is the right call. - Redirecting Squid and Jumper hosts on the Learn pages: FIXED.
learn/bridged-tokens/bridges.mdx:19and:21andlearn/gas-fees.mdx:27now useapp.squidrouter.comandjumper.xyz, host-only edits with the query strings untouched.curl -sILwith a browser user agent: the three new URLs (with their full query strings) return 200 with no further redirect. Old to new, for the record:v2.app.squidrouter.com/?chains=10%2C42220307 toapp.squidrouter.com/?chains=10%2C42220;jumper.exchange/?fromChain=10&toChain=42220301 tojumper.xyz/?fromChain=10&toChain=42220. The rendered Learn pages carry the new hrefs. - Galaxy "is not a bridge" wording: FIXED where it matters. The PR body now strikes the claim and says the reason is the disabled deployment. The original commit message (689865e) still says "it is a DEX, not a bridge", but the repository squash-merges with the PR title as the commit and a blank message body, so that sentence does not reach
main. Nothing to change.
New findings
- Low · PR description and title (not a file)
- Defect: the description was not updated after the follow-up commit, so several statements are now untrue of the head: "Allbridge is unchanged" and the first "Remaining ops steps" box (decide on Allbridge); "The same redirected Squid and Jumper URLs remain ... left for a follow-up" and the second box; file paths
tooling/bridges/bridges.mdx,home/bridged-tokens/bridges.mdx,home/gas-fees.mdx(they arebuild/tools/bridges/bridges.mdx,learn/bridged-tokens/bridges.mdx,learn/gas-fees.mdx); "Branched offmain(50ed71d)" and the #2368 / #2258 stacking notes (both landed); "Questions for the maintainer: should the Allbridge entry go now". The diff stat quoted in the body (2 files, 11 insertions, 36 deletions) is now 3 files, 14 insertions, 52 deletions. The title also omits the Allbridge removal, and the title becomes the squash commit onmain. - Failure scenario: a maintainer deciding on the merge reads that Allbridge is still listed and that a decision is pending, or a later reader of
git logsees a commit message that does not mention one of the three removals. - Fix shape: edit the body to match the head (paths, the Allbridge and Squid/Jumper sections moved from "does NOT do" to "fix", tick or delete the two ops boxes, drop the maintainer question) and retitle to something like
docs(bridges): remove retired Satellite, offline Galaxy and deprecated Allbridge, fix redirected links. Re-paste themint broken-linksresult for the head.
- Defect: the description was not updated after the follow-up commit, so several statements are now untrue of the head: "Allbridge is unchanged" and the first "Remaining ops steps" box (decide on Allbridge); "The same redirected Squid and Jumper URLs remain ... left for a follow-up" and the second box; file paths
Verified good
- Satellite:
https://docs.axelar.dev/resources/satellite/is titled "Satellite (Retired)" and says "Satellite has been retired. Use Axelar's current interface at app.axelar.network for supported transfers. Do not use satellite.money - that domain is no longer controlled by Axelar."curl -sIL https://satellite.money/landerreturns 307 toforsale.godaddy.com/forsale/satellite.money. Not substitutingapp.axelar.network(Celo support not confirmed) is correct. - Galaxy:
galaxy.exchange/and/swapboth return HTTP 402 withx-vercel-error: DEPLOYMENT_DISABLED. - Leftovers: a case-insensitive search of every
.mdx,.mdand.jsonfile (excludingnode_modulesandsubmodules) forsatellite,galaxy,allbridge,usenexus,v2.app.squidandjumper.exchangereturns nothing, includingdocs.json. - Learn and Build pages agree: both give Squid as
app.squidrouter.com, Jumper asjumper.xyz, Portal as Wormhole-based, and Transporter as Chainlink CCIP; neither lists Satellite, Galaxy or Allbridge. The lists differ in membership (Learn has LayerZero, Mach, USDT0; Build has Layerswap and Hyperlane Nexus), which was already the case and is not a contradiction. - Remaining outbound links all return 200:
nexus.hyperlane.xyz,portalbridge.com,layerzero.network,www.transporter.io,www.mach.exchange,smolrefuel.com/?outboundChain=42220,app.squidrouter.com,jumper.xyz. - Portal wording "(built on Wormhole)" fixes the earlier "formerly known as Wormhole" error.
- No page moved or deleted, so no redirect is owed (AGENTS section 2); no orphan; no
#satellite-axelar,#galaxyor#allbridgeanchor is linked anywhere. Rendered section ids on the Build page are superbridge, layerswap, squid-router, jumper-exchange, hyperlane-nexus, portal-wormhole, transporter-chainlink-ccip, smolrefuel. - CI:
gh pr checks 2374lists GitGuardian and the three StepSecurity checks, all passing. The repository'smint broken-linksworkflow does not appear; it is most likely waiting for maintainer approval of the first-time-contributor run. Locally it passes on the head.
What the PR got right
- Three dead or misleading entries removed with a stated rationale each, and no guessed replacement URL for Satellite.
- The "formerly known as Wormhole" correction was a real factual error.
- The follow-up did exactly the narrow edits asked for: host-only changes, query strings preserved, no wording churn.
- The earlier honest residual-risk list is what made the verification quick.
Merge-order hazards
|
Thanks for the second pass, Lena. Low · PR description and title: fixed. The title now names all three removals (
I re-checked the outbound links today: Squid, Jumper (with a browser user agent), Hyperlane Nexus and Portal return 200; Galaxy still returns 402 No file changes in this round, so the head is still |
GigaHierz
left a comment
There was a problem hiding this comment.
Re-reviewed the current head: the findings from the earlier review are resolved, the diff merges cleanly into current main, and the changed claims still check out.
The hole, and the fix
Three bridge routes on the docs sent readers somewhere unsafe, nowhere, or to a product that is shutting down, and several links only worked through redirects.
satellite.moneydomain now forwards toforsale.godaddy.com/forsale/satellite.money, a domain Axelar no longer controls. This PR removes the Satellite section frombuild/tools/bridges/bridges.mdxand the Satellite entry fromlearn/bridged-tokens/bridges.mdx, and drops "Axelar" from the bridge page'sdescriptionbecause the page no longer lists an Axelar app. Squid (which routes over Axelar) stays, and Axelar Network stays in the cross-chain messaging list on the overview page.galaxy.exchangeandgalaxy.exchange/swapboth returnHTTP 402withX-Vercel-Error: DEPLOYMENT_DISABLED. The section is removed because the deployment is disabled.app.allbridge.io), which Allbridge's docs say "is deprecated and will be stopped in November 2026". Allbridge Core does not list Celo (https://api.core.allbridge.io/token-inforeturnsETH,POL,ARB,AVA,OPT,BAS,UNI,BSC,TRX,SOL,SRB,SUI,STX), so there is no current Allbridge route to link. The## AllBridgesection, the "Allbridge" mention in the page description, and the bullet onlearn/bridged-tokens/bridges.mdxare gone.[Portal](https://portalbridge.com/) (built on Wormhole), matching the Transporter entry's pattern.build/tools/bridges/bridges.mdx:v2.app.squidrouter.com->app.squidrouter.com(307),jumper.exchange->jumper.xyz(301),www.usenexus.org->nexus.hyperlane.xyz(301, then 308)learn/bridged-tokens/bridges.mdx:19,:21andlearn/gas-fees.mdx:27: the same Squid and Jumper host changesWhat this does NOT do / residual risk
AGENTS.mddiscourages ("seamless"-style adjectives); a style pass on the page is out of scope.#satellite-axelar,#galaxyor#allbridgeanchors.Issues
Refs: none. No open issue in celo-org/docs covers these bridge links.
Stacking / conflicts
Based on current
main(7b8f97c2, after #2368 and thehome/tolearn/move), so the paths above are the post-restructure paths. Independent of the other open PRs: #2375 touches onlyoperate/specification/*andlearn/protocol/transactions/overview.mdx; no shared files.Verification evidence
Diff against
main: 3 files changed, 14 insertions, 52 deletions (build/tools/bridges/bridges.mdx,learn/bridged-tokens/bridges.mdx,learn/gas-fees.mdx). Files keep their CRLF line endings.Outbound links, re-checked 2026-10-08 with
curl -sLand a browser user agent:Leftovers: a case-insensitive search of every
.mdx,.mdand.jsonfile (excludingnode_modulesandsubmodules) forsatellite,galaxy,allbridge,usenexus,v2.app.squidandjumper.exchangereturns nothing, includingdocs.json.mint broken-links --check-redirectson this head (abc13cc4): success, no broken links found, per the round-two review run. The head is unchanged since that run; this update edits only the title and description.Sources:
Remaining ops steps
None.
Checklist
mainmint broken-links --check-redirectspasses on the current head (review run onabc13cc4)Closesissue (noCloses)🤖 Generated with Claude Code