Skip to content

Conversation

@vedhavyas
Copy link
Contributor

@vedhavyas vedhavyas commented Nov 6, 2025

Fix to increment bundle count for operators.

Previously

OperatorBundleCountInEpoch::<T>::mutate(operator_id, |c| c.saturating_add(1));

is not actually incrementing the count since the value c is not mutated but instead we returned the incremented out of the mutate and mutate never actually updates the state.

I have added a test to simulate this behavior.

Also bumped spec_version for chronos runtime upgrade

Code contributor checklist:

@immunefi-magnus
Copy link

🛡️ Immunefi PR Reviews

We noticed that your project isn't set up for automatic code reviews. If you'd like this PR reviewed by the Immunefi team, you can request it manually using the link below:

🔗 Send this PR in for review

Once submitted, we'll take care of assigning a reviewer and follow up here.

@vedhavyas vedhavyas requested a review from nazar-pc as a code owner November 6, 2025 05:09
Copy link
Member

@jfrank-summit jfrank-summit left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@vedhavyas vedhavyas added this pull request to the merge queue Nov 6, 2025
Merged via the queue into main with commit c3c8bf2 Nov 6, 2025
14 checks passed
@vedhavyas vedhavyas deleted the fix_bundle_count branch November 6, 2025 12:47
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.

3 participants