chore: add react export with context and provider by tomiir · Pull Request #4703 · reown-com/appkit

tomiir · 2025-07-24T08:26:55Z

Description

Adds /react export to universal-connector with a React context and provider serving the connector

Type of change

Chore (non-breaking change that addresses non-functional tasks, maintenance, or code quality improvements)
Bug fix (non-breaking change which fixes an issue)
New feature (non-breaking change which adds functionality)
Breaking change (fix or feature that would cause existing functionality to not work as expected)

Associated Issues

Closes APKT-3375

Checklist

Code in this PR is covered by automated tests (Unit tests, E2E tests)
My changes generate no new warnings
I have reviewed my own code
I have filled out all required sections
I have tested my changes on the preview link
Approver of this PR confirms that the changes are tested on the preview link

linear · 2025-07-24T08:26:58Z

APKT-3375 Single AppKit Core + Universal Provider React Component

changeset-bot · 2025-07-24T08:26:59Z

🦋 Changeset detected

Latest commit: eac5030

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 24 packages

Name	Type
@reown/appkit-universal-connector	Patch
@reown/appkit	Patch
@reown/appkit-adapter-bitcoin	Patch
@reown/appkit-adapter-ethers	Patch
@reown/appkit-adapter-ethers5	Patch
@reown/appkit-adapter-solana	Patch
@reown/appkit-adapter-wagmi	Patch
@reown/appkit-common	Patch
@reown/appkit-ui	Patch
@reown/appkit-controllers	Patch
@reown/appkit-core	Patch
@reown/appkit-utils	Patch
@reown/appkit-scaffold-ui	Patch
@reown/appkit-polyfills	Patch
@reown/appkit-wallet	Patch
@reown/appkit-siwe	Patch
@reown/appkit-siwx	Patch
@reown/appkit-wallet-button	Patch
@reown/appkit-cdn	Patch
@reown/appkit-experimental	Patch
@reown/appkit-testing	Patch
@reown/appkit-cli	Patch
@reown/appkit-pay	Patch
@reown/appkit-codemod	Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

vercel · 2025-07-24T08:27:01Z

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name	Status	Preview	Comments	Updated (UTC)
appkit-basic-html	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Jul 25, 2025 8:21am
appkit-demo	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Jul 25, 2025 8:21am
appkit-gallery	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Jul 25, 2025 8:21am
appkit-laboratory	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Jul 25, 2025 8:21am

10 Skipped Deployments

Name	Status	Updated (UTC)
appkit-basic-example	⬜️ Ignored (Inspect)	Jul 25, 2025 8:21am
appkit-basic-sign-client-example	⬜️ Ignored (Inspect)	Jul 25, 2025 8:21am
appkit-basic-up-example	⬜️ Ignored (Inspect)	Jul 25, 2025 8:21am
appkit-ethers5-bera	⬜️ Ignored (Inspect)	Jul 25, 2025 8:21am
appkit-nansen-demo	⬜️ Ignored (Inspect)	Jul 25, 2025 8:21am
appkit-vue-solana	⬜️ Ignored (Inspect)	Jul 25, 2025 8:21am
appkit-wagmi-cdn-example	⬜️ Ignored (Inspect)	Jul 25, 2025 8:21am
ethereum-provider-wagmi-example	⬜️ Ignored (Inspect)	Jul 25, 2025 8:21am
next-wagmi-solana-bitcoin-example	⬜️ Ignored (Inspect)	Jul 25, 2025 8:21am
vue-wagmi-example	⬜️ Ignored (Inspect)	Jul 25, 2025 8:21am

cursor · 2025-07-24T08:28:52Z

packages/universal-connector/exports/react.tsx

+    memoizeCreateUniversalConnector(props)
+      .then(setConnector)
+      .then(() => setIsReady(true))
+  }, [props])


Bug: UniversalConnectorProvider: Unnecessary Re-renders and Error Handling

The useEffect in UniversalConnectorProvider uses [props] as a dependency. Due to ...props destructuring, props is a new object reference on every render, causing the effect to run unnecessarily and potentially re-initialize the UniversalConnector. Additionally, the useEffect lacks error handling for the memoizeCreateUniversalConnector promise. If initialization fails, the component will remain in a perpetual loading state (isReady stays false) without error indication or recovery.

Locations (1)

packages/universal-connector/exports/react.tsx#L50-L55

Fix in Cursor • Fix in Web

cursor · 2025-07-24T08:28:53Z

packages/universal-connector/exports/react.tsx

+  }
+
+  return universalConnector
+}


Bug: Memoization Ignores Config Changes

The memoizeCreateUniversalConnector function uses a global cache and only initializes the connector once, ignoring subsequent config changes. Consequently, if called with different configurations (e.g., from multiple UniversalConnectorProvider instances or updated props), it returns the connector initialized with the first config, leading to outdated or incorrect configuration.

Locations (1)

packages/universal-connector/exports/react.tsx#L31-L38

Fix in Cursor • Fix in Web

github-actions · 2025-07-24T08:32:17Z

Coverage Report

Status	Category	Percentage	Covered / Total
🔵	Lines	77.57%	31781 / 40968
🔵	Statements	77.57%	31781 / 40968
🔵	Functions	69%	2623 / 3801
🔵	Branches	84.39%	6694 / 7932

File Coverage

No changed files found.

Generated in workflow #13763 for commit eac5030 by the Vitest Coverage Report Action

cloudflare-workers-and-pages · 2025-07-25T08:16:12Z

Deploying with Cloudflare Workers

The latest updates on your project. Learn more about integrating Git with Workers.

Status	Name	Latest Commit	Updated (UTC)
🔵 In progress View logs	pay-test-exchange	`c4563c6`	Jul 25 2025, 08:16 AM

packages/universal-connector/exports/react.tsx

chore: add react export with context and provider

d706e2a

cursor bot reviewed Jul 24, 2025

View reviewed changes

vercel bot deployed to Preview – appkit-demo July 24, 2025 08:32 View deployment

vercel bot deployed to Preview – appkit-gallery July 24, 2025 08:33 View deployment

vercel bot deployed to Preview – appkit-basic-html July 24, 2025 08:33 View deployment

vercel bot deployed to Preview – appkit-laboratory July 24, 2025 08:35 View deployment

chore: refine export

cef6289

vercel bot deployed to Preview – appkit-basic-html July 24, 2025 08:47 View deployment

vercel bot deployed to Preview – appkit-gallery July 24, 2025 08:47 View deployment

vercel bot deployed to Preview – appkit-demo July 24, 2025 08:49 View deployment

vercel bot deployed to Preview – appkit-laboratory July 24, 2025 08:51 View deployment

tomiir requested a review from a team July 24, 2025 09:26

Merge branch 'main' into feat/universal-connector-react

c4563c6

tomiir commented Jul 25, 2025

View reviewed changes

packages/universal-connector/exports/react.tsx Outdated Show resolved Hide resolved

Update packages/universal-connector/exports/react.tsx

eac5030

vercel bot deployed to Preview – appkit-basic-html July 25, 2025 08:18 View deployment

vercel bot deployed to Preview – appkit-demo July 25, 2025 08:20 View deployment

0xmkh reviewed Jul 25, 2025

View reviewed changes

packages/universal-connector/exports/react.tsx Show resolved Hide resolved

vercel bot deployed to Preview – appkit-laboratory July 25, 2025 08:21 View deployment

0xmkh reviewed Jul 25, 2025

View reviewed changes

packages/universal-connector/exports/react.tsx Show resolved Hide resolved

vercel bot deployed to Preview – appkit-gallery July 25, 2025 08:21 View deployment

0xmkh approved these changes Jul 25, 2025

View reviewed changes

tomiir merged commit 3ad29e2 into main Jul 25, 2025
59 of 60 checks passed

tomiir deleted the feat/universal-connector-react branch July 25, 2025 09:19

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

chore: add react export with context and provider#4703

chore: add react export with context and provider#4703
tomiir merged 4 commits intomainfrom
feat/universal-connector-react

tomiir commented Jul 24, 2025

Uh oh!

linear bot commented Jul 24, 2025

Uh oh!

changeset-bot bot commented Jul 24, 2025 •

edited

Loading

Uh oh!

vercel bot commented Jul 24, 2025 •

edited

Loading

Uh oh!

cursor bot Jul 24, 2025

Uh oh!

cursor bot Jul 24, 2025

Uh oh!

github-actions bot commented Jul 24, 2025 •

edited

Loading

Uh oh!

cloudflare-workers-and-pages bot commented Jul 25, 2025

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

Conversation

tomiir commented Jul 24, 2025

Description

Type of change

Associated Issues

Checklist

Uh oh!

linear bot commented Jul 24, 2025

Uh oh!

changeset-bot bot commented Jul 24, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

🦋 Changeset detected

Uh oh!

vercel bot commented Jul 24, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

cursor bot Jul 24, 2025

Choose a reason for hiding this comment

Bug: UniversalConnectorProvider: Unnecessary Re-renders and Error Handling

Uh oh!

cursor bot Jul 24, 2025

Choose a reason for hiding this comment

Bug: Memoization Ignores Config Changes

Uh oh!

github-actions bot commented Jul 24, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Coverage Report

Uh oh!

cloudflare-workers-and-pages bot commented Jul 25, 2025

Deploying with Cloudflare Workers

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

changeset-bot bot commented Jul 24, 2025 •

edited

Loading

vercel bot commented Jul 24, 2025 •

edited

Loading

github-actions bot commented Jul 24, 2025 •

edited

Loading