release: bump the IDL program version

[kskalski]

Problem

The release bumps interface/Cargo.toml without touching the IDL files, so the IDL job fails after every release until someone regenerates them.

Summary of Changes

• regenerate both IDL files from a cargo-release pre-release hook, skipping dry runs
• enable the shared workflow's pnpm input, since the hook needs pnpm

Testing

Ran cargo release minor --execute --no-publish --no-push --no-tag on a scratch branch, from the repo root and from interface/, and checked that each release commit carries both regenerated IDL files at the new version.

[joncinque]

Looks great to me! This is a great way to solve the problem, and I tested that it works. Just some nits to make it more robust.

cc @lorisleiva since we've discussed some sort of action to do this in the past. This seems like a very neat and easy solution

[joncinque]

nit: in case we end up changing the spacing, can we make this slightly more robust?

Suggested change

	{ file = "../idl.json", search = '("publicKey": "Stake11111111111111111111111111111111111111",\n "version": ")[^"]+', replace = "${1}{{version}}", exactly = 1 },
	{ file = "../idl.json", search = '("publicKey":\s*"Stake11111111111111111111111111111111111111",\s*"version":\s*")[^"]+', replace = "${1}{{version}}", exactly = 1 },

[joncinque]

nit: same here

Suggested change

	{ file = "../interface-idl.json", search = '("publicKey": "Stake11111111111111111111111111111111111111",\n "version": ")[^"]+', replace = "${1}{{version}}", exactly = 1 },
	{ file = "../interface-idl.json", search = '("publicKey":\s*"Stake11111111111111111111111111111111111111",\s*"version":\s*")[^"]+', replace = "${1}{{version}}", exactly = 1 },

[lorisleiva]

@joncinque If I understand correctly even though the Codama Macros already do use the Cargo.toml to get the IDL version, this is a post-release timing issue that keeps the IDL version to the previous one until it gets regenerated?

[joncinque]

@joncinque If I understand correctly even though the Codama Macros already do use the Cargo.toml to get the IDL version, this is a post-release timing issue that keeps the IDL version to the previous one until it gets regenerated?

No not quite, this bumps the IDL version to the new / correct version after the crate is released. Currently, every time we bump the interface crate version, we immediately get CI failures because the IDL version is out of sync. This will make it so we avoid that extra manual step every time.

Unless the better option is to perform a post-bump step to re-run codama to pick up the newest version and commit it?

[kskalski]

We could also run the full script as pre-release-hook, but I think the necessary plumbing is not worth it. At least as long as we are sure the full script only updates those release lines that the replacement catches.

[lorisleiva]

Unless the better option is to perform a post-bump step to re-run codama to pick up the newest version and commit it?

That would have been my go-to solution yeah but I'm not sure how feasible it is to add this post-bump step to the Rust pipeline.

[lorisleiva]

I think the necessary plumbing is not worth it

I don't disagree but I'm also not a fan of the fact that we end up with two places to check when it comes to IDL generation. But again I'm not sure how complex that plumbing is in Rust so happy with the current solution is the trade-off is not worth it.

[kskalski]

Seems like it will boil down to something like this

[package.metadata.release]
pre-release-hook = ["sh", "-c", '[ "$DRY_RUN" = true ] || make -C "$WORKSPACE_ROOT" generate-idl-interface']

and the cost will be that the release pipeline needs to install the necessary tools... I agree this is cleaner, so let me prepare this one.

[joncinque]

We can avoid PRs like #572 if we land this, sooo anytime!

[joncinque]

Tested and confirming this works, thanks!