fix: ignore pnpm progress in deadcode guard

This commit is contained in:
Peter Steinberger
2026-05-02 05:59:47 +01:00
parent f269423355
commit 40d50cbbf1
2 changed files with 19 additions and 1 deletions

View File

@@ -28,6 +28,10 @@ function uniqueSorted(values) {
);
}
function isLikelyRepoFilePath(value) {
return /^(apps|docs|extensions|packages|scripts|src|test|ui)\//u.test(normalizeRepoPath(value));
}
export function parseKnipCompactUnusedFiles(output) {
const files = [];
let inUnusedFilesSection = false;
@@ -50,7 +54,10 @@ export function parseKnipCompactUnusedFiles(output) {
if (sawUnusedFilesSection && !inUnusedFilesSection) {
continue;
}
files.push(line.slice(separatorIndex + 2).trim());
const file = line.slice(separatorIndex + 2).trim();
if (isLikelyRepoFilePath(file)) {
files.push(file);
}
}
return uniqueSorted(files);

View File

@@ -29,6 +29,17 @@ left-pad: package.json
]);
});
it("ignores pnpm dlx progress lines in files-only compact output", () => {
expect(
parseKnipCompactUnusedFiles(`
Progress: resolved 21, reused 0, downloaded 0, added 0
src/b.ts: src/b.ts
Progress: resolved 65, reused 20, downloaded 1, added 21, done
src/a.ts: src/a.ts
`),
).toEqual(["src/a.ts", "src/b.ts"]);
});
it("reports unexpected and stale allowlist entries", () => {
expect(
compareUnusedFilesToAllowlist(["src/a.ts", "src/new.ts"], ["src/a.ts", "src/old.ts"]),