Skip to content

node package upgrades to address vulnerabilities#157

Merged
dcdenu4 merged 5 commits intonatcap:developfrom
davemfish:task/156-node-package-upgrades
Feb 11, 2026
Merged

node package upgrades to address vulnerabilities#157
dcdenu4 merged 5 commits intonatcap:developfrom
davemfish:task/156-node-package-upgrades

Conversation

@davemfish
Copy link
Contributor

The critical vulnerability in vm2 was addressed by upgrading puppeteer. I did major version upgrades for most other dependencies as well, except for ones likely to have major breaking changes: react, ol, blueprint. For these I did minor version upgrades if there were any.

yarn audit is now fairly clean, and the outstanding issues I think ought to be dealt with by upstream packages.

audit output yarn audit v1.22.22 ┌───────────────┬──────────────────────────────────────────────────────────────┐ │ moderate │ Babel has inefficient RegExp complexity in generated code │ │ │ with .replace when transpiling named capturing groups │ ├───────────────┼──────────────────────────────────────────────────────────────┤ │ Package │ @babel/runtime │ ├───────────────┼──────────────────────────────────────────────────────────────┤ │ Patched in │ >=7.26.10 │ ├───────────────┼──────────────────────────────────────────────────────────────┤ │ Dependency of │ @testing-library/dom │ ├───────────────┼──────────────────────────────────────────────────────────────┤ │ Path │ @testing-library/dom > @babel/runtime │ ├───────────────┼──────────────────────────────────────────────────────────────┤ │ More info │ https://www.npmjs.com/advisories/1104000 │ └───────────────┴──────────────────────────────────────────────────────────────┘ ┌───────────────┬──────────────────────────────────────────────────────────────┐ │ low │ brace-expansion Regular Expression Denial of Service │ │ │ vulnerability │ ├───────────────┼──────────────────────────────────────────────────────────────┤ │ Package │ brace-expansion │ ├───────────────┼──────────────────────────────────────────────────────────────┤ │ Patched in │ >=1.1.12 │ ├───────────────┼──────────────────────────────────────────────────────────────┤ │ Dependency of │ eslint-plugin-import │ ├───────────────┼──────────────────────────────────────────────────────────────┤ │ Path │ eslint-plugin-import > minimatch > brace-expansion │ ├───────────────┼──────────────────────────────────────────────────────────────┤ │ More info │ https://www.npmjs.com/advisories/1105443 │ └───────────────┴──────────────────────────────────────────────────────────────┘ ┌───────────────┬──────────────────────────────────────────────────────────────┐ │ moderate │ js-yaml has prototype pollution in merge (<<) │ ├───────────────┼──────────────────────────────────────────────────────────────┤ │ Package │ js-yaml │ ├───────────────┼──────────────────────────────────────────────────────────────┤ │ Patched in │ >=4.1.1 │ ├───────────────┼──────────────────────────────────────────────────────────────┤ │ Dependency of │ puppeteer │ ├───────────────┼──────────────────────────────────────────────────────────────┤ │ Path │ puppeteer > cosmiconfig > js-yaml │ ├───────────────┼──────────────────────────────────────────────────────────────┤ │ More info │ https://www.npmjs.com/advisories/1112715 │ └───────────────┴──────────────────────────────────────────────────────────────┘ ┌───────────────┬──────────────────────────────────────────────────────────────┐ │ high │ semver vulnerable to Regular Expression Denial of Service │ ├───────────────┼──────────────────────────────────────────────────────────────┤ │ Package │ semver │ ├───────────────┼──────────────────────────────────────────────────────────────┤ │ Patched in │ >=6.3.1 │ ├───────────────┼──────────────────────────────────────────────────────────────┤ │ Dependency of │ @vitest/coverage-v8 │ ├───────────────┼──────────────────────────────────────────────────────────────┤ │ Path │ @vitest/coverage-v8 > istanbul-reports > istanbul-lib-report │ │ │ > make-dir > semver │ ├───────────────┼──────────────────────────────────────────────────────────────┤ │ More info │ https://www.npmjs.com/advisories/1112922 │ └───────────────┴──────────────────────────────────────────────────────────────┘ ┌───────────────┬──────────────────────────────────────────────────────────────┐ │ high │ semver vulnerable to Regular Expression Denial of Service │ ├───────────────┼──────────────────────────────────────────────────────────────┤ │ Package │ semver │ ├───────────────┼──────────────────────────────────────────────────────────────┤ │ Patched in │ >=6.3.1 │ ├───────────────┼──────────────────────────────────────────────────────────────┤ │ Dependency of │ eslint-config-airbnb │ ├───────────────┼──────────────────────────────────────────────────────────────┤ │ Path │ eslint-config-airbnb > eslint-config-airbnb-base > semver │ ├───────────────┼──────────────────────────────────────────────────────────────┤ │ More info │ https://www.npmjs.com/advisories/1112922 │ └───────────────┴──────────────────────────────────────────────────────────────┘ 5 vulnerabilities found - Packages audited: 771 Severity: 1 Low | 2 Moderate | 2 High Done in 0.74s.

@davemfish davemfish requested a review from dcdenu4 February 11, 2026 15:59
Copy link
Member

@dcdenu4 dcdenu4 left a comment

Choose a reason for hiding this comment

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

Thanks @davemfish

@dcdenu4 dcdenu4 merged commit 81bfa47 into natcap:develop Feb 11, 2026
1 check passed
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