Skip to content

feat(sdk-coin-canton): Support Broadridge standing delegation response parsing - #10030

Merged
ravibitgo merged 1 commit into
masterfrom
CHALO-1921
Oct 22, 2026
Merged

ravibitgo merged 1 commit into
masterfrom
CHALO-1921

Conversation

@ravibitgo

Copy link
Copy Markdown
Contributor

Summary

  • Parse Broadridge StandingDelegationImpl_v1 create responses from Canton prepared transactions.
  • Preserve settlement authorization data for non-transfer commands without requiring sender, receiver, or amount fields.
  • Populate Broadridge descriptor actors from submitterInfo.actAs for CreateCommand transactions.
  • Align BroadridgeProtocolDescriptor with builder fields used for settlement and allocation metadata.
  • Add regression coverage using CantonStandingDelegationPrepareResponse.

Behavior

parseRawCantonTransactionData now extracts:

  • Broadridge protocol descriptor
  • Standing delegation template identity
  • Client and executor parties
  • Metadata
  • Submitter actor set
  • Zero-value transfer envelope for the non-transfer authorization command

This keeps the parsed actor set consistent with the settlement authorization builder’s requested actAs parties.

Verification

  • TypeScript compilation passed.
  • ESLint passed for changed files.
  • Prettier check passed.
  • Direct parser and transaction serialization assertions passed against CantonStandingDelegationPrepareResponse.

Ticket: CHALO-1921

@linear-code

linear-code Bot commented Oct 22, 2026

Copy link
Copy Markdown
Contributor

CHALO-1921

@github-actions

github-actions Bot commented Oct 22, 2026 •

Copy link
Copy Markdown
Contributor

⚠️ Unit tests are failing on Node 26.x (Current release line, non-blocking). This is not an LTS version yet, so it does not block merge, but it signals an incompatibility to fix before Node 26.x becomes LTS.

View run

@ravibitgo
ravibitgo marked this pull request as ready for review October 22, 2026 11:00
@ravibitgo
ravibitgo requested a review from a team as a code owner October 22, 2026 11:00
@ravibitgo
ravibitgo merged commit 5075d96 into master Oct 22, 2026
26 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.

2 participants