fix(test): clean dependency report cli errors · openclaw/openclaw@d6075c1

Recent Commits to openclaw:main

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

Original file line number	Diff line number	Diff line change
`@@ -345,7 +345,7 @@ if (process.argv[1] && path.resolve(process.argv[1]) === path.resolve(import.met`
`345`	`345`	`process.exitCode = exitCode;`
`346`	`346`	`},`
`347`	`347`	`/** @param {unknown} error */ (error) => {`
`348`		-process.stderr.write(`${error.stack ?? error.message ?? String(error)}\n`);
	`348`	+process.stderr.write(`${error instanceof Error ? error.message : String(error)}\n`);
`349`	`349`	`process.exitCode = 1;`
`350`	`350`	`},`
`351`	`351`	`);`

Original file line number	Diff line number	Diff line change
`@@ -303,7 +303,7 @@ if (process.argv[1] && path.resolve(process.argv[1]) === path.resolve(import.met`
`303`	`303`	`process.exitCode = exitCode;`
`304`	`304`	`},`
`305`	`305`	`/** @param {unknown} error */ (error) => {`
`306`		-process.stderr.write(`${error.stack ?? error.message ?? String(error)}\n`);
	`306`	+process.stderr.write(`${error instanceof Error ? error.message : String(error)}\n`);
`307`	`307`	`process.exitCode = 1;`
`308`	`308`	`},`
`309`	`309`	`);`

Original file line number	Diff line number	Diff line change
`@@ -656,7 +656,7 @@ if (process.argv[1] && path.resolve(process.argv[1]) === path.resolve(import.met`
`656`	`656`	`process.exitCode = exitCode;`
`657`	`657`	`},`
`658`	`658`	`/** @param {unknown} error */ (error) => {`
`659`		-process.stderr.write(`${error.stack ?? error.message ?? String(error)}\n`);
	`659`	+process.stderr.write(`${error instanceof Error ? error.message : String(error)}\n`);
`660`	`660`	`process.exitCode = 1;`
`661`	`661`	`},`
`662`	`662`	`);`

Original file line number	Diff line number	Diff line change
`@@ -1,4 +1,6 @@`
`1`	`1`	`// Dependency Changes Report tests cover dependency changes report script behavior.`
	`2`	`+import { spawnSync } from "node:child_process";`
	`3`	`+import path from "node:path";`
`2`	`4`	`import { describe, expect, it } from "vitest";`
`3`	`5`	`import {`
`4`	`6`	`createDependencyChangesReport,`
`@@ -7,6 +9,18 @@ import {`
`7`	`9`	`parseArgs,`
`8`	`10`	`} from "../../scripts/dependency-changes-report.mjs";`
`9`	`11`
	`12`	`+function runCli(...args: string[]) {`
	`13`	`+return spawnSync(process.execPath, ["scripts/dependency-changes-report.mjs", ...args], {`
	`14`	`+cwd: path.resolve("."),`
	`15`	`+encoding: "utf8",`
	`16`	`+});`
	`17`	`+}`
	`18`	`+`
	`19`	`+function expectNoNodeStack(stderr: string) {`
	`20`	`+expect(stderr).not.toContain("Node.js");`
	`21`	`+expect(stderr).not.toContain("\n at ");`
	`22`	`+}`
	`23`	`+`
`10`	`24`	`describe("dependency-changes-report", () => {`
`11`	`25`	`it("reports added, removed, and changed packages", () => {`
`12`	`26`	`const report = createDependencyChangesReport({`
`@@ -72,4 +86,20 @@ describe("dependency-changes-report", () => {`
`72`	`86`	expect(() => parseArgs([flag, "--json"])).toThrow(`${flag} requires a value`);
`73`	`87`	`}`
`74`	`88`	`});`
	`89`	`+`
	`90`	`+it("reports CLI argument errors without a Node stack trace", () => {`
	`91`	`+const missingBase = runCli();`
	`92`	`+expect(missingBase.status).toBe(1);`
	`93`	`+expect(missingBase.stdout).toBe("");`
	`94`	`+expect(missingBase.stderr.trim()).toBe(`
	`95`	`+"Expected --base-ref <git-ref> or --base-lockfile <path>.",`
	`96`	`+);`
	`97`	`+expectNoNodeStack(missingBase.stderr);`
	`98`	`+`
	`99`	`+const unknownArg = runCli("--wat");`
	`100`	`+expect(unknownArg.status).toBe(1);`
	`101`	`+expect(unknownArg.stdout).toBe("");`
	`102`	`+expect(unknownArg.stderr.trim()).toBe("Unsupported argument: --wat");`
	`103`	`+expectNoNodeStack(unknownArg.stderr);`
	`104`	`+});`
`75`	`105`	`});`

Original file line number	Diff line number	Diff line change
`@@ -1,4 +1,5 @@`
`1`	`1`	`// Dependency Vulnerability Gate tests cover dependency vulnerability gate script behavior.`
	`2`	`+import { spawnSync } from "node:child_process";`
`2`	`3`	`import { mkdtemp, rm, writeFile } from "node:fs/promises";`
`3`	`4`	`import { tmpdir } from "node:os";`
`4`	`5`	`import path from "node:path";`
`@@ -9,6 +10,18 @@ import {`
`9`	`10`	`runDependencyVulnerabilityGate,`
`10`	`11`	`} from "../../scripts/dependency-vulnerability-gate.mjs";`
`11`	`12`
	`13`	`+function runCli(...args: string[]) {`
	`14`	`+return spawnSync(process.execPath, ["scripts/dependency-vulnerability-gate.mjs", ...args], {`
	`15`	`+cwd: path.resolve("."),`
	`16`	`+encoding: "utf8",`
	`17`	`+});`
	`18`	`+}`
	`19`	`+`
	`20`	`+function expectNoNodeStack(stderr: string) {`
	`21`	`+expect(stderr).not.toContain("Node.js");`
	`22`	`+expect(stderr).not.toContain("\n at ");`
	`23`	`+}`
	`24`	`+`
`12`	`25`	`function advisory({`
`13`	`26`	`id,`
`14`	`27`	`severity,`
`@@ -53,6 +66,14 @@ snapshots:`
`53`	`66`	`}`
`54`	`67`
`55`	`68`	`describe("dependency-vulnerability-gate", () => {`
	`69`	`+it("reports CLI argument errors without a Node stack trace", () => {`
	`70`	`+const unknownArg = runCli("--wat");`
	`71`	`+expect(unknownArg.status).toBe(1);`
	`72`	`+expect(unknownArg.stdout).toBe("");`
	`73`	`+expect(unknownArg.stderr.trim()).toBe("Unsupported argument: --wat");`
	`74`	`+expectNoNodeStack(unknownArg.stderr);`
	`75`	`+});`
	`76`	`+`
`56`	`77`	`it("blocks critical advisories anywhere and high advisories in the production graph", () => {`
`57`	`78`	`const result = classifyVulnerabilityFindings({`
`58`	`79`	`allAdvisories: {`

Original file line number	Diff line number	Diff line change
`@@ -1,4 +1,6 @@`
`1`	`1`	`// Transitive Manifest Risk Report tests cover transitive manifest risk report script behavior.`
	`2`	`+import { spawnSync } from "node:child_process";`
	`3`	`+import path from "node:path";`
`2`	`4`	`import { describe, expect, it } from "vitest";`
`3`	`5`	`import {`
`4`	`6`	`createTransitiveManifestRiskReport,`
`@@ -7,7 +9,27 @@ import {`
`7`	`9`	`renderTransitiveManifestRiskMarkdownReport,`
`8`	`10`	`} from "../../scripts/transitive-manifest-risk-report.mjs";`
`9`	`11`
	`12`	`+function runCli(...args: string[]) {`
	`13`	`+return spawnSync(process.execPath, ["scripts/transitive-manifest-risk-report.mjs", ...args], {`
	`14`	`+cwd: path.resolve("."),`
	`15`	`+encoding: "utf8",`
	`16`	`+});`
	`17`	`+}`
	`18`	`+`
	`19`	`+function expectNoNodeStack(stderr: string) {`
	`20`	`+expect(stderr).not.toContain("Node.js");`
	`21`	`+expect(stderr).not.toContain("\n at ");`
	`22`	`+}`
	`23`	`+`
`10`	`24`	`describe("transitive-manifest-risk-report", () => {`
	`25`	`+it("reports CLI argument errors without a Node stack trace", () => {`
	`26`	`+const unknownArg = runCli("--wat");`
	`27`	`+expect(unknownArg.status).toBe(1);`
	`28`	`+expect(unknownArg.stdout).toBe("");`
	`29`	`+expect(unknownArg.stderr.trim()).toBe("Unsupported argument: --wat");`
	`30`	`+expectNoNodeStack(unknownArg.stderr);`
	`31`	`+});`
	`32`	`+`
`11`	`33`	`it("reports floating transitive specs, lifecycle scripts, exotic sources, and recently published versions", async () => {`
`12`	`34`	`const report = await createTransitiveManifestRiskReport({`
`13`	`35`	`packageVersions: [`

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

推荐订阅源

Recent Commits to openclaw:main