aligned_layer icon indicating copy to clipboard operation
aligned_layer copied to clipboard

chore: remove dev mode functionality on Proof Aggregation Service contract

Open maximopalopoli opened this issue 1 month ago • 1 comments

Description

This PR removes the mock functionality from the Aligned Proof Aggregation Service contract, since it had no real uses.

How to test

To test this PR, you should test that the normal workflows are working as expected. This means following these steps:

  1. Start ethereum package:
make ethereum_package_start
  1. Start batcher:
make batcher_start_ethereum_package
  1. Send SP1 and Risc0 proofs:
make batcher_send_sp1_burst BURST_SIZE=1
make batcher_send_risc0_burst BURST_SIZE=1
  1. Run proof aggregator for SP1 and RISC0:
make proof_aggregator_start AGGREGATOR=sp1
make proof_aggregator_start AGGREGATOR=risc0
  1. Verify the proofs have been aggregated:
make verify_aggregated_proof_sp1 FROM_BLOCK=0
make verify_aggregated_proof_risc0 FROM_BLOCK=0

Type of change

Please delete options that are not relevant.

  • [ ] New feature
  • [ ] Bug fix
  • [ ] Optimization
  • [ ] Refactor

Checklist

  • [ ] “Hotfix” to testnet, everything else to staging
  • [ ] Linked to Github Issue
  • [ ] This change depends on code or research by an external entity
    • [ ] Acknowledgements were updated to give credit
  • [ ] Unit tests added
  • [ ] This change requires new documentation.
    • [ ] Documentation has been added/updated.
  • [ ] This change is an Optimization
    • [ ] Benchmarks added/run
  • [ ] Has a known issue
    • Link to the open issue addressing it
  • [ ] If your PR changes the Operator compatibility (Ex: Upgrade prover versions)
    • [ ] This PR adds compatibility for operator for both versions and do not change crates/docs/examples
    • [ ] This PR updates batcher and docs/examples to the newer version. This requires the operator are already updated to be compatible

maximopalopoli avatar Nov 25 '25 18:11 maximopalopoli

Changes to gas cost

Generated at commit: d224e75fa6fd51931ee2eb31c5aad49f8a7bb1dc, compared to commit: cb3829aa300cd46e3f548bf18ec433cd1eaa7d4e

🧾 Summary (10% most significant diffs)

Contract Method Avg (+/-) %

Full diff report 👇
Contract Deployment Cost (+/-) Method Min (+/-) % Avg (+/-) % Median (+/-) % Max (+/-) % # Calls (+/-)
AlignedLayerServiceManager 4,398,072 (0) createNewTask
receive
56,874 (0)
23,301 (0)
0.00%
0.00%
76,567 (-78)
46,463 (-93)
-0.10%
-0.20%
77,038 (0)
47,115 (0)
0.00%
0.00%
77,872 (0)
47,115 (0)
0.00%
0.00%
256 (0)
256 (0)

github-actions[bot] avatar Nov 26 '25 21:11 github-actions[bot]