chainwayxyz / citrea

Citrea, Bitcoin's First ZK Rollup 🍊🍋
https://citrea.xyz
GNU General Public License v3.0
95 stars 22 forks source link

Remove sequencer from context #1060

Open eyusufatik opened 2 weeks ago

eyusufatik commented 2 weeks ago

our sequencer is sender, not sequencer in context and we don't need it

rakanalh commented 1 week ago

Could you please provide some more details?

eyusufatik commented 1 week ago

https://github.com/chainwayxyz/citrea/blob/578594afd7b0edb66fa2735d31b7592db3ef4c9e/crates/sovereign-sdk/module-system/sov-modules-api/src/default_context.rs#L21

context is what's passed to hooks or module calls.

In sovereign-sdk it contained a field called sender and a field called sequencer.

In our rollup sender is always the sequencer. And we have a single sequencer. So it makes sense to just remove it.