Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Feat/address checker starknet #5196

Open
wants to merge 36 commits into
base: main
Choose a base branch
from

Conversation

mshojaei-txfusion
Copy link
Collaborator

Description

This PR adds Starknet address utilities and validation functions to the address utils module. New functionality includes:

  • Address validation and normalization for Starknet addresses
  • Checksum validation and parsing
  • Conversion between bytes and Starknet addresses
  • Address equality comparison for Starknet
  • Transaction hash validation for Starknet

All new Starknet functions follow the same patterns as existing address utilities for other protocols (EVM, Cosmos, Sealevel).

Drive-by changes

None

Related issues

N/A

Backward compatibility

Yes - This change only adds new functionality without modifying existing behavior.

Testing

None

mortezashojaei and others added 30 commits October 16, 2024 15:07
Copy link

changeset-bot bot commented Jan 17, 2025

🦋 Changeset detected

Latest commit: ecb73ff

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 9 packages
Name Type
@hyperlane-xyz/utils Minor
@hyperlane-xyz/sdk Minor
@hyperlane-xyz/widgets Minor
@hyperlane-xyz/core Patch
@hyperlane-xyz/cli Minor
@hyperlane-xyz/infra Minor
@hyperlane-xyz/helloworld Minor
@hyperlane-xyz/ccip-server Minor
@hyperlane-xyz/github-proxy Minor

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

Copy link

codecov bot commented Jan 17, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 77.53%. Comparing base (769cec1) to head (ecb73ff).
Report is 7 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #5196   +/-   ##
=======================================
  Coverage   77.53%   77.53%           
=======================================
  Files         103      103           
  Lines        2110     2110           
  Branches      190      190           
=======================================
  Hits         1636     1636           
  Misses        453      453           
  Partials       21       21           
Components Coverage Δ
core 87.80% <ø> (ø)
hooks 79.39% <ø> (ø)
isms 83.68% <ø> (ø)
token 91.27% <ø> (ø)
middlewares 79.80% <ø> (ø)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: In Review
Development

Successfully merging this pull request may close these issues.

3 participants