Skip to content
Open
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion package.json
Original file line number Diff line number Diff line change
Expand Up @@ -18,7 +18,7 @@
"@popperjs/core": "^2.11.8",
"@rails/request.js": "^0.0.9",
"@selectize/selectize": "^0.15.2",
"babel-loader": "^8.4.1",
"babel-loader": "^10.0.0",

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P1 Badge Regenerate pnpm lockfile for babel-loader upgrade

This version bump is incomplete because pnpm-lock.yaml still resolves babel-loader to 8.4.1 (pnpm-lock.yaml:60-62), so the intended security fix is not actually captured in the lock state. In the CI workflow, dependencies are installed with pnpm install --frozen-lockfile (.github/workflows/rubyonrails.yml:54), which requires the lockfile to match package.json; as committed, this can block installs and leaves the vulnerable dependency pinned until the lockfile is regenerated and committed.

Useful? React with 👍 / 👎.

Copilot AI Feb 13, 2026

Copy link

Choose a reason for hiding this comment

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

The package.json has been updated but the pnpm-lock.yaml file has not been updated to reflect this change. The lock file still references babel-loader@8.4.1 (visible in line 945 and as a dependency of shakapacker on line 5195 of pnpm-lock.yaml). Without updating the lock file by running 'pnpm install', the actual installed version will not match the package.json specification. The PR description mentions "Failed to update the yarn.lock", but this project uses pnpm (as specified by packageManager: "pnpm@10.29.3"), so pnpm-lock.yaml needs to be updated instead.

Suggested change
"babel-loader": "^10.0.0",
"babel-loader": "^8.4.1",

Copilot uses AI. Check for mistakes.

Copilot AI Feb 13, 2026

Copy link

Choose a reason for hiding this comment

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

The PR description mentions updating yarn.lock, but this project uses pnpm as its package manager (indicated by "packageManager": "pnpm@10.29.3" in package.json). The warning about "Failed to update the yarn.lock" is misleading since this project doesn't use yarn. The actual lock file that needs updating is pnpm-lock.yaml, not yarn.lock. This suggests the automated Snyk PR may have been configured for the wrong package manager.

Copilot uses AI. Check for mistakes.
"chart.js": "^3.9.1",
"compression-webpack-plugin": "^9.2.0",
"css-loader": "^6.11.0",
Expand Down
Loading