search

celestiaorg / blobstream-contracts

Celestia -> EVM bridge
Apache License 2.0
132 stars 40 forks source link

Investigate BlobstreamX contracts refactoring #317

Open rach-id opened 5 months ago

rach-id commented 5 months ago

We should investigate if it is possible to simplify the BlobstreamX smart contracts architecture so that they're easy to deploy for new chains.

evan-forbes commented 4 months ago

@rach-id when you get the chance, do mind adding / updating the acceptance criteria here to be more specific? Do we know what else is needed here before making a decision? do we know what we need from any refactors to make this worth it? Will switching to sp1 help this?

rach-id commented 4 months ago

Before SP1, we had the following contracts:

  1. Circuit Verifier: verifies the proofs
  2. SuccinctGateway: which is a registry for circuit verifiers and used to access control the verifiers
  3. BlobstreamX contract

With SP1, we will have:

  1. SP1 universal verifier: which verifies SP1 proofs. So we no longer need to deploy a verifier per circuit
  2. BlobstreamX: where the logic in SuccinctGateway got moved to BlobstreamX

I don't think we can simplify the contracts anymore after moving to SP1. So I think we can close this.

Let me know if you agree so that I close.