Skip to content

feat(sdk-coin-sol): V1TransferBuilder for server-side v1 transfers - #9942

Merged
bhavidhingra merged 1 commit into
masterfrom
bhavi/chalo-1603-sdk-v1-transfer-builder
Oct 12, 2026
Merged

bhavidhingra merged 1 commit into
masterfrom
bhavi/chalo-1603-sdk-v1-transfer-builder

Conversation

@bhavidhingra

Copy link
Copy Markdown
Contributor

Summary

Server-side v1 (SIMD-0296/0385) construction surface: V1TransferBuilder builds a native SOL transfer envelope from structured params (sender + recipients/amounts) — callers supply fields, never instructions. Inherits AdvanceNonceAccount injection, message compile, and message-signer-order signature assembly from V1TransactionBuilder.

Exposed via TransactionBuilderFactory.getV1TransferBuilder(). Part of the CHALO-1603 structured-v1-intents rework (Option B) — the WP will construct v1 transfers server-side, eliminating the client-supplied-envelope confused-deputy surface.

Tests

5 new unit tests (structured send surface, build+sign round-trip, durable-nonce 2-sig assembly, config-required, validation); existing v1 suite 13 passing.

Refs: CHALO-1603

@bhavidhingra
bhavidhingra requested a review from a team as a code owner October 12, 2026 10:11
@linear-code

linear-code Bot commented Oct 12, 2026

Copy link
Copy Markdown
Contributor

CHALO-1603

Structured-params surface (Option B): V1PaymentBuilder builds a v1
(SIMD-0296/0385) native SOL payment envelope from sender + recipients/amounts
— the caller supplies fields, never instructions. Named after the payment
intent it serves (WP's 'transfer' refers to tokens). Inherits AdvanceNonceAccount
injection, message compile, and message-signer-order signature assembly from
V1TransactionBuilder. Exposed via TransactionBuilderFactory.getV1PaymentBuilder().

Refs: CHALO-1603
@bhavidhingra
bhavidhingra force-pushed the bhavi/chalo-1603-sdk-v1-transfer-builder branch from 561fb50 to 25bb75c Compare October 12, 2026 11:54
@github-actions

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

@bhavidhingra
bhavidhingra merged commit 1e9ae6d into master Oct 12, 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.

3 participants