Skip to content

Add conversion traits between Felt and StorageKey#130

Merged
franciszekjob merged 1 commit intosoftware-mansion:sequencer-future-compatfrom
AvivYossef-starkware:add-storage-key-felt-conversion
Apr 8, 2026
Merged

Add conversion traits between Felt and StorageKey#130
franciszekjob merged 1 commit intosoftware-mansion:sequencer-future-compatfrom
AvivYossef-starkware:add-storage-key-felt-conversion

Conversation

@AvivYossef-starkware
Copy link
Copy Markdown
Contributor

Summary

  • Add From<Felt> for StorageKey — formats as 0x-prefixed lowercase hex
  • Add TryFrom<&StorageKey> for Felt — parses the hex string back into a Felt

Depends on #129 (starknet-types-core 1.0.0 upgrade).

Test plan

  • test_storage_key_from_felt — round-trip from Felt to StorageKey
  • test_felt_try_from_storage_key — round-trip from StorageKey to Felt
  • test_felt_try_from_storage_key_invalid — invalid hex returns error
  • clippy clean

🤖 Generated with Claude Code

@AvivYossef-starkware AvivYossef-starkware requested a review from a team as a code owner March 30, 2026 12:52
@AvivYossef-starkware AvivYossef-starkware force-pushed the add-storage-key-felt-conversion branch 3 times, most recently from 79dc1b3 to 4b0d913 Compare March 31, 2026 05:54
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@AvivYossef-starkware AvivYossef-starkware force-pushed the add-storage-key-felt-conversion branch from 4b0d913 to 7188899 Compare April 8, 2026 15:35
@AvivYossef-starkware AvivYossef-starkware changed the base branch from master to sequencer-future-compat April 8, 2026 15:35
@franciszekjob franciszekjob merged commit d44dcb2 into software-mansion:sequencer-future-compat Apr 8, 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