Upgrade ts-sdk 0.4.20 - boltz-swap 0.3.21#569
Conversation
|
Warning Rate limit exceeded
To keep reviews running without waiting, you can enable usage-based add-on for your organization. This allows additional reviews beyond the hourly cap. Account admins can enable it under billing. ⌛ How to resolve this issue?After the wait time has elapsed, a review can be triggered using the We recommend that you space out your commits to avoid hitting the rate limit. 🚦 How do rate limits work?CodeRabbit enforces hourly rate limits for each developer per organization. Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout. Please see our FAQ for further information. ℹ️ Review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Run ID: ⛔ Files ignored due to path filters (1)
📒 Files selected for processing (1)
✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Deploying wallet-bitcoin with
|
| Latest commit: |
50603ca
|
| Status: | ✅ Deploy successful! |
| Preview URL: | https://2fedb01c.wallet-bitcoin.pages.dev |
| Branch Preview URL: | https://upgrade-ts-sdk-0-4-20-boltz.wallet-bitcoin.pages.dev |
Deploying wallet-mutinynet with
|
| Latest commit: |
50603ca
|
| Status: | ✅ Deploy successful! |
| Preview URL: | https://a3ced5cc.arkade-wallet.pages.dev |
| Branch Preview URL: | https://upgrade-ts-sdk-0-4-20-boltz.arkade-wallet.pages.dev |
Deploying tmp-boltz-upstream-mainnet-arkade-wallet with
|
| Latest commit: |
50603ca
|
| Status: | ✅ Deploy successful! |
| Preview URL: | https://649fda3c.tmp-boltz-upstream-mainnet-arkade-wallet.pages.dev |
| Branch Preview URL: | https://upgrade-ts-sdk-0-4-20-boltz.tmp-boltz-upstream-mainnet-arkade-wallet.pages.dev |
![arkanaai[bot]](https://avatars.githubusercontent.com/u/170820085?s=60&v=4){kind=small}
There was a problem hiding this comment.
Arkana Code Review — APPROVE ✅
PR: Upgrade @arkade-os/sdk 0.4.19 → 0.4.20, @arkade-os/boltz-swap 0.3.20 → 0.3.21
What changed upstream
ts-sdk 0.4.20 (6 commits since 0.4.19):
feat: NewElectrumOnchainProvider+WsElectrumChainSource— alternative to Esplora for on-chain queriesperf:getTxOutspendsoptimized from O(outputs×history×fetch) to O(4 round trips)feat: Atomic 1P1C package broadcast for TRUC transactions- New dependency:
ws-electrumx-client@1.0.5(→isomorphic-ws,ws)
boltz-swap 0.3.21: ts-sdk bump only, no functional changes.
Analysis
-
No breaking API changes. The ts-sdk diff is purely additive — new exports (
ElectrumOnchainProvider,WsElectrumChainSource, 3 new types). No existing types, interfaces, or methods were modified or removed. The wallet imports (IWallet,ArkAddress,RestIndexerProvider,ExtendedCoin, etc.) are all untouched. -
No wallet code changes. This PR only touches
package.jsonandpnpm-lock.yaml. The wallet does not yet consume the new Electrum provider — it continues usingEsploraProviderandRestIndexerProviderunchanged. -
New transitive dependencies.
ws-electrumx-client@1.0.5pulls inisomorphic-ws@5.0.0and addsbufferutil/utf-8-validateas optional peer deps. These are well-known WebSocket packages. No supply chain concern. -
Not protocol-critical. No changes to VTXO handling, transaction signing, forfeit paths, round lifecycle, or exit paths in this PR.
-
CI: Cloudflare Pages builds pass (3/3). One test job passed (43s), one still pending.
Clean dependency bump. Ship it.
🤖 Reviewed by Arkana
1 participant
No description provided.