Open
Conversation
There was a problem hiding this comment.
Pull Request Overview
This PR updates the dependency versions to support type 4 EIP-7702 transactions and aligns the code with the latest APIs.
- Updated dependency imports and initialization patterns (e.g. KZG setup, chain configuration).
- Replaced deprecated API calls with new methods (e.g. fromRLPTransaction is now replaced with createTxFromRLP).
- Adjusted type definitions and test cases to accommodate the upgraded libraries.
Reviewed Changes
Copilot reviewed 27 out of 30 changed files in this pull request and generated 1 comment.
Show a summary per file
| File | Description |
|---|---|
| src/utils.ts | Updated KZG setup and chain configuration using the new microEthKZG and getPresetChainConfig APIs. |
| src/types.ts | Revised type re-exports and adjusted Blob transaction types; changed import path for Blob4844Tx. |
| src/test/index.ts | Updated transaction signing test with explicit chainId and data format changes. |
| src/stream/txRaw.ts | Updated sender conversion using the new createAddressFromString API. |
| src/stream/tx.ts | Replaced deprecated deserialization with createTxFromRLP and adjusted sender conversion. |
| src/stream/executionPayload.ts | Updated deserialization of execution payloads with new API functions and header creation methods. |
| src/stream/blobTxRaw.ts | Updated sender conversion using createAddressFromString. |
| src/stream/blobTx.ts | Replaced Blob transaction creation with the new createBlob4844Tx and createBlob4844TxFromBytesArray APIs. |
| src/stream/beaconBlock.ts | Updated beacon block deserialization to match new ssz API patterns. |
Files not reviewed (3)
- fiber-proto: Language not supported
- package.json: Language not supported
- tsconfig.tsbuildinfo: Language not supported
Comments suppressed due to low confidence (2)
src/types.ts:71
- Updating BlobTransactionWithSender to use TypedTransaction instead of BlobEIP4844Transaction may affect downstream consumers; please confirm that all related functionality properly supports this change.
transaction: TypedTransaction;
src/stream/tx.ts:42
- Ensure that createTxFromRLP correctly handles all transaction types as a replacement for fromRLPTransaction, and that existing behaviors are maintained with the updated API.
transaction = createTxFromRLP(data.getRlpTransaction_asU8(), { common });
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This PR updates all the dependencies (h/t to ncu).
In particular, we upgrade to libraries that can handle type 4 EIP-7702 transactions in the tx and payload streams.