Repair malformed lockfile entries causing npm install to fail with Invalid Version - #77
Conversation
|
Strix is installed on this repository, but we couldn't run this PR security review because this workspace's trial has ended. Add a card to resume code reviews here. |
| "version": "1.11.2", | ||
| "resolved": "https://registry.npmjs.org/@emnapi/core/-/core-1.11.2.tgz", | ||
| "integrity": "sha512-TC8MkTuZUtcTSiFeuC0ksCh9QIJ5+F21MvZ4Wn4ORfYaFJ/0dsiudv5tVkejgwZlwQ39jL9WWDe2lz8x0WglOA==", | ||
| "dev": true, |
There was a problem hiding this comment.
WARNING: @emnapi/core should not have "peer": true — it is an optional build-time dependency of sharp, not a peer dependency. This misclassification could confuse npm's dependency resolution and bundlers.
Reply with @kilocode-bot fix it to have Kilo Code address this issue.
| "node_modules/@emnapi/runtime": { | ||
| "version": "1.11.2", | ||
| "resolved": "https://registry.npmjs.org/@emnapi/runtime/-/runtime-1.11.2.tgz", | ||
| "integrity": "sha512-kyOl3X0DuTiT1h2ft8r2fYO8JYtU9a9Xis/zBSiGArNaagCOWx90N1k2wxp18czFDH+OgcWGb5ZP/XMt3dcyPA==", |
There was a problem hiding this comment.
WARNING: @emnapi/runtime should not have "peer": true — it is an optional build-time dependency of sharp, not a peer dependency. Additionally, this entry is missing "dev": true, which should be present since it is a development/build dependency.
Reply with @kilocode-bot fix it to have Kilo Code address this issue.
| "version": "1.2.2", | ||
| "resolved": "https://registry.npmjs.org/@emnapi/wasi-threads/-/wasi-threads-1.2.2.tgz", | ||
| "integrity": "sha512-c95qOXkHdydNKhscBTebqEC1CVAZpyqOfVfBzQ1qgzyl3gfeldUjIggDbIZgDKsHLgnsM+igH7TJ/eAasaVuMA==", | ||
| "dev": true, |
There was a problem hiding this comment.
WARNING: @emnapi/wasi-threads should not have "peer": true — it is an optional build-time dependency of sharp, not a peer dependency.
Reply with @kilocode-bot fix it to have Kilo Code address this issue.
| @@ -2302,7 +2366,8 @@ | |||
| "version": "16.2.7", | |||
| "resolved": "https://registry.npmjs.org/@next/env/-/env-16.2.7.tgz", | |||
There was a problem hiding this comment.
WARNING: @next/env should not have "peer": true — it is a direct dependency of next, not a peer dependency. Adding peer: true incorrectly marks it as a peer dependency.
Reply with @kilocode-bot fix it to have Kilo Code address this issue.
| "resolved": "https://registry.npmjs.org/next/-/next-16.2.7.tgz", | ||
| "integrity": "sha512-eMJxgjRzBaj3olkP4cBamHDXL79A8FC6u1GcsO1D1Tsx8bw/LLXUJCaoajVxtnhD3A1IJqIT8IcRJjgBIPJq4w==", | ||
| "license": "MIT", | ||
| "peer": true, |
There was a problem hiding this comment.
WARNING: next should not have "peer": true — it is a direct dependency of the project, not a peer dependency. Adding peer: true incorrectly marks it as a peer dependency.
Reply with @kilocode-bot fix it to have Kilo Code address this issue.
Code Review SummaryStatus: 5 Issues Found | Recommendation: Address before merge Overview
Issue Details (click to expand)WARNING
Files Reviewed (1 files)
Fix these issues in Kilo Cloud Reviewed by ling-3.0-flash:free · Input: 334.9K · Output: 19.5K · Cached: 6.3M |
npm installwas failing immediately withInvalid Version:due to malformed nested entries inpackage-lock.json. The lockfile contained package nodes without required version metadata, which breaks npm/arborist semver handling during dedupe.Root cause
optional/devonly), leavingversioneffectively empty for npm’s resolver path.Lockfile metadata repair
node_modules/@img/sharp-wasm32/node_modules/@emnapi/runtimenode_modules/minipass-fetch/node_modules/encodingversion,resolved,integrity, andlicense.Representative change