chore: Ensure check-types work

Fix issues

Author: nikolaik

.github/workflows/ci.yml

@@ -27,6 +27,7 @@ jobs:
           pnpm install
           pnpm lint
           pnpm test
+          pnpm check-types
 
       - id: docker_meta
         uses: docker/metadata-action@030e881283bb7a6894de51c315a6bfe6a94e05cf # v6.0.0

package.json

@@ -22,6 +22,7 @@
     "typescript": "^6.0.2"
   },
   "devDependencies": {
+    "@tsconfig/node24": "^24.0.4",
     "@vitest/coverage-v8": "^4.1.4",
     "oxfmt": "^0.45.0",
     "oxlint": "^1.60.0",

pnpm-lock.yaml

@@ -36,10 +36,11 @@ export function formatUserList(users: any[]): Set<string> {
 export async function getGithubUsersFromGoogle(): Promise<Set<string>> {
   const service = await mod.getAdminService()
   let githubAccounts = new Set<string>()
-  let pageToken = null
+    // oxlint-disable-next-line no-undefined
+  let pageToken: string | undefined = undefined
 
   do {
-    const userList = await service.users.list({
+    const userList: { data: admin_directory_v1.Schema$Users } = await service.users.list({
       customFieldMask: 'Accounts',
       customer: 'my_customer',
       fields: 'users(customSchemas/Accounts/github(value)),nextPageToken',
@@ -48,8 +49,9 @@ export async function getGithubUsersFromGoogle(): Promise<Set<string>> {
       projection: 'custom',
       query: config.removeSuspendedUsers ? 'isSuspended=false' : '',
     })
-    pageToken = userList.data.nextPageToken
-    githubAccounts = new Set([...githubAccounts, ...formatUserList(userList.data.users)])
+    // oxlint-disable-next-line no-undefined
+    pageToken = userList.data.nextPageToken ?? undefined
+    githubAccounts = new Set([...githubAccounts, ...formatUserList(userList.data.users ?? [])])
   } while (pageToken != null)
   return githubAccounts
 }

src/google.ts

@@ -1,12 +1,8 @@
 {
+  "extends": "@tsconfig/node24/tsconfig.json",
   "compilerOptions": {
     "esModuleInterop": true,
     "declaration": true,
-    "lib": ["es2020"],
-    "target": "es2020",
-    "downlevelIteration": true,
-    "moduleResolution": "NodeNext",
-    "module": "NodeNext",
     "noEmit": true
   },
   "exclude": ["node_modules", "tests/**"]