From 3f23982067c8aabf086db642996c7d1e95b5b30d Mon Sep 17 00:00:00 2001
From: ttbombadil <tom.tiltmann@th-koeln.de>
Date: Mon, 5 Oct 2026 08:02:35 +0200
Subject: [PATCH] fix(ci): make the lint command a read-only check

---
 package.json                          |  2 +-
 tests/server/ci-lint-contract.test.ts | 41 +++++++++++++++++++++++++++
 2 files changed, 42 insertions(+), 1 deletion(-)
 create mode 100644 tests/server/ci-lint-contract.test.ts

diff --git a/package.json b/package.json
index 6f9e2a2cd..4ff5cfebb 100644
--- a/package.json
+++ b/package.json
@@ -60,7 +60,7 @@
     "test:load:50": "LOG_LEVEL=info vitest run --project=load --testNamePattern='Load Test: 50 Concurrent Clients'",
     "test:load:100": "LOG_LEVEL=info vitest run --project=load --testNamePattern='Load Test: 100 Concurrent Clients'",
     "test:load:200": "LOG_LEVEL=info vitest run --project=load --testNamePattern='Load Test: 200 Concurrent Clients'",
-    "lint": "eslint . --ext .ts,.tsx --fix",
+    "lint": "eslint . --ext .ts,.tsx",
     "prepare": "husky"
   },
   "dependencies": {
diff --git a/tests/server/ci-lint-contract.test.ts b/tests/server/ci-lint-contract.test.ts
new file mode 100644
index 000000000..74fb5377f
--- /dev/null
+++ b/tests/server/ci-lint-contract.test.ts
@@ -0,0 +1,41 @@
+import { spawnSync } from "node:child_process";
+import { mkdtempSync, readFileSync, rmSync, writeFileSync } from "node:fs";
+import { tmpdir } from "node:os";
+import path from "node:path";
+import { describe, expect, it } from "vitest";
+
+describe("CI lint command", () => {
+  it("reports a fixable lint error without changing the checked source", () => {
+    const repository = process.cwd();
+    const manifest = JSON.parse(readFileSync(path.join(repository, "package.json"), "utf8"));
+    const fixture = mkdtempSync(path.join(tmpdir(), "unosim-lint-contract-"));
+    const source = "let value = 1; console.log(value);\n";
+
+    try {
+      writeFileSync(path.join(fixture, "package.json"), JSON.stringify({
+        type: "module",
+        scripts: { lint: manifest.scripts.lint },
+      }));
+      writeFileSync(path.join(fixture, "eslint.config.js"),
+        'export default [{ files: ["**/*.ts"], rules: { "prefer-const": "error" } }];\n');
+      writeFileSync(path.join(fixture, "example.ts"), source);
+
+      const result = spawnSync("npm", ["run", "lint"], {
+        cwd: fixture,
+        encoding: "utf8",
+        timeout: 10_000,
+        env: {
+          ...process.env,
+          PATH: `${path.join(repository, "node_modules", ".bin")}${path.delimiter}${process.env.PATH ?? ""}`,
+        },
+      });
+
+      expect(result.error).toBeUndefined();
+      expect(result.status).toBe(1);
+      expect(result.stdout).toContain("prefer-const");
+      expect(readFileSync(path.join(fixture, "example.ts"), "utf8")).toBe(source);
+    } finally {
+      rmSync(fixture, { recursive: true, force: true });
+    }
+  });
+});