Skip to content

feat: port solana grant role#33

Merged
ajaskolski merged 2 commits into
mainfrom
feat-port-solana-grant-role
May 5, 2026
Merged

feat: port solana grant role#33
ajaskolski merged 2 commits into
mainfrom
feat-port-solana-grant-role

Conversation

@ajaskolski
Copy link
Copy Markdown
Contributor

@ajaskolski ajaskolski commented May 5, 2026

Ports grant role from chainlink repo.
https://smartcontract-it.atlassian.net/browse/CLD-1923

@ajaskolski ajaskolski marked this pull request as ready for review May 5, 2026 10:56
@ajaskolski ajaskolski requested a review from a team as a code owner May 5, 2026 10:56
Copilot AI review requested due to automatic review settings May 5, 2026 10:56
@ajaskolski ajaskolski force-pushed the feat-port-solana-grant-role branch from 85f09f9 to a952942 Compare May 5, 2026 11:01
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

Ports the Solana “grant role” functionality (from the Chainlink repo) into this repo’s MCMS changesets/operations layer, enabling granting Timelock roles to one or more Solana accounts either via direct deployer execution or via an MCMS proposal flow.

Changes:

  • Added a Solana operation + sequence to grant a Timelock role to one or multiple accounts (direct-send or MCMS-transaction output).
  • Added a Solana changeset that validates environment/state and builds MCMS batch operations / proposals for role granting.

Reviewed changes

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

File Description
mcms/operations/grant_role_timelock.go Introduces a Solana operation/sequence for granting Timelock roles and optionally producing MCMS transactions.
mcms/changesets/grant_role_timelock.go Adds a changeset wrapper that loads Solana MCMS/Timelock state, executes the sequence, and builds an MCMS proposal when configured.

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

Comment thread mcms/operations/grant_role_timelock.go Outdated
Comment thread mcms/operations/grant_role_timelock.go
Comment thread mcms/operations/grant_role_timelock.go
Comment thread mcms/changesets/grant_role_timelock.go Outdated
Comment thread mcms/changesets/grant_role_timelock.go
Comment thread mcms/changesets/grant_role_timelock.go
Comment thread mcms/changesets/grant_role_timelock.go Outdated
@ecPablo
Copy link
Copy Markdown
Contributor

ecPablo commented May 5, 2026

One suggestion before removing the changesets from core repo is to search on all the org for usages to see if there's any other usages outside of the core repo that we might need to swap import for first.

@ajaskolski
Copy link
Copy Markdown
Contributor Author

ajaskolski commented May 5, 2026

One suggestion before removing the changesets from core repo is to search on all the org for usages to see if there's any other usages outside of the core repo that we might need to swap import for first.

Thanks, roger that. Gladly it was mentioned in the ticket so that's what I planned there !

@ajaskolski ajaskolski merged commit 7eca5be into main May 5, 2026
13 checks passed
@ajaskolski ajaskolski deleted the feat-port-solana-grant-role branch May 5, 2026 12:31
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