chore: linter eslint fix - #9088

bin/linter.mjs (99-107)

In 'getChangedFilesStrict()', if 'err.status' is 1, the catch block does not throw an error and the function implicitly returns 'undefined'. This will cause a 'TypeError' when 'changedTsFiles.length' is accessed in 'run()'. Since the command being run is 'git diff --name-only' (not '--quiet'), any non-zero exit code indicates a failure and should be thrown. Additionally, the error message incorrectly references 'git diff --quiet' instead of the actual command run.

  } catch (err) {
    throw new Error(
      "Strict mode error: git diff " + gitDiffArg + " failed with exit code " + err.status + ".\n" +
      "Ensure that the git reference '" + gitDiffArg + "' exists locally and that you have fetched the required commits/branches.\n" +
      "Details: " + String(err.stderr || err.message || "").trim()
    );
  }

.eslintrc.json (67-68)

There is a duplicate entry for 'handwritten/firestore/dev/test/gapic_firestore_admin_v1.ts' in the 'files' array. Removing the duplicate entry improves readability and maintainability.

        "handwritten/firestore/dev/test/gapic_firestore_admin_v1.ts"

handwritten/spanner-driver/package.json (59-61)

The minimum Node.js version is specified as '>=18', but all other packages in this monorepo are being upgraded to '>=22' in this pull request. To maintain consistency across the repository, please update the engine requirement to '>=22'.

  "engines": {
    "node": ">=22"
  }