Skip to content

x/supernode: persist postpone reason#100

Merged
mateeullahmalik merged 1 commit intoaudit-v1from
supernode-postponement-reason
Feb 6, 2026
Merged

x/supernode: persist postpone reason#100
mateeullahmalik merged 1 commit intoaudit-v1from
supernode-postponement-reason

Conversation

@j-rafique
Copy link
Contributor

No description provided.

@roomote-v0
Copy link

roomote-v0 bot commented Feb 4, 2026

Rooviewer Clock   See task on Roo Cloud

Review complete. No issues found.

The PR correctly adds a reason field to SuperNodeStateRecord to persist the postpone reason. The implementation is backward compatible, properly documented, and consistent with existing patterns.

Mention @roomote in a comment to request specific changes to this pull request or fix all unresolved issues.

@mateeullahmalik mateeullahmalik merged commit c06d3df into audit-v1 Feb 6, 2026
4 of 6 checks passed
@mateeullahmalik mateeullahmalik deleted the supernode-postponement-reason branch February 11, 2026 13:25
mateeullahmalik added a commit that referenced this pull request Mar 3, 2026
* audit: add module

* audit : Reafactor & cleanup

* audit: stable window math across param changes

* audit: enforce postponement + retention

* Add tests, reduce noise in CI test runner, Update CHNAGELOG

* audit: port evidence + staleness

* x/audit: fix query pagination and evidence IDs

* x/audit: avoid evidence ID reuse

* x/audit:  signature and finalization enforcement

* x/audit: lean metada

* x/audit: action finalization postpone+recover

* x/supernode: persist postpone reason (#100)

* audit: epoch-based reporting

* audit : Rename Msgs for clarity, & tweaks

* audit : fixes

* audit: add cascade failure evidence type

* fix audit tests

* audit evidence update

* Address copilot comments

* app/upgrades: prevent double audit init in v1.11.0

* ci: pin buf install to go.mod version

---------

Co-authored-by: J Bilal rafique <113895287+j-rafique@users.noreply.github.com>
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.

2 participants