Updgrade to TS5

firebase · Oct 9, 2024 · ddd4dad · ddd4dad
1 parent 0dbb8e6
commit ddd4dad
Show file tree

Hide file tree

Showing 91 changed files with 21,191 additions and 237 deletions.
diff --git a/.vscode/launch.json b/.vscode/launch.json
@@ -4,6 +4,24 @@
   // For more information, visit: https://go.microsoft.com/fwlink/?linkid=830387
   "version": "0.2.0",
   "configurations": [
+    {
+      "type": "node",
+      "request": "launch",
+      "name": "Prune .d.ts",
+      "program": "${workspaceRoot}/repo-scripts/prune-dts/node_modules/.bin/_mocha",
+      "cwd": "${workspaceRoot}/repo-scripts/prune-dts",
+      "args": [
+        "--require",
+        "ts-node/register",
+        "--timeout",
+        "5000",
+        "${workspaceFolder}/repo-scripts/prune-dts/*.test.ts"
+      ],
+      "env": {
+        "TS_NODE_COMPILER_OPTIONS" : "{\"module\":\"commonjs\"}"
+      },
+      "sourceMaps": true,
+    },
     {
       "type": "node",
       "request": "launch",

diff --git a/e2e/package.json b/e2e/package.json
@@ -34,7 +34,7 @@
     "karma-typescript": "5.5.4",
     "karma-typescript-es6-transform": "5.5.4",
     "mocha": "9.2.2",
-    "typescript": "4.7.4",
+    "typescript": "5.5.4",
     "webpack": "5.76.0",
     "webpack-cli": "4.10.0",
     "webpack-dev-server": "4.11.1"

diff --git a/integration/compat-interop/package.json b/integration/compat-interop/package.json
@@ -24,7 +24,7 @@
     "@firebase/remote-config-compat": "0.2.9"
   },
   "devDependencies": {
-    "typescript": "4.7.4"
+    "typescript": "5.5.4"
   },
   "engines": {
     "node": ">=18.0.0"

diff --git a/integration/compat-typings/package.json b/integration/compat-typings/package.json
@@ -10,7 +10,7 @@
     "firebase": "*"
   },
   "devDependencies": {
-    "typescript": "4.7.4"
+    "typescript": "5.5.4"
   },
   "engines": {
     "node": ">=18.0.0"

diff --git a/integration/firebase/package.json b/integration/firebase/package.json
@@ -20,7 +20,7 @@
     "karma-typescript": "5.5.4",
     "mocha": "9.2.2",
     "npm-run-all": "4.1.5",
-    "typescript": "4.7.4"
+    "typescript": "5.5.4"
   },
   "engines": {
     "node": ">=18.0.0"

diff --git a/package.json b/package.json
@@ -80,7 +80,7 @@
     "@types/long": "4.0.2",
     "@types/mocha": "9.1.1",
     "@types/mz": "2.7.8",
-    "@types/node": "16.18.68",
+    "@types/node": "22.7.5",
     "@types/request": "2.48.12",
     "@types/sinon": "9.0.11",
     "@types/sinon-chai": "3.2.12",
@@ -156,7 +156,7 @@
     "tsec": "0.2.8",
     "tslint": "6.1.3",
     "typedoc": "0.16.11",
-    "typescript": "4.7.4",
+    "typescript": "5.5.4",
     "watch": "1.0.2",
     "webpack": "5.76.0",
     "yargs": "17.7.2"

diff --git a/packages/analytics-compat/package.json b/packages/analytics-compat/package.json
@@ -26,7 +26,7 @@
     "rollup": "2.79.1",
     "@rollup/plugin-json": "4.1.0",
     "rollup-plugin-typescript2": "0.31.2",
-    "typescript": "4.7.4"
+    "typescript": "5.5.4"
   },
   "repository": {
     "directory": "packages/analytics-compat",

diff --git a/packages/analytics-interop-types/package.json b/packages/analytics-interop-types/package.json
@@ -20,6 +20,6 @@
     "url": "https://github.com/firebase/firebase-js-sdk/issues"
   },
   "devDependencies": {
-    "typescript": "4.7.4"
+    "typescript": "5.5.4"
   }
 }
diff --git a/packages/analytics-types/package.json b/packages/analytics-types/package.json
@@ -20,6 +20,6 @@
     "url": "https://github.com/firebase/firebase-js-sdk/issues"
   },
   "devDependencies": {
-    "typescript": "4.7.4"
+    "typescript": "5.5.4"
   }
 }
diff --git a/packages/analytics/package.json b/packages/analytics/package.json
@@ -53,7 +53,7 @@
     "@rollup/plugin-json": "4.1.0",
     "@rollup/plugin-node-resolve": "13.3.0",
     "rollup-plugin-typescript2": "0.31.2",
-    "typescript": "4.7.4"
+    "typescript": "5.5.4"
   },
   "repository": {
     "directory": "packages/analytics",

diff --git a/packages/app-check-compat/package.json b/packages/app-check-compat/package.json
@@ -49,7 +49,7 @@
     "@rollup/plugin-json": "4.1.0",
     "@rollup/plugin-node-resolve": "13.3.0",
     "rollup-plugin-typescript2": "0.31.2",
-    "typescript": "4.7.4"
+    "typescript": "5.5.4"
   },
   "repository": {
     "directory": "packages/app-check",

diff --git a/packages/app-check-interop-types/package.json b/packages/app-check-interop-types/package.json
@@ -20,6 +20,6 @@
     "url": "https://github.com/firebase/firebase-js-sdk/issues"
   },
   "devDependencies": {
-    "typescript": "4.7.4"
+    "typescript": "5.5.4"
   }
 }
diff --git a/packages/app-check-types/package.json b/packages/app-check-types/package.json
@@ -20,6 +20,6 @@
     "url": "https://github.com/firebase/firebase-js-sdk/issues"
   },
   "devDependencies": {
-    "typescript": "4.7.4"
+    "typescript": "5.5.4"
   }
 }
diff --git a/packages/app-check/package.json b/packages/app-check/package.json
@@ -50,7 +50,7 @@
     "@rollup/plugin-json": "4.1.0",
     "@rollup/plugin-node-resolve": "13.3.0",
     "rollup-plugin-typescript2": "0.31.2",
-    "typescript": "4.7.4"
+    "typescript": "5.5.4"
   },
   "repository": {
     "directory": "packages/app-check",

diff --git a/packages/app-compat/package.json b/packages/app-compat/package.json
@@ -48,7 +48,7 @@
     "@rollup/plugin-json": "4.1.0",
     "rollup-plugin-replace": "2.2.0",
     "rollup-plugin-typescript2": "0.31.2",
-    "typescript": "4.7.4"
+    "typescript": "5.5.4"
   },
   "repository": {
     "directory": "packages/app-compat",

diff --git a/packages/app-types/package.json b/packages/app-types/package.json
@@ -24,6 +24,6 @@
     "@firebase/logger": "0.2.6"
   },
   "devDependencies": {
-    "typescript": "4.7.4"
+    "typescript": "5.5.4"
   }
 }
diff --git a/packages/app/package.json b/packages/app/package.json
@@ -50,7 +50,7 @@
     "rollup-plugin-replace": "2.2.0",
     "rollup-plugin-typescript2": "0.31.2",
     "rollup-plugin-dts": "5.3.1",
-    "typescript": "4.7.4"
+    "typescript": "5.5.4"
   },
   "repository": {
     "directory": "packages/app",

diff --git a/packages/auth-compat/demo/package.json b/packages/auth-compat/demo/package.json
@@ -34,7 +34,7 @@
     "rollup-plugin-sourcemaps": "0.6.3",
     "rollup-plugin-typescript2": "0.31.2",
     "rollup-plugin-uglify": "6.0.4",
-    "typescript": "4.7.4"
+    "typescript": "5.5.4"
   },
   "repository": {
     "directory": "packages/auth-compat/demo",

diff --git a/packages/auth-compat/package.json b/packages/auth-compat/package.json
@@ -63,7 +63,7 @@
     "rollup-plugin-replace": "2.2.0",
     "rollup-plugin-typescript2": "0.31.2",
     "selenium-webdriver": "4.8.0",
-    "typescript": "4.7.4"
+    "typescript": "5.5.4"
   },
   "repository": {
     "directory": "packages/auth-compat",

diff --git a/packages/auth-interop-types/package.json b/packages/auth-interop-types/package.json
@@ -20,6 +20,6 @@
     "url": "https://github.com/firebase/firebase-js-sdk/issues"
   },
   "devDependencies": {
-    "typescript": "4.7.4"
+    "typescript": "5.5.4"
   }
 }
diff --git a/packages/auth-types/package.json b/packages/auth-types/package.json
@@ -24,6 +24,6 @@
     "url": "https://github.com/firebase/firebase-js-sdk/issues"
   },
   "devDependencies": {
-    "typescript": "4.7.4"
+    "typescript": "5.5.4"
   }
 }
diff --git a/packages/auth/package.json b/packages/auth/package.json
@@ -141,7 +141,7 @@
     "rollup-plugin-typescript2": "0.31.2",
     "selenium-webdriver": "4.8.0",
     "totp-generator": "0.0.14",
-    "typescript": "4.7.4"
+    "typescript": "5.5.4"
   },
   "repository": {
     "directory": "packages/auth",

diff --git a/packages/auth/src/api/index.ts b/packages/auth/src/api/index.ts
@@ -237,20 +237,20 @@ export async function _performSignInRequest<T, V extends IdTokenResponse>(
   request?: T,
   customErrorMap: Partial<ServerErrorMap<ServerError>> = {}
 ): Promise<V> {
-  const serverResponse = (await _performApiRequest<T, V | IdTokenMfaResponse>(
+  const serverResponse = await _performApiRequest<T, V | IdTokenMfaResponse>(
     auth,
     method,
     path,
     request,
     customErrorMap
-  )) as V;
+  );
   if ('mfaPendingCredential' in serverResponse) {
     _fail(auth, AuthErrorCode.MFA_REQUIRED, {
       _serverResponse: serverResponse
     });
   }
 
-  return serverResponse;
+  return serverResponse as V;
 }
 
 export function _getFinalTarget(

diff --git a/packages/auth/src/platform_browser/recaptcha/recaptcha_enterprise_verifier.ts b/packages/auth/src/platform_browser/recaptcha/recaptcha_enterprise_verifier.ts
@@ -159,22 +159,20 @@ export async function injectRecaptchaFields<T>(
   captchaResp = false
 ): Promise<T> {
   const verifier = new RecaptchaEnterpriseVerifier(auth);
-  let captchaResponse;
+  let captchaResponse: string;
   try {
     captchaResponse = await verifier.verify(action);
   } catch (error) {
     captchaResponse = await verifier.verify(action, true);
   }
-  const newRequest = { ...request };
-  if (!captchaResp) {
-    Object.assign(newRequest, { captchaResponse });
-  } else {
-    Object.assign(newRequest, { 'captchaResp': captchaResponse });
-  }
-  Object.assign(newRequest, { 'clientType': RecaptchaClientType.WEB });
-  Object.assign(newRequest, {
-    'recaptchaVersion': RecaptchaVersion.ENTERPRISE
-  });
+
+  const newRequest: T = {
+    ...request,
+    ...(captchaResp ? { captchaResp: captchaResponse } : { captchaResponse }),
+    clientType: RecaptchaClientType.WEB,
+    recaptchaVersion: RecaptchaVersion.ENTERPRISE
+  };
+
   return newRequest;
 }
 

diff --git a/packages/component/package.json b/packages/component/package.json
@@ -38,7 +38,7 @@
   "devDependencies": {
     "rollup": "2.79.1",
     "rollup-plugin-typescript2": "0.31.2",
-    "typescript": "4.7.4"
+    "typescript": "5.5.4"
   },
   "repository": {
     "directory": "packages/component",

diff --git a/packages/database-compat/package.json b/packages/database-compat/package.json
@@ -58,7 +58,7 @@
   },
   "devDependencies": {
     "@firebase/app-compat": "0.2.42",
-    "typescript": "4.7.4"
+    "typescript": "5.5.4"
   },
   "repository": {
     "directory": "packages/database-compat",

diff --git a/packages/database-types/package.json b/packages/database-types/package.json
@@ -24,6 +24,6 @@
     "url": "https://github.com/firebase/firebase-js-sdk/issues"
   },
   "devDependencies": {
-    "typescript": "4.7.4"
+    "typescript": "5.5.4"
   }
 }
diff --git a/packages/database/package.json b/packages/database/package.json
@@ -60,7 +60,7 @@
     "@firebase/app": "0.10.12",
     "rollup": "2.79.1",
     "rollup-plugin-typescript2": "0.31.2",
-    "typescript": "4.7.4"
+    "typescript": "5.5.4"
   },
   "repository": {
     "directory": "packages/database",

diff --git a/packages/firestore-compat/package.json b/packages/firestore-compat/package.json
@@ -61,7 +61,7 @@
     "rollup-plugin-typescript2": "0.31.2",
     "@rollup/plugin-node-resolve": "13.3.0",
     "ts-node": "10.9.1",
-    "typescript": "4.7.4"
+    "typescript": "5.5.4"
   },
   "license": "Apache-2.0",
   "typings": "dist/src/index.d.ts",

diff --git a/packages/firestore/package.json b/packages/firestore/package.json
@@ -125,7 +125,7 @@
     "rollup-plugin-typescript2": "0.31.2",
     "rollup-plugin-dts": "5.3.1",
     "ts-node": "10.9.1",
-    "typescript": "4.7.4"
+    "typescript": "5.5.4"
   },
   "repository": {
     "directory": "packages/firestore",

diff --git a/packages/firestore/scripts/build-bundle.js b/packages/firestore/scripts/build-bundle.js
diff --git a/packages/firestore/scripts/remove-asserts.js b/packages/firestore/scripts/remove-asserts.js
diff --git a/packages/firestore/scripts/remove-asserts.ts b/packages/firestore/scripts/remove-asserts.ts
@@ -64,17 +64,17 @@ class RemoveAsserts {
         ) {
           const method = declaration.name!.text;
           if (method === 'debugAssert') {
-            updatedNode = ts.createOmittedExpression();
+            updatedNode = ts.factory.createOmittedExpression();
           } else if (method === 'hardAssert') {
             // Remove the log message but keep the assertion
-            updatedNode = ts.createCall(
+            updatedNode = ts.factory.createCallExpression(
               declaration.name!,
               /*typeArgs*/ undefined,
               [node.arguments[0]]
             );
           } else if (method === 'fail') {
             // Remove the log message
-            updatedNode = ts.createCall(
+            updatedNode = ts.factory.createCallExpression(
               declaration.name!,
               /*typeArgs*/ undefined,
               []