chore: migrate to nx 18 (#745)

.env

@@ -0,0 +1,4 @@
+# Nx 18 enables using plugins to infer targets by default
+# This is disabled for existing workspaces to maintain compatibility
+# For more info, see: https://nx.dev/concepts/inferred-tasks
+NX_ADD_PLUGINS=false

.github/workflows/ci.yml

@@ -53,7 +53,7 @@ jobs:
           RELEASE_VERSION: ${{ github.sha }}
       - name: Build
         run: |
-          npx nx run-many -t build --all --parallel=3
+          npx nx run-many -t build --prod --all --parallel=3
           docker build -t kordis-api:${{ github.sha }} -f ./apps/api/Dockerfile --build-arg NODE_VERSION=$(cat .nvmrc | tr -cd '[:digit:].') . &
           docker build -t kordis-spa:${{ github.sha }} -f ./apps/spa/docker/Dockerfile . &
           wait

apps/api/Dockerfile

@@ -9,7 +9,8 @@ ENV NODE_ENV=production
 # Install dependencies separately for caching
 COPY ./dist/apps/api/package.json ./dist/apps/api/package-lock.json ./
 
-RUN npm --omit=dev ci
+# Change to npm ci when https://github.com/nrwl/nx/issues/22386 is fixed
+RUN npm install --omit=dev
 
 COPY ./dist/apps/api ./
 

apps/api/project.json

@@ -15,14 +15,13 @@
 				"tsConfig": "apps/api/tsconfig.app.json",
 				"assets": ["apps/api/src/assets"],
 				"webpackConfig": "apps/api/webpack.config.js",
-				"generatePackageJson": true,
-				"isolatedConfig": true,
-				"babelUpwardRootMode": true
+				"isolatedConfig": true
 			},
 			"configurations": {
 				"production": {
 					"optimization": true,
 					"extractLicenses": true,
+					"generatePackageJson": true,
 					"inspect": false
 				}
 			}
@@ -67,10 +66,6 @@
 			"options": {
 				"jestConfig": "apps/api/jest.config.ts"
 			}
-		},
-		"docker-build": {
-			"dependsOn": ["build"],
-			"command": "docker build --build-arg NODE_VERSION=$(cat .nvmrc | tr -cd [:digit:].) -f apps/api/Dockerfile . -t api"
 		}
 	},
 	"tags": []

migrations.json

@@ -2,11 +2,21 @@
   "migrations": [
     {
       "cli": "nx",
-      "version": "17.2.1-beta.0",
-      "description": "Add webpack.config.js file when webpackConfig is not defined",
-      "implementation": "./src/migrations/update-17-2-1/webpack-config-setup",
-      "package": "@nx/webpack",
-      "name": "update-17-2-1-webpack-config-setup"
+      "version": "18.2.0-beta.0",
+      "requires": {
+        "@angular/core": ">=17.3.0"
+      },
+      "description": "Update the @angular/cli package version to ~17.3.0.",
+      "factory": "./src/migrations/update-18-2-0/update-angular-cli",
+      "package": "@nx/angular",
+      "name": "update-angular-cli-version-17-3-0"
+    },
+    {
+      "version": "17.3.0",
+      "description": "Updates two-way bindings that have an invalid expression to use the longform expression instead.",
+      "factory": "./migrations/invalid-two-way-bindings/bundle",
+      "package": "@angular/core",
+      "name": "invalid-two-way-bindings"
     }
   ]
 }

nx.json

@@ -26,6 +26,10 @@
 					"codeCoverage": true
 				}
 			}
+		},
+		"@nx/eslint:lint": {
+			"inputs": ["default", "{workspaceRoot}/.eslintrc.json"],
+			"cache": true
 		}
 	},
 	"namedInputs": {