Skip to content

Add group metadata persistence and session checkpoint recovery#121

Merged
kwsantiago merged 6 commits intomainfrom
group-metadata-and-signing-session
Jan 19, 2026
Merged

Add group metadata persistence and session checkpoint recovery#121
kwsantiago merged 6 commits intomainfrom
group-metadata-and-signing-session

Conversation

@wksantiago
Copy link
Contributor

@wksantiago wksantiago commented Jan 19, 2026

Summary

  • Add persistent storage for group metadata (threshold, participants, coordinator)
  • Add session checkpoint with group name for crash recovery
  • Fix storage layout overlap between metadata and checkpoint regions
  • Add nonce reuse protection in session resume

Test plan

  • Native tests pass (test_session, test_storage)
  • Manual test DKG flow with metadata persistence
  • Test session checkpoint and resume after crash

@coderabbitai
Copy link

coderabbitai bot commented Jan 19, 2026

Warning

Rate limit exceeded

@kwsantiago has exceeded the limit for the number of commits that can be reviewed per hour. Please wait 2 minutes and 24 seconds before requesting another review.

⌛ How to resolve this issue?

After the wait time has elapsed, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

We recommend that you space out your commits to avoid hitting the rate limit.

🚦 How do rate limits work?

CodeRabbit enforces hourly rate limits for each developer per organization.

Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout.

Please see our FAQ for further information.

📥 Commits

Reviewing files that changed from the base of the PR and between df09711 and 86e7e2b.

⛔ Files ignored due to path filters (1)
  • partitions.csv is excluded by !**/*.csv
📒 Files selected for processing (14)
  • main/frost_dkg.c
  • main/frost_signer.c
  • main/frost_signer.h
  • main/main.c
  • main/protocol.c
  • main/protocol.h
  • main/session.c
  • main/session.h
  • main/storage.c
  • main/storage.h
  • test/native/test_frost_signer_core.c
  • test/native/test_integration.c
  • test/native/test_session.c
  • test/native/test_storage.c

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@kwsantiago kwsantiago force-pushed the group-metadata-and-signing-session branch from b7e423d to 49cbbd1 Compare January 19, 2026 23:06
@kwsantiago
Copy link
Contributor

@coderabbitai review

@coderabbitai
Copy link

coderabbitai bot commented Jan 19, 2026

✅ Actions performed

Review triggered.

Note: CodeRabbit is an incremental review system and does not re-review already reviewed commits. This command is applicable only when automatic reviews are paused.

@kwsantiago kwsantiago merged commit f0bdf0e into main Jan 19, 2026
9 checks passed
@kwsantiago kwsantiago deleted the group-metadata-and-signing-session branch January 19, 2026 23:27
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.

Add signing session persistence for interruption recovery Add group metadata persistence for recovery

2 participants