Skip to content

Raise esbuild, Vitest and happy-dom past their security advisories - #10

Merged
imanimanyara merged 1 commit into
mainfrom
fix/esbuild-security
Oct 5, 2026
Merged

imanimanyara merged 1 commit into
mainfrom
fix/esbuild-security

Conversation

@imanimanyara

@imanimanyara imanimanyara commented Oct 5, 2026 •

Copy link
Copy Markdown
Member

Dependabot reports four advisories on tabar's dev dependencies, and its security updates had been failing on every run:

Package Advisory Severity Raised to
esbuild every release below 0.28.1; ^0.23 stops below 0.24, so the fix was out of range ^0.28.1
vitest below 3.2.6: the UI server can read and run arbitrary files critical ^3.2.6 (+ @vitest/coverage-v8)
happy-dom below 20: VM context escape to code execution critical ^20.8.9
happy-dom below 20.8.9: credentialed fetch sends page-origin cookies high ^20.8.9

Locally, on esbuild 0.28.2, Vitest 3.2.7 and happy-dom 20.14.5: build, 147 of 147 tests, lint and typecheck pass, and Vite now shares the patched esbuild instead of carrying 0.21. The esbuild commit alone passed all 6 checks.

Caveat: Vite 7, which Vitest 3 resolves to, needs Node 20.19+, and happy-dom 20 needs Node 20+, so the test (18) leg may no longer be able to run the suite. Node 18 has been end-of-life since April 2025. The library's own runtime floor (engines.node >=18) is unchanged here.

Every esbuild release below 0.28.1 is covered by an advisory, and ^0.23 stops below 0.24, so Dependabot's security update could not reach the fix and failed on each run. The build, 147 tests, lint and typecheck pass on 0.28.2. Vite 5 inside Vitest 2 still carries esbuild 0.21, which needs the Vitest upgrade.
Copilot AI balanced review requested due to automatic review settings October 5, 2026 14:01

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@imanimanyara
imanimanyara merged commit 3d7da75 into main Oct 5, 2026
6 checks passed
@imanimanyara imanimanyara changed the title Raise esbuild to ^0.28.1 for its security advisory Raise esbuild, Vitest and happy-dom past their security advisories Oct 5, 2026
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