Skip to content

fix(sdk): sync sdk with cairo contract renames (to_addr, enc_user_addr_hash) - #469

Merged
m-kus merged 1 commit into
mainfrom
02-11-fix_sdk_sync_sdk_with_cairo_contract_renames_to_addr_enc_user_addr_hash_
Feb 11, 2026
Merged

m-kus merged 1 commit into
mainfrom
02-11-fix_sdk_sync_sdk_with_cairo_contract_renames_to_addr_enc_user_addr_hash_

Conversation

@m-kus

@m-kus m-kus commented Feb 11, 2026 •

Copy link
Copy Markdown

Improve CI workflow and update SDK parameter names

This PR makes several improvements to the CI workflow and SDK:

  1. Adds separate jobs for fast tests and devnet tests in the CI workflow
  2. Updates parameter names in the ABI to match the latest contract changes:
    • withdrawal_target → to_addr
    • sender_addr → from_addr
    • recipient_addr → to_addr
    • compute_enc_address_hash → compute_enc_user_addr_hash
  3. Adds support for the new Swap client action
  4. Updates npm scripts for better organization:
    • Adds test:devnet script
    • Renames generate:hashes to generate:cairo-hashes
    • Adds check:fast for quicker pre-commit checks

This change is Reviewable

m-kus commented Feb 11, 2026 •

Copy link
Copy Markdown
Author

@m-kus
m-kus marked this pull request as ready for review February 11, 2026 12:31
@m-kus
m-kus force-pushed the 02-11-fix_sdk_sync_sdk_with_cairo_contract_renames_to_addr_enc_user_addr_hash_ branch 3 times, most recently from ad74f46 to b94f895 Compare February 11, 2026 13:25
@m-kus
m-kus requested a review from Yoni-Starkware February 11, 2026 13:26
@m-kus
m-kus force-pushed the 02-11-fix_sdk_sync_sdk_with_cairo_contract_renames_to_addr_enc_user_addr_hash_ branch 6 times, most recently from 37f2bcf to b864bef Compare February 11, 2026 14:21
@m-kus
m-kus enabled auto-merge (squash) February 11, 2026 14:21
@m-kus
m-kus force-pushed the 02-11-fix_sdk_sync_sdk_with_cairo_contract_renames_to_addr_enc_user_addr_hash_ branch 2 times, most recently from bcd6e0d to 9f6fd8b Compare February 11, 2026 16:08

@Yoni-Starkware Yoni-Starkware left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

:lgtm:

@Yoni-Starkware reviewed 15 files and all commit messages, and made 1 comment.
Reviewable status: :shipit: complete! all files reviewed, all discussions resolved (waiting on @ittaysw and @NirLevi-starkware).

@m-kus
m-kus force-pushed the 02-11-fix_sdk_sync_sdk_with_cairo_contract_renames_to_addr_enc_user_addr_hash_ branch from 9f6fd8b to afa1231 Compare February 11, 2026 18:07
@m-kus
m-kus merged commit 98abf78 into main Feb 11, 2026
4 of 6 checks passed
@m-kus
m-kus deleted the 02-11-fix_sdk_sync_sdk_with_cairo_contract_renames_to_addr_enc_user_addr_hash_ branch February 11, 2026 18:09
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