Skip to content

chore: prep for 2.0.4 release#559

Merged
ignaciosantise merged 8 commits into
mainfrom
develop
May 20, 2026
Merged

chore: prep for 2.0.4 release#559
ignaciosantise merged 8 commits into
mainfrom
develop

Conversation

@ignaciosantise
Copy link
Copy Markdown
Collaborator

This pull request introduces a new logger option to the createAppKit API, allowing developers to control the log output of @walletconnect/universal-provider by specifying a log level.

Feature: WalletConnect Logger Option

  • Added an optional logger property to AppKitConfig and ConnectorInitOptions, allowing users to control WalletConnect logging with levels like 'silent', 'error', 'info', etc. This is now accepted in createAppKit and forwarded through the connector initialization flow. [1] [2] [3] [4] [5] [6]
  • Updated the types and documentation to reflect the new logger option, including an explicit type for log levels and usage examples. [1] [2]

@changeset-bot
Copy link
Copy Markdown

changeset-bot Bot commented May 20, 2026

🦋 Changeset detected

Latest commit: f38aada

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

This PR includes changesets to release 9 packages
Name Type
@reown/appkit-react-native Patch
@reown/appkit-bitcoin-react-native Patch
@reown/appkit-coinbase-react-native Patch
@reown/appkit-common-react-native Patch
@reown/appkit-core-react-native Patch
@reown/appkit-ethers-react-native Patch
@reown/appkit-solana-react-native Patch
@reown/appkit-ui-react-native Patch
@reown/appkit-wagmi-react-native 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

@claude
Copy link
Copy Markdown

claude Bot commented May 20, 2026

Claude finished @ignaciosantise's task in 2m 19s —— View job


PR Review: chore: prep for 2.0.4 release

  • Gather context and read changed files
  • Run license compliance subagent
  • Run breaking changes subagent
  • Run data classification subagent
  • Analyze code for issues
  • Post final review

Found 1 issue(s)

Issue 1: Type indirection in AppKitConfig.logger reduces discoverability

ID: types-logger-type-indirection-a3f1
File: packages/appkit/src/types.ts:147
Severity: LOW
Category: code_quality

Context:

  • Pattern: logger?: ConnectorInitOptions['logger'] in AppKitConfig uses indexed access to reference the type rather than directly using the exported WalletConnectLoggerLevel type.
  • Risk: Consumers reading the AppKitConfig interface must trace through ConnectorInitOptions to discover the accepted log level values. Both types are already exported from @reown/appkit-common-react-native.
  • Impact: Reduced API discoverability; IDE hover/autocomplete shows string or a deep reference instead of the union literal values inline.
  • Trigger: Any developer inspecting the AppKitConfig type in their IDE or docs tooling.

Recommendation: Reference the named type directly:

// packages/appkit/src/types.ts
import {
  type WalletConnectLoggerLevel,
  // ...other imports
} from '@reown/appkit-common-react-native';

// In AppKitConfig:
logger?: WalletConnectLoggerLevel;

Fix this →

Subagent findings:

  • License compliance: No issues — newly pinned packages (@xmldom/xmldom, defu, brace-expansion) are all MIT licensed.
  • Breaking changes: No breaking changes — all new logger fields are optional and backward compatible.
  • Data classification: No sensitive data handling concerns.

@sonarqubecloud
Copy link
Copy Markdown

@ignaciosantise ignaciosantise merged commit ffd99fe into main May 20, 2026
20 checks passed
@github-actions github-actions Bot locked and limited conversation to collaborators May 20, 2026
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.

1 participant