Deploy to production 🚀

.github/workflows/build.yml

@@ -21,7 +21,7 @@ jobs:
       - name: Setup Node.js
         uses: actions/setup-node@v4
         with:
-          node-version: 22
+          node-version: 24
       - name: Setup Environment
         run: cp .env.example .env.local
       - name: Install Dependencies

.github/workflows/lint.yml

@@ -18,6 +18,10 @@ jobs:
         uses: actions/checkout@v4
         with:
           persist-credentials: false
+      - name: Setup Node.js
+        uses: actions/setup-node@v4
+        with:
+          node-version: 24
       - name: Install modules
         run: npm install
       - name: Run ESLint

.github/workflows/run-unit-tests.yml

@@ -21,7 +21,7 @@ jobs:
       - name: Setup Node.js
         uses: actions/setup-node@v4
         with:
-          node-version: 22
+          node-version: 24
       - name: Install Dependencies
         run: npm install
       - name: Run Unit Tests

.nvmrc

@@ -1 +1 @@
-v20.13.1
+24

Dockerfile

@@ -1,4 +1,4 @@
-FROM node:20.8.1-alpine AS base
+FROM node:24-alpine AS base
 
 # Install dependencies only when needed
 FROM base AS deps

__test__/auth/AuthjsAccountsOAuthTokenRepository.test.ts

@@ -14,7 +14,7 @@ test("It gets token for user ID and provider", async () => {
           async disconnect() {},
         }
       },
-      async query(_query, values: any[] = []) {
+      async query(_query, values: readonly unknown[] = []) {
         queryProvider = values[0]
         queryUserId = values[1]
         return {
@@ -44,7 +44,7 @@ test("It does not set token", async () => {
           async disconnect() {},
         }
       },
-      async query(_query, _values: any[] = []) {
+      async query(_query, _values: readonly unknown[] = []) {
         didSetToken = true
         return { rows: [] }
       }
@@ -67,7 +67,7 @@ test("It does not delete token", async () => {
           async disconnect() {},
         }
       },
-      async query(_query, _values: any[] = []) {
+      async query(_query, _values: readonly unknown[] = []) {
         didDeleteToken = true
         return { rows: [] }
       }

__test__/auth/OAuthTokenRepository.test.ts

@@ -8,13 +8,13 @@ test("It reads the auth token for the specified user", async () => {
     db: {
       async connect() {
         return {
-          async query(_query: string, _values: any[] = []) {
+          async query(_query: string, _values: readonly unknown[] = []) {
             return { rows: [] }
           },
           async disconnect() {},
         }
       },
-      async query(_query: string, values: any[] = []) {
+      async query(_query: string, values: [string, string]) {
         readProvider = values[0]
         readUserId = values[1]
         return {
@@ -36,20 +36,20 @@ test("It reads the auth token for the specified user", async () => {
 test("It stores the auth token for the specified user", async () => {
   let storedProvider: string | undefined
   let storedUserId: string | undefined
-  let storedAccessToken: any | undefined
-  let storedRefreshToken: any | undefined
+  let storedAccessToken: string | undefined
+  let storedRefreshToken: string | undefined
   const sut = new OAuthTokenRepository({
     provider: "github",
     db: {
       async connect() {
         return {
-          async query(_query: string, _values: any[] = []) {
+          async query(_query: string, _values: readonly unknown[] = []) {
             return { rows: [] }
           },
           async disconnect() {},
         }
       },
-      async query(_query: string, values: any[] = []) {
+      async query(_query: string, values: [string, string, string, string]) {
         storedProvider = values[0]
         storedUserId = values[1]
         storedAccessToken = values[2]
@@ -76,13 +76,13 @@ test("It deletes the auth token for the specified user", async () => {
     db: {
       async connect() {
         return {
-          async query(_query: string, _values: any[] = []) {
+          async query(_query: string, _values: readonly unknown[] = []) {
             return { rows: [] }
           },
           async disconnect() {},
         }
       },
-      async query(_query: string, values: any[] = []) {
+      async query(_query: string, values: [string, string]) {
         deletedUserId = values[1]
         return { rows: [] }
       }

__test__/common/github/OAuthTokenRefreshingGitHubClient.test.ts

@@ -285,6 +285,7 @@ test("It does not refresh an OAuth token when the initial request was successful
     },
     oauthTokenRefresher: {
       async refreshOAuthToken(_refreshToken) {
+        didRefreshOAuthToken = true
         return {
           accessToken: "newAccessToken",
           refreshToken: "newRefreshToken"

__test__/common/utils/saneParseInt.test.ts

@@ -1,7 +1,7 @@
 import { saneParseInt } from "@/common"
 
 test("It parses an integer", async () => {
-  // @ts-ignore
+  // @ts-expect-error - verifying that non-string input is rejected
   const val = saneParseInt(42 as string)
   expect(val).toBe(42)
 })

__test__/projects/GitHubRepositoryDataSource.test.ts

@@ -189,7 +189,7 @@ test("It queries for both .yml and .yaml file extension with no extension", asyn
 })
 
 test("It loads repositories for all logins", async () => {
-  let searchQueries: string[] = []
+  const searchQueries: string[] = []
   const sut = new GitHubRepositoryDataSource({
     repositoryNameSuffix: "-openapi",
     projectConfigurationFilename: ".demo-docs",

eslint.config.mjs

@@ -0,0 +1,31 @@
+import js from "@eslint/js"
+import nextCoreWebVitals from "eslint-config-next/core-web-vitals"
+import tseslint from "typescript-eslint"
+
+const config = [
+  ...nextCoreWebVitals,
+  js.configs.recommended,
+  ...tseslint.configs.recommended,
+  {
+    ignores: ["next-env.d.ts", ".next"],
+    rules: {
+      "array-callback-return": ["error"],
+      "no-await-in-loop": ["error"],
+      "no-constant-binary-expression": ["error"],
+      "no-constructor-return": ["error"],
+      "no-duplicate-imports": ["error"],
+      "no-new-native-nonconstructor": ["error"],
+      "no-self-compare": ["error"],
+      "no-template-curly-in-string": ["error"],
+      "no-unmodified-loop-condition": ["error"],
+      "no-unreachable-loop": ["error"],
+      "no-unused-private-class-members": ["error"],
+      "require-atomic-updates": ["error"],
+      "@typescript-eslint/no-unused-vars": ["error", {
+        argsIgnorePattern: "^_",
+      }],
+    },
+  }
+]
+
+export default config

next.config.js

@@ -1,10 +1,5 @@
 /** @type {import('next').NextConfig} */
 const nextConfig = {
-  eslint: {
-    // Allows production builds to successfully complete even if it has linting errors.
-    // This is only OK because we do linting as part of our CI setup.
-    ignoreDuringBuilds: true,
-  },
   // Output standalone to be used for Docker builds.
   output: 'standalone',
 }