Fix the useEffect dependency in auto-connect (#273)

* Fix dependency array

Update changeset and fix typos in `CONTRIBUTING.md`

Fix more typos

* Clarify changeset reason

.changeset/odd-dingos-lie.md

@@ -0,0 +1,5 @@
+---
+"@aptos-labs/wallet-adapter-react": patch
+---
+
+Fixed the `useEffect` dependency array for auto-connecting to be `[wallets]` instead of `wallets`

CONTRIBUTING.md

@@ -9,7 +9,7 @@ Aptos Wallet Adapter is a monorepo built with [turbo](https://turbo.build/repo/d
 #### Apps Workspace
 
 The App workspace hold different web apps such as the adapter demo app (a nextjs app).
-You are welcome to add different apps (react/vue/etc) in this workspace (make sure you are following turbo best pratices on how to do it)
+You are welcome to add different apps (react/vue/etc) in this workspace (make sure you are following turbo best practices on how to do it)
 
 #### Packages Workspace
 
@@ -18,7 +18,7 @@ The packages workspace holds 2 main packages
 1. `wallet-adapter-core` - the core adapter logic that responsible on the adapter functionality, state and the interaction between the dapp and wallet.
 2. `wallet-adapter-react` - a light react provider that dapps can import and use the adapter.
 
-You are welcome to add packages (vue-provider/UI frameworks/etc) in this workspace (make sure you are following turbo best pratices on how to do it)
+You are welcome to add packages (vue-provider/UI frameworks/etc) in this workspace (make sure you are following turbo best practices on how to do it)
 
 ### Develop Locally
 
@@ -30,7 +30,7 @@ You would need `[email protected]` in order to bootstrap and test a local copy of this
 
 ### Creating a pull request
 
-You are welcome to create a pull reuqest against the `main` branch.
+You are welcome to create a pull request against the `main` branch.
 
 Before creating a PR
 

packages/wallet-adapter-core/src/WalletCore.ts

@@ -126,9 +126,9 @@ export class WalletCore extends EventEmitter<WalletCoreEvents> {
   constructor(plugins: ReadonlyArray<Wallet>) {
     super();
     this._wallets = plugins;
-    // Stretegy to detect legacy wallet adapter v1 wallet plugins
+    // Strategy to detect legacy wallet adapter v1 wallet plugins
     this.scopePollingDetectionStrategy();
-    // Stretegy to detect AIP-62 standard compatible wallets
+    // Strategy to detect AIP-62 standard compatible wallets
     this.fetchAptosWallets();
   }
 
@@ -869,7 +869,7 @@ export class WalletCore extends EventEmitter<WalletCoreEvents> {
         return pendingTransaction;
       }
 
-      // Else have the adpater submits the transaction
+      // Else have the adapter submit the transaction
 
       const aptosConfig = new AptosConfig({
         network: convertNetwork(this.network),

packages/wallet-adapter-react/README.md

@@ -1,4 +1,4 @@
-> **_NOTE:_** This documentation is for Wallet Adapter `v2.0.0` and up that is fully comatible with the Aptos TypeScript SDK V2. For Wallet Adapter `v^1.*.*` refer to [this guide](./READMEV1.md)
+> **_NOTE:_** This documentation is for Wallet Adapter `v2.0.0` and up that is fully compatible with the Aptos TypeScript SDK V2. For Wallet Adapter `v^1.*.*` refer to [this guide](./READMEV1.md)
 
 # Wallet Adapter React Provider
 

packages/wallet-adapter-react/src/WalletProvider.tsx

@@ -160,7 +160,7 @@ export const AptosWalletAdapterProvider: FC<AptosWalletProviderProps> = ({
         setIsLoading(false);
       }
     }
-  }, wallets);
+  }, [wallets]);
 
   useEffect(() => {
     if (connected) {