Merge pull request #615 from shapehq/next16

Updates to Next 16 and Node 24

Author: simonbs

.eslintrc.json

@@ -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/build.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/lint.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

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

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

.nvmrc

@@ -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

Dockerfile

@@ -1,38 +1,13 @@
-import { defineConfig } from "eslint/config"
-import typescriptEslint from "@typescript-eslint/eslint-plugin"
-import tsParser from "@typescript-eslint/parser"
-import path from "node:path"
-import { fileURLToPath } from "node:url"
 import js from "@eslint/js"
-import { FlatCompat } from "@eslint/eslintrc"
+import nextCoreWebVitals from "eslint-config-next/core-web-vitals"
+import tseslint from "typescript-eslint"
 
-const __filename = fileURLToPath(import.meta.url)
-const __dirname = path.dirname(__filename)
-const compat = new FlatCompat({
-  baseDirectory: __dirname,
-  recommendedConfig: js.configs.recommended,
-  allConfig: js.configs.all
-})
-
-export default defineConfig([
+const config = [
+  ...nextCoreWebVitals,
+  js.configs.recommended,
+  ...tseslint.configs.recommended,
   {
     ignores: ["next-env.d.ts", ".next"],
-  },
-  {
-    extends: [
-      ...compat.extends("next/core-web-vitals"),
-      ...compat.extends("eslint:recommended"),
-      ...compat.extends("plugin:@typescript-eslint/recommended")
-    ],
-  
-    plugins: {
-      "@typescript-eslint": typescriptEslint,
-    },
-  
-    languageOptions: {
-      parser: tsParser,
-    },
-  
     rules: {
       "array-callback-return": ["error"],
       "no-await-in-loop": ["error"],
@@ -46,10 +21,11 @@ export default defineConfig([
       "no-unreachable-loop": ["error"],
       "no-unused-private-class-members": ["error"],
       "require-atomic-updates": ["error"],
-  
       "@typescript-eslint/no-unused-vars": ["error", {
         argsIgnorePattern: "^_",
       }],
     },
   }
-])
+]
+
+export default config

eslint.config.mjs

@@ -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',
 }