feat: emit events when the aggregator and verifier bitmap change - #2298
Open
0xrlawrence wants to merge 1 commit into
Open
feat: emit events when the aggregator and verifier bitmap change#22980xrlawrence wants to merge 1 commit into
0xrlawrence wants to merge 1 commit into
Conversation
disableVerifier and enableVerifier each emit an event, but the two other privileged setters on AlignedLayerServiceManager are silent: - setAggregator changes the only address allowed to call respondToTaskV2, so a change there is worth watching for off-chain. - setDisabledVerifiers can rewrite the whole bitmap at once, bypassing the per-index events entirely. Add AggregatorUpdated and DisabledVerifiersUpdated and emit them from both. AggregatorUpdated carries the previous address so indexers can follow the handover without keeping their own copy of the current value. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Description
disableVerifierandenableVerifiereach emit an event, but the two other privileged setters onAlignedLayerServiceManagerare silent:setAggregatorchanges the only address permitted to callrespondToTaskV2. A handover currently leaves no on-chain trace, so anything watching the AVS has to poll the storage slot to notice.setDisabledVerifierscan rewrite the entire bitmap in one call, which bypasses the per-indexVerifierDisabled/VerifierEnabledevents completely.Adds
AggregatorUpdatedandDisabledVerifiersUpdatedand emits them from both.AggregatorUpdatedcarries the previous address as well as the new one so indexers can follow the handover without keeping their own copy of the current value.No storage layout or behaviour change; events only.
Type of change
Checklist
testnet, everything else tostagingVerified with
forge buildonstaging.