Skip to content

Commit bbca5fe

Browse files
fix(review): align lockfile-tamper finding text with lockfile-entry version check
Closes #6633 Co-authored-by: Cursor <cursoragent@cursor.com>
1 parent 8b20ad6 commit bbca5fe

2 files changed

Lines changed: 2 additions & 2 deletions

File tree

src/review/lockfile-tamper.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -224,7 +224,7 @@ export function lockfileTamperRiskFinding(files: PullRequestFileRecord[]): Advis
224224
const hasUnbumped = flagged.some((f) => f.reason === "unbumped_resolved");
225225
const detailParts: string[] = [];
226226
if (hasOffRegistry) detailParts.push("a resolved URL points outside registry.npmjs.org");
227-
if (hasUnbumped) detailParts.push("a resolved/integrity value changed without a matching package.json version bump");
227+
if (hasUnbumped) detailParts.push("a resolved/integrity value changed without a matching version bump in the lockfile entry itself");
228228

229229
return {
230230
code: "lockfile_tamper_risk",

test/unit/lockfile-tamper.test.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -56,7 +56,7 @@ describe("lockfileTamperRiskFinding", () => {
5656
expect(finding).toBeNull();
5757
});
5858

59-
it("triggers on a hand-edited resolved/integrity with NO corresponding package.json version bump", () => {
59+
it("triggers on a hand-edited resolved/integrity with NO corresponding lockfile-entry version bump", () => {
6060
const lockPatch = [
6161
'@@ -100,8 +100,8 @@',
6262
' "node_modules/lodash": {',

0 commit comments

Comments
 (0)