latest changes from sdk-wip

[rnbguy]

Closes: #158

Syncs until Sovereign-Labs/sovereign-sdk-wip@fc1552b44acddc2674a783cd11f301a8b79cc4f3

Description

---

PR author checklist

• Linked to GitHub issue.
• Added tests.
• Updated code comments and documentation (e.g., docs/).
• Tagged one reviewer who will be the one responsible for shepherding this PR.

[rnbguy]

Currently stuck because our test setup uses async runtimes - which requires the MockRollup to be Sync. But Sovereign-Labs/sovereign-sdk-wip#694 introduced Credentials which uses Arc<dyn core::any::Any> which is not bound by Send + Sync.

[rnbguy]

I managed to compile the code, but the tests are failing. Possibly, the mock chains are not initialized with a proper genesis state.

[rnbguy]

Managed to get the tests passed. ProverStorage is now immutable via Sovereign-Labs/sovereign-sdk-wip#613. Needed SimpleStorageManager to persist changes between batches.

[rnbguy]

The only thing is remaining is the cw-check. It is failing with a nix error.

@soareschen, @ljoss17 any idea how I fix the nix error? You can locally reproduce the error by:

nix build .#sov-celestia-cw

[soareschen]

The only thing is remaining is the cw-check. It is failing with a nix error.

@soareschen, @ljoss17 any idea how I fix the nix error? You can locally reproduce the error by:

nix build .#sov-celestia-cw

The Cargo.lock file and the git dependency hash needs to be updated, which you can see from the error log. I have pushed the fix.

[rnbguy]

Thanks so much ! I am still onboarding to nix, so I was confused with the error.

So, the Cargo.lock file needs to stay in sync. I shouldn't update it to the latest crate versions. Gotcha !