Skip to content

Conversation

mariocao
Copy link
Member

Motivation

This PR introduces storage libraries following the ERC-7201 standard to improve code organization and maintainability.

Additionally, I also include dependency updates and TypeScript compatibility fixes.

Explanation of Changes

  • feat(prover): Abstract storage to library - Created Secp256k1ProverStorage.sol library following ERC-7201 standard for better storage management
  • chore(core): Abstract storage to library - Created SedaCoreStorage.sol library following the same pattern for the core contract
  • test: Added proxy upgrade authorization tests to verify that only owners can upgrade contracts
  • fix: Resolved TypeScript function overload issues by using getFunction() workaround for contract method calls
  • chore: Updated dependencies and bumped version to v1.1.0

Testing

Run the test suite to verify all changes:

bun run test

I increased the coverage a little bit 😊:

bun run test:coverage

The changes should maintain backwards compatibility and all existing functionality should work as expected.

@mariocao mariocao changed the title Chore/updates feat: add storage libraries Aug 14, 2025
@mariocao mariocao requested a review from a team August 14, 2025 17:16
Copy link

github-actions bot commented Sep 2, 2025

Test Coverage Report

File % Stmts % Branch % Funcs % Lines Uncovered Lines
core/ 100 100 100 100
SedaCoreV1.sol 100 100 100 100
core/abstract/ 100 100 100 100
RequestHandlerBase.sol 100 100 100 100
ResultHandlerBase.sol 100 100 100 100
fees/ 100 100 100 100
SedaFeeManager.sol 100 100 100 100
provers/ 100 100 100 100
Secp256k1ProverV1.sol 100 100 100 100
provers/abstract/ 100 100 100 100
ProverBase.sol 100 100 100 100
All files 100 100 100 100

@mariocao mariocao self-assigned this Sep 16, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants