Skip to content

feat: Linter, Doku-Gate und Abhängigkeitsprüfung als PR-Checks - #2

Merged
Computerhof merged 1 commit into
mainfrom
feat/lint-doku-gate
Oct 23, 2026
Merged

Computerhof merged 1 commit into
mainfrom
feat/lint-doku-gate

Conversation

@Computerhof

Copy link
Copy Markdown
Owner

Was ändert sich?

Drei neue Pull-Request-Checks und Dependabot für npm:

Check Inhalt
Lint (JS, Markdown, PowerShell) ESLint (Flat Config), markdownlint, PSScriptAnalyzer — npm run lint
Doku-Gate scripts/check-docs.js: CHANGELOG-Eintrag bei Produktänderungen, Entwickler-Doku bei Werkzeug-Änderungen, Konsistenz von npm-Skripten, Links und CHANGELOG. Ausnahmen: Dependabot-Abhängigkeitsupdates, Label keine-doku (Warnung statt Fehler)
Abhängigkeiten prüfen Dependency Review: keine bekannten Sicherheitslücken, keine GPL/AGPL-Lizenzen in neuen Paketen/Actions

Dependabot: npm + GitHub Actions wöchentlich (Mo 06:00 Wien), Minor/Patch gebündelt, Labels dependencies + Ökosystem, Präfix chore(deps). Sicherheitsupdates sind in den Repo-Einstellungen aktiviert.

Bewusste Entscheidung: markdownlint als Bibliothek statt markdownlint-cli2 — die CLI zieht braces (DoS-Meldung, für keine Version behoben) mit; katex (über markdownlint) per overrides auf die behobene 0.19. npm audit: 0 Meldungen.

Lint-Befunde behoben (ohne Verhaltensänderung): unnötige Regex-Escapes in core.js ([\/\] → [/\], durch Test 9c abgedeckt), ungenutzter Import, build.ps1 mit BOM (PS 5.1 zeigte Umlaute falsch) und Write-Output, Markdown-Formatierung.

Prüfung

  • Doku-Gate-Regeln: 24 Tests (test/gates.test.js), dazu Negativtest am echten Repo: Code-Änderung ohne CHANGELOG → blockiert; mit Label → Warnung
  • PSScriptAnalyzer-Negativtest (Probe mit Write-Host) → schlägt an
  • Kern-Tests 229/229, Browser-Smoke-Test (headless Edge) mit test/sample.zip: Sollwerte unverändert

Checkliste

  • Änderungen nur in src/, test/, scripts/ oder Doku — keine gebaute HTML eingecheckt
  • Keine echten Reportdaten im Diff
  • npm run gate:data, npm run lint und npm test lokal grün
  • Im Browser mit test/sample.zip geprüft
  • CHANGELOG.md ergänzt und betroffene Doku im selben PR angepasst

🤖 Generated with Claude Code

- Lint-Job: ESLint (Flat Config), markdownlint (Bibliothek statt CLI, um
  Pakete mit offenen Sicherheitsmeldungen zu vermeiden; katex per override
  auf 0.19), PSScriptAnalyzer
- Doku-Gate (scripts/check-docs.js) mit Regel-Tests: CHANGELOG bei
  Produktänderungen, Entwickler-Doku bei Werkzeug-Änderungen, Konsistenz von
  npm-Skripten, Links und CHANGELOG; Ausnahmen Dependabot und Label keine-doku
- Dependency Review als PR-Check; Dependabot für npm und Actions, wöchentlich
  und gebündelt
- Lint-Befunde behoben: unnötige Regex-Escapes, ungenutzter Import,
  build.ps1 mit BOM und Write-Output, Markdown-Formatierung

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@Computerhof
Computerhof merged commit 63d8c4f into main Oct 23, 2026
4 checks passed
@Computerhof
Computerhof deleted the feat/lint-doku-gate branch October 23, 2026 11:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant