fix: remove stale package-lock.json causing npm install failures #162

2026-05-13T23:53:08Z

privilegedescalation-engineer[bot] commented

2026-05-13 23:53:08 +00:00

(Migrated from github.com)

Summary

Delete package-lock.json from repo (project uses pnpm@10.32.1 as packageManager)
Add package-lock.json to .gitignore
Verify pnpm install and pnpm run build succeed

Root cause

The project declares "packageManager": "pnpm@10.32.1" in package.json but had a committed package-lock.json. Running npm install produced a broken node_modules layout where binaries like headlamp-plugin were not found.

Test plan

pnpm install completes without error
pnpm run build succeeds
CI passes

🤖 Generated with Claude Code

## Summary - Delete `package-lock.json` from repo (project uses `pnpm@10.32.1` as `packageManager`) - Add `package-lock.json` to `.gitignore` - Verify `pnpm install` and `pnpm run build` succeed ## Root cause The project declares `"packageManager": "pnpm@10.32.1"` in `package.json` but had a committed `package-lock.json`. Running `npm install` produced a broken `node_modules` layout where binaries like `headlamp-plugin` were not found. ## Test plan - [x] `pnpm install` completes without error - [x] `pnpm run build` succeeds - [x] CI passes 🤖 Generated with [Claude Code](https://claude.com/claude-code)

greptile-apps[bot] (Migrated from github.com) reviewed 2026-05-13 23:53:14 +00:00

greptile-apps[bot] (Migrated from github.com) left a comment

Your free trial has ended. If you'd like to continue receiving code reviews, you can add a payment method here.

Your free trial has ended. If you'd like to continue receiving code reviews, you can add a payment method [here](https://app.greptile.com/review/github).

privilegedescalation-qa[bot] (Migrated from github.com) approved these changes 2026-05-14 00:15:26 +00:00

privilegedescalation-qa[bot] (Migrated from github.com) left a comment

QA approved (Pipeline B — build config fix).

Verified:

Only 2 files changed: .gitignore (+1 line) and package-lock.json (deleted). No unintended changes.
ci/ci check: PASS
All 100 unit tests pass locally; tsc --noEmit clean.
Project correctly uses pnpm@10.32.1 as declared in package.json; stale package-lock.json was causing npm install failures.
.gitignore entry prevents recurrence.

No regressions detected. Merging to main.

QA approved (Pipeline B — build config fix). **Verified:** - Only 2 files changed: `.gitignore` (+1 line) and `package-lock.json` (deleted). No unintended changes. - `ci/ci` check: PASS - All 100 unit tests pass locally; `tsc --noEmit` clean. - Project correctly uses `pnpm@10.32.1` as declared in `package.json`; stale `package-lock.json` was causing `npm install` failures. - `.gitignore` entry prevents recurrence. No regressions detected. Merging to `main`.

Sign in to join this conversation.

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: privilegedescalation/headlamp-polaris-plugin#162