Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Adding support for declare v0, l1 handler and l1->l2 messages #321

Merged
merged 35 commits into from
Oct 21, 2024
Merged
Changes from 1 commit
Commits
Show all changes
35 commits
Select commit Hold shift + click to select a range
789f457
feat: declare v0 working
Oct 3, 2024
d448b65
debug: debuging declare v0
Oct 4, 2024
b842ae6
main merged
Oct 4, 2024
3da2f1a
adding comments for debugging
Oct 7, 2024
2938bef
debugging with bootstraper
Oct 7, 2024
b8c554c
making l1 handler execute in the mempool
Oct 8, 2024
62e174f
feat: refactored Mempool to accept L1Handler txn
Oct 9, 2024
4c16c8e
fixing test for l1 message and mempool inner
Oct 10, 2024
7f31dde
merged with main
Oct 10, 2024
50dcc7a
formatting and linting
Oct 10, 2024
f9650f5
formatting and changelog
Oct 10, 2024
334c258
comments removed
Oct 10, 2024
7c98ebd
main merged
Oct 10, 2024
bc05e4f
Merge branch 'main' into feat/declare_v0
Trantorian1 Oct 11, 2024
7c63396
validation added for txns
Oct 11, 2024
1485eca
feat: main merged
Oct 15, 2024
c857b79
resolving issues
Oct 15, 2024
8e510c9
refactor: declare v0 refactored with different rpc namespace
Oct 16, 2024
0ca66af
main merged
Oct 16, 2024
501eaf8
merged with upstream
Oct 16, 2024
21d12f5
merged with main
Oct 16, 2024
2ad4d19
msgL2 hash added
Oct 16, 2024
fc16ff3
formatting and linting
Oct 16, 2024
2209885
tests added for proc macros, error added for empty namespace, removed…
Oct 18, 2024
ed5b6c0
fixing comments
Oct 18, 2024
eaf5411
main merged
Oct 18, 2024
f21baad
fix: build fix
Oct 18, 2024
77038a2
feat: added error when namespace is not present or method is not pres…
Oct 18, 2024
7669670
feat: de/serialize tests added for BroadcastedDeclareTransactionV0, c…
Oct 18, 2024
51054f8
fix: state diff special address should have current_block - 10 block …
Oct 19, 2024
b7a2a3e
feat: updating filter of l1 message to check between last and finaliz…
Oct 19, 2024
8ed287c
linting and formatting
Oct 19, 2024
638c645
refactor: reverting the event filter in l1 messaging
Oct 19, 2024
859e50c
Merge branch 'main' into feat/declare_v0
Mohiiit Oct 21, 2024
2fa3b46
extra clones removed
Oct 21, 2024
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
fix: build fix
mohiiit committed Oct 18, 2024
commit f21baadfa05c6be88846d3c967d6fc4c35021ffc
1 change: 1 addition & 0 deletions crates/client/mempool/src/inner.rs
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

there is a

#![proptest_config(ProptestConfig::with_cases(5))] // comment this when developing, this is mostly for faster ci & whole workspace `cargo test`

at the bottom of the file that you should try commenting to see if the proptest passes locally
(don't commit it though)

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is fine for now, but please add a comment saying that the handling of nonces on l1 handler txs is wrong

Also, when I get to fixing it, what tests should I run to see that I did not break l1 messages in the process?

Original file line number Diff line number Diff line change
@@ -159,6 +159,7 @@ impl NonceChain {
match self.transactions.entry(OrderMempoolTransactionByNonce(mempool_tx.clone())) {
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

same

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

removed

btree_map::Entry::Occupied(entry) => {
// duplicate nonce, either it's because the hash is duplicated or nonce conflict with another tx.
#[cfg(debug_assertions)]
if entry.key().0.tx_hash() == mempool_tx_hash {
return Err(TxInsersionError::DuplicateTxn);
} else {