Kein CI und kein Linter: npm test ist reine Ehrensache #7

New issue

Open

opened 2026-10-18 23:23:34 +02:00 by gilli · 0 comments

gilli commented

2026-10-18 23:23:34 +02:00

Owner

Problem

Sieben Test-Suiten laufen nur auf Zuruf. AGENTS.md verlangt npm test vor jedem Commit, durchgesetzt wird es nirgends: kein .github/, keine Gitea-Actions-Datei, kein Linter (eslint.config.*, .prettierrc fehlen).

Ein grünes npm test ist reine Ehrensache — genau die Klasse von Regressionen, die dieses Repo schon einmal still durchgerutscht ist (siehe Commit „Fix two game-shell bugs").

Vorschlag

Eine Workflow-Datei, die npm ci && npm test auf dem PR ausführt. Der komplette Lauf dauert lokal ~5 s.
ESLint mit flacher Config als devDependency, npm run lint als Skript. Regelwerk zunächst minimal (unbekannte Variablen, unerreichter Code, no-unused-vars), damit der erste Durchlauf nicht hundert Meldungen wirft.

Prüfung

npm run lint läuft fehlerfrei; der Workflow ist auf einem Test-PR grün sichtbar.

## Problem Sieben Test-Suiten laufen nur auf Zuruf. `AGENTS.md` verlangt `npm test` vor jedem Commit, durchgesetzt wird es nirgends: kein `.github/`, keine Gitea-Actions-Datei, kein Linter (`eslint.config.*`, `.prettierrc` fehlen). Ein grünes `npm test` ist reine Ehrensache — genau die Klasse von Regressionen, die dieses Repo schon einmal still durchgerutscht ist (siehe Commit „Fix two game-shell bugs"). ## Vorschlag - Eine Workflow-Datei, die `npm ci && npm test` auf dem PR ausführt. Der komplette Lauf dauert lokal ~5 s. - ESLint mit flacher Config als devDependency, `npm run lint` als Skript. Regelwerk zunächst minimal (unbekannte Variablen, unerreichter Code, `no-unused-vars`), damit der erste Durchlauf nicht hundert Meldungen wirft. ## Prüfung `npm run lint` läuft fehlerfrei; der Workflow ist auf einem Test-PR grün sichtbar.

gilli added the

label

2026-10-18 23:23:34 +02:00