Skip to content

refactor(eip712): rename EIP712Repository.signTypedDataEIP712 to signTypedData#52

Merged
Comp0te merged 2 commits into
masterfrom
eip712-rename-repository-sign-typed-data
Jun 26, 2026
Merged

refactor(eip712): rename EIP712Repository.signTypedDataEIP712 to signTypedData#52
Comp0te merged 2 commits into
masterfrom
eip712-rename-repository-sign-typed-data

Conversation

@ost-ptk

@ost-ptk ost-ptk commented Jun 25, 2026

Copy link
Copy Markdown
Member

Summary

Renames the public EIP712Repository signing method consumed by the wallet from signTypedDataEIP712 back to signTypedData.

This reverts the public-facing part of the naming introduced in #47, restoring the original signTypedData method name on the repository contract (IEIP712Repository + EIP712Repository).

Scope

  • Renamed (public, consumed by the wallet): IEIP712Repository.signTypedData, EIP712Repository.signTypedData, JSDoc and the corresponding unit test.
  • Kept (internal): the signing utilities in utils/eip712/sign.ts (signTypedDataEIP712, signTypedDataEIP712DigestWithKey, signTypedDataEIP712WithRawKey), the ...Util alias import and the 'signTypedDataEIP712' error-source tag.

Verification

  • tsc --noEmit clean
  • eslint clean
  • jest — all eip712 suites pass (90 tests)

…TypedData

The public repository method consumed by the wallet is renamed back to `signTypedData`. Internal signing utilities (`signTypedDataEIP712`, `...DigestWithKey`, `...WithRawKey`) and the error-source tag keep the EIP-712 suffix.
@ost-ptk ost-ptk requested a review from Comp0te June 25, 2026 14:48
@ost-ptk ost-ptk self-assigned this Jun 25, 2026
@Comp0te Comp0te merged commit 42ee333 into master Jun 26, 2026
4 checks passed
@Comp0te Comp0te mentioned this pull request Jun 30, 2026
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.

2 participants