Skip to content

Add IDL publishing step - #143

Open
fedgiac wants to merge 1 commit into
mainfrom
add-idl-release-step
Open

Add IDL publishing step#143
fedgiac wants to merge 1 commit into
mainfrom
add-idl-release-step

Conversation

@fedgiac

@fedgiac fedgiac commented Sep 4, 2026

Copy link
Copy Markdown
Contributor

Adds the command used to publish the current IDL.
It was actually originally suggested by Solscan itself when there was no IDL.

It would be nicer to pin a version in some package.json, but for now this is good enough!

How to test

This was tested in a call and seems to be working.

@fedgiac
fedgiac requested a review from a team as a code owner September 4, 2026 16:36

@kaze-cow kaze-cow 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.

In the future this command almost certainly could go in the actual deploy script, but its not much extra at all to run it separately either.

See minor comments about locking the version

Comment thread README.md

```sh
settlement_program_address="the IDL will be registered for the program at the address specified in this string"
npx @solana-program/program-metadata@latest write idl "$settlement_program_address" ./programs/settlement/idl/cow_settlement.json --keypair ./deployer-keypair.json

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.

minor for security:

Suggested change
npx @solana-program/program-metadata@latest write idl "$settlement_program_address" ./programs/settlement/idl/cow_settlement.json --keypair ./deployer-keypair.json
npx @solana-program/program-metadata@v0.9.3 write idl "$settlement_program_address" ./programs/settlement/idl/cow_settlement.json --keypair ./deployer-keypair.json

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