docs: update AppKit terminology to Reown across documentation#738
Open
devin-ai-integration[bot] wants to merge 12 commits intomainfrom
Open
docs: update AppKit terminology to Reown across documentation#738devin-ai-integration[bot] wants to merge 12 commits intomainfrom
devin-ai-integration[bot] wants to merge 12 commits intomainfrom
Conversation
Co-Authored-By: Jem Conlon <jem@walletconnect.com>
Contributor
Author
🤖 Devin AI EngineerI'll be helping with this pull request! Here's what you should know: ✅ I will automatically:
Note: I can only respond to comments from users who have write access to this repository. ⚙️ Control Options:
|
Co-Authored-By: Jem Conlon <jem@walletconnect.com>
…ect-verb agreement) Co-Authored-By: Jem Conlon <jem@walletconnect.com>
Co-Authored-By: Jem Conlon <jem@walletconnect.com>
Co-Authored-By: Jem Conlon <jem@walletconnect.com>
…, and SDK behavior Co-Authored-By: Jem Conlon <jem@walletconnect.com>
…erences, and remaining setup/version terminology Co-Authored-By: Jem Conlon <jem@walletconnect.com>
Jem-reown
reviewed
Feb 12, 2026
| ### Reown | ||
|
|
||
| The following is a **partial CSP** that covers WalletConnect's libraries and services for [AppKit](https://docs.walletconnect.com/appkit/overview). Note that **you may need to define additional sources based on your application's requirements**. | ||
| The following is a **partial CSP** that covers WalletConnect's libraries and services for [Reown](https://docs.walletconnect.com/appkit/overview). Note that **you may need to define additional sources based on your application's requirements**. |
Contributor
There was a problem hiding this comment.
Suggested change
| The following is a **partial CSP** that covers WalletConnect's libraries and services for [Reown](https://docs.walletconnect.com/appkit/overview). Note that **you may need to define additional sources based on your application's requirements**. | |
| The following is a **partial CSP** that covers WalletConnect's libraries and services for [Reown Appkit](https://docs.walletconnect.com/appkit/overview). Note that **you may need to define additional sources based on your application's requirements**. |
Jem-reown
reviewed
Feb 12, 2026
|
|
||
| ### Architecture and Custody | ||
| AppKit Embedded wallets enable applications to provide end-users with a quick onboarding experience by provisioning a non-custodial wallet through social login or email wallets. While the implementation varies per network, Reown relies on a SOC 2 Type 2 compliant third-party vendor for key management. The vendor's [security overview](https://magic.link/docs/home/security/product-security) and [list of audits](https://magic.link/docs/home/security/security-compliance) are available in their documentation. | ||
| Embedded wallets enable applications to provide end-users with a quick onboarding experience by provisioning a non-custodial wallet through social login or email wallets. While the implementation varies per network, Reown relies on a SOC 2 Type 2 compliant third-party vendor for key management. The vendor's [security overview](https://magic.link/docs/home/security/product-security) and [list of audits](https://magic.link/docs/home/security/security-compliance) are available in their documentation. |
Contributor
There was a problem hiding this comment.
Suggested change
| Embedded wallets enable applications to provide end-users with a quick onboarding experience by provisioning a non-custodial wallet through social login or email wallets. While the implementation varies per network, Reown relies on a SOC 2 Type 2 compliant third-party vendor for key management. The vendor's [security overview](https://magic.link/docs/home/security/product-security) and [list of audits](https://magic.link/docs/home/security/security-compliance) are available in their documentation. | |
| Embedded wallets enable applications to provide end-users with a quick onboarding experience by provisioning a non-custodial wallet through social login or email wallets. While the implementation varies per network, Reown Appkit relies on a SOC 2 Type 2 compliant third-party vendor for key management. The vendor's [security overview](https://magic.link/docs/home/security/product-security) and [list of audits](https://magic.link/docs/home/security/security-compliance) are available in their documentation. |
Jem-reown
reviewed
Feb 12, 2026
|
|
||
| ### Audits | ||
| Both the key management vendors and Smart Account vendors have undergone multiple audits (see links above). The integration of the key management provider into AppKit has been audited by Halborn. The complete audit report is available [here](https://drive.google.com/file/d/1LQ6BkcI4PHs_FKAzpCRMqwD--rq6uOy6/view). | ||
| Both the key management vendors and Smart Account vendors have undergone multiple audits (see links above). The integration of the key management provider into Reown has been audited by Halborn. The complete audit report is available [here](https://drive.google.com/file/d/1LQ6BkcI4PHs_FKAzpCRMqwD--rq6uOy6/view). |
Contributor
There was a problem hiding this comment.
Suggested change
| Both the key management vendors and Smart Account vendors have undergone multiple audits (see links above). The integration of the key management provider into Reown has been audited by Halborn. The complete audit report is available [here](https://drive.google.com/file/d/1LQ6BkcI4PHs_FKAzpCRMqwD--rq6uOy6/view). | |
| Both the key management vendors and Smart Account vendors have undergone multiple audits (see links above). The integration of the key management provider into Reown Appkit has been audited by Halborn. The complete audit report is available [here](https://drive.google.com/file/d/1LQ6BkcI4PHs_FKAzpCRMqwD--rq6uOy6/view). |
Jem-reown
reviewed
Feb 12, 2026
| ## Third-Party Reviews | ||
|
|
||
| The security infrastructure of Reown has undergone multiple rounds of audits by independent security auditing firms, including Trail of Bits, Halborn, and Spearbit. These audits cover both AppKit and WalletKit, along with a comprehensive company-wide Threat Model. | ||
| The security infrastructure of Reown has undergone multiple rounds of audits by independent security auditing firms, including Trail of Bits, Halborn, and Spearbit. These audits cover both Reown and WalletKit, along with a comprehensive company-wide Threat Model. |
Contributor
There was a problem hiding this comment.
Suggested change
| The security infrastructure of Reown has undergone multiple rounds of audits by independent security auditing firms, including Trail of Bits, Halborn, and Spearbit. These audits cover both Reown and WalletKit, along with a comprehensive company-wide Threat Model. | |
| Reown's security infrastructure of Reown has undergone multiple rounds of audits by independent security auditing firms, including Trail of Bits, Halborn, and Spearbit. These audits cover both Reown Appkit and WalletConnect's WalletKit, along with a comprehensive company-wide Threat Model. |
Jem-reown
reviewed
Feb 12, 2026
|
|
||
| - [Awesome WalletConnect](https://github.com/WalletConnect/awesome-walletconnect) - Community-curated collection of WalletConnect-enabled wallets, libraries, and tools. | ||
| - [AppKit Laboratory](https://appkit-lab.reown.com/) - A place to test your wallet integrations against various setups of AppKit. | ||
| - [Reown Laboratory](https://appkit-lab.reown.com/) - A place to test your wallet integrations against various setups of Reown. |
Contributor
There was a problem hiding this comment.
Suggested change
| - [Reown Laboratory](https://appkit-lab.reown.com/) - A place to test your wallet integrations against various setups of Reown. | |
| - [Reown Laboratory](https://appkit-lab.reown.com/) - A place to test your wallet integrations against various setups of Reown Appkit. |
Jem-reown
reviewed
Feb 12, 2026
appkit/android/core/actions.mdx
Outdated
| ### Get Active Account | ||
|
|
||
| Returns the current active account connected via AppKit | ||
| Returns the current active account connected via Reown |
Contributor
There was a problem hiding this comment.
Suggested change
| Returns the current active account connected via Reown | |
| Returns the current active account connected via the Reown SDK |
Jem-reown
reviewed
Feb 12, 2026
appkit/android/core/components.mdx
Outdated
| ### Reown State | ||
|
|
||
| AppKitState is an object that ensures communication between your application and the state of the AppKit. | ||
| AppKitState is an object that ensures communication between your application and the state of the Reown. |
Contributor
There was a problem hiding this comment.
Suggested change
| AppKitState is an object that ensures communication between your application and the state of the Reown. | |
| AppKitState is an object that ensures communication between your application and the state of your Reown Appkit instance. |
Jem-reown
reviewed
Feb 12, 2026
appkit/android/core/components.mdx
Outdated
| ``` | ||
|
|
||
| returns `StateFlow<Boolean>` whose value depends on the active session in AppKit | ||
| returns `StateFlow<Boolean>` whose value depends on the active session in Reown |
Contributor
There was a problem hiding this comment.
Suggested change
| returns `StateFlow<Boolean>` whose value depends on the active session in Reown | |
| returns `StateFlow<Boolean>` whose value depends on the active session in your Reown Appkit instance. |
Jem-reown
reviewed
Feb 12, 2026
| ## Configure your AppKit Client | ||
|
|
||
| To integrate SIWE with AppKit, you need to configure your AppKit client using `Modal.Model.AuthPayloadParams`, which are required to create a SIWE message for the user to sign: | ||
| To integrate SIWE with Reown, you need to configure your AppKit client using `Modal.Model.AuthPayloadParams`, which are required to create a SIWE message for the user to sign: |
Contributor
There was a problem hiding this comment.
Suggested change
| To integrate SIWE with Reown, you need to configure your AppKit client using `Modal.Model.AuthPayloadParams`, which are required to create a SIWE message for the user to sign: | |
| To integrate SIWE, you must configure your Reown AppKit client using `Modal.Model.AuthPayloadParams`. This is required to create a SIWE message for the user to sign: |
Jem-reown
reviewed
Feb 12, 2026
| ## Getting Started | ||
|
|
||
| **SIWX** works as a plugin system for AppKit and you are going to add the plugin in the AppKit configuration. There are some ways to implement the **SIWX** feature: | ||
| **SIWX** works as a plugin system for Reown and you are going to add the plugin in the Reown AppKit configuration. There are some ways to implement the **SIWX** feature: |
Contributor
There was a problem hiding this comment.
Suggested change
| **SIWX** works as a plugin system for Reown and you are going to add the plugin in the Reown AppKit configuration. There are some ways to implement the **SIWX** feature: | |
| **SIWX** works as a plugin, which can be added in your Reown AppKit configuration. There are several ways to implement the **SIWX** feature: |
Jem-reown
reviewed
Feb 12, 2026
| you can configure the `siwx` parameters expected to receive. This is what allows you to customize the feature to suit your specific requirements. | ||
|
|
||
| The defined interface must follow specific rules to make sure that AppKit can interact with it correctly. Read more about how to have your [Custom Implementation](./siwx-custom). | ||
| The defined interface must follow specific rules to make sure that Reown can interact with it correctly. Read more about how to have your [Custom Implementation](./siwx-custom). |
Contributor
There was a problem hiding this comment.
Suggested change
| The defined interface must follow specific rules to make sure that Reown can interact with it correctly. Read more about how to have your [Custom Implementation](./siwx-custom). | |
| The defined interface must follow specific rules to make sure that the SDK can interact with it correctly. Read more about how to have your [Custom Implementation](./siwx-custom). |
Jem-reown
reviewed
Feb 12, 2026
| <CardGroup cols={2}> | ||
| <Card title="React" icon="react" href="/appkit/react/core/siwe"> | ||
| Get started with AppKit in React. | ||
| Get started with Reown AppKit in React. |
Contributor
There was a problem hiding this comment.
Suggested change
| Get started with Reown AppKit in React. | |
| Get started with Reown in React. |
Jem-reown
reviewed
Feb 12, 2026
|
|
||
| <Card title="Next.js" icon="square-n" href="/appkit/next/core/siwe"> | ||
| Get started with AppKit in Next.js. | ||
| Get started with Reown AppKit in Next.js. |
Contributor
There was a problem hiding this comment.
Suggested change
| Get started with Reown AppKit in Next.js. | |
| Get started with Reown in Next.js. |
Jem-reown
reviewed
Feb 12, 2026
|
|
||
| <Card title="Vue" icon="vuejs" href="/appkit/vue/core/siwe"> | ||
| Get started with AppKit in Vue. | ||
| Get started with Reown AppKit in Vue. |
Contributor
There was a problem hiding this comment.
Suggested change
| Get started with Reown AppKit in Vue. | |
| Get started with Reown in Vue. |
Jem-reown
reviewed
Feb 12, 2026
|
|
||
| <Card title="JavaScript" icon="js" href="/appkit/javascript/core/siwe"> | ||
| Get started with AppKit in JavaScript. | ||
| Get started with Reown AppKit in JavaScript. |
Contributor
There was a problem hiding this comment.
Suggested change
| Get started with Reown AppKit in JavaScript. | |
| Get started with Reown in JavaScript. |
Jem-reown
reviewed
Feb 13, 2026
| - Using cookies is completely optional and by default Wagmi will use `localStorage` instead if the `storage` param is not defined. | ||
| - The `ssr` flag will delay the hydration of Wagmi's store to avoid hydration mismatch errors. | ||
| - AppKit doesn't fully support the `ssr` flag. | ||
| - Reown doesn't fully support the `ssr` flag. |
Contributor
There was a problem hiding this comment.
Suggested change
| - Reown doesn't fully support the `ssr` flag. | |
| - Reown AppKit doesn't fully support the `ssr` flag. |
Jem-reown
reviewed
Feb 13, 2026
| ### Context Provider | ||
|
|
||
| Let's create now a context provider that will wrap our application and initialized AppKit (`createAppKit` needs to be called inside a Next Client Component file). | ||
| Let's create now a context provider that will wrap our application and initialized Reown (`createAppKit` needs to be called inside a Next Client Component file). |
Contributor
There was a problem hiding this comment.
Suggested change
| Let's create now a context provider that will wrap our application and initialized Reown (`createAppKit` needs to be called inside a Next Client Component file). | |
| Now we'll create a context provider that will wrap our application and initialized Reown AppKit instance (`createAppKit` needs to be called inside a Next Client Component file). |
Jem-reown
reviewed
Feb 13, 2026
| ``` | ||
|
|
||
| Learn more about the AppKit web components [here](/appkit/next/core/components) | ||
| Learn more about the Reown web components [here](/appkit/next/core/components) |
Contributor
There was a problem hiding this comment.
Suggested change
| Learn more about the Reown web components [here](/appkit/next/core/components) | |
| Learn more about Reown's web components [here](/appkit/next/core/components) |
Jem-reown
reviewed
Feb 13, 2026
| ``` | ||
|
|
||
| Learn more about the AppKit composables [here](/appkit/nuxt/core/composables) | ||
| Learn more about the Reown composables [here](/appkit/nuxt/core/composables) |
Contributor
There was a problem hiding this comment.
Suggested change
| Learn more about the Reown composables [here](/appkit/nuxt/core/composables) | |
| Learn more about Reown's composables [here](/appkit/nuxt/core/composables) |
Jem-reown
reviewed
Feb 13, 2026
| ``` | ||
|
|
||
| Learn more about the AppKit web components [here](/appkit/react/core/components) | ||
| Learn more about the Reown web components [here](/appkit/react/core/components) |
Contributor
There was a problem hiding this comment.
Suggested change
| Learn more about the Reown web components [here](/appkit/react/core/components) | |
| Learn more about Reown's web components [here](/appkit/react/core/components) |
Jem-reown
reviewed
Feb 13, 2026
| @@ -1,4 +1,4 @@ | |||
| The theme for the AppKit integration in your dApp can be fully customized. | |||
| The theme for the Reown integration in your dApp can be fully customized. | |||
Contributor
There was a problem hiding this comment.
Suggested change
| The theme for the Reown integration in your dApp can be fully customized. | |
| You can customize the theme of your Reown AppKit integration fully. |
Jem-reown
reviewed
Feb 13, 2026
|
|
||
| <Note> | ||
| The `browser` check ensures AppKit is only initialized in the browser environment, which is important for SvelteKit's SSR compatibility. | ||
| The `browser` check ensures Reown is only initialized in the browser environment, which is important for SvelteKit's SSR compatibility. |
Contributor
There was a problem hiding this comment.
Suggested change
| The `browser` check ensures Reown is only initialized in the browser environment, which is important for SvelteKit's SSR compatibility. | |
| The `browser` check ensures Reown AppKit is only initialized in the browser environment, which is important for SvelteKit's SSR compatibility. |
Jem-reown
reviewed
Feb 13, 2026
|
|
||
| <Note> | ||
| The `browser` check ensures AppKit is only initialized in the browser environment, which is important for SvelteKit's SSR compatibility. | ||
| The `browser` check ensures Reown is only initialized in the browser environment, which is important for SvelteKit's SSR compatibility. |
Contributor
There was a problem hiding this comment.
Suggested change
| The `browser` check ensures Reown is only initialized in the browser environment, which is important for SvelteKit's SSR compatibility. | |
| The `browser` check ensures Reown AppKit is only initialized in the browser environment, which is important for SvelteKit's SSR compatibility. |
Jem-reown
reviewed
Feb 13, 2026
|
|
||
| <Note> | ||
| The `browser` check ensures AppKit is only initialized in the browser environment, which is important for SvelteKit's SSR compatibility. | ||
| The `browser` check ensures Reown is only initialized in the browser environment, which is important for SvelteKit's SSR compatibility. |
Contributor
There was a problem hiding this comment.
Suggested change
| The `browser` check ensures Reown is only initialized in the browser environment, which is important for SvelteKit's SSR compatibility. | |
| The `browser` check ensures Reown AppKit is only initialized in the browser environment, which is important for SvelteKit's SSR compatibility. |
Jem-reown
reviewed
Feb 13, 2026
|
|
||
| <Note> | ||
| The `browser` check ensures AppKit is only initialized in the browser environment, which is important for SvelteKit's SSR compatibility. | ||
| The `browser` check ensures Reown is only initialized in the browser environment, which is important for SvelteKit's SSR compatibility. |
Contributor
There was a problem hiding this comment.
Suggested change
| The `browser` check ensures Reown is only initialized in the browser environment, which is important for SvelteKit's SSR compatibility. | |
| The `browser` check ensures Reown AppKit is only initialized in the browser environment, which is important for SvelteKit's SSR compatibility. |
Jem-reown
reviewed
Feb 13, 2026
|
|
||
| <Note> | ||
| The `browser` check ensures AppKit is only initialized in the browser environment, which is important for SvelteKit's SSR compatibility. | ||
| The `browser` check ensures Reown is only initialized in the browser environment, which is important for SvelteKit's SSR compatibility. |
Contributor
There was a problem hiding this comment.
Suggested change
| The `browser` check ensures Reown is only initialized in the browser environment, which is important for SvelteKit's SSR compatibility. | |
| The `browser` check ensures Reown AppKit is only initialized in the browser environment, which is important for SvelteKit's SSR compatibility. |
Jem-reown
reviewed
Feb 13, 2026
| ``` | ||
|
|
||
| Learn more about the AppKit web components [here](/appkit/vue/core/components) | ||
| Learn more about the Reown web components [here](/appkit/vue/core/components) |
Contributor
There was a problem hiding this comment.
Suggested change
| Learn more about the Reown web components [here](/appkit/vue/core/components) | |
| Learn more about Reown's web components [here](/appkit/vue/core/components) |
Jem-reown
reviewed
Feb 13, 2026
| ``` | ||
|
|
||
| Learn more about the AppKit composables [here](/appkit/vue/core/composables) | ||
| Learn more about the Reown composables [here](/appkit/vue/core/composables) |
Contributor
There was a problem hiding this comment.
Suggested change
| Learn more about the Reown composables [here](/appkit/vue/core/composables) | |
| Learn more about Reown's composables [here](/appkit/vue/core/composables) |
Jem-reown
reviewed
Feb 13, 2026
| ``` | ||
|
|
||
| Learn more about the Appkit web components [here](/appkit/vue/core/components) | ||
| Learn more about the Reown web components [here](/appkit/vue/core/components) |
Contributor
There was a problem hiding this comment.
Suggested change
| Learn more about the Reown web components [here](/appkit/vue/core/components) | |
| Learn more about Reown's web components [here](/appkit/vue/core/components) |
Jem-reown
reviewed
Feb 13, 2026
| ``` | ||
|
|
||
| Learn more about the AppKit composables [here](/appkit/vue/core/composables) | ||
| Learn more about the Reown composables [here](/appkit/vue/core/composables) |
Contributor
There was a problem hiding this comment.
Suggested change
| Learn more about the Reown composables [here](/appkit/vue/core/composables) | |
| Learn more about Reown's composables [here](/appkit/vue/core/composables) |
Jem-reown
reviewed
Feb 13, 2026
| ``` | ||
|
|
||
| Learn more about the AppKit web components [here](/appkit/vue/core/components) | ||
| Learn more about the Reown web components [here](/appkit/vue/core/components) |
Contributor
There was a problem hiding this comment.
Suggested change
| Learn more about the Reown web components [here](/appkit/vue/core/components) | |
| Learn more about Reown's web components [here](/appkit/vue/core/components) |
Jem-reown
reviewed
Feb 13, 2026
| Learn more about the Blockchain API [here](./blockchain-api) | ||
|
|
||
| ### AppKit | ||
| ### Reown |
Contributor
There was a problem hiding this comment.
Suggested change
| ### Reown | |
| ### Your App Performance |
Jem-reown
reviewed
Feb 13, 2026
| #### Avg. Daily Visitors | ||
|
|
||
| Indicates the daily average of unique visitors to your app’s AppKit. | ||
| Indicates the daily average of unique visitors to your app’s Reown. |
Contributor
There was a problem hiding this comment.
Suggested change
| Indicates the daily average of unique visitors to your app’s Reown. | |
| Indicates the daily average of unique visitors to your app. |
Jem-reown
reviewed
Feb 13, 2026
| #### All Events | ||
|
|
||
| This table and chart shows the count of various events that are triggered as the users interact with AppKit. | ||
| This table and chart shows the count of various events that are triggered as the users interact with the modal. |
Contributor
There was a problem hiding this comment.
Suggested change
| This table and chart shows the count of various events that are triggered as the users interact with the modal. | |
| This table and chart shows the count of various events triggered by your users. |
Jem-reown
reviewed
Feb 13, 2026
| #### Visitors | ||
|
|
||
| Shows the daily trend of unique visitors to your app’s AppKit. | ||
| Shows the daily trend of unique visitors to your app’s Reown. |
Contributor
There was a problem hiding this comment.
Suggested change
| Shows the daily trend of unique visitors to your app’s Reown. | |
| Shows the daily trend of unique visitors to your app. |
Member
|
@claude review |
|
Claude finished @bkrem's task in 2m 23s —— View job Code Review Complete🚨 Critical Issues FoundFound 5 issue(s)Issue 1: PR exceeds recommended sizeID: pr-size-limit-exceeded-7f3a Context:
Recommendation: While this PR is nearly complete and has undergone multiple review rounds, future terminology updates should be split into focused PRs:
Issue 2: Inconsistent capitalization - "Appkit" vs "AppKit"ID: security-csp-appkit-case-f3a2 Context:
Recommendation: Clarify the official capitalization standard. Current PR uses "Reown AppKit" (capital K) throughout. If "Reown Appkit" is preferred, update all 100+ files. The review suggestion shows: [Reown Appkit](https://docs.walletconnect.com/appkit/overview)But current code uses Issue 3: FAQ answer doesn't match questionID: payments-faq-monitor-f8d1 Context:
Recommendation: Review comment #90 suggests simplifying to availability statement, but the answer still doesn't address monitoring. Should be: <Accordion>
Yes, transaction monitoring is available through [Analytics dashboard/API endpoint]. You can track transaction status, confirmations, and receive webhooks for state changes.
</Accordion>Or remove the FAQ item if monitoring isn't supported yet. Issue 4: Incorrect GitHub link referenceID: ios-usage-wrong-repo-a4b7 Context:
Current code: Reown AppKit uses the Sign SDK to implement the WalletConnect Protocol. Most Sign SDK functionality is exposed through the [`AppKit`](https://github.com/reown-com/reown-dotnet/blob/main/src/Reown.AppKit.Unity/Runtime/AppKit.cs#L17-L45) class, the main entry point for developers.Recommendation: Fix link to point to iOS/Swift repository: Reown AppKit uses the Sign SDK to implement the WalletConnect Protocol. Most Sign SDK functionality is exposed through the [`AppKit`](https://github.com/reown-com/reown-swift/tree/develop/Sources/ReownAppKit) class, the main entry point for developers.Issue 5: Unapplied review suggestions from Jem-reownID: review-suggestions-pending-9c2f Context:
Files with pending suggestions (sample):
Recommendation: Apply remaining review suggestions systematically:
✅ Positive ObservationsTerminology consistency: The core replacement rules are well-executed:
Documentation quality:
No automated violations detected:
Recommended Actions
|
![github-actions[bot]](https://avatars.githubusercontent.com/in/15368?s=60&v=4){kind=small}
| ### Reown | ||
|
|
||
| The following is a **partial CSP** that covers WalletConnect's libraries and services for [AppKit](https://docs.walletconnect.com/appkit/overview). Note that **you may need to define additional sources based on your application's requirements**. | ||
| The following is a **partial CSP** that covers WalletConnect's libraries and services for [Reown AppKit](https://docs.walletconnect.com/appkit/overview). Note that **you may need to define additional sources based on your application's requirements**. |
There was a problem hiding this comment.
🤖 Auto Review Issue: Inconsistent capitalization - "Appkit" vs "AppKit"
Severity: MEDIUM
Category: consistency
Tool: Claude Auto Review
Context:
- Pattern: Line 13 uses "Reown AppKit" (capital K), but review comment Home UI #1 from Jem-reown suggests "Reown Appkit" (lowercase k)
- Risk: Brand inconsistency across documentation
- Impact: Confused developers, inconsistent brand presentation
- Trigger: Any reference to the product name in documentation
Recommendation: Clarify the official capitalization standard. Current PR uses "Reown AppKit" (capital K) throughout. If "Reown Appkit" is preferred, update all 100+ files. The review suggestion shows:
[Reown Appkit](https://docs.walletconnect.com/appkit/overview)But current code uses Reown AppKit everywhere else. Standardize one way.
| @@ -182,19 +182,19 @@ This feature is currently in early access. If you are **interested in getting ea | |||
| </Accordion> | |||
|
|
|||
| <Accordion title="Can we monitor transactions in real time?"> | |||
There was a problem hiding this comment.
🤖 Auto Review Issue: FAQ answer doesn't match question
Severity: HIGH
Category: content_quality
Tool: Claude Auto Review
Context:
- Pattern: "Can we monitor transactions in real time?" answers with "Pay with Exchange is available as a modal and a headless flow." (lines 185, 282)
- Risk: Answer is completely unrelated to the question about transaction monitoring
- Impact: Developers cannot find information about transaction monitoring capabilities
- Trigger: Users reading FAQ to understand monitoring capabilities
Recommendation: Review comment #90 suggests simplifying to availability statement, but the answer still doesn't address monitoring. Should be:
<Accordion>
Yes, transaction monitoring is available through [Analytics dashboard/API endpoint]. You can track transaction status, confirmations, and receive webhooks for state changes.
</Accordion>Or remove the FAQ item if monitoring isn't supported yet.
| ### Reown | ||
|
|
||
| The following is a **partial CSP** that covers WalletConnect's libraries and services for [AppKit](https://docs.walletconnect.com/appkit/overview). Note that **you may need to define additional sources based on your application's requirements**. | ||
| The following is a **partial CSP** that covers WalletConnect's libraries and services for [Reown AppKit](https://docs.walletconnect.com/appkit/overview). Note that **you may need to define additional sources based on your application's requirements**. |
There was a problem hiding this comment.
🤖 Auto Review Issue: Unapplied review suggestions from Jem-reown
Severity: MEDIUM
Category: code_quality
Tool: Claude Auto Review
Context:
- Pattern: 90+ review suggestions from Jem-reown remain unapplied since Feb 12-13
- Risk: PR description claims "accepted and applied" but current code doesn't reflect all changes
- Impact: Inconsistent terminology, missed improvements, stale PR state
- Trigger: Final merge will include unresolved feedback
Recommendation: Apply remaining review suggestions systematically:
- Decide on "AppKit" vs "Appkit" capitalization (see Issue re:org ui: updating welcome page UI and general theme #2)
- Apply all Jem-reown suggestions from comments Fix wagmiAdapter spelling. Update implementation.mdx #69-111
- Verify each change matches the suggestion exactly
- Re-test preview site to ensure no broken links
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Description
Updates product terminology across the documentation, replacing "AppKit" references with "Reown" in all prose/descriptive text per the new branding guidelines. 230+ files modified.
Replacement rules applied:
Preserved unchanged:
Latest updates (since last revision) — Comments 69-111
Accepted and applied additional suggestions from Jem-reown, and propagated same changes to equivalent pages across frameworks where applicable.
Latest updates (18 files) — Comments 60-67
Accepted additional PR suggestions from Jem-reown and applied to all framework pages:
iOS Usage page:
Custom Connectors pages (JavaScript, React, Vue, Next):
Custom Networks pages (JavaScript, React, Vue, Next):
Installation pages (JavaScript, React, Vue, Next):
SIWE pages (JavaScript, React, Vue, Next, Unity):
Updates since last revision (18 files) — Comments 30-58
Accepted additional PR suggestions from Jem-reown:
Solana Adapter page:
One-Click Auth descriptions (Android, Flutter, iOS):
Features pages:
Flutter docs:
iOS docs:
SIWX page:
Requested by: Jem Conlon
Link to Devin run: https://app.devin.ai/sessions/bea6e7101b56464ba2c591bd4230f8f3
Key areas for reviewer attention
Local Testing Screenshots
Homepage with updated terminology:
![Homepage]()
Overview page:
![Overview]()
Pay with Exchange page:
![Pay with Exchange]()
Tests
Direct link to the deployed preview files