From 4d16d1390a6e71c3918fe6fdd3e7979c72360cc6 Mon Sep 17 00:00:00 2001 From: Peter Steinberger Date: Fri, 13 Mar 2026 23:39:44 +0000 Subject: [PATCH] fix: tighten package json coverage --- src/infra/package-json.test.ts | 9 ++++++++- src/infra/package-json.ts | 3 ++- 2 files changed, 10 insertions(+), 2 deletions(-) diff --git a/src/infra/package-json.test.ts b/src/infra/package-json.test.ts index 664fcaa4f14..19793929636 100644 --- a/src/infra/package-json.test.ts +++ b/src/infra/package-json.test.ts @@ -9,7 +9,7 @@ describe("package-json helpers", () => { await withTempDir({ prefix: "openclaw-package-json-" }, async (root) => { await fs.writeFile( path.join(root, "package.json"), - JSON.stringify({ version: "1.2.3", name: " @openclaw/demo " }), + JSON.stringify({ version: " 1.2.3 ", name: " @openclaw/demo " }), "utf8", ); @@ -34,6 +34,13 @@ describe("package-json helpers", () => { ); await expect(readPackageVersion(root)).resolves.toBeNull(); await expect(readPackageName(root)).resolves.toBeNull(); + + await fs.writeFile( + path.join(root, "package.json"), + JSON.stringify({ version: " ", name: "@openclaw/demo" }), + "utf8", + ); + await expect(readPackageVersion(root)).resolves.toBeNull(); }); }); }); diff --git a/src/infra/package-json.ts b/src/infra/package-json.ts index f0007a3c04d..8da5108938a 100644 --- a/src/infra/package-json.ts +++ b/src/infra/package-json.ts @@ -5,7 +5,8 @@ export async function readPackageVersion(root: string): Promise { try { const raw = await fs.readFile(path.join(root, "package.json"), "utf-8"); const parsed = JSON.parse(raw) as { version?: string }; - return typeof parsed?.version === "string" ? parsed.version : null; + const version = parsed?.version?.trim(); + return version ? version : null; } catch { return null; }