chore: enable esModuleInterop

- Enable esModuleInterop in tsconfig.release.json for better CJS/ESM interop.
- Update unit tests and source code to use default imports for CJS modules (fs, nock, jsonwebtoken, etc.) to align with new compiler settings.

Author: taeold

mocha/setup.ts

@@ -1,17 +1,6 @@
-import * as chai from "chai";
-import * as chaiAsPromisedModule from "chai-as-promised";
-import * as nockModule from "nock";
-
-// Normalize CommonJS exports so ts-node (Node.js 20) and Node.js 22's strip-only loader
-// both receive callable modules without relying on esModuleInterop.
-type ChaiPlugin = Parameters<typeof chai.use>[0];
-type NockModule = typeof nockModule;
-
-const chaiAsPromisedExport = chaiAsPromisedModule as ChaiPlugin & { default?: ChaiPlugin };
-const chaiAsPromised = chaiAsPromisedExport.default ?? chaiAsPromisedExport;
-const nockExport = nockModule as NockModule & { default?: NockModule };
-const nock = nockExport.default ?? nockExport;
+import chai from "chai";
+import chaiAsPromised from "chai-as-promised";
+import nock from "nock";
 
 chai.use(chaiAsPromised);
-
-nock.disableNetConnect();
+nock.disableNetConnect();

spec/common/config.spec.ts

@@ -21,8 +21,7 @@
 // SOFTWARE.
 
 import { expect } from "chai";
-import * as fs from "fs";
-import * as process from "process";
+import fs from "fs";
 import * as sinon from "sinon";
 
 import { firebaseConfig, resetCache } from "../../src/common/config";

spec/fixtures/mockrequest.ts

@@ -1,8 +1,8 @@
 import { EventEmitter } from 'node:stream';
 
-import * as jwt from 'jsonwebtoken';
-import * as jwkToPem from 'jwk-to-pem';
-import * as nock from 'nock';
+import jwt from 'jsonwebtoken';
+import jwkToPem from 'jwk-to-pem';
+import nock from 'nock';
 import * as mockJWK from '../fixtures/credential/jwk.json';
 import * as mockKey from '../fixtures/credential/key.json';
 

src/bin/firebase-functions.ts

@@ -23,7 +23,7 @@
 // SOFTWARE.
 
 import * as http from "http";
-import * as express from "express";
+import express from "express";
 import * as fs from "fs/promises";
 import * as path from "path";
 import { loadStack } from "../runtime/loader";

src/common/config.ts

@@ -1,5 +1,5 @@
 import { AppOptions } from "firebase-admin/app";
-import { readFileSync } from "fs";
+import fs from "fs";
 import * as path from "path";
 
 import * as logger from "../logger";
@@ -29,7 +29,7 @@ export function firebaseConfig(): AppOptions | null {
     // explicitly state that the user can set the env to a file:
     // https://firebase.google.com/docs/admin/setup#initialize-without-parameters
     if (!env.startsWith("{")) {
-      env = readFileSync(path.join(process.env.PWD, env)).toString("utf8");
+      env = fs.readFileSync(path.join(process.env.PWD, env)).toString("utf8");
     }
 
     cache = JSON.parse(env);

src/common/providers/https.ts

@@ -20,7 +20,7 @@
 // OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE
 // SOFTWARE.
 
-import * as cors from "cors";
+import cors from "cors";
 import * as express from "express";
 import { DecodedAppCheckToken } from "firebase-admin/app-check";
 

src/v2/providers/https.ts

@@ -25,7 +25,7 @@
  * @packageDocumentation
  */
 
-import * as cors from "cors";
+import cors from "cors";
 import * as express from "express";
 import { convertIfPresent, convertInvoker, copyIfPresent } from "../../common/encoding";
 import { wrapTraceContext } from "../trace";
@@ -48,7 +48,8 @@ import * as options from "../options";
 import { withInit } from "../../common/onInit";
 import * as logger from "../../logger";
 
-export { Request, CallableRequest, CallableResponse, FunctionsErrorCode, HttpsError };
+export { HttpsError };
+export type { Request, CallableRequest, CallableResponse, FunctionsErrorCode };
 
 /**
  * Options that can be set on an onRequest HTTPS function.

tsconfig.release.json

@@ -9,6 +9,7 @@
     "stripInternal": true,
     "target": "es2022",
     "useDefineForClassFields": false,
+    "esModuleInterop": true,
     "typeRoots": ["./node_modules/@types"]
   },
   "files": [