fix(ci): bound deadcode knip scan · openclaw/openclaw@27097be

Recent Commits to openclaw:main

vincentkoc · 2026-06-01 · via Recent Commits to openclaw:main

Original file line number	Diff line number	Diff line change
`@@ -7,6 +7,8 @@ import {`
`7`	`7`	`} from "./deadcode-unused-files.allowlist.mjs";`
`8`	`8`
`9`	`9`	`const KNIP_VERSION = "6.8.0";`
	`10`	`+export const KNIP_TIMEOUT_MS = 10 * 60 * 1000;`
	`11`	`+export const KNIP_MAX_BUFFER_BYTES = 16 * 1024 * 1024;`
`10`	`12`	`const KNIP_ARGS = [`
`11`	`13`	`"--config",`
`12`	`14`	`"config/knip.config.ts",`
`@@ -108,18 +110,28 @@ export function formatUnusedFileComparison(comparison) {`
`108`	`110`	`return lines.join("\n");`
`109`	`111`	`}`
`110`	`112`
`111`		`-export function runKnipUnusedFiles() {`
`112`		`-const result = spawnSync(`
	`113`	`+function spawnErrorCode(error) {`
	`114`	`+return error && typeof error === "object" && "code" in error ? String(error.code) : undefined;`
	`115`	`+}`
	`116`	`+`
	`117`	`+export function runKnipUnusedFiles(params = {}) {`
	`118`	`+const run = params.spawnSyncCommand ?? spawnSync;`
	`119`	`+const result = run(`
`113`	`120`	`"pnpm",`
`114`	`121`	["--config.minimum-release-age=0", "dlx", `knip@${KNIP_VERSION}`, ...KNIP_ARGS],
`115`	`122`	`{`
`116`	`123`	`encoding: "utf8",`
	`124`	`+killSignal: "SIGTERM",`
	`125`	`+maxBuffer: params.maxBufferBytes ?? KNIP_MAX_BUFFER_BYTES,`
`117`	`126`	`stdio: ["ignore", "pipe", "pipe"],`
	`127`	`+timeout: params.timeoutMs ?? KNIP_TIMEOUT_MS,`
`118`	`128`	`},`
`119`	`129`	`);`
`120`	`130`	`return {`
`121`	`131`	`status: result.status,`
`122`	`132`	`signal: result.signal,`
	`133`	`+errorCode: spawnErrorCode(result.error),`
	`134`	`+errorMessage: result.error?.message,`
`123`	`135`	output: `${result.stdout ?? ""}${result.stderr ?? ""}`,
`124`	`136`	`};`
`125`	`137`	`}`
`@@ -144,6 +156,18 @@ export function checkUnusedFiles(`
`144`	`156`
`145`	`157`	`function main() {`
`146`	`158`	`const result = runKnipUnusedFiles();`
	`159`	`+if (result.errorCode \|\| result.status === null) {`
	`160`	`+console.error(`
	`161`	+`deadcode unused-file scan failed: ${result.errorCode ?? result.signal ?? "unknown"}${
	`162`	+ result.errorMessage ? `: ${result.errorMessage}` : ""
	`163`	+ }`,
	`164`	`+);`
	`165`	`+if (result.output) {`
	`166`	`+console.error(result.output);`
	`167`	`+}`
	`168`	`+process.exitCode = 1;`
	`169`	`+return;`
	`170`	`+}`
`147`	`171`	`const check = checkUnusedFiles(result.output);`
`148`	`172`	`if (!check.ok) {`
`149`	`173`	`if (check.message) {`

Original file line number	Diff line number	Diff line change
`@@ -2,7 +2,10 @@ import { describe, expect, it } from "vitest";`
`2`	`2`	`import {`
`3`	`3`	`checkUnusedFiles,`
`4`	`4`	`compareUnusedFilesToAllowlist,`
	`5`	`+KNIP_MAX_BUFFER_BYTES,`
	`6`	`+KNIP_TIMEOUT_MS,`
`5`	`7`	`parseKnipCompactUnusedFiles,`
	`8`	`+runKnipUnusedFiles,`
`6`	`9`	`} from "../../scripts/check-deadcode-unused-files.mjs";`
`7`	`10`
`8`	`11`	`describe("check-deadcode-unused-files", () => {`
`@@ -107,4 +110,54 @@ src/a.ts: src/a.ts`
`107`	`110`	`unexpected: [],`
`108`	`111`	`});`
`109`	`112`	`});`
	`113`	`+`
	`114`	`+it("bounds Knip execution and reports spawn errors", () => {`
	`115`	`+const calls: unknown[] = [];`
	`116`	`+const timeoutError = Object.assign(new Error("spawnSync pnpm ETIMEDOUT"), {`
	`117`	`+code: "ETIMEDOUT",`
	`118`	`+});`
	`119`	`+`
	`120`	`+const result = runKnipUnusedFiles({`
	`121`	`+spawnSyncCommand(command: string, args: string[], options: unknown) {`
	`122`	`+calls.push({ args, command, options });`
	`123`	`+return {`
	`124`	`+error: timeoutError,`
	`125`	`+signal: "SIGTERM",`
	`126`	`+status: null,`
	`127`	`+stderr: "partial stderr",`
	`128`	`+stdout: "partial stdout",`
	`129`	`+};`
	`130`	`+},`
	`131`	`+});`
	`132`	`+`
	`133`	`+expect(calls).toHaveLength(1);`
	`134`	`+expect(calls[0]).toMatchObject({`
	`135`	`+args: [`
	`136`	`+"--config.minimum-release-age=0",`
	`137`	`+"dlx",`
	`138`	`+"knip@6.8.0",`
	`139`	`+"--config",`
	`140`	`+"config/knip.config.ts",`
	`141`	`+"--production",`
	`142`	`+"--no-progress",`
	`143`	`+"--reporter",`
	`144`	`+"compact",`
	`145`	`+"--files",`
	`146`	`+"--no-config-hints",`
	`147`	`+],`
	`148`	`+command: "pnpm",`
	`149`	`+options: {`
	`150`	`+killSignal: "SIGTERM",`
	`151`	`+maxBuffer: KNIP_MAX_BUFFER_BYTES,`
	`152`	`+timeout: KNIP_TIMEOUT_MS,`
	`153`	`+},`
	`154`	`+});`
	`155`	`+expect(result).toStrictEqual({`
	`156`	`+errorCode: "ETIMEDOUT",`
	`157`	`+errorMessage: "spawnSync pnpm ETIMEDOUT",`
	`158`	`+output: "partial stdoutpartial stderr",`
	`159`	`+signal: "SIGTERM",`
	`160`	`+status: null,`
	`161`	`+});`
	`162`	`+});`
`110`	`163`	`});`

此内容由惯性聚合(RSS阅读器)自动聚合整理，仅供阅读参考。原文来自 — 版权归原作者所有。

推荐订阅源

Recent Commits to openclaw:main