Skip to content

#68 Add networkId and its verification, reorder mint transaction params#69

Open
martti007 wants to merge 4 commits into
mainfrom
issue-68
Open

#68 Add networkId and its verification, reorder mint transaction params#69
martti007 wants to merge 4 commits into
mainfrom
issue-68

Conversation

@martti007
Copy link
Copy Markdown
Contributor

No description provided.

Copy link
Copy Markdown

@gemini-code-assist gemini-code-assist Bot left a comment

Choose a reason for hiding this comment

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

Code Review

This pull request introduces a formal NetworkId enum and refactors the minting and splitting logic to use a combination of NetworkId and a new TokenSalt to derive TokenId deterministically. It renames getBlockHeight to getLatestBlockNumber across the aggregator client and updates verification rules to ensure network consistency between transactions and trust bases. Additionally, the token split API was refactored to use SplitTokenRequest and SplitToken objects. Feedback was provided regarding the non-deterministic ordering of proofs in the split logic and a redundant collection allocation.

Comment thread src/main/java/org/unicitylabs/sdk/payment/TokenSplit.java
Comment thread src/main/java/org/unicitylabs/sdk/payment/TokenSplit.java Outdated
@martti007 martti007 requested a review from MastaP May 28, 2026 11:50
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.

[TGE] [Java] Migrate minting from caller-supplied token ids to networkId + salt

3 participants