chore(deps): bump @ar.io/sdk to 4.0.0-solana.26 (exact, parity)#763
Conversation
Parity/currency bump alongside ar-io-observer (full-size Epoch decoder fix). The node uses @ar.io/sdk only for ArNS resolution (arns-names-cache, *-arns-resolver) — it does NOT decode the Epoch account, so it is NOT affected by the shrunk-Epoch-decoder bug that blinded the observer (solana.24's devnet-shrunk failure_counts=[u16;30]). Still worth aligning onto the same full-size client (@ar.io/solana-contracts@0.5.0-staging.15) the rest of the stack uses, and swapping the loose caret for an exact pin (caret drift is what stranded the observer on a stale build). yarn.lock refreshed with classic yarn 1.22.22 (stays lockfile v1, not berry). NOTE: .22→.26 jump touches ArNS-resolution behavior — let CI run the resolver suite before merge. (Local install hit an unrelated better-sqlite3 node-gyp error under node 24; lockfile resolution is unaffected.) Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Repository UI Review profile: CHILL Plan: Pro Run ID: ⛔ Files ignored due to path filters (1)
📒 Files selected for processing (1)
📝 WalkthroughWalkthroughUpdates the ChangesDependency Updates
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~2 minutes Possibly related PRs
Suggested reviewers
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ 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 |
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## develop #763 +/- ##
===========================================
+ Coverage 79.41% 79.43% +0.01%
===========================================
Files 125 125
Lines 46457 46457
Branches 3560 3560
===========================================
+ Hits 36896 36903 +7
+ Misses 9514 9507 -7
Partials 47 47 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
Parity/currency bump alongside ar-io-observer#93 (the observer's actual fix).
Context
The observer was blind on Solana staging because
@ar.io/sdk@solana.24bundled the pre-ADR-024devnet-shrunkcontracts client, whoseEpochdecoder usedfailure_counts: [u16; 30]instead of the deployed[u16; 3000]→ every following field shifted →prescribed_observers/namesdecoded as zeros.ar-io-node is NOT affected by that bug — it uses
@ar.io/sdkonly for ArNS resolution (arns-names-cache,composite/on-demand-arns-resolver); it never decodesEpoch/prescribed names.Why bump anyway
@ar.io/solana-contracts@0.5.0-staging.15) the rest of the stack now uses.^4.0.0-solana.22) with an exact pin — caret drift is exactly what stranded the observer on a stale build..22.Notes / caveats
.22 → .26touches ArNS-resolution behavior — please let CI run the resolver suite before merge.yarn.lockrefreshed with classic yarn 1.22.22 (stayslockfile v1, not berry).better-sqlite3node-gyp error under node 24 (native build/env), not a lockfile issue — resolution succeeded and@ar.io/sdk@4.0.0-solana.26is pinned. CI/Docker builds it in a proper toolchain.🤖 Generated with Claude Code