惯性聚合 高效追踪和阅读你感兴趣的博客、新闻、科技资讯
阅读原文 在惯性聚合中打开

推荐订阅源

L
LangChain Blog
B
Blog RSS Feed
阮一峰的网络日志
阮一峰的网络日志
Cyber Security Advisories - MS-ISAC
Cyber Security Advisories - MS-ISAC
H
Help Net Security
MyScale Blog
MyScale Blog
WordPress大学
WordPress大学
Microsoft Azure Blog
Microsoft Azure Blog
GbyAI
GbyAI
小众软件
小众软件
大猫的无限游戏
大猫的无限游戏
Martin Fowler
Martin Fowler
Vercel News
Vercel News
S
SegmentFault 最新的问题
M
MIT News - Artificial intelligence
Microsoft Security Blog
Microsoft Security Blog
G
Google Developers Blog
Last Week in AI
Last Week in AI
Hugging Face - Blog
Hugging Face - Blog
酷 壳 – CoolShell
酷 壳 – CoolShell
博客园 - 【当耐特】
Google DeepMind News
Google DeepMind News
Engineering at Meta
Engineering at Meta
云风的 BLOG
云风的 BLOG

Recent Commits to openclaw:main

test: merge chat side-result checks · openclaw/openclaw@ddd2c2a test: merge cron history checks · openclaw/openclaw@f7eb746 test: merge responsive navigation shell checks · openclaw/openclaw@c2e4b47 docs(changelog): add codex oauth fixes · openclaw/openclaw@628e6cd test: merge navigation routing cases · openclaw/openclaw@5d8cecb Tests: mock channel registry bundled fallback · openclaw/openclaw@2b08233 Secrets: avoid broad web search discovery for single plugin config · openclaw/openclaw@a464f59 test: merge config view browser checks · openclaw/openclaw@20cf511 fix(status): align oauth health with runtime · openclaw/openclaw@eed7116 feat: add macOS screen snapshots for monitor preview (#67954) thanks … · openclaw/openclaw@f377db1 fix: report shared auth scopes in hello-ok (#67810) thanks @BunsDev · openclaw/openclaw@0b6c39b Auto-reply: avoid eager bundled route fallback · openclaw/openclaw@3ea1bf4 Tests: narrow session binding contract setup · openclaw/openclaw@54e4e16 fix(macOS): enable undo/redo in webchat composer text input (#34962) · openclaw/openclaw@00951dc Tests: speed up channel setup promotion · openclaw/openclaw@82b529a Docs: refresh agent instructions · openclaw/openclaw@5775fe2 fix(auth): serialize OAuth refresh across agents to fix #26322 (#67876) · openclaw/openclaw@8e79080 test: allow ollama public surface boundary test · openclaw/openclaw@7d4f1a6 Docs: add test performance guardrails · openclaw/openclaw@89706d3 Tests: restore context-engine usage proof · openclaw/openclaw@e4c4f95 Tests: slim context engine runtime coverage · openclaw/openclaw@74c198f ci: retry failed custom checkouts · openclaw/openclaw@0ee5baf test: trim duplicate provider auth onboarding cases · openclaw/openclaw@1ffc02e matrix: fix sessions_spawn --thread subagent session spawning (#67643) · openclaw/openclaw@1ce2596 test: reduce auth choice fixture churn · openclaw/openclaw@857b9cd test: mock health status config boundaries · openclaw/openclaw@9d5ab4a test: mock onboard config io boundary · openclaw/openclaw@299694d test: mock legacy state plugin boundaries · openclaw/openclaw@2713089 test: mock channel install boundaries · openclaw/openclaw@b945248 test: mock doctor preview channel boundaries · openclaw/openclaw@b1a3ad4
test: clear file transfer write broad matchers · openclaw...
steipete · 2026-05-10 · via Recent Commits to openclaw:main

@@ -20,32 +20,59 @@ function b64(s: string): string {

2020

return Buffer.from(s, "utf-8").toString("base64");

2121

}

222223+

function expectFailure(result: Awaited<ReturnType<typeof handleFileWrite>>, code: string) {

24+

expect(result.ok).toBe(false);

25+

if (result.ok) {

26+

throw new Error("expected file write failure");

27+

}

28+

expect(result.code).toBe(code);

29+

}

30+31+

function expectSuccessFields(

32+

result: Awaited<ReturnType<typeof handleFileWrite>>,

33+

fields: Record<string, unknown>,

34+

) {

35+

expect(result.ok).toBe(true);

36+

if (!result.ok) {

37+

throw new Error(`expected ok, got ${result.code}: ${result.message}`);

38+

}

39+

for (const [key, value] of Object.entries(fields)) {

40+

expect(result[key as keyof typeof result]).toEqual(value);

41+

}

42+

}

43+44+

async function expectAccessMissing(target: string) {

45+

try {

46+

await fs.access(target);

47+

} catch (error) {

48+

expect((error as NodeJS.ErrnoException).code).toBe("ENOENT");

49+

return;

50+

}

51+

throw new Error(`expected ${target} to be missing`);

52+

}

53+2354

describe("handleFileWrite — input validation", () => {

2455

it("rejects empty / non-string path", async () => {

25-

expect(await handleFileWrite({ path: "", contentBase64: b64("x") })).toMatchObject({

26-

ok: false,

27-

code: "INVALID_PATH",

28-

});

56+

expectFailure(await handleFileWrite({ path: "", contentBase64: b64("x") }), "INVALID_PATH");

2957

});

30583159

it("rejects relative paths", async () => {

3260

const r = await handleFileWrite({ path: "relative.txt", contentBase64: b64("x") });

33-

expect(r).toMatchObject({ ok: false, code: "INVALID_PATH" });

61+

expectFailure(r, "INVALID_PATH");

3462

});

35633664

it("rejects paths with NUL bytes", async () => {

3765

const r = await handleFileWrite({ path: "/tmp/foo\0bar", contentBase64: b64("x") });

38-

expect(r).toMatchObject({ ok: false, code: "INVALID_PATH" });

66+

expectFailure(r, "INVALID_PATH");

3967

});

40684169

it("requires contentBase64 but allows an empty encoded payload", async () => {

4270

const missing = await handleFileWrite({ path: path.join(tmpRoot, "missing.bin") });

43-

expect(missing).toMatchObject({ ok: false, code: "INVALID_BASE64" });

71+

expectFailure(missing, "INVALID_BASE64");

44724573

const target = path.join(tmpRoot, "empty.bin");

4674

const empty = await handleFileWrite({ path: target, contentBase64: "" });

47-

expect(empty).toMatchObject({

48-

ok: true,

75+

expectSuccessFields(empty, {

4976

size: 0,

5077

sha256: "e3b0c44298fc1c149afbf4c8996fb92427ae41e4649b934ca495991b7852b855",

5178

});

@@ -91,7 +118,7 @@ describe("handleFileWrite — overwrite policy", () => {

91118

contentBase64: b64("after"),

92119

overwrite: false,

93120

});

94-

expect(r).toMatchObject({ ok: false, code: "EXISTS_NO_OVERWRITE" });

121+

expectFailure(r, "EXISTS_NO_OVERWRITE");

95122

expect(await fs.readFile(target, "utf-8")).toBe("before");

96123

});

97124

@@ -120,7 +147,7 @@ describe("handleFileWrite — parent directory handling", () => {

120147

contentBase64: b64("x"),

121148

createParents: false,

122149

});

123-

expect(r).toMatchObject({ ok: false, code: "PARENT_NOT_FOUND" });

150+

expectFailure(r, "PARENT_NOT_FOUND");

124151

});

125152126153

it("creates missing parents when createParents=true", async () => {

@@ -147,7 +174,7 @@ describe("handleFileWrite — symlink protection", () => {

147174

contentBase64: b64("evil"),

148175

overwrite: true,

149176

});

150-

expect(r).toMatchObject({ ok: false, code: "SYMLINK_TARGET_DENIED" });

177+

expectFailure(r, "SYMLINK_TARGET_DENIED");

151178

// The original file must be unchanged.

152179

expect(await fs.readFile(real, "utf-8")).toBe("untouched");

153180

});

@@ -169,14 +196,12 @@ describe("handleFileWrite — symlink protection", () => {

169196

path: path.join(allowed, "new-file.txt"),

170197

contentBase64: b64("payload"),

171198

});

172-

expect(r).toMatchObject({ ok: false, code: "SYMLINK_REDIRECT" });

199+

expectFailure(r, "SYMLINK_REDIRECT");

173200

// The error includes the canonical target so the operator can

174201

// either update allowWritePaths or set followSymlinks=true.

175202

expect(r.ok ? null : r.canonicalPath).toBe(path.join(realDir, "new-file.txt"));

176203

// No file was created at the canonical target.

177-

await expect(fs.access(path.join(realDir, "new-file.txt"))).rejects.toMatchObject({

178-

code: "ENOENT",

179-

});

204+

await expectAccessMissing(path.join(realDir, "new-file.txt"));

180205

// Sentinel must be untouched.

181206

expect(await fs.readFile(sentinel, "utf-8")).toBe("DO_NOT_TOUCH");

182207

});

@@ -193,11 +218,9 @@ describe("handleFileWrite — symlink protection", () => {

193218

createParents: true,

194219

});

195220196-

expect(r).toMatchObject({ ok: false, code: "SYMLINK_REDIRECT" });

221+

expectFailure(r, "SYMLINK_REDIRECT");

197222

expect(r.ok ? null : r.canonicalPath).toBe(path.join(realDir, "new", "child.txt"));

198-

await expect(fs.access(path.join(realDir, "new"))).rejects.toMatchObject({

199-

code: "ENOENT",

200-

});

223+

await expectAccessMissing(path.join(realDir, "new"));

201224

});

202225203226

it("follows the parent symlink when followSymlinks=true", async () => {

@@ -230,14 +253,11 @@ describe("handleFileWrite — symlink protection", () => {

230253

preflightOnly: true,

231254

});

232255233-

expect(r).toMatchObject({

234-

ok: true,

256+

expectSuccessFields(r, {

235257

path: path.join(realDir, "new", "child.txt"),

236258

size: "payload".length,

237259

});

238-

await expect(fs.access(path.join(realDir, "new"))).rejects.toMatchObject({

239-

code: "ENOENT",

240-

});

260+

await expectAccessMissing(path.join(realDir, "new"));

241261

});

242262243263

it("refuses to overwrite a directory", async () => {

@@ -249,7 +269,7 @@ describe("handleFileWrite — symlink protection", () => {

249269

contentBase64: b64("x"),

250270

overwrite: true,

251271

});

252-

expect(r).toMatchObject({ ok: false, code: "IS_DIRECTORY" });

272+

expectFailure(r, "IS_DIRECTORY");

253273

});

254274

});

255275

@@ -261,9 +281,9 @@ describe("handleFileWrite — integrity check", () => {

261281

contentBase64: b64("real-content"),

262282

expectedSha256: "0".repeat(64),

263283

});

264-

expect(r).toMatchObject({ ok: false, code: "INTEGRITY_FAILURE" });

284+

expectFailure(r, "INTEGRITY_FAILURE");

265285

// The file must never be created on a mismatch.

266-

await expect(fs.access(target)).rejects.toMatchObject({ code: "ENOENT" });

286+

await expectAccessMissing(target);

267287

});

268288269289

it("does NOT replace or delete an existing file when overwrite=true and expectedSha256 mismatches", async () => {

@@ -276,7 +296,7 @@ describe("handleFileWrite — integrity check", () => {

276296

overwrite: true,

277297

expectedSha256: "0".repeat(64),

278298

});

279-

expect(r).toMatchObject({ ok: false, code: "INTEGRITY_FAILURE" });

299+

expectFailure(r, "INTEGRITY_FAILURE");

280300

// Critical: the original must survive. A bad caller hash must not

281301

// be a primitive for replacing-then-deleting an existing file.

282302

expect(await fs.readFile(target, "utf-8")).toBe("ORIGINAL_CONTENT_DO_NOT_TOUCH");

@@ -319,8 +339,8 @@ describe("handleFileWrite — base64 round-trip validation", () => {

319339

path: target,

320340

contentBase64: "AAA@@@",

321341

});

322-

expect(r).toMatchObject({ ok: false, code: "INVALID_BASE64" });

323-

await expect(fs.access(target)).rejects.toMatchObject({ code: "ENOENT" });

342+

expectFailure(r, "INVALID_BASE64");

343+

await expectAccessMissing(target);

324344

});

325345326346

it("accepts standard base64 with and without padding", async () => {

@@ -352,6 +372,6 @@ describe("handleFileWrite — size cap", () => {

352372

path: target,

353373

contentBase64: big.toString("base64"),

354374

});

355-

expect(r).toMatchObject({ ok: false, code: "FILE_TOO_LARGE" });

375+

expectFailure(r, "FILE_TOO_LARGE");

356376

});

357377

});