Skip to content

update package-lock#453

Merged
Bojan131 merged 2 commits intomainfrom
package-lock-check
Feb 9, 2026
Merged

update package-lock#453
Bojan131 merged 2 commits intomainfrom
package-lock-check

Conversation

@Bojan131
Copy link
Contributor

@Bojan131 Bojan131 commented Feb 9, 2026

Note

Low Risk
CI-only workflow changes that tighten PR validation around dependency/lockfile consistency; risk is limited to potential false positives/negatives in the git/jq diff checks.

Overview
Tightens the check-package-lock GitHub Action by fetching full git history and, on pull requests, detecting whether package.json/package-lock.json changed and whether dependency fields actually changed via a jq-sorted comparison.

If dependency fields changed but package-lock.json did not, the workflow now fails the PR. Node setup and npm ci --dry-run validation are also made conditional so they only run on pushes to main or when the package files changed in the PR.

Written by Cursor Bugbot for commit 3f56e34. This will update automatically on new commits. Configure here.

Copy link

@cursor cursor bot left a comment

Choose a reason for hiding this comment

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

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Bugbot Autofix is OFF. To automatically fix reported issues with Cloud Agents, enable Autofix in the Cursor dashboard.

This PR is being reviewed by Cursor Bugbot

Details

Your team is on the Bugbot Free tier. On this plan, Bugbot will review limited PRs each billing cycle for each member of your team.

To receive Bugbot reviews on all of your PRs, visit the Cursor dashboard to activate Pro and start your 14-day free trial.

@Bojan131 Bojan131 merged commit d81189b into main Feb 9, 2026
5 checks 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