Skip to content

fix: npm audit fix (brace-expansion DoS) - #347

Merged
S540d merged 2 commits into
testingfrom
fix/audit-brace-expansion
Oct 2, 2026
Merged

S540d merged 2 commits into
testingfrom
fix/audit-brace-expansion

Conversation

@S540d

@S540d S540d commented Oct 2, 2026

Copy link
Copy Markdown
Owner

Behebt die behebbare Schwachstelle brace-expansion per npm audit fix (nur package-lock.json). Tests: 460/460 grün.

Verbleibende hohe Findings (node-forge via expo → @expo/cli) haben keinen Upstream-Fix und bleiben, der Check „Security Audit" bleibt daher vorerst rot.

Zusätzlich Prettier-Formatierung für CLAUDE.md und docs/ROADMAP.md (vorbestehende Abweichungen blockierten den Pre-Push-Hook).

🤖 Generated with Claude Code

https://claude.ai/code/session_01TeuiRgABX9dgDxZYEze7aA


Generated by Claude Code

claude added 2 commits October 2, 2026 01:50
Behebt die behebbare Schwachstelle brace-expansion. Die verbleibenden
hohen Findings (node-forge via @expo/cli) haben keinen Upstream-Fix.

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01TeuiRgABX9dgDxZYEze7aA
Vorbestehende Abweichungen blockierten den Pre-Push-Hook.

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01TeuiRgABX9dgDxZYEze7aA
@github-actions

github-actions Bot commented Oct 2, 2026

Copy link
Copy Markdown

🔀 Mergeability-Report

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

CI-Checks: ✅ keine fehlgeschlagenen Checks

⏳ 🔍 Code Quality & Linting
⏳ ⚙️ Workflow Linting / actionlint
⏳ 🔒 Security Audit
✅ 🔑 Keystore & Credential Scan
⏳ 🔐 Docs Privacy Check
✅ 🔧 Platform Compatibility
⏳ mergeability / mergeability
⏳ 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 commented Oct 2, 2026

Copy link
Copy Markdown
Owner Author

🔒 Security Audit weiterhin rot – erwartet.

Dieser PR behebt brace-expansion (zuvor 6 hohe Findings, jetzt 5). Die verbleibenden 5 hohen Findings hängen alle an node-forge (via expo → @expo/cli → @expo/code-signing-certificates); node-forge hat in der Version 1.4.0 keinen Upstream-Fix, und npm audit fix --force würde expo@44 installieren. Bis dahin lässt sich der Check aus diesem PR nicht grün bekommen.


Generated by Claude Code

@S540d
S540d merged commit 04689b2 into testing Oct 2, 2026
13 of 14 checks passed
@S540d
S540d deleted the fix/audit-brace-expansion branch October 11, 2026 05:52
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.

2 participants