SC-015: Implement verifier slashing engine - #342
Conversation
|
Hi maintainers, SC-015 is ready for review. The required Fuzz Tests, Invariant Tests, Lint, and Test workflows are currently awaiting maintainer approval because this PR comes from a fork. Please approve the workflows when convenient. Thank you. |
|
resolve conflicts @Adedoyinjr |
e75228e to
46ba69a
Compare
|
Hi @dDevAhmed, the merge conflicts have been resolved and the branch is now mergeable. The latest CI run installs dependencies and passes lint successfully. The remaining checks fail during compilation on an existing The separate Foundry workflows also encounter existing invalid/missing import paths in the repository test suite. These failures occur before the SC-015 tests execute and are unrelated to the changes in this PR. Could you please confirm whether you would prefer the baseline CI issues to be fixed separately, or whether I should include those unrelated corrections in this PR? |
Summary
Implements SC-015 by upgrading the verifier slashing engine with deterministic claim-aware penalties, Treasury settlement, duplicate prevention, reputation notifications, immutable audit records, and fuzz/invariant coverage.
Changes
slashVerifier(...)Staking.forceSlashSafeERC20transfersslash()andbatchSlash()behaviorValidation
npm run compilepasses32 passing21 passing7 passing4 passingGas Benchmarks
410,025 gas400,389 gasInvariant
Treasury balance increases by exactly the confiscated collateral amount.
Closes #295