Skip to content

chore(deps): resolve critical and high npm advisories in /cdk - #24

Open
robsable wants to merge 1 commit into
mainfrom
triage/audit-fix
Open

robsable wants to merge 1 commit into
mainfrom
triage/audit-fix

Conversation

@robsable

Copy link
Copy Markdown
Contributor

What

Resolves every open critical and high npm advisory in /cdk with one lockfile refresh. It replaces the individual Dependabot PRs that had piled up.

  • npm audit fix within the existing semver ranges: handlebars 4.7.10, js-yaml 3.15.2, brace-expansion 5.0.12+, browserslist, baseline-browser-mapping, fast-uri, and aws-cdk-lib 2.276.0.
  • aws-cdk CLI raised to ^2.1148.1. aws-cdk-lib 2.276.0 writes cloud assembly schema 55, which older CLIs can't read, so synth fails without this bump.

Verified

tsc, jest and cdk synth of all stacks pass locally. CI runs the same checks on this PR.

Remaining (moderate/low, not addressed here)

  • The jest toolchain (glob, minimatch, test-exclude, deepmerge) is dev-only and never deployed. npm's suggested "fix" downgrades jest to 25, which would be wrong.
  • aws-cdk-lib bundles its own minimatch, which only an upstream aws-cdk-lib release can fix.

Once this merges, Dependabot closes its superseded PRs on its next run.

npm audit fix within existing semver ranges, plus aws-cdk CLI ^2.1148.1 so the CLI can read the cloud assembly schema from aws-cdk-lib 2.276.0.
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