You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Updating commitlint alone does not remove the vulnerable transitive, because latest commitlint currently still depends on ajv@8.20.0.
The effective fix here is the root pnpm override to fast-uri@3.1.4, which is semver-compatible with ajv’s ^3.0.1 range and removes the vulnerable 3.1.3.
fast-uri: 3.1.4 exact-pin override added to root pnpm overrides
Skipped (out of scope): pnpm-lock.yaml
⚠️Note: CI checks could not be retrieved automatically — please confirm the commitlint hook passes on a test commit before merging.
Findings
⚠️ Low — PR title missing conventional-commit prefix
File: PR metadata Note: The title "Update commitlint and fast-uri dependencies" does not follow the conventional commits format enforced by commitlint itself (ironic). Prefer chore: update commitlint and fast-uri dependencies to satisfy the hook and match repo conventions.
⚠️ Low — fast-uri override is exact-pinned rather than a range
File:package.json line 74 Note:"fast-uri": "3.1.4" is an exact pin. If a future patch (3.1.5+) fixes another CVE, this override will silently block it. The other overrides in this file use ranges (>=, ^). Consider:
"fast-uri": ">=3.1.4"
This still excludes the vulnerable 3.1.3 while allowing forward-compatible patches. Exact pinning is not wrong here, but it requires a manual follow-up PR for every future fast-uri patch.
Positives
The PR description precisely explains the two-layer fix (commitlint bump + override) and why upgrading commitlint alone is insufficient — excellent context for future readers.
Deprecated git-raw-commits (which was already deprecated upstream) is fully removed by the commitlint v21 transitive graph — good cleanup side-effect with no action required.
Multiple heavy lodash utilities (lodash.kebabcase, lodash.snakecase, lodash.startcase, lodash.mergewith, etc.) are replaced by es-toolkit, a smaller, tree-shakeable modern alternative.
xlsx CDN tarball now gains an integrity SHA512 hash in the lockfile (sha512-oLDq3...), improving supply chain security for a package that previously had no integrity check.
The fix correctly targets the right layer: pnpm.overrides forces resolution of the vulnerable transitive without patching upstream packages.
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
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.
Pull request type
Dependency changes (any modification to dependencies in
package.json)Description
This PR tries to resolve a vulnerable transitive dependency: https://security.snyk.io/vuln/SNYK-JS-FASTURI-18021349
Updating commitlint alone does not remove the vulnerable transitive, because latest commitlint currently still depends on ajv@8.20.0.
The effective fix here is the root pnpm override to fast-uri@3.1.4, which is semver-compatible with ajv’s ^3.0.1 range and removes the vulnerable 3.1.3.