Skip to content

Modernize record actions into plugin instruction plans#336

Merged
lorisleiva merged 1 commit into
mainfrom
loris/modernize-record-actions
Jun 9, 2026
Merged

Modernize record actions into plugin instruction plans#336
lorisleiva merged 1 commit into
mainfrom
loris/modernize-record-actions

Conversation

@lorisleiva

Copy link
Copy Markdown
Member

This PR replaces record's hand-written actions.ts helpers with instruction-plan getters wired into splRecordProgram() under client.splRecord.instructions.*, matching the Kit-plugin convention used across the other Solana program clients. It adds a chunked getWriteInstructionPlan that splits a write of any length across multiple transactions, exposes the record payload via a new payload field on the RecordData account in the IDL, and restructures the tests into one focused file per instruction (to mirror other repos).

This PR replaces record's hand-written `actions.ts` helpers with instruction-plan getters wired into `splRecordProgram()` under `client.splRecord.instructions.*`, matching the Kit-plugin convention used across the other Solana program clients. It adds a chunked `getWriteInstructionPlan` that splits a write of any length across multiple transactions, exposes the record payload via a new `payload` field on the `RecordData` account in the IDL, and restructures the tests into one focused file per instruction (to mirror other repos).
@lorisleiva lorisleiva marked this pull request as ready for review June 8, 2026 15:33
@lorisleiva lorisleiva requested a review from joncinque June 8, 2026 15:33
@lorisleiva

Copy link
Copy Markdown
Member Author

@joncinque I know this is mainly a JS client change but I wanted your approval since it touches a little bit on the IDL and it refactors quite a lot of JS tests.

@joncinque joncinque 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.

This looks great!

Sorry, I had to read up a bit on Typescript fundamentals to understand the plugin definition and creation, but after that, all smooth sailing 😁

@lorisleiva lorisleiva merged commit 6184c76 into main Jun 9, 2026
22 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