Skip to content

V5 Ecosystem Wallet #323

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.

Sign up for GitHub

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

Jump to bottom
Merged
andygruening merged 273 commits into from
Dec 12, 2025
Merged

V5 Ecosystem Wallet #323

andygruening merged 273 commits into from
Dec 12, 2025

Conversation

@andygruening
Copy link
Contributor

@andygruening andygruening commented Aug 15, 2025

Version Increment

Please ensure you have incremented the package version in the package.json as necessary.

  • I have incremented the package.json according to semantic versioning
  • No version increment is needed; the change does not impact SDK or Sample code/assets

Docs Checklist

Please ensure you have addressed documentation updates if needed as part of this PR:

  • I have created a separate PR on the sequence docs repository for documentation updates: Link to docs PR
  • No documentation update is needed for this change.

BellringerQuinn and others added 30 commits May 29, 2025 10:44
@BellringerQuinn
Fix address comparison such that address instances that represent the…
3b4948f 
… 0 address will be equal regardless of if they have the same value string (one may have more 0s than the other) - for example, previously 0x00 and 0x0000 would not be considered equal like they should've been, this is now fixed.
@BellringerQuinn
Added a LocalChain because the V3 tests return this chain id frequently.
83e301b
@BellringerQuinn
Update FixedBytesCoder and StaticBytesCoder such that they both can h…
7ccb5ec 
…andle either byte[] or FixedByte types
@BellringerQuinn
Fix NumberCoder such that it can handle other types of number other t…
8bce78e 
…han just BigIntegers; also, don't throw an ArgumentOutOfRangeException due to having a larger bigint than the desired size by clamping the padding to a min of 0
@BellringerQuinn
Fix broken test
c586762
@BellringerQuinn
Fix encoding runtime exceptions that occur when running Payload tests
653552c
@BellringerQuinn
Added additional unit tests to help determine why the Payload integra…
a2dce77 
…tion tests are failing
@BellringerQuinn
Fix TestPackCalls and implementation
ced124a
@BellringerQuinn
Fix BytesCoder - simply removing the trailing 0s can cause overflows …
c4169b9 
…when trying to take a substring in FixedBytesCoder if we have our byte array ending with 0 values; instead, we should just leave them and take the substring as in the spec
@andygruening
added permission primitives and its decoding/encoding functions
d69ee91
@BellringerQuinn
Added a unit test to confirm we can successfully build a TypedDataToS…
64d262a 
…ign object from a ConfigUpdate Parented payload
@BellringerQuinn
Added a unit test to confirm we can successfully build a TypedDataToS…
6024e1e 
…ign object from a Message Parented payload
@BellringerQuinn
Added a unit test to confirm we can successfully created a TypedDataT…
4695b82 
…oSign from a Calls Parented Payload
@BellringerQuinn
Log a bunch of stuff to debug eip712 signing
42446e1
@BellringerQuinn
Fix how we handle byte[] encoding
dd28a8d
@BellringerQuinn
Include embedded wallet address when using list sessions
9de346c
@BellringerQuinn
Add additional unit test EIP712 signing and fix how we are EIP712 enc…
871466d 
…oding strings
@BellringerQuinn
Get TestGetSignedPayload to pass using nethereum
d44d3be
@BellringerQuinn
Fix our conversion of Calls payloads into a type that nethereum can u…
4919b6a 
…nderstand, adding some unit tests in the process.
@BellringerQuinn
Change how we get minimum bytes needed such that tests are now passing
9895e81
@BellringerQuinn
Cleaner Calls.Encode implementation
0432540
@BellringerQuinn
Cleanup Calls.Decode implementation
755ed34
@BellringerQuinn
Create CallsEncoder and CallEncoder as we've done with decoding as th…
55aac5c 
…is is much cleaner to read
@BellringerQuinn
Cleanup how we convert from our data types (TypedDataToSign + NamedTy…
dbd42bf 
…pe) to the Nethereum equivalents
@BellringerQuinn
More cleanup
6067f63
@BellringerQuinn
Added a readme explaining how to update Nethereum if needed
c148c78
@BellringerQuinn
Fix broken unit tests by fixing class access level
9a1e238
@BellringerQuinn
Added FindSignerLeaf and GetSigners methods
9aa2418
@BellringerQuinn
Added ability to get weight of a topology or config - adding Weight c…
4ae0790 
…lass
@BellringerQuinn
Added HashConfiguration method
9980b2e
@andygruening andygruening changed the title Ecosystem Wallets V5 Ecosystem Wallet Dec 11, 2025
@andygruening andygruening marked this pull request as ready for review December 12, 2025 12:19
@andygruening andygruening requested a review from a team as a code owner December 12, 2025 12:19
@andygruening andygruening merged commit 9cdc67d into master Dec 12, 2025
3 checks passed
@andygruening andygruening deleted the V5-EcosystemWallet branch December 12, 2025 19:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@taylanpince taylanpince taylanpince approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@andygruening @taylanpince @BellringerQuinn @pkieltyka