chore(deps): clear all open Dependabot alerts, refresh in-range dependencies - #255
Merged
Conversation
Closes all eight open Dependabot alerts (two high, six moderate) plus two more that npm audit reports: ten advisories across five packages, every one fixed inside the existing version constraints. Five of them are guzzlehttp/guzzle (7.12.1 -> 7.15.1): proxy-authorization headers sent to origin servers, URI fragments disclosed in redirect Referer headers, unbounded response cookies, host-only cookie scope not preserved, and cookie disclosure via IP-address domains. One is guzzlehttp/psr7 (2.12.1 -> 2.13.0), host confusion via weak URI host validation (CVE-2026-59882). Both packages reach LWT through league/oauth2-google, so these affect the running application rather than only the toolchain. The rest are JS toolchain, not shipped to users: brace-expansion (two DoS advisories), linkify-it and postcss. The brace-expansion override floor moves >=5.0.6 -> >=5.0.8. The lock already pinned 5.0.8, but the newer advisory covers everything up to 5.0.7, so the floor no longer guaranteed what it was added to guarantee. Everything else is a routine in-range refresh (phpmailer 7.1.1, commonmark 2.8.3, phpunit 11.5.56, vite 8.1.5, vitest 4.1.10, eslint 10.8.0, lucide 1.26.0, and others). Three majors held back as separate decisions: typescript 6 -> 7, purgecss 4 -> 8, @types/node 25 -> 26. The eslint bump flagged a dead eslint-disable directive in vite.app.config.ts; no-console is not configured anywhere, so the directive was suppressing nothing. lucide 1.11 -> 1.26 is a wide jump for a package addressed by string name, so all 118 icon names used in src/ were resolved through the IconHelper and icons.ts alias maps and checked against the installed package, along with the alias targets: 148 names, all resolve.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Clears all eight open Dependabot alerts (2 high, 6 moderate) plus two more that
npm auditreports — ten advisories across five packages. Every one is fixed inside the existing version constraints, so nocomposer.json/package.jsonrange edits were needed.Security
guzzlehttp/guzzleguzzlehttp/psr7brace-expansionlinkify-itpostcssThe guzzle and psr7 ones are runtime, not tooling — both reach LWT through
league/oauth2-google, so they affect the running application. The guzzle advisories are: proxy-authorization headers sent to origin servers, URI fragments disclosed in redirectRefererheaders, unbounded response cookies, host-only cookie scope not preserved, and cookie disclosure via IP-address domains.psr7is host confusion via weak URI host validation.brace-expansion,linkify-itandpostcssare build/test toolchain only, not shipped to users.One deliberate constraint change
The
brace-expansionoverride floor moves from>=5.0.6to>=5.0.8. The lock already pinned 5.0.8 so audit was green either way, but the newer advisory covers everything up to 5.0.7 — the floor had quietly stopped guaranteeing what it was added to guarantee.Routine refresh
All in-range:
phpmailer/phpmailer7.1.1,league/commonmark2.8.3,thenetworg/oauth2-azure2.2.6,phpunit/phpunit11.5.56, plus Symfony and amphp transitives. On the JS side:vite8.1.5,vitest4.1.10,eslint10.8.0,cypress15.19.0,lucide1.26.0,@alpinejs/csp3.15.12,typescript-eslint8.65.0,prettier3.9.6,vue3.5.40.Held back as separate decisions:
typescript6 → 7,purgecss4 → 8,@types/node25 → 26. Alsoguzzle8 andpsr73, whichleague/oauth2-clientconstrains anyway.Verification
typecheck,lint,build:allcleancomposer auditandnpm auditboth report zeroTwo things worth a reviewer's attention:
lucide jumped 1.11 → 1.26, which is wide for a package addressed by string name — a removed icon fails silently at runtime and nothing in the test suite covers it. All 118 icon names used in
src/were resolved through theIconHelperandicons.tsalias maps, the alias targets added, and all 148 checked against the installed package. All resolve.The eslint bump flagged a dead
eslint-disable no-consoledirective invite.app.config.ts.no-consoleis not configured anywhere ineslint.config.ts, so it was suppressing nothing. Removing it is the only source change in this PR.Not addressed here
PHPUnit reports 4 deprecations for doc-comment metadata in the dictionary import tests. Verified against the previous PHPUnit by reinstalling the old lock — they predate this update and are not fallout from it. They become errors in PHPUnit 12, which is separate work.