Firebase Functions 7.0.0 (#1758)

* BREAKING: Upgrade TypeScript to v5.9.3 (#1746)

Upgrades from TypeScript v4.3.5 to v5.9.3. All tests pass successfully.

* BREAKING: Drop Node.js 14/16 support, require Node.js >=18 (#1747)

* BREAKING: Drop Node.js 14/16 support, require Node.js >=18

- Update package.json engines to require Node.js >=18.0.0
- Upgrade @types/node from v14 to v18
- Update TypeScript target from ES2019 to ES2022
- Add useDefineForClassFields: false for backward compatibility
- Update CI/CD workflows to test Node.js 18, 20, 22, and 24
- Set default CI/CD Node.js version to 24
- Update integration test documentation

* BREAKING: remove functions.config() implementation (#1748)

Replace functions.config() implementation with an error that provides
clear migration guidance to the params module. This removes the deprecated
API as planned for the next major release while helping developers migrate.

The error message directs users to:
https://firebase.google.com/docs/functions/config-env#migrate-config

* chore: migrate to ESLint v9 and Flat Config (#1753)

Upgraded eslint to v9 and replaced the legacy .eslintrc.js with the new eslint.config.js format.

Applied minor fixes to 4 files (renaming unused catch variables to _e and one small refactor) to satisfy slightly stricter baseline rules from updated plugins.

Mostly doing this due to our upgrade in TS to latest v5 version which the old versions of eslint ts plugins no longer support.

* chore: enable esModuleInterop (#1751)

* feat: add async error handling to onRequest (#1755)

BREAKING CHANGE: Unhandled errors in async onRequest handlers now immediately return a 500 Internal Server Error instead of hanging until timeout. This may affect clients relying on 504 Gateway Timeout for retry logic.

* BREAKING: Break TS build that uses functions.config() (#1757)

Type out `functions.config()` API. Use of `functions.config()` API should now cause a BUILD error at compile time.

```ts
import * as functions from "firebase-functions";
import {onRequest} from "firebase-functions/https";
import * as logger from "firebase-functions/logger";

const cfg = functions.config()

export const helloWorld = onRequest((request, response) => {
  logger.info("Hello logs!", cfg);
  response.send("Hello from Firebase!");
});
```
```bash
$ npm run build

> build
> tsc

src/index.ts:5:23 - error TS2349: This expression is not callable.
  Type 'never' has no call signatures.

5 const cfg = functions.config()
                        ~~~~~~


Found 1 error in src/index.ts:5
```

* Add Firebase Data Connect v2 support (#1727)

* Add Firebase Data Connect v2 support

* Export dataconnect

* Fix export

* Rename Extract to VarName, since Extract is a built in type

* Include auth context in event payload

* Stronger typing for data connect params

* Fix formatting and address comments

* Apply another suggestion

* Add region and fix bug in which event.service, event.connector, event.operation gets populated

* Remove locations/... segment

* Address comments

* Fix linter

* feat: enable ESM support via dual-publishing (#1750)

- Configure tsdown to build both CJS and ESM outputs.
- Update package.json exports to support both 'require' and 'import'.
- Fix type hygiene issues by using explicit 'export type'.
- Update protos/update.sh to generate ESM version of compiledFirestore and fix its imports for Node.js compatibility.
- Configure build aliases to handle relative paths to protos correctly in both CJS and ESM builds.
- Add packaging integration test to CI.

~Note: I enabled `esModuleInterop` in tsconfig.release.json to support default imports from CommonJS modules (like cors), which is required for proper ESM interop. This caused some chain-effect that required updating several unit tests and source files to use default imports for other CJS dependencies (e.g., fs, nock, jsonwebtoken) to align with the new compiler configuration.~ Fixed in #1751.

* lock down permission per GitHub alert.

* update lint rule to enforce prettier rules.

* chore: update CHANGELOG for v7.0.0

* format.

---------

Co-authored-by: Lisa Jian <[email protected]>

Author: taeold

.eslintignore

@@ -12,7 +12,7 @@ jobs:
       - uses: actions/checkout@v4
       - uses: actions/setup-node@v4
         with:
-          node-version: "20"
+          node-version: "24"
       - name: Cache npm
         uses: actions/cache@v4
         with:

.eslintrc.js

@@ -30,7 +30,7 @@ jobs:
       - uses: actions/checkout@v3
       - uses: actions/setup-node@v3
         with:
-          node-version: 18
+          node-version: 24
 
       - uses: google-github-actions/auth@v0
         with:

.github/workflows/docs.yaml

@@ -4,6 +4,9 @@ on:
   - pull_request
   - push
 
+permissions:
+  contents: read
+
 env:
   CI: true
 
@@ -14,17 +17,36 @@ jobs:
     strategy:
       matrix:
         node-version:
-          - 18.x
+          - 20.x
     steps:
-      - uses: actions/checkout@v3
-        with:
-          fetch-depth: 0
-      - uses: actions/setup-node@v3
+      - uses: actions/checkout@v4
+      - uses: actions/setup-node@v4
         with:
           node-version: ${{ matrix.node-version }}
           cache: npm
       - run: npm ci
       - run: npm run lint
+
+  build:
+    runs-on: ubuntu-latest
+    steps:
+      - uses: actions/checkout@v4
+      - uses: actions/setup-node@v4
+        with:
+          node-version: 20.x
+          cache: npm
+      - run: npm ci
+      - run: npm run build
+      - run: npm pack
+      - uses: actions/upload-artifact@v4
+        with:
+          name: lib
+          path: lib/
+      - uses: actions/upload-artifact@v4
+        with:
+          name: tarball
+          path: firebase-functions-*.tgz
+
   unit:
     runs-on: ubuntu-latest
     strategy:
@@ -33,36 +55,58 @@ jobs:
           - 18.x
           - 20.x
           - 22.x
+          - 24.x
     steps:
-      - uses: actions/checkout@v1
-      - uses: actions/setup-node@v1
+      - uses: actions/checkout@v4
+      - uses: actions/setup-node@v4
         with:
           node-version: ${{ matrix.node-version }}
-      - name: Cache npm
-        uses: actions/cache@v4
-        with:
-          path: ~/.npm
-          key: ${{ runner.os }}-node-${{ matrix.node-version }}-${{ hashFiles('**/package-lock.json') }}
+          cache: npm
       - run: npm ci
       - run: npm run test
+
   integration:
-    needs: "unit"
+    needs: build
     runs-on: ubuntu-latest
     strategy:
       matrix:
         node-version:
           - 18.x
           - 20.x
           - 22.x
+          - 24.x
     steps:
-      - uses: actions/checkout@v1
-      - uses: actions/setup-node@v1
+      - uses: actions/checkout@v4
+      - uses: actions/setup-node@v4
         with:
           node-version: ${{ matrix.node-version }}
-      - name: Cache npm
-        uses: actions/cache@v4
-        with:
-          path: ~/.npm
-          key: ${{ runner.os }}-node-${{ matrix.node-version }}-${{ hashFiles('**/package-lock.json') }}
+          cache: npm
       - run: npm ci
+      - uses: actions/download-artifact@v4
+        with:
+          name: lib
+          path: lib
       - run: npm run test:bin
+        env:
+          SKIP_BUILD: true
+
+  packaging:
+    needs: build
+    runs-on: ubuntu-latest
+    strategy:
+      matrix:
+        node-version:
+          - 18.x
+          - 20.x
+          - 22.x
+          - 24.x
+    steps:
+      - uses: actions/checkout@v4
+      - uses: actions/setup-node@v4
+        with:
+          node-version: ${{ matrix.node-version }}
+      - uses: actions/download-artifact@v4
+        with:
+          name: tarball
+          path: .
+      - run: chmod +x ./scripts/test-packaging.sh && ./scripts/test-packaging.sh firebase-functions-*.tgz

.github/workflows/postmerge.yaml

@@ -0,0 +1,6 @@
+- BREAKING: Drop support for Node.js 16. Minimum supported version is now Node.js 18. (#1747)
+- BREAKING: Remove deprecated `functions.config()` API. Use `params` module for environment variables instead. (#1748)
+- BREAKING: Upgrade to TypeScript v5 and target ES2022. (#1746)
+- BREAKING: Unhandled errors in async `onRequest` handlers in the Emulator now return a 500 error immediately. (#1755)
+- Add support for ESM (ECMAScript Modules) alongside CommonJS. (#1750)
+- Add `onMutationExecuted()` trigger for Firebase Data Connect. (#1727)

.github/workflows/test.yaml

@@ -0,0 +1,112 @@
+const { FlatCompat } = require("@eslint/eslintrc");
+const js = require("@eslint/js");
+const path = require("path");
+
+const compat = new FlatCompat({
+    baseDirectory: __dirname,
+    recommendedConfig: js.configs.recommended,
+    allConfig: js.configs.all
+});
+
+module.exports = [
+    {
+        ignores: [
+            "lib/",
+            "dev/",
+            "node_modules/",
+            "coverage/",
+            "docgen/",
+            "v1/",
+            "v2/",
+            "logger/",
+            "dist/",
+            "spec/fixtures/",
+            "scripts/**/*.js",
+            "scripts/**/*.mjs",
+            "protos/",
+            ".prettierrc.js",
+            "eslint.config.*",
+            "tsdown.config.*",
+            "scripts/bin-test/sources/esm-ext/index.mjs",
+        ],
+    },
+    ...compat.extends(
+        "eslint:recommended",
+        "plugin:@typescript-eslint/recommended",
+        "plugin:@typescript-eslint/recommended-requiring-type-checking",
+        "plugin:jsdoc/recommended",
+        "google",
+        "prettier"
+    ),
+    {
+        languageOptions: {
+            parser: require("@typescript-eslint/parser"),
+            parserOptions: {
+                project: "tsconfig.json",
+                tsconfigRootDir: __dirname,
+            },
+            ecmaVersion: 2022
+        },
+        plugins: {
+            "prettier": require("eslint-plugin-prettier"),
+        },
+        rules: {
+            "jsdoc/newline-after-description": "off",
+            "jsdoc/require-jsdoc": ["warn", { publicOnly: true }],
+            "jsdoc/check-tag-names": ["warn", { definedTags: ["alpha", "remarks", "typeParam", "packageDocumentation", "hidden"] }],
+            "no-restricted-globals": ["error", "name", "length"],
+            "prefer-arrow-callback": "error",
+            "prettier/prettier": "error",
+            "require-atomic-updates": "off", // This rule is so noisy and isn't useful: https://github.com/eslint/eslint/issues/11899
+            "require-jsdoc": "off", // This rule is deprecated and superseded by jsdoc/require-jsdoc.
+            "valid-jsdoc": "off", // This is deprecated but included in recommended configs.
+            "no-prototype-builtins": "warn",
+            "no-useless-escape": "warn",
+            "prefer-promise-reject-errors": "warn",
+        },
+    },
+    {
+        files: ["**/*.ts"],
+        rules: {
+            "jsdoc/require-param-type": "off",
+            "jsdoc/require-returns-type": "off",
+            // Google style guide allows us to omit trivial parameters and returns
+            "jsdoc/require-param": "off",
+            "jsdoc/require-returns": "off",
+
+            "@typescript-eslint/no-invalid-this": "error",
+            "@typescript-eslint/no-unused-vars": ["error", { argsIgnorePattern: "^_", caughtErrorsIgnorePattern: "^_" }], // Unused vars should not exist.
+            "@typescript-eslint/no-misused-promises": "warn", // rule does not work with async handlers for express.
+            "no-invalid-this": "off", // Turned off in favor of @typescript-eslint/no-invalid-this.
+            "no-unused-vars": "off", // Off in favor of @typescript-eslint/no-unused-vars.
+            eqeqeq: ["error", "always", { null: "ignore" }],
+            camelcase: ["error", { properties: "never" }], // snake_case allowed in properties iif to satisfy an external contract / style
+
+            // Ideally, all these warning should be error - let's fix them in the future.
+            "@typescript-eslint/no-unsafe-argument": "warn",
+            "@typescript-eslint/no-unsafe-assignment": "warn",
+            "@typescript-eslint/no-unsafe-call": "warn",
+            "@typescript-eslint/no-unsafe-member-access": "warn",
+            "@typescript-eslint/no-unsafe-return": "warn",
+            "@typescript-eslint/restrict-template-expressions": "warn",
+            "@typescript-eslint/no-explicit-any": "warn",
+            "@typescript-eslint/no-redundant-type-constituents": "warn",
+            "@typescript-eslint/no-base-to-string": "warn",
+            "@typescript-eslint/no-duplicate-type-constituents": "warn",
+            "@typescript-eslint/no-require-imports": "warn",
+            "@typescript-eslint/no-empty-object-type": "warn",
+            "@typescript-eslint/prefer-promise-reject-errors": "warn",
+        },
+    },
+    {
+        files: ["**/*.spec.ts", "**/*.spec.js", "spec/helper.ts", "scripts/bin-test/**/*.ts", "integration_test/**/*.ts"],
+        languageOptions: {
+            globals: {
+                mocha: true,
+            },
+        },
+        rules: {
+            "@typescript-eslint/no-unused-expressions": "off",
+        }
+    },
+];

CHANGELOG.md

@@ -3,7 +3,7 @@ import { GoogleAuth } from "google-auth-library";
 import { Request, Response } from "express";
 import * as admin from "firebase-admin";
 import * as functions from "firebase-functions";
-import * as fs from "fs";
+import fs from "fs";
 import fetch from "node-fetch";
 
 import * as v1 from "./v1";

eslint.config.js

@@ -13,8 +13,8 @@ export const pubsubTests: any = functions
     let testId: string;
     try {
       testId = m.json.testId;
-    } catch (e) {
-      /* Ignored. Covered in another test case that `event.data.json` works. */
+    } catch (_e) {
+      // Ignored. Covered in another test case that `event.data.json` works.
     }
 
     return new TestSuite<PubsubMessage>("pubsub onPublish")

integration_test/functions/src/index.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();