Skip to content

[CLD-1916]: feat(solana): port over mcms pda loader#11

Merged
graham-chainlink merged 1 commit into
mainfrom
ggoh/cld-1916/mcms-pda
Apr 23, 2026
Merged

[CLD-1916]: feat(solana): port over mcms pda loader#11
graham-chainlink merged 1 commit into
mainfrom
ggoh/cld-1916/mcms-pda

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Adds Solana MCMS/Timelock PDA derivation helpers (ported from Chainlink) plus unit tests to validate that each helper derives the expected PDA from its seed prefix and that the derived PDAs are distinct.

Changes:

  • Introduce PDA generator functions for MCMS signer/config/root metadata/expiring root+opcount and Timelock config/signer.
  • Add tests validating PDA derivations match solana.FindProgramAddress for each seed prefix.
  • Add a distinctness test to ensure no collisions between the different PDA generators for the same seed.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 3 comments.

File Description
pkg/family/solana/mcms_pda.go Adds exported PDA helper functions and a shared internal getPDA wrapper.
pkg/family/solana/mcms_pda_test.go Adds unit tests covering correctness and uniqueness of derived PDAs.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread pkg/family/solana/mcms_pda.go Outdated
Comment thread pkg/family/solana/mcms_pda.go Outdated
Comment thread pkg/family/solana/mcms_pda.go Outdated
@graham-chainlink graham-chainlink marked this pull request as ready for review April 23, 2026 05:46
@graham-chainlink graham-chainlink requested a review from a team as a code owner April 23, 2026 05:46
Copilot AI review requested due to automatic review settings April 23, 2026 05:46
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread pkg/family/solana/mcms_pda.go
@graham-chainlink graham-chainlink enabled auto-merge (squash) April 23, 2026 09:33
@graham-chainlink graham-chainlink merged commit 7170ddc into main Apr 23, 2026
14 checks passed
@graham-chainlink graham-chainlink deleted the ggoh/cld-1916/mcms-pda branch April 23, 2026 16:12
jkongie added a commit that referenced this pull request May 6, 2026
🤖 I have created a release *beep* *boop*
---


##
[0.1.0](cld-changesets-v0.0.1...cld-changesets-v0.1.0)
(2026-05-06)


### Features

* add "catalog create chain metadata" changeset
([#36](#36))
([c12bb51](c12bb51))
* add catalog create contract metadata changeset
([#27](#27))
([8ec1cbc](8ec1cbc))
* add CRE workflow deploy changeset and operation
([#1](#1))
([960fde1](960fde1))
* add target name param to cre deploy changeset
([#3](#3))
([8b25662](8b25662))
* fund mcms pdas
([#22](#22))
([821e542](821e542))
* **jobspec:** port jobspec changesets from chainlink
([#21](#21))
([0ce3f74](0ce3f74))
* link token
([#30](#30))
([07345c1](07345c1))
* **pkg:** add contract constants and Solana MCMS state loading
([#5](#5))
([6a2bbee](6a2bbee))
* port BuildProposalFromBatchesV2
([#24](#24))
([28d53d7](28d53d7))
* port run changeset from chainlink
([#32](#32))
([693922f](693922f))
* port solana grant role
([#33](#33))
([7eca5be](7eca5be))
* port token approve
([#40](#40))
([04130de](04130de))
* **port:** firedrill mcms with operations api refactor
([#25](#25))
([6d9010c](6d9010c))
* **solana:** add SOL funding helpers for deployer transfers
([#19](#19))
([63bebf8](63bebf8))
* **solana:** port over mcms pda loader
([#11](#11))
([7170ddc](7170ddc))


### Bug Fixes

* **aptos:** move state load
([#8](#8))
([de58102](de58102))
* **evm:** port state load evm funcs
([#9](#9))
([99279f1](99279f1))
* **state:** restore addressbook usage
([#17](#17))
([017cd5e](017cd5e))

---
This PR was generated with [Release
Please](https://github.com/googleapis/release-please). See
[documentation](https://github.com/googleapis/release-please#release-please).

Co-authored-by: app-token-issuer-ops-platform[bot] <275822481+app-token-issuer-ops-platform[bot]@users.noreply.github.com>
Co-authored-by: James Kong <61834+jkongie@users.noreply.github.com>
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