Skip to content

fix(deps): security package updates - #5

Open
Nisarg48 wants to merge 7 commits into
mainfrom
fix-package-vulnerabilities
Open

fix(deps): security package updates#5
Nisarg48 wants to merge 7 commits into
mainfrom
fix-package-vulnerabilities

Conversation

@Nisarg48

@Nisarg48 Nisarg48 commented Jun 3, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • Upgrade Nx from 20.7.2 → 23.1.1 (with migrations)
  • Standardize on Yarn Classic as the package manager: remove package-lock.json, keep yarn.lock, set "packageManager": "yarn@1.22.22", and update CI to yarn install --frozen-lockfile on Node 22
  • Bump vulnerable tooling parents without overrides / resolutions: vite, vitest, postcss, @babel/core, @swc/cli, verdaccio, @playwright/test, vite-plugin-dts, Jest/jsdom stack
  • Align lib @nx/vite / vite pins to the workspace (no nested Nx 20)
  • Align Jest 30 types/env (@types/jest, jest-environment-node) with Jest 30.4.1

This comment was marked as outdated.

This comment was marked as outdated.

Bump tooling parents and migrate the workspace from Nx 20 to 23 without
overrides so CI stays green and Vanta findings from outdated packages are cleared.

This comment was marked as outdated.

KrishKashiwala

This comment was marked as resolved.

Comment thread nx.json Outdated
Use yarn.lock as the single lockfile, drop package-lock.json, and bump
parents (postcss, jest/jsdom, vite-plugin-dts) so audits clear without
resolutions. Format Prettier breaks so lint stays green.
Verdaccio 6.10 requires Node >=22; align CI with local and the lockfile.

This comment was marked as resolved.

@Nisarg48
Nisarg48 requested a balanced review from Copilot August 25, 2026 12:01

This comment was marked as off-topic.

Add packageManager for Corepack and bump @types/jest plus
jest-environment-node to match Jest 30.

This comment was marked as resolved.

@Brainic477 Brainic477 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.

LGTM

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.

4 participants