Skip to content

fix(deps): raise smol-toml floor to >=1.8.0 for GHSA-7w5x-hrqm-74c2 - #106

Merged
acamarata merged 1 commit into
mainfrom
fix/smol-toml-ghsa-7w5x-hrqm-74c2
Sep 10, 2026
Merged

acamarata merged 1 commit into
mainfrom
fix/smol-toml-ghsa-7w5x-hrqm-74c2

Conversation

@acamarata

Copy link
Copy Markdown
Contributor

Raises the smol-toml override floor to >=1.8.0.

GHSA-7w5x-hrqm-74c2 — HIGH, CWE-835. smol-toml.parse() never returns when a value inside an array or inline table is followed by a comment with no trailing newline. parse("a=[1 #") pins a core at 100%. Vulnerable <=1.7.0, first patched 1.7.1.

Reached through @astrojs/internal-helpers@0.10.0, pulled by astro, @astrojs/react and @astrojs/vercel alike. The lockfile resolved 1.7.0.

Why 1.8.0 and not the advisory minimum. @astrojs/internal-helpers declares ^1.6.0, so 1.7.1, 1.7.2 and 1.8.0 all satisfy it, and 1.8.0 is current latest. A floor pinned at exactly the first patched version goes stale the moment 1.8.x carries its own advisory.

Why this is not a regression. The advisory was published after the 2026-09-09 hardening pass, which is why the scheduled audit runs went red overnight on commits that were green when merged. An override floor only protects against advisories that exist when it is written.

Verified: all four repo install roots resolve smol-toml@1.8.0; islamwiki/web astro build exits 0 against it with zero errors.

Merging this is also the ASI Policy 9.0 deploy approval for the repo it lands in.

smol-toml <=1.7.0 never returns from parse() when a value inside an array
or inline table is followed by a comment with no trailing newline. The
parser resets its cursor to the start of the string instead of exiting an
internal loop, so `parse('a=[1 #')` pins a core at 100 percent. HIGH,
CWE-835.

Reached through @astrojs/internal-helpers@0.10.0, which astro, @astrojs/react
and @astrojs/vercel all pull in. The lockfile resolved 1.7.0.

Floor set at 1.8.0 rather than the advisory's 1.7.1 minimum.
@astrojs/internal-helpers declares ^1.6.0, so the whole 1.7.1-1.8.0 range
satisfies it, and 1.8.0 is current latest. A floor pinned at exactly the
first patched version goes stale the moment 1.8.x carries its own advisory.

Published after the 2026-09-09 hardening pass, which is why three scheduled
audit runs went red overnight on commits that were green when merged.

Verified by islamwiki/web astro build against 1.8.0. smol-toml is only
invoked by Astro's build-time TOML parsing.

Refs D-2026-09-10-SMOLTOML-FLOOR
@vercel

vercel Bot commented Sep 10, 2026

Copy link
Copy Markdown

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

2 Skipped Deployments
Project Deployment Actions Updated
ummat-praycalc Ignored Ignored Sep 10, 2026 12:16pm UTC
ummat-praycalc-org Ignored Ignored Sep 10, 2026 12:16pm UTC

Request Review

@github-actions

Copy link
Copy Markdown

🟢 Rampart Security Gate — CLEAN

Tool CRITICAL HIGH MEDIUM LOW
(no findings)

Totals: 0 critical · 0 high · 0 medium · 0 low

Mode: RAMPART_ENFORCE=warn

@acamarata
acamarata merged commit 8f1e99c into main Sep 10, 2026
18 checks passed
@acamarata
acamarata deleted the fix/smol-toml-ghsa-7w5x-hrqm-74c2 branch September 10, 2026 12:27
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