chore!: support node 18

.github/sync-repo-settings.yaml

@@ -7,12 +7,11 @@ branchProtectionRules:
     requiredStatusCheckContexts:
       - "ci/kokoro: Samples test"
       - "ci/kokoro: System test"
-      - docs
       - lint
-      - test (14)
-      - test (16)
       - test (18)
       - test (20)
+      - test (22)
+      - test (24)
       - cla/google
       - windows
       - OwlBot Post Processor

.github/workflows/ci.yaml

@@ -9,10 +9,10 @@ jobs:
     runs-on: ubuntu-latest
     strategy:
       matrix:
-        node: [14, 16, 18, 20]
+        node: [18, 20, 22, 24]
     steps:
-      - uses: actions/checkout@v3
-      - uses: actions/setup-node@v3
+      - uses: actions/checkout@v5
+      - uses: actions/setup-node@v4
         with:
           node-version: ${{ matrix.node }}
       - run: node --version
@@ -29,32 +29,33 @@ jobs:
   windows:
     runs-on: windows-latest
     steps:
-      - uses: actions/checkout@v3
-      - uses: actions/setup-node@v3
+      - uses: actions/checkout@v5
+      - uses: actions/setup-node@v4
         with:
-          node-version: 14
+          node-version: 18
       - run: npm install
       - run: npm test
         env:
           MOCHA_THROW_DEPRECATION: false
   lint:
     runs-on: ubuntu-latest
     steps:
-      - uses: actions/checkout@v3
-      - uses: actions/setup-node@v3
+      - uses: actions/checkout@v5
+      - uses: actions/setup-node@v4
         with:
-          node-version: 14
+          node-version: 18
       - run: npm install
       - run: npm run lint
-  docs:
-    runs-on: ubuntu-latest
-    steps:
-      - uses: actions/checkout@v3
-      - uses: actions/setup-node@v3
-        with:
-          node-version: 14
-      - run: npm install
-      - run: npm run docs
-      - uses: JustinBeckwith/linkinator-action@v1
-        with:
-          paths: docs/
+# TODO(https://github.com/googleapis/nodejs-error-reporting/issues/777): enable docs
+#  docs:
+#    runs-on: ubuntu-latest
+#    steps:
+#      - uses: actions/checkout@v5
+#      - uses: actions/setup-node@v4
+#        with:
+#          node-version: 18
+#      - run: npm install
+#      - run: npm run docs
+#      - uses: JustinBeckwith/linkinator-action@v1
+#        with:
+#          paths: docs/

package.json

@@ -5,7 +5,7 @@
   "license": "Apache-2.0",
   "author": "Google Inc.",
   "engines": {
-    "node": ">=14.0.0"
+    "node": ">=18"
   },
   "repository": "googleapis/nodejs-error-reporting",
   "main": "./build/src/index.js",
@@ -31,39 +31,42 @@
     "precompile": "gts clean"
   },
   "dependencies": {
-    "@google-cloud/common": "^5.0.0",
+    "@google-cloud/common": "^6.0.0",
     "console-log-level": "^1.4.1"
   },
   "devDependencies": {
-    "@compodoc/compodoc": "1.1.23",
-    "@hapi/hapi": "^21.0.0",
-    "@types/boom": "^7.2.1",
+    "@compodoc/compodoc": "1.1.25",
+    "@hapi/hapi": "^21.4.4",
+    "@types/boom": "^7.3.5",
     "@types/console-log-level": "^1.4.0",
-    "@types/express": "^4.17.0",
-    "@types/hapi__hapi": "^20.0.5",
-    "@types/json-stable-stringify": "^1.0.32",
-    "@types/koa": "^2.0.48",
-    "@types/mocha": "^9.0.0",
-    "@types/node": "^20.4.9",
-    "@types/once": "^1.4.0",
-    "@types/proxyquire": "^1.3.28",
+    "@types/express": "^4.17.21",
+    "@types/json-stable-stringify": "^1.2.0",
+    "@types/koa": "^3.0.1",
+    "@types/mocha": "^10.0.10",
+    "@types/node": "^24.10.1",
+    "@types/once": "^1.4.5",
+    "@types/proxyquire": "^1.3.31",
     "@types/restify": "^8.0.0",
-    "@types/uuid": "^9.0.0",
-    "boom": "^7.3.0",
-    "c8": "^9.0.0",
-    "codecov": "^3.5.0",
+    "@types/uuid": "^8.3.0",
+    "boom": "^7.2.0",
+    "c8": "^10.1.3",
+    "codecov": "^3.6.2",
     "express": "^4.17.1",
-    "gts": "^5.0.0",
+    "gts": "^6.0.2",
+    "joi": "^17.0.0",
     "js-green-licenses": "^4.0.0",
-    "json-stable-stringify": "^1.0.1",
-    "koa": "^2.7.0",
-    "linkinator": "^4.0.0",
-    "mocha": "^9.2.2",
-    "nock": "^13.0.0",
-    "post-install-check": "0.0.1",
-    "proxyquire": "^2.1.0",
+    "json-stable-stringify": "^1.3.0",
+    "koa": "^3.1.1",
+    "linkinator": "^6.1.2",
+    "mocha": "^11.7.5",
+    "nock": "^14.0.10",
+    "pack-n-play": "^2.0.0",
+    "proxyquire": "^2.1.3",
     "restify": "^11.0.0",
-    "typescript": "^5.1.6",
-    "uuid": "^10.0.0"
+    "typescript": "^5.9.3",
+    "uuid": "^8.3.2"
+  },
+  "overrides": {
+    "undici": "5.28.4"
   }
 }

src/configuration.ts

@@ -306,7 +306,7 @@ export class Configuration {
     if (!isReportModeValid) {
       throw new Error(
         'config.reportMode must a string that is one ' +
-          'of "production", "always", or "never".'
+          'of "production", "always", or "never".',
       );
     }
 
@@ -316,15 +316,15 @@ export class Configuration {
     if (hasEnvCheck) {
       this._logger.warn(
         'The "ignoreEnvironmentCheck" config option is deprecated.  ' +
-          'Use the "reportMode" config option instead.'
+          'Use the "reportMode" config option instead.',
       );
     }
     if (hasEnvCheck && hasReportMode) {
       this._logger.warn(
         [
           'Both the "ignoreEnvironmentCheck" and "reportMode" configuration options',
           'have been specified.  The "reportMode" option will take precedence.',
-        ].join(' ')
+        ].join(' '),
       );
       this._determineReportMode();
     } else if (hasEnvCheck) {
@@ -349,7 +349,7 @@ export class Configuration {
           'if and only if the NODE_ENV environment variable is set to "production".',
           'Errors will not be reported.  To have errors always reported, regardless of the',
           'value of NODE_ENV, set the reportMode configuration option to "always".',
-        ].join(' ')
+        ].join(' '),
       );
     }
 

src/google-apis/auth-client.ts

@@ -105,14 +105,14 @@ export class RequestHandler extends Service {
         scopes: SCOPES,
         projectIdRequired: true,
       },
-      serviceOptions
+      serviceOptions,
     );
     this._config = config;
     this._logger = logger;
 
     if (!this._config.getShouldReportErrorsToAPI()) {
       this._logger.info(
-        'Not configured to send errors to the API; skipping Google Cloud API Authentication.'
+        'Not configured to send errors to the API; skipping Google Cloud API Authentication.',
       );
     } else if (tryAuthenticate) {
       this.authClient.getAccessToken().then(
@@ -123,9 +123,9 @@ export class RequestHandler extends Service {
               'Unable to find credential information on instance. This library',
               'will be unable to communicate with the Google Cloud API to save',
               'errors.  Message: ' + err.message,
-            ].join(' ')
+            ].join(' '),
           );
-        }
+        },
       );
     } else {
       this.request(
@@ -147,10 +147,10 @@ export class RequestHandler extends Service {
                 'Encountered an error while attempting to validate the provided',
                 'API key',
               ].join(' '),
-              err
+              err,
             );
           }
-        }
+        },
       );
       this._logger.info('API key provided; skipping OAuth2 token request.');
     }
@@ -172,8 +172,8 @@ export class RequestHandler extends Service {
     userCb?: (
       err: Error | null,
       response: http.ServerResponse | null,
-      body: {}
-    ) => void
+      body: {},
+    ) => void,
   ) {
     const cb: Function = (
       typeof userCb === 'function' ? userCb : RequestHandler.noOp
@@ -197,11 +197,11 @@ export class RequestHandler extends Service {
                 'Encountered an error while attempting to transmit an error to',
                 'the Error Reporting API.',
               ].join(' '),
-              err
+              err,
             );
           }
           cb(err, response, body);
-        }
+        },
       );
     } else {
       cb(
@@ -211,10 +211,10 @@ export class RequestHandler extends Service {
             'if and only if the NODE_ENV environment variable is set to "production".',
             'Errors will not be reported.  To have errors always reported, regardless of the',
             'value of NODE_ENV, set the reportMode configuration option to "always".',
-          ].join(' ')
+          ].join(' '),
         ),
         null,
-        null
+        null,
       );
     }
   }