fix: update various dependencies and linting (#1622)

Thank you for opening a Pull Request! Before submitting your PR, there are a few things you can do to make sure it goes smoothly:
- [ ] Make sure to open an issue as a [bug/issue](https://github.com/googleapis/gax-nodejs/issues/new/choose) before writing your code!  That way we can discuss the change, evaluate designs, and agree on the general idea
- [ ] Ensure the tests and linter pass
- [ ] Code coverage does not decrease (if any source code was changed)
- [ ] Appropriate docs were updated (if necessary)

Fixes #<issue_number_goes_here> 🦕

gax/package.json

@@ -10,18 +10,18 @@
     "!build/src/**/*.map"
   ],
   "dependencies": {
-    "@grpc/grpc-js": "~1.10.3",
+    "@grpc/grpc-js": "^1.10.9",
     "@grpc/proto-loader": "^0.7.13",
     "@types/long": "^4.0.0",
     "abort-controller": "^3.0.0",
     "duplexify": "^4.0.0",
     "google-auth-library": "^9.3.0",
     "node-fetch": "^2.6.1",
     "object-hash": "^3.0.0",
-    "proto3-json-serializer": "^2.0.0",
+    "proto3-json-serializer": "^2.0.2",
     "retry-request": "^7.0.0",
     "uuid": "^9.0.1",
-    "protobufjs": "7.3.0"
+    "protobufjs": "^7.3.2"
   },
   "devDependencies": {
     "@types/uuid": "^9.0.7",
@@ -51,7 +51,7 @@
     "proxyquire": "^2.0.1",
     "pumpify": "^2.0.0",
     "rimraf": "^5.0.1",
-    "sinon": "^17.0.0",
+    "sinon": "^18.0.0",
     "stream-events": "^1.0.4",
     "ts-loader": "^8.0.0",
     "typescript": "^5.1.6",

gax/src/fallbackRest.ts

@@ -83,7 +83,7 @@ export function decodeResponse(
   ok: boolean,
   response: Buffer | ArrayBuffer
 ): {} {
-  // eslint-disable-next-line node/no-unsupported-features/node-builtins
+  // eslint-disable-next-line n/no-unsupported-features/node-builtins
   const decodedString = new TextDecoder().decode(response);
   const json = JSON.parse(decodedString);
   if (!ok) {

gax/src/featureDetection.ts

@@ -21,9 +21,9 @@ const features = {
     typeof window !== 'undefined' &&
     window?.fetch &&
     typeof window?.fetch === 'function',
-  // eslint-disable-next-line node/no-unsupported-features/node-builtins
+  // eslint-disable-next-line n/no-unsupported-features/node-builtins
   textEncoder: typeof TextEncoder !== 'undefined',
-  // eslint-disable-next-line node/no-unsupported-features/node-builtins
+  // eslint-disable-next-line n/no-unsupported-features/node-builtins
   textDecoder: typeof TextDecoder !== 'undefined',
   nodeJS: typeof process !== 'undefined' && process?.versions?.node,
   abortController: typeof AbortController !== 'undefined',

gax/test/browser-test/karma.conf.js

@@ -22,7 +22,7 @@ const isDocker = require('is-docker')();
 const webpackConfig = require('./webpack.config.js');
 webpackConfig.performance = {hints: false};
 delete webpackConfig.entry;
-// eslint-disable-next-line node/no-missing-require
+// eslint-disable-next-line n/no-missing-require
 process.env.CHROME_BIN = require('puppeteer').executablePath();
 
 module.exports = function (config) {
@@ -38,7 +38,7 @@ module.exports = function (config) {
       childProcess: {
         path: path.relative(
           process.cwd(),
-          // eslint-disable-next-line node/no-missing-require
+          // eslint-disable-next-line n/no-missing-require
           require.resolve('showcase-server/build/src/index.js')
         ),
         args: [],

tools/src/compileProtos.ts

@@ -452,7 +452,7 @@ function usage() {
 if (require.main === module) {
   if (process.argv.length <= 2) {
     usage();
-    // eslint-disable-next-line no-process-exit
+    // eslint-disable-next-line n/no-process-exit
     process.exit(1);
   }
   // argv[0] is node.js binary, argv[1] is script path

tools/src/minify.ts

@@ -72,7 +72,7 @@ if (require.main === module) {
 
   if (process.argv[2] === '--help') {
     usage();
-    // eslint-disable-next-line no-process-exit
+    // eslint-disable-next-line n/no-process-exit
     process.exit(1);
   }
 

tools/src/prepublish.ts

@@ -67,7 +67,7 @@ if (require.main === module) {
 
   if (process.argv[2] === '--help') {
     usage();
-    // eslint-disable-next-line no-process-exit
+    // eslint-disable-next-line n/no-process-exit
     process.exit(1);
   }
 

tools/src/replaceImportMetaUrl.ts

@@ -12,7 +12,7 @@
 // See the License for the specific language governing permissions and
 // limitations under the License.
 
-// eslint-disable-next-line node/no-extraneous-import
+// eslint-disable-next-line n/no-extraneous-import
 import {smart} from '@babel/template';
 import {Statement} from '@babel/types';
 import {Visitor} from '@babel/core';