Skip to content
This repository was archived by the owner on Oct 17, 2025. It is now read-only.

Update appkit#23

Merged
CBrather merged 3 commits into
mainfrom
update-appkit
Aug 31, 2025
Merged

Update appkit#23
CBrather merged 3 commits into
mainfrom
update-appkit

Conversation

@CBrather

Copy link
Copy Markdown
Contributor

No description provided.

Copilot AI review requested due to automatic review settings August 31, 2025 19:58

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 PR updates the appkit library from version 1.7.20 to 1.8.1 and adds corresponding mock handlers and tests for the wallet functionality.

  • Updates @reown/appkit and @reown/appkit-adapter-solana packages to version 1.8.1
  • Adds MSW mock handlers for appkit API endpoints
  • Creates comprehensive test coverage for the appkit service configuration

Reviewed Changes

Copilot reviewed 5 out of 6 changed files in this pull request and generated 2 comments.

Show a summary per file
File Description
package.json Updates appkit packages to version 1.8.1
src/mocks/server.ts Integrates new appkit handlers into MSW server setup
src/mocks/appkit.handlers.ts Adds mock handlers for walletconnect and web3modal endpoints
src/lib/services/authn/client.ts Fixes import path to use absolute path format
src/lib/services/wallet/appkit.test.ts Adds comprehensive tests for appkit service configuration
Files not reviewed (1)
  • pnpm-lock.yaml: Language not supported

Comment thread src/mocks/appkit.handlers.ts Outdated
Comment thread src/mocks/appkit.handlers.ts Outdated
@codecov

codecov Bot commented Aug 31, 2025

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@CBrather CBrather merged commit 006017d into main Aug 31, 2025
6 checks passed
@CBrather CBrather deleted the update-appkit branch August 31, 2025 20:08
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants