Skip to content

fix: npm audit fix gegen High-Severity in js-yaml - #333

Merged
S540d merged 1 commit into
testingfrom
fix/js-yaml-security-audit
Sep 20, 2026
Merged

S540d merged 1 commit into
testingfrom
fix/js-yaml-security-audit

Conversation

@S540d

@S540d S540d commented Sep 20, 2026

Copy link
Copy Markdown
Owner

Problem

CI-Job "Security Audit" schlug auf testing fehl (PR #331, Run 35110374165): neue high-severity Lücke in js-yaml (CPU-DoS via maxTotalMergeKeys, GHSA-2883-xcg3-v3hh), transitiv über eslint/@eslint/eslintrc — reine devDependency, nicht im Laufzeit-Bundle.

Fix

npm audit fix (ohne --force) — kein Breaking Change laut npm. Nur package-lock.json betroffen.

Verifiziert

  • npm audit --audit-level=high → exit 0 (16 moderate verbleibend, bekannt/dokumentiert)
  • npx tsc --noEmit → clean
  • npm run lint → clean
  • npm run test:ci → 453/453 Tests grün, Coverage über Schwellenwerten
  • npm run build:web → erfolgreich

🤖 Generated with Claude Code

CI Security-Audit-Job schlug auf testing fehl (js-yaml CPU-DoS via
maxTotalMergeKeys, transitiv über eslint/@eslint/eslintrc). Nur
devDependency-seitig betroffen. npm audit fix (ohne --force) behebt es
ohne Breaking Change; verbleibend 16 moderate Findings (bekannt,
dokumentiert in CLAUDE.md).

Verifiziert: npm audit --audit-level=high exit 0, tsc --noEmit clean,
lint clean, 453 Tests grün, Web-Build erfolgreich.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
@github-actions

Copy link
Copy Markdown

🔀 Mergeability-Report

✅ Konfliktfrei mergebar (unstable).
✅ Ziel-Branch testing ist policy-konform.

CI-Checks: ✅ keine fehlgeschlagenen Checks

⏳ ⚙️ Workflow Linting / actionlint
⏳ 🔑 Keystore & Credential Scan
⏳ 🔍 Code Quality & Linting
⏳ mergeability / mergeability
⏳ 🔐 Docs Privacy Check
⏳ 🔧 Platform Compatibility
⏳ 🔒 Security Audit
⏳ Analyze (javascript-typescript)
⏳ Analyze (actions)

Merge-Checkliste

  • CI-Checks grün
  • Keine Merge-Konflikte
  • Ziel-Branch korrekt
  • (Optional) tiefes KI-Review via /review aus Claude Code (Pro-Abo, ohne API-Kosten)

Kostenlos generiert von reusable-mergeability.yml · keine API-Kosten · review-gate: success

@S540d
S540d merged commit a9c4383 into testing Sep 20, 2026
14 checks passed
@S540d
S540d deleted the fix/js-yaml-security-audit branch September 20, 2026 15:54
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