Skip to content

Conversation

@kronosapiens
Copy link
Contributor

@kronosapiens kronosapiens commented May 19, 2025

Issue / feature description

Add a connector for the Cartridge Controller

Changes

Add a new directory under src/controllers defining a starknetkit-compatible connector for Controller

Checklist

  • Rebased to the last commit of the target branch (or merged)
  • Code self-reviewed
  • Code self-tested

Limitations

Currently the Controller has support for transactions and message-signing (the primary gaming use-cases). Additional wallet functionality can be added in response to user requests.

The Controller supports only revision 1 domain separators when signing messages. Dapps will need to upgrade to use revision 1 in order to use Controller. See SNIP-12 for more information about domain separators.

@kronosapiens kronosapiens marked this pull request as draft May 19, 2025 17:58
@rsodre
Copy link

rsodre commented Jun 9, 2025

why so draft? 😆

@kronosapiens kronosapiens marked this pull request as ready for review June 12, 2025 22:27
@kronosapiens
Copy link
Contributor Author

Thanks @bluecco. It says that the workflow also needs approval?

@bluecco
Copy link
Contributor

bluecco commented Jul 9, 2025

hey @kronosapiens, should start automatically
just started ;)

@kronosapiens
Copy link
Contributor Author

Hi @bluecco, I rebased the PR onto develop. Can you re-approve the workflow? Something in the build failed, but it didn't seem related to my PR.

@kronosapiens
Copy link
Contributor Author

Hi @bluecco, it looks like the build issue is coming from here. Would welcome your thoughts when you have a minute.

@Cussone Cussone self-requested a review July 14, 2025 07:56
@Cussone
Copy link
Member

Cussone commented Jul 14, 2025

Hi @bluecco, it looks like the build issue is coming from here. Would welcome your thoughts when you have a minute.

@kronosapiens fixed the issue

@Cussone Cussone merged commit e628a16 into argentlabs:develop Jul 14, 2025
1 check passed
@kronosapiens
Copy link
Contributor Author

@Cussone thank you!

@rsodre
Copy link

rsodre commented Jul 15, 2025

@kronosapiens I tried to use the new connector, but for some reason it didn't connect, probably I didn't configure it properly.
The docs website has no mention of Controller yet, it would be good to have an example setup there.

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.

4 participants