Skip to content

fix(governance): return CC voters as CIP-129 cc_hot ids in proposal votes - #467

Merged
slowbackspace merged 5 commits into
masterfrom
fix/proposal-votes-cc-cip129
Sep 1, 2026
Merged

fix(governance): return CC voters as CIP-129 cc_hot ids in proposal votes#467
slowbackspace merged 5 commits into
masterfrom
fix/proposal-votes-cc-cip129

Conversation

@slowbackspace

Copy link
Copy Markdown
Contributor

Summary

For "voter_role": "constitutional_committee", the voter field of /governance/proposals/{tx_hash}/{cert_index}/votes carried the raw 28-byte hot credential hash in hex, with no way to tell a key hash from a script hash. DRep voters in the same response are already CIP-129 bech32 and SPO voters are pool bech32, so committee voters were the only ambiguous role.

This documents the voter field as a Bech32 identifier for every role and switches the committee example to a CIP-129 hot credential (cc_hot1...), whose header byte encodes the key/script distinction — matching the representation the /governance/committee endpoints already use.

Backend counterpart: blockfrost/blockfrost-backend-ryo#351 (the actual conversion; verified byte-for-byte against Koios voter_id for mainnet CC votes).

Note: this changes the value format of voter for constitutional committee entries only (raw hex → cc_hot1...).

Fixes #465

…otes

For "voter_role": "constitutional_committee" the voter field carried the
raw 28-byte hash in hex, with no way to tell a key hash from a script
hash. DRep voters in the same response are already CIP-129 bech32 and
SPO voters are pool bech32, so committee voters were the only role left
ambiguous.

Document the voter field as a Bech32 identifier for every role and
switch the committee example to a CIP-129 hot credential (cc_hot1...),
whose header byte encodes the key/script distinction — matching the
representation already used by the /governance/committee endpoints.

Fixes #465
@vercel

vercel Bot commented Sep 1, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated
openapi Ready Ready Preview Sep 1, 2026 3:39pm UTC

Request Review

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

@slowbackspace
slowbackspace merged commit 47d3054 into master Sep 1, 2026
6 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.

GET /governance/proposals/{tx_hash}/{cert_index}/votes returns ambiguous credentials for CC

2 participants