coderabbitai[bot]
commented
2 months ago [!IMPORTANT]
Review skipped
Review was skipped as selected files did not have any reviewable changes.
💤 Files selected but had no reviewable changes (19)
* packages/andrjs/src/AndromedaClient.ts * packages/cli/.gitignore * packages/cli/package.json * packages/cli/src/config/index.ts * packages/cli/src/handlers/ado/common.ts * packages/cli/src/handlers/ado/index.ts * packages/cli/src/handlers/ado/modules.ts * packages/cli/src/handlers/bank.ts * packages/cli/src/handlers/chain.ts * packages/cli/src/handlers/env/index.ts * packages/cli/src/handlers/os/db.ts * packages/cli/src/handlers/os/index.ts * packages/cli/src/handlers/os/vfs.ts * packages/cli/src/handlers/tx.ts * packages/cli/src/handlers/user.ts * packages/cli/src/handlers/utils.ts * packages/cli/src/handlers/wallet.ts * packages/cli/src/handlers/wasm.ts * packages/cli/src/schema/SchemaPrompt.tsYou can disable this status message by setting the
reviews.review_statustofalsein the CodeRabbit configuration file.
Walkthrough
The changes include the introduction of a new configuration file for managing review processes, updates to the package.json file for script commands and dependency versions, and the addition of files for changeset management. Key modifications involve path filters for reviews, adjustments to various dependencies, and a structured approach to version tracking within the project.
Changes
| Files | Change Summary |
|---|---|
.coderabbit.yaml, .changeset/pre.json, .changeset/shaggy-carrots-serve.md |
Introduced configuration for review processes with path filters, and added changeset management structure with version tracking. |
package.json |
Updated script commands, modified rimraf path, and changed dependency versions for @changesets/cli and rimraf. Added updates to pnpm overrides for compatibility. |
Sequence Diagram(s)
sequenceDiagram
participant Developer
participant ReviewSystem
participant Codebase
Developer->>ReviewSystem: Submit changes for review
ReviewSystem->>Codebase: Apply path filters
Codebase-->>ReviewSystem: Return relevant changes
ReviewSystem-->>Developer: Provide review results🐇 "In the meadow where code does play,
New filters guide the review each day.
Dependencies updated, paths refined,
A smoother journey for all aligned!
Hops of joy in every line,
Progress blooms, oh how divine!" 🐇
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media?
SlayerAnsh
Note - Ignore adocodegen diffs as these are autogerated
ANDRJS
Cosmwasm Client
As there are multiple chains AOS is interacting with, it was hard to manage clients using their own packages specially because it all comes to signer and pubkey. Major refactor is done to
CosmWasmClient using byAndromedaClientto support multiple pubkey. With this update, custom packages for terra and archway clients were removed.Wallet Client
Wallet Client is also updated to now use mnemonic or private key and encrypt it using its own encryption method. This will help clients to use same wallet for multiple chains which was not possible with previous Client. (This feature has more impact on CLI wallet feature which we will discuss below)
Schema Client and GQL Client
Schema client was internally using env variable for fallback which was not visible cleary to library users. Now Andromeda Client needs schema url as constructor. Gql client is not part of AndromedaClient but it was also using env variable, you can update gql client now using
setGQLSdkUri.Bug Fixes
With latest updates to chains, there were some bugs with msg parsing related to store code mainly. Now with new Cosmwasm client implementation, all messages are wrapped inside there own encoder and we have implemented event parsing logic with all the fallbacks that are needed to support events from multiple chains
CLI
Wallet Command
Wallet command is refactored completely and now supports both mnemonic and private key. In contrast to previous version were you need to store your wallet for each chain, now same wallet can be used for multiple chain. Encryption method for wallet store has been updated (Any suggestion regarding other, more secure encyption method are highly welcomed).
ENV Command
One of the issues that I personally face with any wallet provider is isolation of different env. I want to use 5-6 testing wallets but don't want to accidently use these for any mainnet stuff. With new env you can isolate your keys and configs inside an env and quickly shift between them.
Chain Command
Updated localchains storage to fix all the local config related bugs. Localchains are also part of env so you can set different chains per env.
Summary by CodeRabbit
Summary by CodeRabbit
New Features
Chores
package.jsonfor improved clarity and compatibility.