Skip to content

chore: CON-1682 Rename VetKdPayload to ChainKeyPayload#9420

Open
eichhorl wants to merge 7 commits intomasterfrom
eichhorl/rename-vetkd-payload-rs-only
Open

chore: CON-1682 Rename VetKdPayload to ChainKeyPayload#9420
eichhorl wants to merge 7 commits intomasterfrom
eichhorl/rename-vetkd-payload-rs-only

Conversation

@eichhorl
Copy link
Contributor

@eichhorl eichhorl commented Mar 17, 2026

Background

The online phases of the VetKd and tECDSA/tSchnorr protocols are very similar: for each request context, combine and include all validated shares into a block. For historical reasons, the tECDSA/tSchnorr implementation is part of the IDKG payload, whereas the VetKd implementation is part of the batch payload.

Using the batch payload (over the IDKG payload) also for tECDSA/tSchnorr comes with the following advantages:

  • Maximum payload size limitation out of the box
  • No more incompatibility issues, since there is no batch payload in summary blocks
  • Established location for user data/requests in the block
  • Less code duplication by re-using VetKD implementation

For these reasons, this PR begins the process of generalizing the VetKD batch payload, such that it can also hold tECDSA/tSchnorr agreements in the future.

Proposed Changes

In particular, this PR renames the following types:

  • VetKdPayload.* -> ChainKeyPayload.*
  • VetKdAgreement -> ChainKeyAgreement
  • VetKdErrorCode -> ChainKeyErrorCode
  • vetkd_payload_to_bytes -> chain_key_payload_to_bytes
  • bytes_to_vetkd_payload -> bytes_to_chain_key_payload

Additionally, the following metrics were renamed:

  • vetkd_payload_build_duration -> chain_key_payload_build_duration
  • vetkd_payload_metrics -> chain_key_payload_metrics
  • vetkd_payload_errors -> chain_key_payload_errors

The alerts and dashboards corresponding to these metrics will be updated in a follow up.

@eichhorl eichhorl added the CI_ALL_BAZEL_TARGETS Runs all bazel targets label Mar 17, 2026
@eichhorl eichhorl changed the title Draft: rename VetKdPayload to ChainKeyPayload chore: CON-1682 Rename VetKdPayload to ChainKeyPayload Mar 17, 2026
@github-actions github-actions bot added the chore label Mar 17, 2026
@eichhorl eichhorl marked this pull request as ready for review March 17, 2026 13:36
@eichhorl eichhorl requested review from a team as code owners March 17, 2026 13:36
@eichhorl eichhorl requested a review from fspreiss March 17, 2026 13:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants