fix: Bradbury V06 ABI transaction decoding#70
Conversation
Add Bradbury testnet with v0.6 ConsensusMain and ConsensusData ABIs. Includes smoke tests verifying RPC connectivity and contract calls against the live Bradbury deployment.
- GenLayerRawTransaction.from_transaction_data auto-detects V06 (23 fields) vs V04 (21 fields) by tuple length and maps positions correctly. Bradbury splits txData into txExecutionHash/txCalldata/ eqBlocksOutputs and shifts all subsequent fields. - Update testnet RPC URLs to GenLayer RPC nodes - Update smoke test URL assertions
|
Warning Rate limit exceeded
⌛ 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: defaults Review profile: CHILL Plan: Pro Run ID: ⛔ Files ignored due to path filters (1)
📒 Files selected for processing (5)
✨ Finishing Touches🧪 Generate unit tests (beta)
📝 Coding Plan
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 Tip Flake8 can be used to improve the quality of Python code reviews.Flake8 is a Python linter that wraps PyFlakes, pycodestyle and Ned Batchelder's McCabe script. To configure Flake8, add a '.flake8' or 'setup.cfg' file to your project root. See Flake8 Documentation for more details. |
Summary
GenLayerRawTransaction.from_transaction_datafor Bradbury V06 ABI which has 23 fields vs Asimov's 21txData/txReceiptintotxExecutionHash/txCalldata/eqBlocksOutputs, shifting all subsequent field indicesTest plan
uv run pytest tests/unit/— 36 tests pass