Skip to content

ICS Assessment findings - #862

Merged
madlabman merged 6 commits into
developfrom
fix/ics-assessment-integrity
Aug 5, 2026
Merged

ICS Assessment findings#862
madlabman merged 6 commits into
developfrom
fix/ics-assessment-integrity

Conversation

@skhomuti

Copy link
Copy Markdown
Contributor

Description

  • Fixed inclusive Hoodi frame counting: end_epoch - start_epoch + 1; operator 96 was added to the eligibility artifact.
  • Added chain ID validation for every RPC before running a sync job.
  • Fixed operator ID loss when a single address controls multiple operators.
  • Made owner snapshot synchronization fail closed: individual read failures are no longer suppressed.

Checklist

  • Appropriate PR labels applied
  • Test coverage maintained (just coverage)
    • No need to add/update tests
    • Tests are added/updated
  • Documentation maintained
    • No need to update
    • Updated

@skhomuti
skhomuti requested a review from a team as a code owner July 27, 2026 11:05
@skhomuti
skhomuti requested review from dgusakov and madlabman July 27, 2026 11:05
@skhomuti skhomuti added the chore label Jul 27, 2026
@skhomuti skhomuti changed the title Fix/ics assessment integrity ICS Assessment findings Jul 27, 2026
@madlabman
madlabman merged commit 09dc602 into develop Aug 5, 2026
6 checks passed
@madlabman
madlabman deleted the fix/ics-assessment-integrity branch August 5, 2026 08:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants