fix: don't set clipboard + move validation to hook

src/components/walletconnect/HeaderWidget/index.tsx

@@ -19,6 +19,7 @@ const usePrepopulatedUri = (): [string, () => void] => {
 
   const clearUri = useCallback(() => {
     setSearchParamWcUri(null)
+    // This does not clear the system clipboard, just state
     setClipboardWcUri('')
   }, [setClipboardWcUri, setSearchParamWcUri])
 

src/components/walletconnect/WcInput/index.tsx

@@ -43,9 +43,7 @@ const WcInput = ({ uri }: { uri: string }): ReactElement => {
   )
 
   useEffect(() => {
-    if (uri.startsWith('wc:')) {
-      onInput(uri)
-    }
+    onInput(uri)
   }, [onInput, uri])
 
   const onPaste = useCallback(async () => {

src/services/exceptions/ErrorCodes.ts

@@ -49,7 +49,6 @@ enum ErrorCodes {
   _706 = '706: Failed to write to IndexedDB',
   _707 = '707: Error requesting clipboard permissions',
   _708 = '708: Failed to read clipboard',
-  _709 = '709: Failed to write to clipboard',
 
   _800 = '800: Safe creation tx failed',
   _801 = '801: Failed to send a tx with a spending limit',

src/services/walletconnect/useWalletConnectClipboardUri.ts

@@ -1,9 +1,10 @@
 import { useState, useEffect } from 'react'
+import type { Dispatch, SetStateAction } from 'react'
 
 import { getClipboard, isClipboardGranted } from '@/utils/clipboard'
-import { Errors, logError } from '../exceptions'
+import { isPairingUri } from './utils'
 
-export const useWalletConnectClipboardUri = (): [string, (data: string) => Promise<void>] => {
+export const useWalletConnectClipboardUri = (): [string, Dispatch<SetStateAction<string>>] => {
   const [state, setState] = useState('')
 
   useEffect(() => {
@@ -20,7 +21,7 @@ export const useWalletConnectClipboardUri = (): [string, (data: string) => Promi
 
       const clipboard = await getClipboard()
 
-      if (clipboard.startsWith('wc:')) {
+      if (isPairingUri(clipboard)) {
         setState(clipboard)
       }
     }
@@ -34,16 +35,5 @@ export const useWalletConnectClipboardUri = (): [string, (data: string) => Promi
     }
   }, [])
 
-  const setClipboard = async (data: string) => {
-    await navigator.clipboard
-      .writeText(data)
-      .then(() => {
-        setState(data)
-      })
-      .catch((e) => {
-        logError(Errors._709, e)
-      })
-  }
-
-  return [state, setClipboard]
+  return [state, setState]
 }

src/services/walletconnect/useWalletConnectSearchParamUri.ts

@@ -1,13 +1,16 @@
 import { useRouter } from 'next/router'
 import { useCallback } from 'react'
 
+import { isPairingUri } from './utils'
+
 const WC_URI_SEARCH_PARAM = 'wc'
 
 export function useWalletConnectSearchParamUri(): [string | null, (wcUri: string | null) => void] {
   const router = useRouter()
 
   const wcUriQuery = router.query[WC_URI_SEARCH_PARAM]
-  const wcUri = wcUriQuery ? (Array.isArray(wcUriQuery) ? wcUriQuery[0] : wcUriQuery) : null
+  const value = wcUriQuery ? (Array.isArray(wcUriQuery) ? wcUriQuery[0] : wcUriQuery) : null
+  const wcUri = value && isPairingUri(value) ? value : null
 
   const setWcUri = useCallback(
     (wcUri: string | null) => {

src/services/walletconnect/utils.ts

@@ -1,5 +1,9 @@
 import { EIP155 } from './constants'
 
+export const isPairingUri = (uri: string): boolean => {
+  return uri.startsWith('wc:')
+}
+
 export const getEip155ChainId = (chainId: string): string => {
   return `${EIP155}:${chainId}`
 }

src/utils/clipboard.ts

@@ -32,11 +32,3 @@ export const getClipboard = async (): Promise<string> => {
 
   return clipboard
 }
-
-export const setClipboard = async (data: string): Promise<void> => {
-  try {
-    await navigator.clipboard.writeText(data)
-  } catch (e) {
-    logError(Errors._709, e)
-  }
-}