Skip to content

Add wagmi as a peer dependency in evm package.json#59

Merged
GiovaniGuizzo merged 2 commits into
mainfrom
build/fix-evm-dep
Oct 29, 2025
Merged

Add wagmi as a peer dependency in evm package.json#59
GiovaniGuizzo merged 2 commits into
mainfrom
build/fix-evm-dep

Conversation

@GiovaniGuizzo

Copy link
Copy Markdown
Collaborator

Description

Moves wagmi to peer dependency in EVM package.json.
This fixes a bug with mismatch of wagmi versions.

Type of change

Please delete options that are not relevant.

  • New feature (non-breaking change which adds functionality)
  • Bug fix (non-breaking change which fixes an issue)
  • Documentation (updates documentation on the project)
  • chore (Updates on dependencies, gitignore, etc)
  • test (For updates on tests)

Copilot AI review requested due to automatic review settings October 29, 2025 19:19
@coderabbitai

coderabbitai Bot commented Oct 29, 2025

Copy link
Copy Markdown

Caution

Review failed

The pull request is closed.

Walkthrough

The packages/evm/package.json file was updated with a version bump from 0.5.0 to 0.5.1. The wagmi dependency was relocated from the top-level dependencies block to peerDependencies, joining the existing viem peer dependency. The version specification for wagmi remains "^2.18.2". No functional code changes were made.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

  • Single file modification with straightforward structural reorganization
  • Homogeneous changes limited to dependency declaration repositioning and version bump
  • No logic alterations or code impact
✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch build/fix-evm-dep

📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 4913322 and 538ec81.

⛔ Files ignored due to path filters (2)
  • .yarn/install-state.gz is excluded by !**/.yarn/**, !**/*.gz
  • yarn.lock is excluded by !**/yarn.lock, !**/*.lock
📒 Files selected for processing (1)
  • packages/evm/package.json (3 hunks)

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull Request Overview

This pull request moves the wagmi dependency from dependencies to peerDependencies in the @kiichain/kiijs-evm package and bumps the version from 0.5.0 to 0.5.1. The change also removes a massive number of transitive dependencies from the yarn.lock file (primarily related to wallet connectors, WalletConnect, MetaMask SDK, and other blockchain wallet integration libraries) that were previously pulled in by wagmi.

Key changes:

  • Version bump: Package version updated from 0.5.0 to 0.5.1
  • Dependency reorganization: wagmi moved from dependencies to peerDependencies with version constraint ^2.18.2
  • Dependency cleanup: Extensive removal of transitive dependencies from yarn.lock (approximately 500+ packages removed)

Reviewed Changes

Copilot reviewed 1 out of 3 changed files in this pull request and generated 1 comment.

File Description
packages/evm/package.json Moved wagmi from dependencies to peerDependencies and bumped version to 0.5.1
yarn.lock Removed extensive transitive dependencies that were previously pulled in by wagmi

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread packages/evm/package.json
@GiovaniGuizzo GiovaniGuizzo merged commit 8cdbc3f into main Oct 29, 2025
8 of 13 checks passed
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.

3 participants