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

推荐订阅源

云风的 BLOG
云风的 BLOG
V
Visual Studio Blog
人人都是产品经理
人人都是产品经理
The GitHub Blog
The GitHub Blog
月光博客
月光博客
T
Tailwind CSS Blog
小众软件
小众软件
Y
Y Combinator Blog
钛媒体:引领未来商业与生活新知
钛媒体:引领未来商业与生活新知
P
Proofpoint News Feed
B
Blog RSS Feed
博客园 - 司徒正美
A
About on SuperTechFans
Cyber Security Advisories - MS-ISAC
Cyber Security Advisories - MS-ISAC
博客园 - 聂微东
Microsoft Security Blog
Microsoft Security Blog
Recent Announcements
Recent Announcements
博客园 - Franky
U
Unit 42
奇客Solidot–传递最新科技情报
奇客Solidot–传递最新科技情报
Microsoft Azure Blog
Microsoft Azure Blog
T
The Blog of Author Tim Ferriss
GbyAI
GbyAI
Apple Machine Learning Research
Apple Machine Learning Research

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: guard msteams reply dispatcher mock calls · opencla...
steipete · 2026-05-12 · via Recent Commits to openclaw:main

@@ -159,6 +159,51 @@ describe("createMSTeamsReplyDispatcher", () => {

159159

return lastContextSendActivity;

160160

}

161161162+

type DispatcherOptions = {

163+

onReplyStart?: () => Promise<void> | void;

164+

deliver: (payload: { text: string }) => Promise<void> | void;

165+

};

166+167+

type PipelineArgs = {

168+

typing?: {

169+

keepaliveIntervalMs?: number;

170+

maxDurationMs?: number;

171+

start?: () => Promise<void>;

172+

};

173+

};

174+175+

function dispatcherOptions(): DispatcherOptions {

176+

const [call] = createReplyDispatcherWithTypingMock.mock.calls;

177+

if (!call) {

178+

throw new Error("expected reply dispatcher factory call");

179+

}

180+

return call[0] as DispatcherOptions;

181+

}

182+183+

function pipelineArgs(): PipelineArgs {

184+

const [call] = createChannelMessageReplyPipelineMock.mock.calls;

185+

if (!call) {

186+

throw new Error("expected reply pipeline factory call");

187+

}

188+

return call[0] as PipelineArgs;

189+

}

190+191+

function pipelineTypingStart(): () => Promise<void> {

192+

const sendTyping = pipelineArgs().typing?.start;

193+

if (typeof sendTyping !== "function") {

194+

throw new Error("expected typing start callback");

195+

}

196+

return sendTyping;

197+

}

198+199+

function firstSystemEventCall(): [string, unknown] {

200+

const [call] = enqueueSystemEventMock.mock.calls;

201+

if (!call) {

202+

throw new Error("expected system event call");

203+

}

204+

return call as [string, unknown];

205+

}

206+162207

async function triggerPartialReply(text: string): Promise<void> {

163208

if (!lastCreatedDispatcher) {

164209

throw new Error("createDispatcher must be called first");

@@ -168,7 +213,7 @@ describe("createMSTeamsReplyDispatcher", () => {

168213169214

it("sends an informative status update once work expands in personal chats", async () => {

170215

const dispatcher = createDispatcher("personal", { streaming: { mode: "progress" } });

171-

const options = createReplyDispatcherWithTypingMock.mock.calls[0]?.[0];

216+

const options = dispatcherOptions();

172217173218

await options.onReplyStart?.();

174219

await dispatcher.replyOptions.onToolStart?.({ name: "exec" });

@@ -180,7 +225,7 @@ describe("createMSTeamsReplyDispatcher", () => {

180225181226

it("starts the typing keepalive in personal chats so the TurnContext survives long tool chains", async () => {

182227

createDispatcher("personal");

183-

const options = createReplyDispatcherWithTypingMock.mock.calls[0]?.[0];

228+

const options = dispatcherOptions();

184229185230

await options.onReplyStart?.();

186231

@@ -192,7 +237,7 @@ describe("createMSTeamsReplyDispatcher", () => {

192237193238

it("skips the typing keepalive in personal chats when typingIndicator=false", async () => {

194239

createDispatcher("personal", { typingIndicator: false });

195-

const options = createReplyDispatcherWithTypingMock.mock.calls[0]?.[0];

240+

const options = dispatcherOptions();

196241197242

await options.onReplyStart?.();

198243

@@ -203,18 +248,17 @@ describe("createMSTeamsReplyDispatcher", () => {

203248

it("passes a longer keepalive TTL so the loop survives long tool chains", () => {

204249

createDispatcher("personal");

205250206-

const pipelineArgs = createChannelMessageReplyPipelineMock.mock.calls[0]?.[0];

207-

expect(pipelineArgs?.typing?.keepaliveIntervalMs).toBeGreaterThan(3_000);

208-

expect(pipelineArgs?.typing?.keepaliveIntervalMs).toBeLessThanOrEqual(10_000);

251+

const args = pipelineArgs();

252+

expect(args.typing?.keepaliveIntervalMs).toBeGreaterThan(3_000);

253+

expect(args.typing?.keepaliveIntervalMs).toBeLessThanOrEqual(10_000);

209254

// Issue #59731 reports 60s+ tool chains — the default 60s TTL is too

210255

// tight so the dispatcher passes its own generous ceiling.

211-

expect(pipelineArgs?.typing?.maxDurationMs).toBeGreaterThanOrEqual(300_000);

256+

expect(args.typing?.maxDurationMs).toBeGreaterThanOrEqual(300_000);

212257

});

213258214259

it("allows typing keepalive sends before any stream tokens arrive", async () => {

215260

createDispatcher("personal");

216-

const pipelineArgs = createChannelMessageReplyPipelineMock.mock.calls[0]?.[0];

217-

const sendTyping = pipelineArgs?.typing?.start as () => Promise<void>;

261+

const sendTyping = pipelineTypingStart();

218262219263

// No onPartialReply has been called yet, so the stream is not active.

220264

// The typing keepalive should be allowed to warm the TurnContext.

@@ -226,8 +270,7 @@ describe("createMSTeamsReplyDispatcher", () => {

226270227271

it("suppresses typing keepalive sends while the stream card is actively chunking", async () => {

228272

createDispatcher("personal");

229-

const pipelineArgs = createChannelMessageReplyPipelineMock.mock.calls[0]?.[0];

230-

const sendTyping = pipelineArgs?.typing?.start as () => Promise<void>;

273+

const sendTyping = pipelineTypingStart();

231274232275

// Simulate the stream actively receiving a partial chunk. While the

233276

// stream card is live we do not want a plain "..." typing indicator

@@ -242,8 +285,7 @@ describe("createMSTeamsReplyDispatcher", () => {

242285243286

it("resumes typing keepalive sends once the stream finalizes between tool rounds", async () => {

244287

createDispatcher("personal");

245-

const pipelineArgs = createChannelMessageReplyPipelineMock.mock.calls[0]?.[0];

246-

const sendTyping = pipelineArgs?.typing?.start as () => Promise<void>;

288+

const sendTyping = pipelineTypingStart();

247289248290

// First segment: tokens flow, stream is active, typing is gated off.

249291

await triggerPartialReply("first segment tokens");

@@ -271,8 +313,7 @@ describe("createMSTeamsReplyDispatcher", () => {

271313272314

it("fires native typing in group chats (no stream) because the gate never applies", async () => {

273315

createDispatcher("groupchat");

274-

const pipelineArgs = createChannelMessageReplyPipelineMock.mock.calls[0]?.[0];

275-

const sendTyping = pipelineArgs?.typing?.start as () => Promise<void>;

316+

const sendTyping = pipelineTypingStart();

276317277318

// In group chats we don't create a stream, so isStreamActive() always

278319

// returns false and the typing indicator still fires normally.

@@ -284,8 +325,7 @@ describe("createMSTeamsReplyDispatcher", () => {

284325285326

it("is a no-op for channel conversations (typing unsupported)", async () => {

286327

createDispatcher("channel");

287-

const pipelineArgs = createChannelMessageReplyPipelineMock.mock.calls[0]?.[0];

288-

const sendTyping = pipelineArgs?.typing?.start as () => Promise<void>;

328+

const sendTyping = pipelineTypingStart();

289329290330

const contextSendActivity = getContextSendActivity();

291331

contextSendActivity.mockClear();

@@ -297,7 +337,7 @@ describe("createMSTeamsReplyDispatcher", () => {

297337298338

it("sends native typing indicator for channel conversations by default", async () => {

299339

createDispatcher("channel");

300-

const options = createReplyDispatcherWithTypingMock.mock.calls[0]?.[0];

340+

const options = dispatcherOptions();

301341302342

await options.onReplyStart?.();

303343

@@ -307,7 +347,7 @@ describe("createMSTeamsReplyDispatcher", () => {

307347308348

it("skips native typing indicator when typingIndicator=false", async () => {

309349

createDispatcher("channel", { typingIndicator: false });

310-

const options = createReplyDispatcherWithTypingMock.mock.calls[0]?.[0];

350+

const options = dispatcherOptions();

311351312352

await options.onReplyStart?.();

313353

@@ -393,7 +433,7 @@ describe("createMSTeamsReplyDispatcher", () => {

393433

sendMSTeamsMessagesMock.mockResolvedValue(["id-1"] as never);

394434395435

const dispatcher = createDispatcher("personal", { streaming: { mode: "block" } });

396-

const options = createReplyDispatcherWithTypingMock.mock.calls[0]?.[0];

436+

const options = dispatcherOptions();

397437398438

await options.deliver({ text: "block content" });

399439

@@ -420,7 +460,7 @@ describe("createMSTeamsReplyDispatcher", () => {

420460

sendMSTeamsMessagesMock.mockResolvedValue(["id-1"] as never);

421461422462

createDispatcher("personal", { blockStreaming: true });

423-

const options = createReplyDispatcherWithTypingMock.mock.calls[0]?.[0];

463+

const options = dispatcherOptions();

424464425465

// Call deliver — with blockStreaming enabled it should flush immediately

426466

await options.deliver({ text: "block content" });

@@ -432,7 +472,7 @@ describe("createMSTeamsReplyDispatcher", () => {

432472

renderReplyPayloadsToMessagesMock.mockReturnValue([{ content: "hello" }] as never);

433473434474

createDispatcher("personal", { blockStreaming: false });

435-

const options = createReplyDispatcherWithTypingMock.mock.calls[0]?.[0];

475+

const options = dispatcherOptions();

436476437477

await options.deliver({ text: "block content" });

438478

@@ -455,14 +495,14 @@ describe("createMSTeamsReplyDispatcher", () => {

455495

{ blockStreaming: false },

456496

{ onSentMessageIds },

457497

);

458-

const options = createReplyDispatcherWithTypingMock.mock.calls[0]?.[0];

498+

const options = dispatcherOptions();

459499460500

await options.deliver({ text: "block content" });

461501

await dispatcher.markDispatchIdle();

462502463503

expect(onSentMessageIds).toHaveBeenCalledWith(["id-1"]);

464504

expect(enqueueSystemEventMock).toHaveBeenCalledTimes(1);

465-

const [message, context] = enqueueSystemEventMock.mock.calls[0] ?? [];

505+

const [message, context] = firstSystemEventCall();

466506

expect(message).toContain("Microsoft Teams delivery failed");

467507

expect(message).toContain("1 of 2 message blocks were not delivered");

468508

expect(message).toContain("The user may not have received the full reply");

@@ -478,7 +518,7 @@ describe("createMSTeamsReplyDispatcher", () => {

478518

sendMSTeamsMessagesMock.mockResolvedValue(["id-1"] as never);

479519480520

const dispatcher = createDispatcher("personal", { blockStreaming: false });

481-

const options = createReplyDispatcherWithTypingMock.mock.calls[0]?.[0];

521+

const options = dispatcherOptions();

482522483523

await options.deliver({ text: "block content" });

484524

await dispatcher.markDispatchIdle();