refactor(mock-utils): consume relayer-sdk core entrypoint#101
refactor(mock-utils): consume relayer-sdk core entrypoint#101gustavovalverde wants to merge 1 commit intozama-ai:mainfrom
Conversation
|
Thank you for your pull request. We require contributors to sign our Contributor License Agreement / Terms and Conditions, and we don't seem to have the users @gustavovalverde on file. In order for us to review and merge your code, please sign:
If you already signed one of this document, just wait to be added to the bot config. |
gustavovalverde
force-pushed
the
fix/mock-utils-browser-compat
branch
from
ed7a95d to
27a6bcd
Compare
|
Thank you for your pull request. We require contributors to sign our Contributor License Agreement / Terms and Conditions, and we don't seem to have the users @gustavovalverde on file. In order for us to review and merge your code, please sign:
If you already signed one of this document, just wait to be added to the bot config. |
gustavovalverde
force-pushed
the
fix/mock-utils-browser-compat
branch
from
27a6bcd to
84176dc
Compare
|
Thank you for your pull request. We require contributors to sign our Contributor License Agreement / Terms and Conditions, and we don't seem to have the users @gustavovalverde on file. In order for us to review and merge your code, please sign:
If you already signed one of this document, just wait to be added to the bot config. |
gustavovalverde
changed the title
gustavovalverde
force-pushed
the
fix/mock-utils-browser-compat
branch
from
84176dc to
9ae1933
Compare
|
Thank you for your pull request. We require contributors to sign our Contributor License Agreement / Terms and Conditions, and we don't seem to have the users @gustavovalverde on file. In order for us to review and merge your code, please sign:
If you already signed one of this document, just wait to be added to the bot config. |
1 similar comment
|
Thank you for your pull request. We require contributors to sign our Contributor License Agreement / Terms and Conditions, and we don't seem to have the users @gustavovalverde on file. In order for us to review and merge your code, please sign:
If you already signed one of this document, just wait to be added to the bot config. |
gustavovalverde
changed the title
Use the side-effect-free `@zama-fhe/relayer-sdk/core` surface for KmsEIP712 in mock-utils and keep local mock keypair generation WASM-free. This aligns mock-utils with the upstream modular relayer-sdk architecture and removes vendored helper drift.
gustavovalverde
force-pushed
the
fix/mock-utils-browser-compat
branch
from
cf55c67 to
6fb3ecd
Compare
|
Thank you for your pull request. We require contributors to sign our Contributor License Agreement / Terms and Conditions, and we don't seem to have the users @gustavovalverde on file. In order for us to review and merge your code, please sign:
If you already signed one of this document, just wait to be added to the bot config. |
|
@cla-bot check |
|
The cla-bot has been summoned, and re-checked this pull request! |
|
I'm marking this as draft so it doesn't get mistakenly merged, considering it depends on zama-ai/relayer-sdk#412 (which has to be merged first) |
2 participants
Summary
mock-utilsdecryption EIP-712 construction toKmsEIP712from@zama-fhe/relayer-sdk/coregenerateKeypair()(no WASM bootstrap path)KmsEIP712copy and rely on upstream implementationWhy
mock-utilsneeds the EIP-712 helper surface but should avoid pulling runtime WASM/native bootstrap paths through@zama-fhe/relayer-sdk/node.Using the new side-effect-free
coreentrypoint keeps the integration aligned with upstream relayer-sdk logic while avoiding vendored helper drift.Dependency
This PR depends on relayer-sdk PR #412, which introduces
@zama-fhe/relayer-sdk/core:This branch is expected to merge after that PR is merged and released.
Validation
npm -C packages/mock-utils run buildnpm -C packages/mock-utils run test(Validation run with a local relayer-sdk checkout that includes PR #412 changes.)