Skip to content

chore: bump eth.zig to v0.8.1#12

Merged
koko1123 merged 1 commit into
mainfrom
chore/bump-eth-v0.8.1
Jul 9, 2026
Merged

chore: bump eth.zig to v0.8.1#12
koko1123 merged 1 commit into
mainfrom
chore/bump-eth-v0.8.1

Conversation

@lukemacauley

@lukemacauley lukemacauley commented Jul 9, 2026

Copy link
Copy Markdown
Contributor

Bumps the eth dependency from v0.8.0 to v0.8.1 (StrobeLabs/eth.zig#104): KMS stale-connection retry, verified broadcast, and real signer-error propagation.

Why this SDK needs the bump

gator-liquidators depends on both this SDK and eth directly. Zig identifies packages by content hash, so if this SDK stays on eth v0.8.0 while gator moves to v0.8.1, gator compiles two different eth copies and the vendored C crypto (secp256k1 / blst / c-kzg) collides with duplicate-symbol link errors. Both must reference the identical eth version to dedupe.

Testing

  • zig build — clean
  • zig build test — 342/342 pass
  • No source changes; the WalletError.SigningFailed removed upstream is unused here.

Follow-up

After merge, tag v0.5.1 so gator can pin it; gator's bump PR references both v0.5.1 and eth v0.8.1.

🤖 Generated with Claude Code

Summary by CodeRabbit

  • Chores
    • Updated a bundled dependency to a newer revision for improved stability and compatibility.

Picks up the KMS stale-connection retry, verified broadcast, and real
signer errors from eth.zig v0.8.1 (StrobeLabs/eth.zig#104). Required so
downstream consumers (gator-liquidators) can bump their own eth to
v0.8.1 without pulling two divergent eth copies: Zig dedupes packages by
hash, so a consumer and this SDK must reference the identical eth version
or the vendored C crypto compiles twice and collides at link.

No source changes; the removed WalletError.SigningFailed is unused here.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
@coderabbitai

coderabbitai Bot commented Jul 9, 2026

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro Plus

Run ID: 13e4de49-4798-46e2-baa3-5f86a4e07f83

📥 Commits

Reviewing files that changed from the base of the PR and between eb82d3c and 3dcc46d.

📒 Files selected for processing (1)
  • build.zig.zon

📝 Walkthrough

Walkthrough

The pull request updates the eth dependency entry in build.zig.zon, changing the git ref/commit in the .url field and updating the corresponding .hash value to reflect a newer revision of eth.zig.

Changes

Dependency Update

Layer / File(s) Summary
Update eth.zig dependency pin
build.zig.zon
Updated the .url ref/commit and .hash for the eth dependency to point to a newer eth.zig revision.

Estimated code review effort: 1 (Trivial) | ~2 minutes

Possibly related PRs

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly states the dependency bump to eth.zig v0.8.1, which matches the only change in the PR.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch chore/bump-eth-v0.8.1

Comment @coderabbitai help to get the list of available commands.

@koko1123
koko1123 merged commit e8728fd into main Jul 9, 2026
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants