Skip to content

fix(deps): Next.js 16.3.8 and patched transitive packages; dated braces exception (#115) - #116

Merged
Fluory merged 1 commit into
mainfrom
claude/fix-deps-audit-115
Oct 9, 2026
Merged

Fluory merged 1 commit into
mainfrom
claude/fix-deps-audit-115

Conversation

@Fluory

@Fluory Fluory commented Oct 9, 2026 •

Copy link
Copy Markdown
Owner

Warum

Fixes #115 · blockiert jeden Code-PR (gefunden in #114)

Arbeitsstand

  • Ziel: pnpm audit --audit-level high wieder grün.
  • Nicht-Ziele: moderate/low ohne High (esbuild via drizzle-kit).
  • Erledigt: next + eslint-config-next 16.3.8, Lockfile: sharp 0.35.5, source-map-js 1.2.2; braces-Ausnahme (nur diese GHSA) + Register-Eintrag; CHANGELOG.
  • Offen: Merge nach Review.
  • Annahmen: keine.
  • Nächster kleinster Schritt: Review.

Was ist passiert (Klartext)

Seit Ende September sind Sicherheitsmeldungen zu Next.js und zwei Unterpaketen erschienen; die automatische Sicherheitsprüfung hat deshalb jede Code-Änderung blockiert. Die Pakete sind jetzt auf gepatchten Versionen. Für ein Paket (braces) gibt es noch keinen Fix; es steckt nur im Lint-Werkzeug, nicht in der ausgelieferten App, und ist darum als befristete, dokumentierte Ausnahme eingetragen.

Plan-Pflicht (SYSTEM.md §4)

  • Kein Auslöser – keine Modulgrenze, öffentliche API, Migration, Auth/Rechte, kein Zahlungs-/Daten-/Infrapfad, höchstens zwei Module, keine Architekturvarianten, umkehrbar
  • Auslöser zutreffend – Impact Manifest ausgefüllt (Plan vor Code)

Geändert

  • package.json: next/eslint-config-next 16.3.8; pnpm.auditConfig.ignoreGhsas = GHSA-vfj7-8cjw-p6xm
  • pnpm-lock.yaml: dazu Patch-Sprünge im vite-Teilbaum (postcss 8.5.28→8.5.29, rolldown 1.2.9→1.2.13), innerhalb der Bereiche
  • docs/technical/architecture.md: Ausnahmen-Register; CHANGELOG.md: Security

Nachweis (SYSTEM.md §11)

  • verify:changed: grün – pnpm audit --audit-level high Exit 0 (1 high ignoriert, 1 moderate)
  • verify: grün – CI check Run 37975167713 (inkl. E2E, Label verify-full)
  • verify:full / E2E-Spec: Label verify-full (UI-Framework-Upgrade)
  • Manueller Prüfnachweis: pnpm why braces --prod leer
  • Frischer Review (P1 vor Ready-for-review; Architektur/API/DB immer): noch offen

Doku-Entscheidung (genau eine)

Entferntes oder Umbenanntes: docs/ + README gegrept, Treffer bereinigt: nichts entfernt

Dateigrößen und neue Bausteine (SYSTEM.md §7)

Dateien über 500 Zeilen im Diff (Ausnahmen: generierter Code, Lockfiles, Fixtures, Migrationen, Schemas, Ressourcen, Doku, Konfiguration):

  • keine
  • bewusst belassen – Begründung: –
  • im selben PR nach fachlicher Verantwortung geteilt
  • Folge-Issue –

Über 800 Zeilen mit neuer Fachlogik oder über 1000 Zeilen (P1/P2): nicht betroffen

Neue Shared-Komponente, Utility-Datei, Adapter oder fachlicher Service:

  • nein
  • ja – gesucht nach: –

Risiken / offene Punkte

  • Patch-Upgrade des UI-Frameworks; Rollback = Revert.

🤖 Generated with Claude Code

…races exception (#115)

pnpm audit --audit-level high failed on advisories published after
2026-09-28 and blocked every code PR. braces has no patched version and is
reached only through lint tooling, so it is ignored by GHSA and registered
as a dated exception.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@Fluory Fluory added the verify-full Run verify:full (integration + E2E) in CI label Oct 9, 2026
@vercel

vercel Bot commented Oct 9, 2026 •

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated
requestflow Ready Ready Preview Oct 9, 2026 6:48pm UTC
requestflow-ai Error Error Oct 9, 2026 6:48pm UTC

@Fluory

Fluory commented Oct 9, 2026

Copy link
Copy Markdown
Owner Author

Frischer Review (Reviewer-Rolle)

  • [Note] CI – check (inkl. E2E) und compose-smoke grün. Vercel – requestflow-ai rot, aber identisch schon auf main – nicht durch diesen PR verursacht.
  • [Note] package.json – nur next und eslint-config-next 16.3.6 → 16.3.8 (im Gleichschritt) plus pnpm.auditConfig.ignoreGhsas mit genau GHSA-vfj7-8cjw-p6xm. Keine weiteren Versionssprünge.
  • [Note] pnpm-lock.yaml – sharp 0.35.5 (+ libvips 1.3.4), source-map-js 1.2.2 (laut Beschreibung), keine Overrides; zusätzlich Patch-Sprünge rolldown 1.2.9→1.2.13 / postcss 8.5.29 im vite-Teilbaum, innerhalb der Bereiche, in der PR-Beschreibung offengelegt – unkritisch.
  • [Note] docs/technical/architecture.md – Register-Eintrag mit Begründung (nur Dev-Tooling-Pfad, pnpm why braces --prod leer), Owner und Ablauf spätestens 2026-12-31. Erfüllt das Kriterium.
  • [Note] CHANGELOG.md – Security-Eintrag unter [Unreleased] mit Verweis security(deps): high audit advisories on next, sharp, source-map-js; braces has no fix #115.
  • [Note] Secrets – keine im Diff. PR-Klartext verständlich; Manifest-Plan-Pflicht korrekt als nicht ausgelöst begründet.

Verdict: mergeable – keine Blocker; alle Akzeptanzkriterien aus #115 erfüllt.

@Fluory
Fluory merged commit 30b3d4a into main Oct 9, 2026
7 of 11 checks passed
@Fluory
Fluory deleted the claude/fix-deps-audit-115 branch October 9, 2026 18:57

This branch had an error being deployed

1 failed and 1 active deployments
Preview – requestflow — fa879038 Deployed Oct 9, 2026 by vercel[bot]
Preview – requestflow-ai — fa879038 Deployed Oct 9, 2026 by vercel[bot]
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

verify-full Run verify:full (integration + E2E) in CI

Projects

None yet

Development

Successfully merging this pull request may close these issues.

security(deps): high audit advisories on next, sharp, source-map-js; braces has no fix

1 participant