Skip to content

refactor(clearnode): migrate Signature{r,s,v} to []byte#283

Merged
nksazonov merged 1 commit intofeat/mm-eip191-signaturesfrom
refactor/clearnode-migrate-sig-to-bytes
Jul 17, 2025
Merged

refactor(clearnode): migrate Signature{r,s,v} to []byte#283
nksazonov merged 1 commit intofeat/mm-eip191-signaturesfrom
refactor/clearnode-migrate-sig-to-bytes

Conversation

@nksazonov
Copy link
Contributor

@nksazonov nksazonov commented Jul 16, 2025

Copy link
Contributor

@gemini-code-assist gemini-code-assist bot left a comment

Choose a reason for hiding this comment

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

Summary of Changes

Hello @nksazonov, I'm Gemini Code Assist1! I'm currently reviewing this pull request and will post my feedback shortly. In the meantime, here's a summary to help you and other reviewers quickly get up to speed!

This pull request introduces a significant refactoring effort to standardize the representation of cryptographic signatures throughout the project. By migrating from a custom Signature struct to a raw bytes array, the codebase achieves greater consistency between the Go client and Solidity contracts. This change simplifies signature handling, reduces boilerplate code, and aligns the project with common Ethereum signature formats, ultimately improving maintainability and clarity.

Highlights

  • Go Client Simplification: The Go Signer and nitrolite.Sign functions have been refactored to directly return []byte signatures, eliminating the need for intermediate struct conversions. The NitroSign helper function has also been removed, consolidating signature generation logic.
  • Solidity Contract Alignment: All relevant Solidity contracts and interfaces, including Custody.sol, Utils.sol, IChannel.sol, Types.sol, and various adjudicators, have been updated to use bytes for signature parameters and storage. This ensures consistency with the Go implementation and streamlines signature verification on-chain.
  • Test Suite Adaptation: The comprehensive test suites for both Go and Solidity have been updated to reflect the new bytes signature type, ensuring that all signing and verification logic remains correct and robust after the refactoring.
Using Gemini Code Assist

The full guide for Gemini Code Assist can be found on our documentation page, here are some quick tips.

Invoking Gemini

You can request assistance from Gemini at any point in your pull request via creating an issue comment (i.e. comment on the pull request page) using either /gemini <command> or @gemini-code-assist <command>. Below is a summary of the supported commands.

Feature Command Description
Code Review /gemini review Performs a code review for the current pull request in its current state.
Pull Request Summary /gemini summary Provides a summary of the current pull request in its current state.
Comment @gemini-code-assist Responds in comments when explicitly tagged, both in issue comments and review comments.
Help /gemini help Displays a list of available commands.

Customization

To customize Gemini Code Assist for GitHub experience, repository maintainers can create a configuration file and/or provide a custom code review style guide (such as PEP-8 for Python) by creating and adding files to a .gemini/ folder in the base of the repository. Detailed instructions can be found here.

Limitations & Feedback

Gemini Code Assist is currently in preview and may make mistakes. Please leave feedback on any instances where its feedback is incorrect or counter productive. You can react with 👍 and 👎 on @gemini-code-assist comments to provide feedback.

You can also get AI-powered code generation, chat, as well as code reviews directly in the IDE at no cost with the Gemini Code Assist IDE Extension.

Footnotes

  1. Review the Privacy Notices, Generative AI Prohibited Use Policy, Terms of Service, and learn how to configure Gemini Code Assist in GitHub here. Gemini can make mistakes, so double check it and use code with caution.

Copy link
Contributor

@gemini-code-assist gemini-code-assist bot left a comment

Choose a reason for hiding this comment

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

Code Review

This pull request refactors the signature representation from structs to byte slices across the codebase, enhancing code clarity and maintainability by simplifying signature handling logic and removing redundant conversions.

@nksazonov nksazonov force-pushed the refactor/clearnode-migrate-sig-to-bytes branch from 6abf68b to 3b8b8b5 Compare July 17, 2025 07:23
@nksazonov nksazonov requested review from dimast-x and mnaichuk July 17, 2025 07:25
@nksazonov nksazonov changed the title refactor(clearnode): migrate sig to bytes refactor(clearnode): migrate Signature{r,s,v} to []byte Jul 17, 2025
@nksazonov nksazonov merged commit fed4418 into feat/mm-eip191-signatures Jul 17, 2025
@nksazonov nksazonov deleted the refactor/clearnode-migrate-sig-to-bytes branch July 17, 2025 12:29
nksazonov added a commit that referenced this pull request Jul 31, 2025
…1271,6492) (#294)

* feat: move erc7824 documentation to nitrolite (#272)

* feat: move erc7824 documentation to nitrolite

* feat: move GH actions to common scope

* fix: comment GITHUB_TOKEN

* feat(custody): add EIP-191, EIP-712 signature support (#257)

* feat(custody): add EIP-191, EIP-712 support

* feat(custody): correct EIP-712 sig impl

* feat(utils): add POLA principle comment

* feat(contract): add EIP712AdjudicatorBase

* feat(contract): integrate EIP712AdjudicatorBase to supported adjs

* fix(contract): tests with updated adjs

* test(contract): EIP712AdjudicatorBase

* test(contract): SimpleConsensus EIP191, EIP712 sigs

* test(contract): Utils signatures

* test(contract): Custody, SimpleConsensus integration tests for sigs

* refactor(contract): migrate sig to bytes (#279)

* refactor(Types): replace Signature{v,r,s} with bytes

* style(contract): add empty lines at the end

* refactor(clearnode): replace Signature{r,s,v} with []byte (#283)

* refactor(sdk): migrate `Signature{r,s,v}` to `Hex` (#284)

* refactor(sdk): replace Signature{r,s,v} with Hex

* fix(NitroliteService): perform convertStateFromContract correctly

* fix(rpc): expect updated signature type from rpc

* style(sdk/rpc): remove ServerSignatureSchema var

* feat(sdk/rpc): add empty signatures array by default

* feat(sdk): add build:force command

* feat(sdk): remove npm build:force

* refactor(clearnode): define `Signature []byte` type (#287)

* refactor(clearnode): define `Signature []byte` type

* refactor(clearnode::RPCEntry): replace sig `[]string` with `[]Signature`

* feat(clearnode/nitrolite): add Sigs2Strings, Strings2Sigs helpers

* fix(integration): tests sig migration (#289)

* feat(docker-compose): add ability to pass logger visibility to clearnode

* fix(clearnode): raw ECDSA sign and verify

* fix(clearnode): try to extract req id on ummarshal error

* fix(integration): update sig to new type

* feat(sdk): change type of sig array to contain only Hex

* refactor(clearnode): remove check on nil Req, zero-init instead

* fix(clearnode:nitrolite): remove unintended side-effect of sig param modif

* feat(custody): add note about ephemeral final state

* style(contract): run forge fmt

* fix(clearnode/docs): remove legacy signature format

* feat(custody): eip-1271, eip-6492 sigs support (#293)

* feat(contract): add ERC-1271,6492 support

* test(contract/Utils.sol): add ERC-1271, 6492 tests

* refactor(contract:Utils.t.sol): separate tests into contracts

* refactor(UtilsHarness): do NOT expose constants

* fix(contracts/adjudicators): use verifyStateSignature instead of just EOA sig

* test(contract/Custody): add ERC-1271, 6492 sig to integration test

* test(contract/Custody): add challenge with EIP-712, EIP-1271 tests

* refactor(contract/Utils): reorder recoverStateEIP712Signer params for consistency

* test(contract): remove console.logs

* feat(contract): clarify Utils comments

* refactor(contract): optimize Custody and Utils sig verification functions

* style(contract): run forge fmt

* feat: replace `stateHash` with `packedState` in message signing (#295)

* feat(contract): replace stateHash with packedState in signing

* test(contract): migrate to getPackedState

* feat(contract): remove getStateHash

* feat(clearnode): remove state_hash, rename EncodeState to Pack

* feat(sdk): remove state_hash, add getPackedState

* refactor(sdk): supply channelId and state to signState

* docs(website): migrate Sig to Hex, add supported signature formats

---------

Co-authored-by: MaxMoskalenko <mx.msklnk@gmail.com>
nksazonov added a commit that referenced this pull request Jul 31, 2025
…1271,6492) (#294)

* feat: move erc7824 documentation to nitrolite (#272)

* feat: move erc7824 documentation to nitrolite

* feat: move GH actions to common scope

* fix: comment GITHUB_TOKEN

* feat(custody): add EIP-191, EIP-712 signature support (#257)

* feat(custody): add EIP-191, EIP-712 support

* feat(custody): correct EIP-712 sig impl

* feat(utils): add POLA principle comment

* feat(contract): add EIP712AdjudicatorBase

* feat(contract): integrate EIP712AdjudicatorBase to supported adjs

* fix(contract): tests with updated adjs

* test(contract): EIP712AdjudicatorBase

* test(contract): SimpleConsensus EIP191, EIP712 sigs

* test(contract): Utils signatures

* test(contract): Custody, SimpleConsensus integration tests for sigs

* refactor(contract): migrate sig to bytes (#279)

* refactor(Types): replace Signature{v,r,s} with bytes

* style(contract): add empty lines at the end

* refactor(clearnode): replace Signature{r,s,v} with []byte (#283)

* refactor(sdk): migrate `Signature{r,s,v}` to `Hex` (#284)

* refactor(sdk): replace Signature{r,s,v} with Hex

* fix(NitroliteService): perform convertStateFromContract correctly

* fix(rpc): expect updated signature type from rpc

* style(sdk/rpc): remove ServerSignatureSchema var

* feat(sdk/rpc): add empty signatures array by default

* feat(sdk): add build:force command

* feat(sdk): remove npm build:force

* refactor(clearnode): define `Signature []byte` type (#287)

* refactor(clearnode): define `Signature []byte` type

* refactor(clearnode::RPCEntry): replace sig `[]string` with `[]Signature`

* feat(clearnode/nitrolite): add Sigs2Strings, Strings2Sigs helpers

* fix(integration): tests sig migration (#289)

* feat(docker-compose): add ability to pass logger visibility to clearnode

* fix(clearnode): raw ECDSA sign and verify

* fix(clearnode): try to extract req id on ummarshal error

* fix(integration): update sig to new type

* feat(sdk): change type of sig array to contain only Hex

* refactor(clearnode): remove check on nil Req, zero-init instead

* fix(clearnode:nitrolite): remove unintended side-effect of sig param modif

* feat(custody): add note about ephemeral final state

* style(contract): run forge fmt

* fix(clearnode/docs): remove legacy signature format

* feat(custody): eip-1271, eip-6492 sigs support (#293)

* feat(contract): add ERC-1271,6492 support

* test(contract/Utils.sol): add ERC-1271, 6492 tests

* refactor(contract:Utils.t.sol): separate tests into contracts

* refactor(UtilsHarness): do NOT expose constants

* fix(contracts/adjudicators): use verifyStateSignature instead of just EOA sig

* test(contract/Custody): add ERC-1271, 6492 sig to integration test

* test(contract/Custody): add challenge with EIP-712, EIP-1271 tests

* refactor(contract/Utils): reorder recoverStateEIP712Signer params for consistency

* test(contract): remove console.logs

* feat(contract): clarify Utils comments

* refactor(contract): optimize Custody and Utils sig verification functions

* style(contract): run forge fmt

* feat: replace `stateHash` with `packedState` in message signing (#295)

* feat(contract): replace stateHash with packedState in signing

* test(contract): migrate to getPackedState

* feat(contract): remove getStateHash

* feat(clearnode): remove state_hash, rename EncodeState to Pack

* feat(sdk): remove state_hash, add getPackedState

* refactor(sdk): supply channelId and state to signState

* docs(website): migrate Sig to Hex, add supported signature formats

---------

Co-authored-by: MaxMoskalenko <mx.msklnk@gmail.com>
nksazonov added a commit that referenced this pull request Jul 31, 2025
…1271,6492) (#294)

* feat: move erc7824 documentation to nitrolite (#272)

* feat: move erc7824 documentation to nitrolite

* feat: move GH actions to common scope

* fix: comment GITHUB_TOKEN

* feat(custody): add EIP-191, EIP-712 signature support (#257)

* feat(custody): add EIP-191, EIP-712 support

* feat(custody): correct EIP-712 sig impl

* feat(utils): add POLA principle comment

* feat(contract): add EIP712AdjudicatorBase

* feat(contract): integrate EIP712AdjudicatorBase to supported adjs

* fix(contract): tests with updated adjs

* test(contract): EIP712AdjudicatorBase

* test(contract): SimpleConsensus EIP191, EIP712 sigs

* test(contract): Utils signatures

* test(contract): Custody, SimpleConsensus integration tests for sigs

* refactor(contract): migrate sig to bytes (#279)

* refactor(Types): replace Signature{v,r,s} with bytes

* style(contract): add empty lines at the end

* refactor(clearnode): replace Signature{r,s,v} with []byte (#283)

* refactor(sdk): migrate `Signature{r,s,v}` to `Hex` (#284)

* refactor(sdk): replace Signature{r,s,v} with Hex

* fix(NitroliteService): perform convertStateFromContract correctly

* fix(rpc): expect updated signature type from rpc

* style(sdk/rpc): remove ServerSignatureSchema var

* feat(sdk/rpc): add empty signatures array by default

* feat(sdk): add build:force command

* feat(sdk): remove npm build:force

* refactor(clearnode): define `Signature []byte` type (#287)

* refactor(clearnode): define `Signature []byte` type

* refactor(clearnode::RPCEntry): replace sig `[]string` with `[]Signature`

* feat(clearnode/nitrolite): add Sigs2Strings, Strings2Sigs helpers

* fix(integration): tests sig migration (#289)

* feat(docker-compose): add ability to pass logger visibility to clearnode

* fix(clearnode): raw ECDSA sign and verify

* fix(clearnode): try to extract req id on ummarshal error

* fix(integration): update sig to new type

* feat(sdk): change type of sig array to contain only Hex

* refactor(clearnode): remove check on nil Req, zero-init instead

* fix(clearnode:nitrolite): remove unintended side-effect of sig param modif

* feat(custody): add note about ephemeral final state

* style(contract): run forge fmt

* fix(clearnode/docs): remove legacy signature format

* feat(custody): eip-1271, eip-6492 sigs support (#293)

* feat(contract): add ERC-1271,6492 support

* test(contract/Utils.sol): add ERC-1271, 6492 tests

* refactor(contract:Utils.t.sol): separate tests into contracts

* refactor(UtilsHarness): do NOT expose constants

* fix(contracts/adjudicators): use verifyStateSignature instead of just EOA sig

* test(contract/Custody): add ERC-1271, 6492 sig to integration test

* test(contract/Custody): add challenge with EIP-712, EIP-1271 tests

* refactor(contract/Utils): reorder recoverStateEIP712Signer params for consistency

* test(contract): remove console.logs

* feat(contract): clarify Utils comments

* refactor(contract): optimize Custody and Utils sig verification functions

* style(contract): run forge fmt

* feat: replace `stateHash` with `packedState` in message signing (#295)

* feat(contract): replace stateHash with packedState in signing

* test(contract): migrate to getPackedState

* feat(contract): remove getStateHash

* feat(clearnode): remove state_hash, rename EncodeState to Pack

* feat(sdk): remove state_hash, add getPackedState

* refactor(sdk): supply channelId and state to signState

* docs(website): migrate Sig to Hex, add supported signature formats

---------

Co-authored-by: MaxMoskalenko <mx.msklnk@gmail.com>
nksazonov added a commit that referenced this pull request Jul 31, 2025
…1271,6492) (#294)

* feat: move erc7824 documentation to nitrolite (#272)

* feat: move erc7824 documentation to nitrolite

* feat: move GH actions to common scope

* fix: comment GITHUB_TOKEN

* feat(custody): add EIP-191, EIP-712 signature support (#257)

* feat(custody): add EIP-191, EIP-712 support

* feat(custody): correct EIP-712 sig impl

* feat(utils): add POLA principle comment

* feat(contract): add EIP712AdjudicatorBase

* feat(contract): integrate EIP712AdjudicatorBase to supported adjs

* fix(contract): tests with updated adjs

* test(contract): EIP712AdjudicatorBase

* test(contract): SimpleConsensus EIP191, EIP712 sigs

* test(contract): Utils signatures

* test(contract): Custody, SimpleConsensus integration tests for sigs

* refactor(contract): migrate sig to bytes (#279)

* refactor(Types): replace Signature{v,r,s} with bytes

* style(contract): add empty lines at the end

* refactor(clearnode): replace Signature{r,s,v} with []byte (#283)

* refactor(sdk): migrate `Signature{r,s,v}` to `Hex` (#284)

* refactor(sdk): replace Signature{r,s,v} with Hex

* fix(NitroliteService): perform convertStateFromContract correctly

* fix(rpc): expect updated signature type from rpc

* style(sdk/rpc): remove ServerSignatureSchema var

* feat(sdk/rpc): add empty signatures array by default

* feat(sdk): add build:force command

* feat(sdk): remove npm build:force

* refactor(clearnode): define `Signature []byte` type (#287)

* refactor(clearnode): define `Signature []byte` type

* refactor(clearnode::RPCEntry): replace sig `[]string` with `[]Signature`

* feat(clearnode/nitrolite): add Sigs2Strings, Strings2Sigs helpers

* fix(integration): tests sig migration (#289)

* feat(docker-compose): add ability to pass logger visibility to clearnode

* fix(clearnode): raw ECDSA sign and verify

* fix(clearnode): try to extract req id on ummarshal error

* fix(integration): update sig to new type

* feat(sdk): change type of sig array to contain only Hex

* refactor(clearnode): remove check on nil Req, zero-init instead

* fix(clearnode:nitrolite): remove unintended side-effect of sig param modif

* feat(custody): add note about ephemeral final state

* style(contract): run forge fmt

* fix(clearnode/docs): remove legacy signature format

* feat(custody): eip-1271, eip-6492 sigs support (#293)

* feat(contract): add ERC-1271,6492 support

* test(contract/Utils.sol): add ERC-1271, 6492 tests

* refactor(contract:Utils.t.sol): separate tests into contracts

* refactor(UtilsHarness): do NOT expose constants

* fix(contracts/adjudicators): use verifyStateSignature instead of just EOA sig

* test(contract/Custody): add ERC-1271, 6492 sig to integration test

* test(contract/Custody): add challenge with EIP-712, EIP-1271 tests

* refactor(contract/Utils): reorder recoverStateEIP712Signer params for consistency

* test(contract): remove console.logs

* feat(contract): clarify Utils comments

* refactor(contract): optimize Custody and Utils sig verification functions

* style(contract): run forge fmt

* feat: replace `stateHash` with `packedState` in message signing (#295)

* feat(contract): replace stateHash with packedState in signing

* test(contract): migrate to getPackedState

* feat(contract): remove getStateHash

* feat(clearnode): remove state_hash, rename EncodeState to Pack

* feat(sdk): remove state_hash, add getPackedState

* refactor(sdk): supply channelId and state to signState

* docs(website): migrate Sig to Hex, add supported signature formats

---------

Co-authored-by: MaxMoskalenko <mx.msklnk@gmail.com>
dimast-x pushed a commit that referenced this pull request Jul 31, 2025
feat(custody): add EIP-191, EIP-712 signature support (#257)

* feat(custody): add EIP-191, EIP-712 support

* feat(custody): correct EIP-712 sig impl

* feat(utils): add POLA principle comment

* feat(contract): add EIP712AdjudicatorBase

* feat(contract): integrate EIP712AdjudicatorBase to supported adjs

* fix(contract): tests with updated adjs

* test(contract): EIP712AdjudicatorBase

* test(contract): SimpleConsensus EIP191, EIP712 sigs

* test(contract): Utils signatures

* test(contract): Custody, SimpleConsensus integration tests for sigs

refactor(contract): migrate sig to bytes (#279)

* refactor(Types): replace Signature{v,r,s} with bytes

* style(contract): add empty lines at the end

refactor(clearnode): replace Signature{r,s,v} with []byte (#283)

refactor(sdk): migrate `Signature{r,s,v}` to `Hex` (#284)

* refactor(sdk): replace Signature{r,s,v} with Hex

* fix(NitroliteService): perform convertStateFromContract correctly

* fix(rpc): expect updated signature type from rpc

* style(sdk/rpc): remove ServerSignatureSchema var

* feat(sdk/rpc): add empty signatures array by default

* feat(sdk): add build:force command

* feat(sdk): remove npm build:force

refactor(clearnode): define `Signature []byte` type (#287)

* refactor(clearnode): define `Signature []byte` type

* refactor(clearnode::RPCEntry): replace sig `[]string` with `[]Signature`

* feat(clearnode/nitrolite): add Sigs2Strings, Strings2Sigs helpers

fix(integration): tests sig migration (#289)

* feat(docker-compose): add ability to pass logger visibility to clearnode

* fix(clearnode): raw ECDSA sign and verify

* fix(clearnode): try to extract req id on ummarshal error

* fix(integration): update sig to new type

* feat(sdk): change type of sig array to contain only Hex

* refactor(clearnode): remove check on nil Req, zero-init instead

* fix(clearnode:nitrolite): remove unintended side-effect of sig param modif

feat(custody): add note about ephemeral final state

style(contract): run forge fmt

fix(clearnode/docs): remove legacy signature format

feat(custody): eip-1271, eip-6492 sigs support (#293)

* feat(contract): add ERC-1271,6492 support

* test(contract/Utils.sol): add ERC-1271, 6492 tests

* refactor(contract:Utils.t.sol): separate tests into contracts

* refactor(UtilsHarness): do NOT expose constants

* fix(contracts/adjudicators): use verifyStateSignature instead of just EOA sig

* test(contract/Custody): add ERC-1271, 6492 sig to integration test

* test(contract/Custody): add challenge with EIP-712, EIP-1271 tests

* refactor(contract/Utils): reorder recoverStateEIP712Signer params for consistency

* test(contract): remove console.logs

* feat(contract): clarify Utils comments

* refactor(contract): optimize Custody and Utils sig verification functions

* style(contract): run forge fmt

feat(custody): add ability to create channel w/ server sig

feat(contract/Custody): add ability to broker deposit in create
dimast-x pushed a commit that referenced this pull request Jul 31, 2025
feat(custody): add EIP-191, EIP-712 signature support (#257)

* feat(custody): add EIP-191, EIP-712 support

* feat(custody): correct EIP-712 sig impl

* feat(utils): add POLA principle comment

* feat(contract): add EIP712AdjudicatorBase

* feat(contract): integrate EIP712AdjudicatorBase to supported adjs

* fix(contract): tests with updated adjs

* test(contract): EIP712AdjudicatorBase

* test(contract): SimpleConsensus EIP191, EIP712 sigs

* test(contract): Utils signatures

* test(contract): Custody, SimpleConsensus integration tests for sigs

refactor(contract): migrate sig to bytes (#279)

* refactor(Types): replace Signature{v,r,s} with bytes

* style(contract): add empty lines at the end

refactor(clearnode): replace Signature{r,s,v} with []byte (#283)

refactor(sdk): migrate `Signature{r,s,v}` to `Hex` (#284)

* refactor(sdk): replace Signature{r,s,v} with Hex

* fix(NitroliteService): perform convertStateFromContract correctly

* fix(rpc): expect updated signature type from rpc

* style(sdk/rpc): remove ServerSignatureSchema var

* feat(sdk/rpc): add empty signatures array by default

* feat(sdk): add build:force command

* feat(sdk): remove npm build:force

refactor(clearnode): define `Signature []byte` type (#287)

* refactor(clearnode): define `Signature []byte` type

* refactor(clearnode::RPCEntry): replace sig `[]string` with `[]Signature`

* feat(clearnode/nitrolite): add Sigs2Strings, Strings2Sigs helpers

fix(integration): tests sig migration (#289)

* feat(docker-compose): add ability to pass logger visibility to clearnode

* fix(clearnode): raw ECDSA sign and verify

* fix(clearnode): try to extract req id on ummarshal error

* fix(integration): update sig to new type

* feat(sdk): change type of sig array to contain only Hex

* refactor(clearnode): remove check on nil Req, zero-init instead

* fix(clearnode:nitrolite): remove unintended side-effect of sig param modif

feat(custody): add note about ephemeral final state

style(contract): run forge fmt

fix(clearnode/docs): remove legacy signature format

feat(custody): eip-1271, eip-6492 sigs support (#293)

* feat(contract): add ERC-1271,6492 support

* test(contract/Utils.sol): add ERC-1271, 6492 tests

* refactor(contract:Utils.t.sol): separate tests into contracts

* refactor(UtilsHarness): do NOT expose constants

* fix(contracts/adjudicators): use verifyStateSignature instead of just EOA sig

* test(contract/Custody): add ERC-1271, 6492 sig to integration test

* test(contract/Custody): add challenge with EIP-712, EIP-1271 tests

* refactor(contract/Utils): reorder recoverStateEIP712Signer params for consistency

* test(contract): remove console.logs

* feat(contract): clarify Utils comments

* refactor(contract): optimize Custody and Utils sig verification functions

* style(contract): run forge fmt

feat(custody): add ability to create channel w/ server sig

feat(contract/Custody): add ability to broker deposit in create
nksazonov added a commit that referenced this pull request Aug 6, 2025
feat(custody): add EIP-191, EIP-712 signature support (#257)

* feat(custody): add EIP-191, EIP-712 support

* feat(custody): correct EIP-712 sig impl

* feat(utils): add POLA principle comment

* feat(contract): add EIP712AdjudicatorBase

* feat(contract): integrate EIP712AdjudicatorBase to supported adjs

* fix(contract): tests with updated adjs

* test(contract): EIP712AdjudicatorBase

* test(contract): SimpleConsensus EIP191, EIP712 sigs

* test(contract): Utils signatures

* test(contract): Custody, SimpleConsensus integration tests for sigs

refactor(contract): migrate sig to bytes (#279)

* refactor(Types): replace Signature{v,r,s} with bytes

* style(contract): add empty lines at the end

refactor(clearnode): replace Signature{r,s,v} with []byte (#283)

refactor(sdk): migrate `Signature{r,s,v}` to `Hex` (#284)

* refactor(sdk): replace Signature{r,s,v} with Hex

* fix(NitroliteService): perform convertStateFromContract correctly

* fix(rpc): expect updated signature type from rpc

* style(sdk/rpc): remove ServerSignatureSchema var

* feat(sdk/rpc): add empty signatures array by default

* feat(sdk): add build:force command

* feat(sdk): remove npm build:force

refactor(clearnode): define `Signature []byte` type (#287)

* refactor(clearnode): define `Signature []byte` type

* refactor(clearnode::RPCEntry): replace sig `[]string` with `[]Signature`

* feat(clearnode/nitrolite): add Sigs2Strings, Strings2Sigs helpers

fix(integration): tests sig migration (#289)

* feat(docker-compose): add ability to pass logger visibility to clearnode

* fix(clearnode): raw ECDSA sign and verify

* fix(clearnode): try to extract req id on ummarshal error

* fix(integration): update sig to new type

* feat(sdk): change type of sig array to contain only Hex

* refactor(clearnode): remove check on nil Req, zero-init instead

* fix(clearnode:nitrolite): remove unintended side-effect of sig param modif

feat(custody): add note about ephemeral final state

style(contract): run forge fmt

fix(clearnode/docs): remove legacy signature format

feat(custody): eip-1271, eip-6492 sigs support (#293)

* feat(contract): add ERC-1271,6492 support

* test(contract/Utils.sol): add ERC-1271, 6492 tests

* refactor(contract:Utils.t.sol): separate tests into contracts

* refactor(UtilsHarness): do NOT expose constants

* fix(contracts/adjudicators): use verifyStateSignature instead of just EOA sig

* test(contract/Custody): add ERC-1271, 6492 sig to integration test

* test(contract/Custody): add challenge with EIP-712, EIP-1271 tests

* refactor(contract/Utils): reorder recoverStateEIP712Signer params for consistency

* test(contract): remove console.logs

* feat(contract): clarify Utils comments

* refactor(contract): optimize Custody and Utils sig verification functions

* style(contract): run forge fmt

feat(custody): add ability to create channel w/ server sig

feat(contract/Custody): add ability to broker deposit in create
nksazonov added a commit that referenced this pull request Aug 6, 2025
…ints (#290)

* feat: custody updates

feat(custody): add EIP-191, EIP-712 signature support (#257)

* feat(custody): add EIP-191, EIP-712 support

* feat(custody): correct EIP-712 sig impl

* feat(utils): add POLA principle comment

* feat(contract): add EIP712AdjudicatorBase

* feat(contract): integrate EIP712AdjudicatorBase to supported adjs

* fix(contract): tests with updated adjs

* test(contract): EIP712AdjudicatorBase

* test(contract): SimpleConsensus EIP191, EIP712 sigs

* test(contract): Utils signatures

* test(contract): Custody, SimpleConsensus integration tests for sigs

refactor(contract): migrate sig to bytes (#279)

* refactor(Types): replace Signature{v,r,s} with bytes

* style(contract): add empty lines at the end

refactor(clearnode): replace Signature{r,s,v} with []byte (#283)

refactor(sdk): migrate `Signature{r,s,v}` to `Hex` (#284)

* refactor(sdk): replace Signature{r,s,v} with Hex

* fix(NitroliteService): perform convertStateFromContract correctly

* fix(rpc): expect updated signature type from rpc

* style(sdk/rpc): remove ServerSignatureSchema var

* feat(sdk/rpc): add empty signatures array by default

* feat(sdk): add build:force command

* feat(sdk): remove npm build:force

refactor(clearnode): define `Signature []byte` type (#287)

* refactor(clearnode): define `Signature []byte` type

* refactor(clearnode::RPCEntry): replace sig `[]string` with `[]Signature`

* feat(clearnode/nitrolite): add Sigs2Strings, Strings2Sigs helpers

fix(integration): tests sig migration (#289)

* feat(docker-compose): add ability to pass logger visibility to clearnode

* fix(clearnode): raw ECDSA sign and verify

* fix(clearnode): try to extract req id on ummarshal error

* fix(integration): update sig to new type

* feat(sdk): change type of sig array to contain only Hex

* refactor(clearnode): remove check on nil Req, zero-init instead

* fix(clearnode:nitrolite): remove unintended side-effect of sig param modif

feat(custody): add note about ephemeral final state

style(contract): run forge fmt

fix(clearnode/docs): remove legacy signature format

feat(custody): eip-1271, eip-6492 sigs support (#293)

* feat(contract): add ERC-1271,6492 support

* test(contract/Utils.sol): add ERC-1271, 6492 tests

* refactor(contract:Utils.t.sol): separate tests into contracts

* refactor(UtilsHarness): do NOT expose constants

* fix(contracts/adjudicators): use verifyStateSignature instead of just EOA sig

* test(contract/Custody): add ERC-1271, 6492 sig to integration test

* test(contract/Custody): add challenge with EIP-712, EIP-1271 tests

* refactor(contract/Utils): reorder recoverStateEIP712Signer params for consistency

* test(contract): remove console.logs

* feat(contract): clarify Utils comments

* refactor(contract): optimize Custody and Utils sig verification functions

* style(contract): run forge fmt

feat(custody): add ability to create channel w/ server sig

feat(contract/Custody): add ability to broker deposit in create

* feat: clearnode updates

feat: improve and simplify channel joining by broker

drop join by broker

feat: add create channel endpoint draft

feat: use proper state format

feat: code improvements and fix tests

add doc

feat: use state as encode state arg

address comments

feat: add tests for request create method

update docs

feat: refactor channel service tests

rename encode state into encode state data

feat: update response

feat: unify channel operation responses

feat: alling app session service structure to channel service

update docs

improve errors consistency

* feat(rpc): add create_channel method and unify channel response types (#296)

* feat: remove state hash from channel operation response

* update erc-7824 docs

* feat: include channel in channel operation response

* fix: use chain id as key in network config

* fix(Custody::join): allow 2 signatures

* feat: add an optional session key for create_channel request

* return state data as an encoded string

* fix(Custody::join): add Server wallet

* test(custody::join): broker auto-join tests (#303)

* test(Custody::join): broker auto-join

* refactor(contract/test): signState usage

* refactor(Custody.t.sol): enhance broker auto join test name readability

* feat: add sdk and integration support for simplified join (#304)

---------

Co-authored-by: Sazonov Nikita <35502225+nksazonov@users.noreply.github.com>
Co-authored-by: Max Pushkarov <mpushkarov@yellow.org>
Co-authored-by: nksazonov <nsazonov@openware.com>
Co-authored-by: MaxMoskalenko <mx.msklnk@gmail.com>
nksazonov added a commit that referenced this pull request Aug 6, 2025
…ints (#290)

* feat: custody updates

feat(custody): add EIP-191, EIP-712 signature support (#257)

* feat(custody): add EIP-191, EIP-712 support

* feat(custody): correct EIP-712 sig impl

* feat(utils): add POLA principle comment

* feat(contract): add EIP712AdjudicatorBase

* feat(contract): integrate EIP712AdjudicatorBase to supported adjs

* fix(contract): tests with updated adjs

* test(contract): EIP712AdjudicatorBase

* test(contract): SimpleConsensus EIP191, EIP712 sigs

* test(contract): Utils signatures

* test(contract): Custody, SimpleConsensus integration tests for sigs

refactor(contract): migrate sig to bytes (#279)

* refactor(Types): replace Signature{v,r,s} with bytes

* style(contract): add empty lines at the end

refactor(clearnode): replace Signature{r,s,v} with []byte (#283)

refactor(sdk): migrate `Signature{r,s,v}` to `Hex` (#284)

* refactor(sdk): replace Signature{r,s,v} with Hex

* fix(NitroliteService): perform convertStateFromContract correctly

* fix(rpc): expect updated signature type from rpc

* style(sdk/rpc): remove ServerSignatureSchema var

* feat(sdk/rpc): add empty signatures array by default

* feat(sdk): add build:force command

* feat(sdk): remove npm build:force

refactor(clearnode): define `Signature []byte` type (#287)

* refactor(clearnode): define `Signature []byte` type

* refactor(clearnode::RPCEntry): replace sig `[]string` with `[]Signature`

* feat(clearnode/nitrolite): add Sigs2Strings, Strings2Sigs helpers

fix(integration): tests sig migration (#289)

* feat(docker-compose): add ability to pass logger visibility to clearnode

* fix(clearnode): raw ECDSA sign and verify

* fix(clearnode): try to extract req id on ummarshal error

* fix(integration): update sig to new type

* feat(sdk): change type of sig array to contain only Hex

* refactor(clearnode): remove check on nil Req, zero-init instead

* fix(clearnode:nitrolite): remove unintended side-effect of sig param modif

feat(custody): add note about ephemeral final state

style(contract): run forge fmt

fix(clearnode/docs): remove legacy signature format

feat(custody): eip-1271, eip-6492 sigs support (#293)

* feat(contract): add ERC-1271,6492 support

* test(contract/Utils.sol): add ERC-1271, 6492 tests

* refactor(contract:Utils.t.sol): separate tests into contracts

* refactor(UtilsHarness): do NOT expose constants

* fix(contracts/adjudicators): use verifyStateSignature instead of just EOA sig

* test(contract/Custody): add ERC-1271, 6492 sig to integration test

* test(contract/Custody): add challenge with EIP-712, EIP-1271 tests

* refactor(contract/Utils): reorder recoverStateEIP712Signer params for consistency

* test(contract): remove console.logs

* feat(contract): clarify Utils comments

* refactor(contract): optimize Custody and Utils sig verification functions

* style(contract): run forge fmt

feat(custody): add ability to create channel w/ server sig

feat(contract/Custody): add ability to broker deposit in create

* feat: clearnode updates

feat: improve and simplify channel joining by broker

drop join by broker

feat: add create channel endpoint draft

feat: use proper state format

feat: code improvements and fix tests

add doc

feat: use state as encode state arg

address comments

feat: add tests for request create method

update docs

feat: refactor channel service tests

rename encode state into encode state data

feat: update response

feat: unify channel operation responses

feat: alling app session service structure to channel service

update docs

improve errors consistency

* feat(rpc): add create_channel method and unify channel response types (#296)

* feat: remove state hash from channel operation response

* update erc-7824 docs

* feat: include channel in channel operation response

* fix: use chain id as key in network config

* fix(Custody::join): allow 2 signatures

* feat: add an optional session key for create_channel request

* return state data as an encoded string

* fix(Custody::join): add Server wallet

* test(custody::join): broker auto-join tests (#303)

* test(Custody::join): broker auto-join

* refactor(contract/test): signState usage

* refactor(Custody.t.sol): enhance broker auto join test name readability

* feat: add sdk and integration support for simplified join (#304)

---------

Co-authored-by: Sazonov Nikita <35502225+nksazonov@users.noreply.github.com>
Co-authored-by: Max Pushkarov <mpushkarov@yellow.org>
Co-authored-by: nksazonov <nsazonov@openware.com>
Co-authored-by: MaxMoskalenko <mx.msklnk@gmail.com>
dimast-x pushed a commit that referenced this pull request Aug 13, 2025
…1271,6492) (#294)

* feat: move erc7824 documentation to nitrolite (#272)

* feat: move erc7824 documentation to nitrolite

* feat: move GH actions to common scope

* fix: comment GITHUB_TOKEN

* feat(custody): add EIP-191, EIP-712 signature support (#257)

* feat(custody): add EIP-191, EIP-712 support

* feat(custody): correct EIP-712 sig impl

* feat(utils): add POLA principle comment

* feat(contract): add EIP712AdjudicatorBase

* feat(contract): integrate EIP712AdjudicatorBase to supported adjs

* fix(contract): tests with updated adjs

* test(contract): EIP712AdjudicatorBase

* test(contract): SimpleConsensus EIP191, EIP712 sigs

* test(contract): Utils signatures

* test(contract): Custody, SimpleConsensus integration tests for sigs

* refactor(contract): migrate sig to bytes (#279)

* refactor(Types): replace Signature{v,r,s} with bytes

* style(contract): add empty lines at the end

* refactor(clearnode): replace Signature{r,s,v} with []byte (#283)

* refactor(sdk): migrate `Signature{r,s,v}` to `Hex` (#284)

* refactor(sdk): replace Signature{r,s,v} with Hex

* fix(NitroliteService): perform convertStateFromContract correctly

* fix(rpc): expect updated signature type from rpc

* style(sdk/rpc): remove ServerSignatureSchema var

* feat(sdk/rpc): add empty signatures array by default

* feat(sdk): add build:force command

* feat(sdk): remove npm build:force

* refactor(clearnode): define `Signature []byte` type (#287)

* refactor(clearnode): define `Signature []byte` type

* refactor(clearnode::RPCEntry): replace sig `[]string` with `[]Signature`

* feat(clearnode/nitrolite): add Sigs2Strings, Strings2Sigs helpers

* fix(integration): tests sig migration (#289)

* feat(docker-compose): add ability to pass logger visibility to clearnode

* fix(clearnode): raw ECDSA sign and verify

* fix(clearnode): try to extract req id on ummarshal error

* fix(integration): update sig to new type

* feat(sdk): change type of sig array to contain only Hex

* refactor(clearnode): remove check on nil Req, zero-init instead

* fix(clearnode:nitrolite): remove unintended side-effect of sig param modif

* feat(custody): add note about ephemeral final state

* style(contract): run forge fmt

* fix(clearnode/docs): remove legacy signature format

* feat(custody): eip-1271, eip-6492 sigs support (#293)

* feat(contract): add ERC-1271,6492 support

* test(contract/Utils.sol): add ERC-1271, 6492 tests

* refactor(contract:Utils.t.sol): separate tests into contracts

* refactor(UtilsHarness): do NOT expose constants

* fix(contracts/adjudicators): use verifyStateSignature instead of just EOA sig

* test(contract/Custody): add ERC-1271, 6492 sig to integration test

* test(contract/Custody): add challenge with EIP-712, EIP-1271 tests

* refactor(contract/Utils): reorder recoverStateEIP712Signer params for consistency

* test(contract): remove console.logs

* feat(contract): clarify Utils comments

* refactor(contract): optimize Custody and Utils sig verification functions

* style(contract): run forge fmt

* feat: replace `stateHash` with `packedState` in message signing (#295)

* feat(contract): replace stateHash with packedState in signing

* test(contract): migrate to getPackedState

* feat(contract): remove getStateHash

* feat(clearnode): remove state_hash, rename EncodeState to Pack

* feat(sdk): remove state_hash, add getPackedState

* refactor(sdk): supply channelId and state to signState

* docs(website): migrate Sig to Hex, add supported signature formats

---------

Co-authored-by: MaxMoskalenko <mx.msklnk@gmail.com>
dimast-x added a commit that referenced this pull request Aug 13, 2025
…ints (#290)

* feat: custody updates

feat(custody): add EIP-191, EIP-712 signature support (#257)

* feat(custody): add EIP-191, EIP-712 support

* feat(custody): correct EIP-712 sig impl

* feat(utils): add POLA principle comment

* feat(contract): add EIP712AdjudicatorBase

* feat(contract): integrate EIP712AdjudicatorBase to supported adjs

* fix(contract): tests with updated adjs

* test(contract): EIP712AdjudicatorBase

* test(contract): SimpleConsensus EIP191, EIP712 sigs

* test(contract): Utils signatures

* test(contract): Custody, SimpleConsensus integration tests for sigs

refactor(contract): migrate sig to bytes (#279)

* refactor(Types): replace Signature{v,r,s} with bytes

* style(contract): add empty lines at the end

refactor(clearnode): replace Signature{r,s,v} with []byte (#283)

refactor(sdk): migrate `Signature{r,s,v}` to `Hex` (#284)

* refactor(sdk): replace Signature{r,s,v} with Hex

* fix(NitroliteService): perform convertStateFromContract correctly

* fix(rpc): expect updated signature type from rpc

* style(sdk/rpc): remove ServerSignatureSchema var

* feat(sdk/rpc): add empty signatures array by default

* feat(sdk): add build:force command

* feat(sdk): remove npm build:force

refactor(clearnode): define `Signature []byte` type (#287)

* refactor(clearnode): define `Signature []byte` type

* refactor(clearnode::RPCEntry): replace sig `[]string` with `[]Signature`

* feat(clearnode/nitrolite): add Sigs2Strings, Strings2Sigs helpers

fix(integration): tests sig migration (#289)

* feat(docker-compose): add ability to pass logger visibility to clearnode

* fix(clearnode): raw ECDSA sign and verify

* fix(clearnode): try to extract req id on ummarshal error

* fix(integration): update sig to new type

* feat(sdk): change type of sig array to contain only Hex

* refactor(clearnode): remove check on nil Req, zero-init instead

* fix(clearnode:nitrolite): remove unintended side-effect of sig param modif

feat(custody): add note about ephemeral final state

style(contract): run forge fmt

fix(clearnode/docs): remove legacy signature format

feat(custody): eip-1271, eip-6492 sigs support (#293)

* feat(contract): add ERC-1271,6492 support

* test(contract/Utils.sol): add ERC-1271, 6492 tests

* refactor(contract:Utils.t.sol): separate tests into contracts

* refactor(UtilsHarness): do NOT expose constants

* fix(contracts/adjudicators): use verifyStateSignature instead of just EOA sig

* test(contract/Custody): add ERC-1271, 6492 sig to integration test

* test(contract/Custody): add challenge with EIP-712, EIP-1271 tests

* refactor(contract/Utils): reorder recoverStateEIP712Signer params for consistency

* test(contract): remove console.logs

* feat(contract): clarify Utils comments

* refactor(contract): optimize Custody and Utils sig verification functions

* style(contract): run forge fmt

feat(custody): add ability to create channel w/ server sig

feat(contract/Custody): add ability to broker deposit in create

* feat: clearnode updates

feat: improve and simplify channel joining by broker

drop join by broker

feat: add create channel endpoint draft

feat: use proper state format

feat: code improvements and fix tests

add doc

feat: use state as encode state arg

address comments

feat: add tests for request create method

update docs

feat: refactor channel service tests

rename encode state into encode state data

feat: update response

feat: unify channel operation responses

feat: alling app session service structure to channel service

update docs

improve errors consistency

* feat(rpc): add create_channel method and unify channel response types (#296)

* feat: remove state hash from channel operation response

* update erc-7824 docs

* feat: include channel in channel operation response

* fix: use chain id as key in network config

* fix(Custody::join): allow 2 signatures

* feat: add an optional session key for create_channel request

* return state data as an encoded string

* fix(Custody::join): add Server wallet

* test(custody::join): broker auto-join tests (#303)

* test(Custody::join): broker auto-join

* refactor(contract/test): signState usage

* refactor(Custody.t.sol): enhance broker auto join test name readability

* feat: add sdk and integration support for simplified join (#304)

---------

Co-authored-by: Sazonov Nikita <35502225+nksazonov@users.noreply.github.com>
Co-authored-by: Max Pushkarov <mpushkarov@yellow.org>
Co-authored-by: nksazonov <nsazonov@openware.com>
Co-authored-by: MaxMoskalenko <mx.msklnk@gmail.com>
nksazonov added a commit that referenced this pull request Aug 21, 2025
* feat: add system support for EIP-191, contracts support for EIP-(712,1271,6492) (#294)

* feat: move erc7824 documentation to nitrolite (#272)

* feat: move erc7824 documentation to nitrolite

* feat: move GH actions to common scope

* fix: comment GITHUB_TOKEN

* feat(custody): add EIP-191, EIP-712 signature support (#257)

* feat(custody): add EIP-191, EIP-712 support

* feat(custody): correct EIP-712 sig impl

* feat(utils): add POLA principle comment

* feat(contract): add EIP712AdjudicatorBase

* feat(contract): integrate EIP712AdjudicatorBase to supported adjs

* fix(contract): tests with updated adjs

* test(contract): EIP712AdjudicatorBase

* test(contract): SimpleConsensus EIP191, EIP712 sigs

* test(contract): Utils signatures

* test(contract): Custody, SimpleConsensus integration tests for sigs

* refactor(contract): migrate sig to bytes (#279)

* refactor(Types): replace Signature{v,r,s} with bytes

* style(contract): add empty lines at the end

* refactor(clearnode): replace Signature{r,s,v} with []byte (#283)

* refactor(sdk): migrate `Signature{r,s,v}` to `Hex` (#284)

* refactor(sdk): replace Signature{r,s,v} with Hex

* fix(NitroliteService): perform convertStateFromContract correctly

* fix(rpc): expect updated signature type from rpc

* style(sdk/rpc): remove ServerSignatureSchema var

* feat(sdk/rpc): add empty signatures array by default

* feat(sdk): add build:force command

* feat(sdk): remove npm build:force

* refactor(clearnode): define `Signature []byte` type (#287)

* refactor(clearnode): define `Signature []byte` type

* refactor(clearnode::RPCEntry): replace sig `[]string` with `[]Signature`

* feat(clearnode/nitrolite): add Sigs2Strings, Strings2Sigs helpers

* fix(integration): tests sig migration (#289)

* feat(docker-compose): add ability to pass logger visibility to clearnode

* fix(clearnode): raw ECDSA sign and verify

* fix(clearnode): try to extract req id on ummarshal error

* fix(integration): update sig to new type

* feat(sdk): change type of sig array to contain only Hex

* refactor(clearnode): remove check on nil Req, zero-init instead

* fix(clearnode:nitrolite): remove unintended side-effect of sig param modif

* feat(custody): add note about ephemeral final state

* style(contract): run forge fmt

* fix(clearnode/docs): remove legacy signature format

* feat(custody): eip-1271, eip-6492 sigs support (#293)

* feat(contract): add ERC-1271,6492 support

* test(contract/Utils.sol): add ERC-1271, 6492 tests

* refactor(contract:Utils.t.sol): separate tests into contracts

* refactor(UtilsHarness): do NOT expose constants

* fix(contracts/adjudicators): use verifyStateSignature instead of just EOA sig

* test(contract/Custody): add ERC-1271, 6492 sig to integration test

* test(contract/Custody): add challenge with EIP-712, EIP-1271 tests

* refactor(contract/Utils): reorder recoverStateEIP712Signer params for consistency

* test(contract): remove console.logs

* feat(contract): clarify Utils comments

* refactor(contract): optimize Custody and Utils sig verification functions

* style(contract): run forge fmt

* feat: replace `stateHash` with `packedState` in message signing (#295)

* feat(contract): replace stateHash with packedState in signing

* test(contract): migrate to getPackedState

* feat(contract): remove getStateHash

* feat(clearnode): remove state_hash, rename EncodeState to Pack

* feat(sdk): remove state_hash, add getPackedState

* refactor(sdk): supply channelId and state to signState

* docs(website): migrate Sig to Hex, add supported signature formats

---------

Co-authored-by: MaxMoskalenko <mx.msklnk@gmail.com>

* feat: replace stateWalletClient with StateSigner (#297)

* feat: redesign and improve joining channels by broker, refactor endpoints (#290)

* feat: custody updates

feat(custody): add EIP-191, EIP-712 signature support (#257)

* feat(custody): add EIP-191, EIP-712 support

* feat(custody): correct EIP-712 sig impl

* feat(utils): add POLA principle comment

* feat(contract): add EIP712AdjudicatorBase

* feat(contract): integrate EIP712AdjudicatorBase to supported adjs

* fix(contract): tests with updated adjs

* test(contract): EIP712AdjudicatorBase

* test(contract): SimpleConsensus EIP191, EIP712 sigs

* test(contract): Utils signatures

* test(contract): Custody, SimpleConsensus integration tests for sigs

refactor(contract): migrate sig to bytes (#279)

* refactor(Types): replace Signature{v,r,s} with bytes

* style(contract): add empty lines at the end

refactor(clearnode): replace Signature{r,s,v} with []byte (#283)

refactor(sdk): migrate `Signature{r,s,v}` to `Hex` (#284)

* refactor(sdk): replace Signature{r,s,v} with Hex

* fix(NitroliteService): perform convertStateFromContract correctly

* fix(rpc): expect updated signature type from rpc

* style(sdk/rpc): remove ServerSignatureSchema var

* feat(sdk/rpc): add empty signatures array by default

* feat(sdk): add build:force command

* feat(sdk): remove npm build:force

refactor(clearnode): define `Signature []byte` type (#287)

* refactor(clearnode): define `Signature []byte` type

* refactor(clearnode::RPCEntry): replace sig `[]string` with `[]Signature`

* feat(clearnode/nitrolite): add Sigs2Strings, Strings2Sigs helpers

fix(integration): tests sig migration (#289)

* feat(docker-compose): add ability to pass logger visibility to clearnode

* fix(clearnode): raw ECDSA sign and verify

* fix(clearnode): try to extract req id on ummarshal error

* fix(integration): update sig to new type

* feat(sdk): change type of sig array to contain only Hex

* refactor(clearnode): remove check on nil Req, zero-init instead

* fix(clearnode:nitrolite): remove unintended side-effect of sig param modif

feat(custody): add note about ephemeral final state

style(contract): run forge fmt

fix(clearnode/docs): remove legacy signature format

feat(custody): eip-1271, eip-6492 sigs support (#293)

* feat(contract): add ERC-1271,6492 support

* test(contract/Utils.sol): add ERC-1271, 6492 tests

* refactor(contract:Utils.t.sol): separate tests into contracts

* refactor(UtilsHarness): do NOT expose constants

* fix(contracts/adjudicators): use verifyStateSignature instead of just EOA sig

* test(contract/Custody): add ERC-1271, 6492 sig to integration test

* test(contract/Custody): add challenge with EIP-712, EIP-1271 tests

* refactor(contract/Utils): reorder recoverStateEIP712Signer params for consistency

* test(contract): remove console.logs

* feat(contract): clarify Utils comments

* refactor(contract): optimize Custody and Utils sig verification functions

* style(contract): run forge fmt

feat(custody): add ability to create channel w/ server sig

feat(contract/Custody): add ability to broker deposit in create

* feat: clearnode updates

feat: improve and simplify channel joining by broker

drop join by broker

feat: add create channel endpoint draft

feat: use proper state format

feat: code improvements and fix tests

add doc

feat: use state as encode state arg

address comments

feat: add tests for request create method

update docs

feat: refactor channel service tests

rename encode state into encode state data

feat: update response

feat: unify channel operation responses

feat: alling app session service structure to channel service

update docs

improve errors consistency

* feat(rpc): add create_channel method and unify channel response types (#296)

* feat: remove state hash from channel operation response

* update erc-7824 docs

* feat: include channel in channel operation response

* fix: use chain id as key in network config

* fix(Custody::join): allow 2 signatures

* feat: add an optional session key for create_channel request

* return state data as an encoded string

* fix(Custody::join): add Server wallet

* test(custody::join): broker auto-join tests (#303)

* test(Custody::join): broker auto-join

* refactor(contract/test): signState usage

* refactor(Custody.t.sol): enhance broker auto join test name readability

* feat: add sdk and integration support for simplified join (#304)

---------

Co-authored-by: Sazonov Nikita <35502225+nksazonov@users.noreply.github.com>
Co-authored-by: Max Pushkarov <mpushkarov@yellow.org>
Co-authored-by: nksazonov <nsazonov@openware.com>
Co-authored-by: MaxMoskalenko <mx.msklnk@gmail.com>

* refactor: API protocol (#285)

* refactor: API protocol

* fix: resolve issues after rebase

* feat(Custody): update domain version to 0.3.0

* config(contract): set optimizer runs to 45 000

* docs(contract/deployments): add UAT 137, 11155111, PROD 137

* fix: add missing exports

* bump sdk version

* fix(sdk::RPCNetworkInfo): remove chain name

* update uat release values

* feat: migrate cerebro to v0.3.0 (#306)

* fix: channel amount

* feat(examples): remove snake, tictactoe, aura

* fix(main-pr): remove test-examples action

* build(contract): add separate config for linea

* docs(contract): add v0.3.0 deployments for mainnet

* add linea support & update prod config

---------

Co-authored-by: MaxMoskalenko <mx.msklnk@gmail.com>
Co-authored-by: Dmytro Steblyna <80773046+dimast-x@users.noreply.github.com>
Co-authored-by: Max Pushkarov <mpushkarov@yellow.org>
Co-authored-by: Anton Filonenko <phil@yellow.org>
gabrielantonyxaviour pushed a commit to gabrielantonyxaviour/nitrolite that referenced this pull request Sep 27, 2025
* feat: add system support for EIP-191, contracts support for EIP-(712,1271,6492) (layer-3#294)

* feat: move erc7824 documentation to nitrolite (layer-3#272)

* feat: move erc7824 documentation to nitrolite

* feat: move GH actions to common scope

* fix: comment GITHUB_TOKEN

* feat(custody): add EIP-191, EIP-712 signature support (layer-3#257)

* feat(custody): add EIP-191, EIP-712 support

* feat(custody): correct EIP-712 sig impl

* feat(utils): add POLA principle comment

* feat(contract): add EIP712AdjudicatorBase

* feat(contract): integrate EIP712AdjudicatorBase to supported adjs

* fix(contract): tests with updated adjs

* test(contract): EIP712AdjudicatorBase

* test(contract): SimpleConsensus EIP191, EIP712 sigs

* test(contract): Utils signatures

* test(contract): Custody, SimpleConsensus integration tests for sigs

* refactor(contract): migrate sig to bytes (layer-3#279)

* refactor(Types): replace Signature{v,r,s} with bytes

* style(contract): add empty lines at the end

* refactor(clearnode): replace Signature{r,s,v} with []byte (layer-3#283)

* refactor(sdk): migrate `Signature{r,s,v}` to `Hex` (layer-3#284)

* refactor(sdk): replace Signature{r,s,v} with Hex

* fix(NitroliteService): perform convertStateFromContract correctly

* fix(rpc): expect updated signature type from rpc

* style(sdk/rpc): remove ServerSignatureSchema var

* feat(sdk/rpc): add empty signatures array by default

* feat(sdk): add build:force command

* feat(sdk): remove npm build:force

* refactor(clearnode): define `Signature []byte` type (layer-3#287)

* refactor(clearnode): define `Signature []byte` type

* refactor(clearnode::RPCEntry): replace sig `[]string` with `[]Signature`

* feat(clearnode/nitrolite): add Sigs2Strings, Strings2Sigs helpers

* fix(integration): tests sig migration (layer-3#289)

* feat(docker-compose): add ability to pass logger visibility to clearnode

* fix(clearnode): raw ECDSA sign and verify

* fix(clearnode): try to extract req id on ummarshal error

* fix(integration): update sig to new type

* feat(sdk): change type of sig array to contain only Hex

* refactor(clearnode): remove check on nil Req, zero-init instead

* fix(clearnode:nitrolite): remove unintended side-effect of sig param modif

* feat(custody): add note about ephemeral final state

* style(contract): run forge fmt

* fix(clearnode/docs): remove legacy signature format

* feat(custody): eip-1271, eip-6492 sigs support (layer-3#293)

* feat(contract): add ERC-1271,6492 support

* test(contract/Utils.sol): add ERC-1271, 6492 tests

* refactor(contract:Utils.t.sol): separate tests into contracts

* refactor(UtilsHarness): do NOT expose constants

* fix(contracts/adjudicators): use verifyStateSignature instead of just EOA sig

* test(contract/Custody): add ERC-1271, 6492 sig to integration test

* test(contract/Custody): add challenge with EIP-712, EIP-1271 tests

* refactor(contract/Utils): reorder recoverStateEIP712Signer params for consistency

* test(contract): remove console.logs

* feat(contract): clarify Utils comments

* refactor(contract): optimize Custody and Utils sig verification functions

* style(contract): run forge fmt

* feat: replace `stateHash` with `packedState` in message signing (layer-3#295)

* feat(contract): replace stateHash with packedState in signing

* test(contract): migrate to getPackedState

* feat(contract): remove getStateHash

* feat(clearnode): remove state_hash, rename EncodeState to Pack

* feat(sdk): remove state_hash, add getPackedState

* refactor(sdk): supply channelId and state to signState

* docs(website): migrate Sig to Hex, add supported signature formats

---------

Co-authored-by: MaxMoskalenko <mx.msklnk@gmail.com>

* feat: replace stateWalletClient with StateSigner (layer-3#297)

* feat: redesign and improve joining channels by broker, refactor endpoints (layer-3#290)

* feat: custody updates

feat(custody): add EIP-191, EIP-712 signature support (layer-3#257)

* feat(custody): add EIP-191, EIP-712 support

* feat(custody): correct EIP-712 sig impl

* feat(utils): add POLA principle comment

* feat(contract): add EIP712AdjudicatorBase

* feat(contract): integrate EIP712AdjudicatorBase to supported adjs

* fix(contract): tests with updated adjs

* test(contract): EIP712AdjudicatorBase

* test(contract): SimpleConsensus EIP191, EIP712 sigs

* test(contract): Utils signatures

* test(contract): Custody, SimpleConsensus integration tests for sigs

refactor(contract): migrate sig to bytes (layer-3#279)

* refactor(Types): replace Signature{v,r,s} with bytes

* style(contract): add empty lines at the end

refactor(clearnode): replace Signature{r,s,v} with []byte (layer-3#283)

refactor(sdk): migrate `Signature{r,s,v}` to `Hex` (layer-3#284)

* refactor(sdk): replace Signature{r,s,v} with Hex

* fix(NitroliteService): perform convertStateFromContract correctly

* fix(rpc): expect updated signature type from rpc

* style(sdk/rpc): remove ServerSignatureSchema var

* feat(sdk/rpc): add empty signatures array by default

* feat(sdk): add build:force command

* feat(sdk): remove npm build:force

refactor(clearnode): define `Signature []byte` type (layer-3#287)

* refactor(clearnode): define `Signature []byte` type

* refactor(clearnode::RPCEntry): replace sig `[]string` with `[]Signature`

* feat(clearnode/nitrolite): add Sigs2Strings, Strings2Sigs helpers

fix(integration): tests sig migration (layer-3#289)

* feat(docker-compose): add ability to pass logger visibility to clearnode

* fix(clearnode): raw ECDSA sign and verify

* fix(clearnode): try to extract req id on ummarshal error

* fix(integration): update sig to new type

* feat(sdk): change type of sig array to contain only Hex

* refactor(clearnode): remove check on nil Req, zero-init instead

* fix(clearnode:nitrolite): remove unintended side-effect of sig param modif

feat(custody): add note about ephemeral final state

style(contract): run forge fmt

fix(clearnode/docs): remove legacy signature format

feat(custody): eip-1271, eip-6492 sigs support (layer-3#293)

* feat(contract): add ERC-1271,6492 support

* test(contract/Utils.sol): add ERC-1271, 6492 tests

* refactor(contract:Utils.t.sol): separate tests into contracts

* refactor(UtilsHarness): do NOT expose constants

* fix(contracts/adjudicators): use verifyStateSignature instead of just EOA sig

* test(contract/Custody): add ERC-1271, 6492 sig to integration test

* test(contract/Custody): add challenge with EIP-712, EIP-1271 tests

* refactor(contract/Utils): reorder recoverStateEIP712Signer params for consistency

* test(contract): remove console.logs

* feat(contract): clarify Utils comments

* refactor(contract): optimize Custody and Utils sig verification functions

* style(contract): run forge fmt

feat(custody): add ability to create channel w/ server sig

feat(contract/Custody): add ability to broker deposit in create

* feat: clearnode updates

feat: improve and simplify channel joining by broker

drop join by broker

feat: add create channel endpoint draft

feat: use proper state format

feat: code improvements and fix tests

add doc

feat: use state as encode state arg

address comments

feat: add tests for request create method

update docs

feat: refactor channel service tests

rename encode state into encode state data

feat: update response

feat: unify channel operation responses

feat: alling app session service structure to channel service

update docs

improve errors consistency

* feat(rpc): add create_channel method and unify channel response types (layer-3#296)

* feat: remove state hash from channel operation response

* update erc-7824 docs

* feat: include channel in channel operation response

* fix: use chain id as key in network config

* fix(Custody::join): allow 2 signatures

* feat: add an optional session key for create_channel request

* return state data as an encoded string

* fix(Custody::join): add Server wallet

* test(custody::join): broker auto-join tests (layer-3#303)

* test(Custody::join): broker auto-join

* refactor(contract/test): signState usage

* refactor(Custody.t.sol): enhance broker auto join test name readability

* feat: add sdk and integration support for simplified join (layer-3#304)

---------

Co-authored-by: Sazonov Nikita <35502225+nksazonov@users.noreply.github.com>
Co-authored-by: Max Pushkarov <mpushkarov@yellow.org>
Co-authored-by: nksazonov <nsazonov@openware.com>
Co-authored-by: MaxMoskalenko <mx.msklnk@gmail.com>

* refactor: API protocol (layer-3#285)

* refactor: API protocol

* fix: resolve issues after rebase

* feat(Custody): update domain version to 0.3.0

* config(contract): set optimizer runs to 45 000

* docs(contract/deployments): add UAT 137, 11155111, PROD 137

* fix: add missing exports

* bump sdk version

* fix(sdk::RPCNetworkInfo): remove chain name

* update uat release values

* feat: migrate cerebro to v0.3.0 (layer-3#306)

* fix: channel amount

* feat(examples): remove snake, tictactoe, aura

* fix(main-pr): remove test-examples action

* build(contract): add separate config for linea

* docs(contract): add v0.3.0 deployments for mainnet

* add linea support & update prod config

---------

Co-authored-by: MaxMoskalenko <mx.msklnk@gmail.com>
Co-authored-by: Dmytro Steblyna <80773046+dimast-x@users.noreply.github.com>
Co-authored-by: Max Pushkarov <mpushkarov@yellow.org>
Co-authored-by: Anton Filonenko <phil@yellow.org>
gabrielantonyxaviour pushed a commit to gabrielantonyxaviour/nitrolite that referenced this pull request Sep 27, 2025
* feat: add system support for EIP-191, contracts support for EIP-(712,1271,6492) (layer-3#294)

* feat: move erc7824 documentation to nitrolite (layer-3#272)

* feat: move erc7824 documentation to nitrolite

* feat: move GH actions to common scope

* fix: comment GITHUB_TOKEN

* feat(custody): add EIP-191, EIP-712 signature support (layer-3#257)

* feat(custody): add EIP-191, EIP-712 support

* feat(custody): correct EIP-712 sig impl

* feat(utils): add POLA principle comment

* feat(contract): add EIP712AdjudicatorBase

* feat(contract): integrate EIP712AdjudicatorBase to supported adjs

* fix(contract): tests with updated adjs

* test(contract): EIP712AdjudicatorBase

* test(contract): SimpleConsensus EIP191, EIP712 sigs

* test(contract): Utils signatures

* test(contract): Custody, SimpleConsensus integration tests for sigs

* refactor(contract): migrate sig to bytes (layer-3#279)

* refactor(Types): replace Signature{v,r,s} with bytes

* style(contract): add empty lines at the end

* refactor(clearnode): replace Signature{r,s,v} with []byte (layer-3#283)

* refactor(sdk): migrate `Signature{r,s,v}` to `Hex` (layer-3#284)

* refactor(sdk): replace Signature{r,s,v} with Hex

* fix(NitroliteService): perform convertStateFromContract correctly

* fix(rpc): expect updated signature type from rpc

* style(sdk/rpc): remove ServerSignatureSchema var

* feat(sdk/rpc): add empty signatures array by default

* feat(sdk): add build:force command

* feat(sdk): remove npm build:force

* refactor(clearnode): define `Signature []byte` type (layer-3#287)

* refactor(clearnode): define `Signature []byte` type

* refactor(clearnode::RPCEntry): replace sig `[]string` with `[]Signature`

* feat(clearnode/nitrolite): add Sigs2Strings, Strings2Sigs helpers

* fix(integration): tests sig migration (layer-3#289)

* feat(docker-compose): add ability to pass logger visibility to clearnode

* fix(clearnode): raw ECDSA sign and verify

* fix(clearnode): try to extract req id on ummarshal error

* fix(integration): update sig to new type

* feat(sdk): change type of sig array to contain only Hex

* refactor(clearnode): remove check on nil Req, zero-init instead

* fix(clearnode:nitrolite): remove unintended side-effect of sig param modif

* feat(custody): add note about ephemeral final state

* style(contract): run forge fmt

* fix(clearnode/docs): remove legacy signature format

* feat(custody): eip-1271, eip-6492 sigs support (layer-3#293)

* feat(contract): add ERC-1271,6492 support

* test(contract/Utils.sol): add ERC-1271, 6492 tests

* refactor(contract:Utils.t.sol): separate tests into contracts

* refactor(UtilsHarness): do NOT expose constants

* fix(contracts/adjudicators): use verifyStateSignature instead of just EOA sig

* test(contract/Custody): add ERC-1271, 6492 sig to integration test

* test(contract/Custody): add challenge with EIP-712, EIP-1271 tests

* refactor(contract/Utils): reorder recoverStateEIP712Signer params for consistency

* test(contract): remove console.logs

* feat(contract): clarify Utils comments

* refactor(contract): optimize Custody and Utils sig verification functions

* style(contract): run forge fmt

* feat: replace `stateHash` with `packedState` in message signing (layer-3#295)

* feat(contract): replace stateHash with packedState in signing

* test(contract): migrate to getPackedState

* feat(contract): remove getStateHash

* feat(clearnode): remove state_hash, rename EncodeState to Pack

* feat(sdk): remove state_hash, add getPackedState

* refactor(sdk): supply channelId and state to signState

* docs(website): migrate Sig to Hex, add supported signature formats

---------

Co-authored-by: MaxMoskalenko <mx.msklnk@gmail.com>

* feat: replace stateWalletClient with StateSigner (layer-3#297)

* feat: redesign and improve joining channels by broker, refactor endpoints (layer-3#290)

* feat: custody updates

feat(custody): add EIP-191, EIP-712 signature support (layer-3#257)

* feat(custody): add EIP-191, EIP-712 support

* feat(custody): correct EIP-712 sig impl

* feat(utils): add POLA principle comment

* feat(contract): add EIP712AdjudicatorBase

* feat(contract): integrate EIP712AdjudicatorBase to supported adjs

* fix(contract): tests with updated adjs

* test(contract): EIP712AdjudicatorBase

* test(contract): SimpleConsensus EIP191, EIP712 sigs

* test(contract): Utils signatures

* test(contract): Custody, SimpleConsensus integration tests for sigs

refactor(contract): migrate sig to bytes (layer-3#279)

* refactor(Types): replace Signature{v,r,s} with bytes

* style(contract): add empty lines at the end

refactor(clearnode): replace Signature{r,s,v} with []byte (layer-3#283)

refactor(sdk): migrate `Signature{r,s,v}` to `Hex` (layer-3#284)

* refactor(sdk): replace Signature{r,s,v} with Hex

* fix(NitroliteService): perform convertStateFromContract correctly

* fix(rpc): expect updated signature type from rpc

* style(sdk/rpc): remove ServerSignatureSchema var

* feat(sdk/rpc): add empty signatures array by default

* feat(sdk): add build:force command

* feat(sdk): remove npm build:force

refactor(clearnode): define `Signature []byte` type (layer-3#287)

* refactor(clearnode): define `Signature []byte` type

* refactor(clearnode::RPCEntry): replace sig `[]string` with `[]Signature`

* feat(clearnode/nitrolite): add Sigs2Strings, Strings2Sigs helpers

fix(integration): tests sig migration (layer-3#289)

* feat(docker-compose): add ability to pass logger visibility to clearnode

* fix(clearnode): raw ECDSA sign and verify

* fix(clearnode): try to extract req id on ummarshal error

* fix(integration): update sig to new type

* feat(sdk): change type of sig array to contain only Hex

* refactor(clearnode): remove check on nil Req, zero-init instead

* fix(clearnode:nitrolite): remove unintended side-effect of sig param modif

feat(custody): add note about ephemeral final state

style(contract): run forge fmt

fix(clearnode/docs): remove legacy signature format

feat(custody): eip-1271, eip-6492 sigs support (layer-3#293)

* feat(contract): add ERC-1271,6492 support

* test(contract/Utils.sol): add ERC-1271, 6492 tests

* refactor(contract:Utils.t.sol): separate tests into contracts

* refactor(UtilsHarness): do NOT expose constants

* fix(contracts/adjudicators): use verifyStateSignature instead of just EOA sig

* test(contract/Custody): add ERC-1271, 6492 sig to integration test

* test(contract/Custody): add challenge with EIP-712, EIP-1271 tests

* refactor(contract/Utils): reorder recoverStateEIP712Signer params for consistency

* test(contract): remove console.logs

* feat(contract): clarify Utils comments

* refactor(contract): optimize Custody and Utils sig verification functions

* style(contract): run forge fmt

feat(custody): add ability to create channel w/ server sig

feat(contract/Custody): add ability to broker deposit in create

* feat: clearnode updates

feat: improve and simplify channel joining by broker

drop join by broker

feat: add create channel endpoint draft

feat: use proper state format

feat: code improvements and fix tests

add doc

feat: use state as encode state arg

address comments

feat: add tests for request create method

update docs

feat: refactor channel service tests

rename encode state into encode state data

feat: update response

feat: unify channel operation responses

feat: alling app session service structure to channel service

update docs

improve errors consistency

* feat(rpc): add create_channel method and unify channel response types (layer-3#296)

* feat: remove state hash from channel operation response

* update erc-7824 docs

* feat: include channel in channel operation response

* fix: use chain id as key in network config

* fix(Custody::join): allow 2 signatures

* feat: add an optional session key for create_channel request

* return state data as an encoded string

* fix(Custody::join): add Server wallet

* test(custody::join): broker auto-join tests (layer-3#303)

* test(Custody::join): broker auto-join

* refactor(contract/test): signState usage

* refactor(Custody.t.sol): enhance broker auto join test name readability

* feat: add sdk and integration support for simplified join (layer-3#304)

---------

Co-authored-by: Sazonov Nikita <35502225+nksazonov@users.noreply.github.com>
Co-authored-by: Max Pushkarov <mpushkarov@yellow.org>
Co-authored-by: nksazonov <nsazonov@openware.com>
Co-authored-by: MaxMoskalenko <mx.msklnk@gmail.com>

* refactor: API protocol (layer-3#285)

* refactor: API protocol

* fix: resolve issues after rebase

* feat(Custody): update domain version to 0.3.0

* config(contract): set optimizer runs to 45 000

* docs(contract/deployments): add UAT 137, 11155111, PROD 137

* fix: add missing exports

* bump sdk version

* fix(sdk::RPCNetworkInfo): remove chain name

* update uat release values

* feat: migrate cerebro to v0.3.0 (layer-3#306)

* fix: channel amount

* feat(examples): remove snake, tictactoe, aura

* fix(main-pr): remove test-examples action

* build(contract): add separate config for linea

* docs(contract): add v0.3.0 deployments for mainnet

* add linea support & update prod config

---------

Co-authored-by: MaxMoskalenko <mx.msklnk@gmail.com>
Co-authored-by: Dmytro Steblyna <80773046+dimast-x@users.noreply.github.com>
Co-authored-by: Max Pushkarov <mpushkarov@yellow.org>
Co-authored-by: Anton Filonenko <phil@yellow.org>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants