Compare commits

..
Author SHA1 Message Date
Clay GoodandClaude Opus 5.5 9111a7654d fix(security): accept the unpatched braces advisory in pnpm audit (#2048)
* fix(security): clear the unpatched braces advisory on main

pnpm audit --prod fails on main for GHSA-vfj7-8cjw-p6xm (braces <=3.0.3,
stack exhaustion on deeply nested patterns). braces ships at runtime via
fast-glob > micromatch, and no patched version exists, so no override can
fix it.

Reject artifact output patterns that nest braces more than 16 levels deep
before they reach fast-glob, and record the advisory in auditConfig with
that mitigation and a removal check.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>

* fix(security): keep only the audit exception for the braces advisory

Move the brace-nesting guard to a follow-up PR: it adds a user-visible
limit to schema `generates` that needs a docs-lab contract update and a
spec change. The audit exception alone clears main's Security workflow.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>

* fix(security): record the braces advisory's residual risk accurately

Name both inputs that reach fast-glob (generates and apply.tracks) and
state that a crafted schema can still crash the CLI, instead of relying on
the input cap or a failed local reproduction.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>

* fix(security): drop unsupported claims from the braces risk record

Schemas resolve from the project, user, or package directories, not a
store, and the input-length cap does not prevent stack exhaustion. State
only the accepted risk and the removal check.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>

---------

Co-authored-by: Claude Opus 5.5 <noreply@anthropic.com>
2026-10-06 00:21:34 +00:00
+12
View File
@@ -26,3 +26,15 @@ overrides:
# Dev-only (transitive via @vitest/ui); never in the published CLI. Remove once
# transitive fflate is >=0.8.3 (check: pnpm why fflate).
fflate@<0.8.3: '>=0.8.3 <0.9'
auditConfig:
ignoreGhsas:
# GHSA-vfj7-8cjw-p6xm — braces <=3.0.3 stack exhaustion on deeply nested
# patterns. Runtime (fast-glob > micromatch > braces), and no patched braces
# exists, so no override can fix it. Accepted residual risk: schema artifact
# `generates` and `apply.tracks` patterns reach fast-glob through
# resolveArtifactOutputs without a brace-depth guard, and the 10,000-character
# input limit does not prevent stack exhaustion. A crafted project-local or
# user schema can still terminate the openspec process. Remove this exception
# once braces ships a patched release (check: pnpm why braces; pnpm audit --prod).
- GHSA-vfj7-8cjw-p6xm