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

推荐订阅源

爱范儿
爱范儿
H
Help Net Security
Jina AI
Jina AI
T
The Blog of Author Tim Ferriss
宝玉的分享
宝玉的分享
博客园 - 叶小钗
Y
Y Combinator Blog
罗磊的独立博客
大猫的无限游戏
大猫的无限游戏
WordPress大学
WordPress大学
C
Check Point Blog
Recent Announcements
Recent Announcements
IT之家
IT之家
奇客Solidot–传递最新科技情报
奇客Solidot–传递最新科技情报
美团技术团队
云风的 BLOG
云风的 BLOG
雷峰网
雷峰网
H
Hackread – Cybersecurity News, Data Breaches, AI and More
S
SegmentFault 最新的问题
MyScale Blog
MyScale Blog
Apple Machine Learning Research
Apple Machine Learning Research
Microsoft Azure Blog
Microsoft Azure Blog
V
Visual Studio Blog
B
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: tighten feishu comment handler assertions · opencla...
shakkernerd · 2026-05-11 · via Recent Commits to openclaw:main

@@ -263,23 +263,32 @@ describe("handleFeishuCommentEvent", () => {

263263

typeof vi.fn

264264

>;

265265266-

expect(finalizeInboundContext).toHaveBeenCalledWith(

267-

expect.objectContaining({

268-

From: "feishu:ou_sender",

269-

To: "comment:docx:doc_token_1:comment_1",

270-

Surface: "feishu-comment",

271-

OriginatingChannel: "feishu",

272-

OriginatingTo: "comment:docx:doc_token_1:comment_1",

273-

MessageSid: "drive-comment:evt_1",

274-

MessageThreadId: "reply_1",

275-

}),

276-

);

266+

expect(finalizeInboundContext).toHaveBeenCalledTimes(1);

267+

const finalizedContext = finalizeInboundContext.mock.calls[0]?.[0] as

268+

| Record<string, unknown>

269+

| undefined;

270+

expect({

271+

from: finalizedContext?.From,

272+

to: finalizedContext?.To,

273+

surface: finalizedContext?.Surface,

274+

originatingChannel: finalizedContext?.OriginatingChannel,

275+

originatingTo: finalizedContext?.OriginatingTo,

276+

messageSid: finalizedContext?.MessageSid,

277+

messageThreadId: finalizedContext?.MessageThreadId,

278+

}).toEqual({

279+

from: "feishu:ou_sender",

280+

to: "comment:docx:doc_token_1:comment_1",

281+

surface: "feishu-comment",

282+

originatingChannel: "feishu",

283+

originatingTo: "comment:docx:doc_token_1:comment_1",

284+

messageSid: "drive-comment:evt_1",

285+

messageThreadId: "reply_1",

286+

});

277287

expect(recordInboundSession).toHaveBeenCalledTimes(1);

278-

expect(recordInboundSession).toHaveBeenCalledWith(

279-

expect.objectContaining({

280-

sessionKey: "agent:main:feishu:direct:comment-doc:docx:doc_token_1",

281-

}),

282-

);

288+

const recordArgs = recordInboundSession.mock.calls[0]?.[0] as

289+

| { sessionKey?: string }

290+

| undefined;

291+

expect(recordArgs?.sessionKey).toBe("agent:main:feishu:direct:comment-doc:docx:doc_token_1");

283292

expect(dispatchReplyFromConfig).toHaveBeenCalledTimes(1);

284293

});

285294

@@ -342,12 +351,12 @@ describe("handleFeishuCommentEvent", () => {

342351

} as never,

343352

});

344353345-

expect(maybeCreateDynamicAgentMock).toHaveBeenCalledWith(

346-

expect.objectContaining({

347-

senderOpenId: "ou_sender",

348-

configWritesAllowed: false,

349-

}),

350-

);

354+

expect(maybeCreateDynamicAgentMock).toHaveBeenCalledTimes(1);

355+

const dynamicAgentArgs = maybeCreateDynamicAgentMock.mock.calls[0]?.[0] as

356+

| { configWritesAllowed?: boolean; senderOpenId?: string }

357+

| undefined;

358+

expect(dynamicAgentArgs?.senderOpenId).toBe("ou_sender");

359+

expect(dynamicAgentArgs?.configWritesAllowed).toBe(false);

351360

const dispatchReplyFromConfig = runtime.channel.reply.dispatchReplyFromConfig as ReturnType<

352361

typeof vi.fn

353362

>;

@@ -377,15 +386,30 @@ describe("handleFeishuCommentEvent", () => {

377386

} as never,

378387

});

379388380-

expect(deliverCommentThreadTextMock).toHaveBeenCalledWith(

381-

expect.anything(),

382-

expect.objectContaining({

383-

file_token: "doc_token_1",

384-

file_type: "docx",

385-

comment_id: "comment_1",

386-

is_whole_comment: false,

387-

}),

388-

);

389+

expect(deliverCommentThreadTextMock).toHaveBeenCalledTimes(1);

390+

const [pairingClient, pairingReply] = deliverCommentThreadTextMock.mock.calls[0] ?? [];

391+

expect(pairingClient).toBe(createFeishuClientMock.mock.results[0]?.value);

392+

expect(pairingReply).toEqual({

393+

file_token: "doc_token_1",

394+

file_type: "docx",

395+

comment_id: "comment_1",

396+

content: [

397+

"OpenClaw: access not configured.",

398+

"",

399+

"Your Feishu user id: ou_sender",

400+

"Pairing code:",

401+

"```",

402+

"TESTCODE",

403+

"```",

404+

"",

405+

"Ask the bot owner to approve with:",

406+

"openclaw pairing approve feishu TESTCODE",

407+

"```",

408+

"openclaw pairing approve feishu TESTCODE",

409+

"```",

410+

].join("\n"),

411+

is_whole_comment: false,

412+

});

389413

const dispatchReplyFromConfig = runtime.channel.reply.dispatchReplyFromConfig as ReturnType<

390414

typeof vi.fn

391415

>;

@@ -424,15 +448,21 @@ describe("handleFeishuCommentEvent", () => {

424448

} as never,

425449

});

426450427-

expect(createFeishuCommentReplyDispatcherMock).toHaveBeenCalledWith(

428-

expect.objectContaining({

429-

commentId: "comment_whole",

430-

fileToken: "doc_token_1",

431-

fileType: "docx",

432-

replyId: "reply_whole",

433-

isWholeComment: true,

434-

}),

435-

);

451+

expect(createFeishuCommentReplyDispatcherMock).toHaveBeenCalledTimes(1);

452+

const dispatcherArgs = createFeishuCommentReplyDispatcherMock.mock.calls[0]?.[0] as

453+

| {

454+

commentId?: string;

455+

fileToken?: string;

456+

fileType?: string;

457+

isWholeComment?: boolean;

458+

replyId?: string;

459+

}

460+

| undefined;

461+

expect(dispatcherArgs?.commentId).toBe("comment_whole");

462+

expect(dispatcherArgs?.fileToken).toBe("doc_token_1");

463+

expect(dispatcherArgs?.fileType).toBe("docx");

464+

expect(dispatcherArgs?.replyId).toBe("reply_whole");

465+

expect(dispatcherArgs?.isWholeComment).toBe(true);

436466

});

437467438468

it("always finalizes comment typing cleanup even when dispatch fails", async () => {