normalize some constants #753
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
hayes-mysten
had a problem deploying
to
sui-typescript-aws-kms-test-env
GitHub Actions
Failure
Contributor
|
|
williamrobertson13
approved these changes
Jan 6, 2026
hayes-mysten
added a commit
that referenced
this pull request
Jan 9, 2026
hayes-mysten
added a commit
that referenced
this pull request
Jan 15, 2026
hayes-mysten
added a commit
that referenced
this pull request
Jan 23, 2026
hayes-mysten
added a commit
that referenced
this pull request
Jan 28, 2026
hayes-mysten
added a commit
that referenced
this pull request
Jan 28, 2026
hayes-mysten
added a commit
that referenced
this pull request
Jan 29, 2026
* Remove default client used for verifying zklogin signatures (#625) Co-authored-by: Michael Hayes <[email protected]> * require legacy address arg (#627) make legacyAddress parameter required Co-authored-by: Michael Hayes <[email protected]> * make network required (#628) Co-authored-by: Michael Hayes <[email protected]> * remove reportTransactionEffects (#630) Co-authored-by: Michael Hayes <[email protected]> * remove asClientExtension * remove sui client exports (#633) remove json rpc exports from @mysten/sui/client Co-authored-by: Michael Hayes <[email protected]> * remove deprecated methods (#634) remove deprecated methdos Co-authored-by: Michael Hayes <[email protected]> * core api tests (#644) add test coverage for core api methods * Export a single graphql schema (#646) * fix fetching package objects * Add transaction resolution support for grpc client (#649) * Add transaction resolution support for grpc client * remove experimental client exports (#650) * remove experimental prefix for core client apis * fix some todos and inconsistencies in the core apis (#654) * Add include options for core APIs (#656) * Add include options for core APIs * migrate tests to grpc * PR feedback * remove eslint (#698) * rename listObjects to getObjects (#696) * move sui sdk to packages/sui (#697) * dApp Kit Next docs (#529) * dApp Kit Next docs * fix links and wording * linting * migration guide improvements & general refactor * remove more hallucinations + fix code (after manual tests) and text (after re-reading) * refactor + add missing global type registration in the migration guide * global type registration comment in react/next-js examples * polish and fixes * Apply suggestions from code review Co-authored-by: William Robertson <[email protected]> * merge dapp-kit-next docs into the old ones; refactor * address PR comments * more pr feedback fixes * restructure docs * fix lint --------- Co-authored-by: William Robertson <[email protected]> Co-authored-by: Michael Hayes <[email protected]> * Add top level transaction status to transaction response (#702) * Add top level transaction status to transaction response update examples and docs add transaction result type * pr feedback * fix broken test * update create-dapp (#703) * update create-dapp Add codegen docs * PR feedback * rename `Commands` -> `TransactionCommands` to fix react-native ESM resolution (#724) * remove deepbook from changset * Add objectBcs to core API and update signers sdk to work with core cl… (#727) * Add objectBcs to core API and update signers sdk to work with core clients * address PR comments * Migrate more thigns to grpc (#728) * Add commandResults to simulate api (#730) * Add commandResults to simulate api * simulate unresolved transactions * add missing wait for transaction * test fixes * codegen: support on-chain package summary generation Add support for generating code from on-chain packages by using `sui move summary --package-id` to fetch summaries automatically. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <[email protected]> * fix packageId normalization issue in tests * Mh less publishing (#739) less package publishing in tests * fix(codegen): preserve original type parameter index when filtering When generic type parameters are filtered (e.g., to exclude phantom parameters), the codegen was incorrectly using the new array index instead of the original index. This caused generated code to reference wrong type parameters (e.g., T1 instead of T0). This fix maps type parameters to `{ param, originalIndex }` before filtering, ensuring the original index is preserved and used in the generated output. Also adds generated BCS contracts for kiosk package. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude Opus 4.5 <[email protected]> * Codegen import improvements * fix broken test * fix(codegen): use lowercase module name for object::ID check The module name in the type tag is 'object' (lowercase), not 'Object'. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude Opus 4.5 <[email protected]> * chore: regenerate codegen output 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude Opus 4.5 <[email protected]> * chore: sync branch histories * update a few examples * [bcs] add map sorting (#747) * make packages esm only (#749) * convert to tsdown and esm only packages * fix deps * cleanup dapp-kit-next build config * update crypto deps * upgrade deepbook to sdk v2 (#731) * upgrade deepbook to sdk v2 * pr feedback * fix some todos * normalize some constants (#753) * remove old subpath package.json files (#752) * export grpc types (#751) * expose core methods on grpc and graphql (#750) * expose core methods on grpc and graphql clients * update examples and docs * accept TransactionResult in waitForTransaction (#746) * remove nesting of uid fields (#745) * fix lint * fix build issue * Remove data fetching helpers from zkSend SDK (#676) * Migrate suins to sdk-v2 (#743) * Update zksend sdk (#769) Update zklogin sdk * update executors to work with core APIs (#744) * Add clever error support (#754) * Add clever error support * pr feedback * Add withdrawal inputs (#768) * fix any * fix rebase * kiosk refactor (#740) * wip - kiosk refactor * transaction building cleanup * add changeset and migration * update schemas * remove entire balance option (#779) * use peer deps for @mysten/sui (#781) * use network instead of env in deepbook sdk (#783) * Improve migration docs (#772) * Improve migration docs * PR feedback * fix lint * fix seal test timeouts * update deps * Add getter to codegen bcs types (#797) * Add address balance support to CoinWithBalance and core API (#773) * Add address balance support to CoinWithBalance and core API * Add json rpc address balances * change expiration default * Add transction resolution for graphql * add graphql address balances * fix rebase * Support address balacnes in executors (#804) * disable gas selection on simulate * dapp-kit css migration docs * docs verification feedback * esm migration doc * fix version bumps * fix bumps for wallet-sdk and wallet-standard * fix lockfile * fix lint * update deps * Add banner * fix audit * Add core include json option (#808) * update seal tests --------- Co-authored-by: Michael Hayes <[email protected]> Co-authored-by: Greg Wróbel <[email protected]> Co-authored-by: William Robertson <[email protected]> Co-authored-by: Claude <[email protected]> Co-authored-by: Jordan Gensler <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description
Describe the changes or additions included in this PR.
Test plan
How did you test the new or updated feature?
AI Assistance Notice