Skip to content

Add tuple array ABI support#369

Merged
q9f merged 6 commits intomainfrom
codex/implement-abi-encoding-for-tuples
Aug 12, 2025
Merged

Add tuple array ABI support#369
q9f merged 6 commits intomainfrom
codex/implement-abi-encoding-for-tuples

Conversation

@q9f
Copy link
Owner

@q9f q9f commented Aug 6, 2025 •
edited
Loading

  • allow tuple ABI encoding with array args and handle nested tuples
  • support decoding arrays of tuples and nested dynamic components
  • parse canonical tuple syntax
  • coerce numeric strings for integer/fixed types and enable ethers.js vectors
  • guard decoder offsets for malformed tuple data
  • fix eth/abi: decode/encode array of tuples #102

@q9f q9f added the codex Codex is a durable, decentralised data storage protocol label Aug 6, 2025 — with ChatGPT Codex Connector
@bolom
Copy link

bolom commented Aug 7, 2025

THANKS for this PR, Tuple encoding is working perfectly !!!!

@q9f
Copy link
Owner Author

q9f commented Aug 12, 2025

Thank you for testing. I'm still working on this, it's not "perfect" yet but I see some early wins already.

@codecov
Copy link

codecov bot commented Aug 12, 2025 •
edited
Loading

Codecov Report

❌ Patch coverage is 82.35294% with 15 lines in your changes missing coverage. Please review.
✅ Project coverage is 99.49%. Comparing base (1fc93e0) to head (c38a949).
⚠️ Report is 1 commits behind head on main.

Files with missing lines Patch % Lines
lib/eth/abi/decoder.rb 68.00% 8 Missing ⚠️
lib/eth/abi/encoder.rb 71.42% 4 Missing ⚠️
spec/eth/abi_spec.rb 89.28% 3 Missing ⚠️
Additional details and impacted files 
@@            Coverage Diff             @@
##             main     #369      +/-   ##
==========================================
- Coverage   99.59%   99.49%   -0.10%     
==========================================
  Files          87       87              
  Lines        6149     6198      +49     
==========================================
+ Hits         6124     6167      +43     
- Misses         25       31       +6

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@q9f q9f merged commit 18897f7 into main Aug 12, 2025
7 of 8 checks passed
@q9f q9f deleted the codex/implement-abi-encoding-for-tuples branch August 12, 2025 08:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

codex Codex is a durable, decentralised data storage protocol

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

eth/abi: decode/encode array of tuples

3 participants

@q9f @bolom @l5yth