Update typescript v4.8 (#1718)

## Explanation The `typescript` package has been updated to v4.8.4 to align with the module template. This was done to simplify the process of migrating libraries into the monorepo.
MetaMask · Oct 11, 2023 · b3eaf2d · b3eaf2d
1 parent debeb7f
commit b3eaf2d
Show file tree

Hide file tree

Showing 29 changed files with 93 additions and 61 deletions.
diff --git a/package.json b/package.json
@@ -63,7 +63,7 @@
     "rimraf": "^3.0.2",
     "simple-git-hooks": "^2.8.0",
     "ts-node": "^10.9.1",
-    "typescript": "~4.6.3",
+    "typescript": "~4.8.4",
     "which": "^3.0.0"
   },
   "packageManager": "yarn@3.3.0",

diff --git a/packages/accounts-controller/package.json b/packages/accounts-controller/package.json
@@ -50,7 +50,7 @@
     "ts-jest": "^27.1.4",
     "typedoc": "^0.23.15",
     "typedoc-plugin-missing-exports": "^0.23.0",
-    "typescript": "~4.6.3"
+    "typescript": "~4.8.4"
   },
   "peerDependencies": {
     "@metamask/keyring-controller": "^8.0.0"

diff --git a/packages/address-book-controller/package.json b/packages/address-book-controller/package.json
@@ -40,7 +40,7 @@
     "ts-jest": "^27.1.4",
     "typedoc": "^0.23.15",
     "typedoc-plugin-missing-exports": "^0.23.0",
-    "typescript": "~4.6.3"
+    "typescript": "~4.8.4"
   },
   "engines": {
     "node": ">=16.0.0"

diff --git a/packages/announcement-controller/package.json b/packages/announcement-controller/package.json
@@ -39,7 +39,7 @@
     "ts-jest": "^27.1.4",
     "typedoc": "^0.23.15",
     "typedoc-plugin-missing-exports": "^0.23.0",
-    "typescript": "~4.6.3"
+    "typescript": "~4.8.4"
   },
   "engines": {
     "node": ">=16.0.0"

diff --git a/packages/approval-controller/package.json b/packages/approval-controller/package.json
@@ -43,7 +43,7 @@
     "ts-jest": "^27.1.4",
     "typedoc": "^0.23.15",
     "typedoc-plugin-missing-exports": "^0.23.0",
-    "typescript": "~4.6.3"
+    "typescript": "~4.8.4"
   },
   "engines": {
     "node": ">=16.0.0"

diff --git a/packages/assets-controllers/package.json b/packages/assets-controllers/package.json
@@ -64,7 +64,7 @@
     "ts-jest": "^27.1.4",
     "typedoc": "^0.23.15",
     "typedoc-plugin-missing-exports": "^0.23.0",
-    "typescript": "~4.6.3"
+    "typescript": "~4.8.4"
   },
   "peerDependencies": {
     "@metamask/approval-controller": "^3.5.1",

diff --git a/packages/base-controller/package.json b/packages/base-controller/package.json
@@ -41,7 +41,7 @@
     "ts-jest": "^27.1.4",
     "typedoc": "^0.23.15",
     "typedoc-plugin-missing-exports": "^0.23.0",
-    "typescript": "~4.6.3"
+    "typescript": "~4.8.4"
   },
   "engines": {
     "node": ">=16.0.0"

diff --git a/packages/composable-controller/package.json b/packages/composable-controller/package.json
@@ -40,7 +40,7 @@
     "ts-jest": "^27.1.4",
     "typedoc": "^0.23.15",
     "typedoc-plugin-missing-exports": "^0.23.0",
-    "typescript": "~4.6.3"
+    "typescript": "~4.8.4"
   },
   "engines": {
     "node": ">=16.0.0"

diff --git a/packages/controller-utils/package.json b/packages/controller-utils/package.json
@@ -46,7 +46,7 @@
     "ts-jest": "^27.1.4",
     "typedoc": "^0.23.15",
     "typedoc-plugin-missing-exports": "^0.23.0",
-    "typescript": "~4.6.3"
+    "typescript": "~4.8.4"
   },
   "engines": {
     "node": ">=16.0.0"

diff --git a/packages/ens-controller/package.json b/packages/ens-controller/package.json
@@ -44,7 +44,7 @@
     "ts-jest": "^27.1.4",
     "typedoc": "^0.23.15",
     "typedoc-plugin-missing-exports": "^0.23.0",
-    "typescript": "~4.6.3"
+    "typescript": "~4.8.4"
   },
   "peerDependencies": {
     "@metamask/network-controller": "^13.0.0"

diff --git a/packages/gas-fee-controller/package.json b/packages/gas-fee-controller/package.json
@@ -51,7 +51,7 @@
     "ts-jest": "^27.1.4",
     "typedoc": "^0.23.15",
     "typedoc-plugin-missing-exports": "^0.23.0",
-    "typescript": "~4.6.3"
+    "typescript": "~4.8.4"
   },
   "peerDependencies": {
     "@metamask/network-controller": "^13.0.0"

diff --git a/packages/keyring-controller/package.json b/packages/keyring-controller/package.json
@@ -53,7 +53,7 @@
     "ts-jest": "^27.1.4",
     "typedoc": "^0.23.15",
     "typedoc-plugin-missing-exports": "^0.23.0",
-    "typescript": "~4.6.3",
+    "typescript": "~4.8.4",
     "uuid": "^8.3.2"
   },
   "peerDependencies": {

diff --git a/packages/logging-controller/package.json b/packages/logging-controller/package.json
@@ -40,7 +40,7 @@
     "ts-jest": "^27.1.4",
     "typedoc": "^0.23.15",
     "typedoc-plugin-missing-exports": "^0.23.0",
-    "typescript": "~4.6.3"
+    "typescript": "~4.8.4"
   },
   "engines": {
     "node": ">=16.0.0"

diff --git a/packages/message-manager/package.json b/packages/message-manager/package.json
@@ -45,7 +45,7 @@
     "ts-jest": "^27.1.4",
     "typedoc": "^0.23.15",
     "typedoc-plugin-missing-exports": "^0.23.0",
-    "typescript": "~4.6.3"
+    "typescript": "~4.8.4"
   },
   "engines": {
     "node": ">=16.0.0"

diff --git a/packages/name-controller/package.json b/packages/name-controller/package.json
@@ -42,7 +42,7 @@
     "ts-jest": "^27.1.4",
     "typedoc": "^0.23.15",
     "typedoc-plugin-missing-exports": "^0.23.0",
-    "typescript": "~4.6.3"
+    "typescript": "~4.8.4"
   },
   "engines": {
     "node": ">=16.0.0"

diff --git a/packages/name-controller/src/providers/etherscan.ts b/packages/name-controller/src/providers/etherscan.ts
@@ -9,7 +9,7 @@ import type {
   NameProviderResult,
 } from '../types';
 import { NameType } from '../types';
-import { handleFetch } from '../util';
+import { handleFetch, assertIsError } from '../util';
 
 const ID = 'etherscan';
 const LABEL = 'Etherscan (Verified Contract Name)';
@@ -134,7 +134,10 @@ export class EtherscanNameProvider implements NameProvider {
     }
   }
 
-  async #sendRequest(url: string) {
+  async #sendRequest(url: string): Promise<{
+    responseData?: EtherscanGetSourceCodeResponse;
+    error?: Error;
+  }> {
     try {
       log('Sending request', url);
 
@@ -144,6 +147,7 @@ export class EtherscanNameProvider implements NameProvider {
 
       return { responseData };
     } catch (error) {
+      assertIsError(error);
       return { error };
     } finally {
       this.#lastRequestTime = Date.now();

diff --git a/packages/name-controller/src/util.ts b/packages/name-controller/src/util.ts
@@ -57,3 +57,19 @@ export async function successfulFetch(request: string, options?: RequestInit) {
   }
   return response;
 }
+
+/**
+ * Assert that a value is an error. If it's not an error, throw an
+ * error that wraps the given value.
+ *
+ * TODO: Migrate this to @metamask/utils
+ *
+ * @param error - The value that we expect to be an error.
+ * @throws Throws an error wrapping the given value if it's not an error.
+ */
+export function assertIsError(error: unknown): asserts error is Error {
+  if (error instanceof Error) {
+    return;
+  }
+  throw new Error(`Invalid error of type '${typeof error}'`);
+}
diff --git a/packages/name-controller/src/utils.test.ts b/packages/name-controller/src/utils.test.ts
@@ -1,4 +1,4 @@
-import { graphQL } from './util';
+import { assertIsError, graphQL } from './util';
 
 describe('Utils', () => {
   describe('graphQL', () => {
@@ -51,4 +51,16 @@ describe('Utils', () => {
       );
     });
   });
+
+  describe('assertIsError', () => {
+    it('does not throw if given an error', () => {
+      expect(() => assertIsError(new Error('test'))).not.toThrow();
+    });
+
+    it('throws if passed something that is not an error', () => {
+      expect(() => assertIsError('test')).toThrow(
+        `Invalid error of type 'string'`,
+      );
+    });
+  });
 });
diff --git a/packages/network-controller/package.json b/packages/network-controller/package.json
@@ -58,7 +58,7 @@
     "ts-jest": "^27.1.4",
     "typedoc": "^0.23.15",
     "typedoc-plugin-missing-exports": "^0.23.0",
-    "typescript": "~4.6.3"
+    "typescript": "~4.8.4"
   },
   "engines": {
     "node": ">=16.0.0"

diff --git a/packages/notification-controller/package.json b/packages/notification-controller/package.json
@@ -41,7 +41,7 @@
     "ts-jest": "^27.1.4",
     "typedoc": "^0.23.15",
     "typedoc-plugin-missing-exports": "^0.23.0",
-    "typescript": "~4.6.3"
+    "typescript": "~4.8.4"
   },
   "engines": {
     "node": ">=16.0.0"

diff --git a/packages/permission-controller/package.json b/packages/permission-controller/package.json
@@ -47,7 +47,7 @@
     "ts-jest": "^27.1.4",
     "typedoc": "^0.23.15",
     "typedoc-plugin-missing-exports": "^0.23.0",
-    "typescript": "~4.6.3"
+    "typescript": "~4.8.4"
   },
   "peerDependencies": {
     "@metamask/approval-controller": "^3.5.1"

diff --git a/packages/phishing-controller/package.json b/packages/phishing-controller/package.json
@@ -44,7 +44,7 @@
     "ts-jest": "^27.1.4",
     "typedoc": "^0.23.15",
     "typedoc-plugin-missing-exports": "^0.23.0",
-    "typescript": "~4.6.3"
+    "typescript": "~4.8.4"
   },
   "engines": {
     "node": ">=16.0.0"

diff --git a/packages/polling-controller/package.json b/packages/polling-controller/package.json
@@ -43,7 +43,7 @@
     "ts-jest": "^27.1.4",
     "typedoc": "^0.23.15",
     "typedoc-plugin-missing-exports": "^0.23.0",
-    "typescript": "~4.6.3"
+    "typescript": "~4.8.4"
   },
   "peerDependencies": {
     "@metamask/network-controller": "^13.0.0"

diff --git a/packages/preferences-controller/package.json b/packages/preferences-controller/package.json
@@ -39,7 +39,7 @@
     "ts-jest": "^27.1.4",
     "typedoc": "^0.23.15",
     "typedoc-plugin-missing-exports": "^0.23.0",
-    "typescript": "~4.6.3"
+    "typescript": "~4.8.4"
   },
   "engines": {
     "node": ">=16.0.0"

diff --git a/packages/rate-limit-controller/package.json b/packages/rate-limit-controller/package.json
@@ -40,7 +40,7 @@
     "ts-jest": "^27.1.4",
     "typedoc": "^0.23.15",
     "typedoc-plugin-missing-exports": "^0.23.0",
-    "typescript": "~4.6.3"
+    "typescript": "~4.8.4"
   },
   "engines": {
     "node": ">=16.0.0"

diff --git a/packages/selected-network-controller/package.json b/packages/selected-network-controller/package.json
@@ -44,7 +44,7 @@
     "ts-jest": "^27.1.4",
     "typedoc": "^0.23.15",
     "typedoc-plugin-missing-exports": "^0.23.0",
-    "typescript": "~4.6.3"
+    "typescript": "~4.8.4"
   },
   "peerDependencies": {
     "@metamask/network-controller": "^13.0.0"

diff --git a/packages/signature-controller/package.json b/packages/signature-controller/package.json
@@ -48,7 +48,7 @@
     "ts-jest": "^27.1.4",
     "typedoc": "^0.23.15",
     "typedoc-plugin-missing-exports": "^0.23.0",
-    "typescript": "~4.6.3"
+    "typescript": "~4.8.4"
   },
   "peerDependencies": {
     "@metamask/approval-controller": "^3.5.1",

diff --git a/packages/transaction-controller/package.json b/packages/transaction-controller/package.json
@@ -59,7 +59,7 @@
     "ts-jest": "^27.1.4",
     "typedoc": "^0.23.15",
     "typedoc-plugin-missing-exports": "^0.23.0",
-    "typescript": "~4.6.3"
+    "typescript": "~4.8.4"
   },
   "peerDependencies": {
     "@metamask/approval-controller": "^3.5.1",