Skip to content

[#549] Make attestation metrics internally consistent - #562

Merged
1nonlypiece merged 1 commit into
Commitlabs-Org:masterfrom
Baskarayelu:feat/issue-549-metric-consistency
Aug 27, 2026
Merged

1nonlypiece merged 1 commit into
Commitlabs-Org:masterfrom
Baskarayelu:feat/issue-549-metric-consistency

Conversation

@Baskarayelu

Copy link
Copy Markdown
Contributor

Summary

  • enforce bounded fee and drawdown inputs across single and batch attestation paths
  • use checked arithmetic for fee aggregates and all attestation counters
  • preserve atomic rollback when a metric update is rejected
  • add deterministic consistency tests and contract metric documentation

Issue coverage

  • Counters and scores stay within documented bounds
  • Equivalent operation sequences produce equivalent totals
  • Rejected records leave metrics and history unchanged
  • Empty, single, mixed, repeated, maximum, and overflow histories are covered

Validation

  • cargo +1.88.0 test -p attestation_engine metric_consistency_tests
  • cargo +1.88.0 test -p attestation_engine --lib (56 passed)
  • git diff --cached --check

Closes #549

@1nonlypiece
1nonlypiece merged commit ee700c9 into Commitlabs-Org:master Aug 27, 2026
0 of 2 checks passed
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.

[Quality] Make health metrics updates internally consistent

2 participants