fix(clawsweeper): address review for automerge-openclaw-openclaw-84878 (validation-1)

clawsweeper[bot] · clawsweeper[bot] · commit 1dd8a88d218d · 2026-05-23T23:50:00.000Z
diff --git a/extensions/meeting-notes/index.test.ts b/extensions/meeting-notes/index.test.ts
@@ -429,15 +429,19 @@ describe("meeting-notes plugin", () => {
       ],
     });
     const logger = { debug: vi.fn(), error: vi.fn(), info: vi.fn(), warn: vi.fn() };
+    const service = services[0];
+    if (!service?.stop) {
+      throw new Error("Expected meeting notes service with stop hook");
+    }
 
-    await services[0]?.start({ config: {}, logger, stateDir });
+    await service.start({ config: {}, logger, stateDir });
     await vi.waitFor(() => {
       expect(start).toHaveBeenCalledOnce();
     });
     const request = start.mock.calls[0]?.[0];
     expect(request.abortSignal?.aborted).toBe(false);
 
-    await services[0]?.stop({ config: {}, logger, stateDir });
+    await service.stop({ config: {}, logger, stateDir });
 
     expect(request.abortSignal?.aborted).toBe(true);
     expect(stop).not.toHaveBeenCalled();
diff --git a/extensions/meeting-notes/src/tool.ts b/extensions/meeting-notes/src/tool.ts
@@ -106,7 +106,7 @@ async function waitForPendingAutoStartsToSettle(
   let timeout: ReturnType<typeof setTimeout> | undefined;
   try {
     return await Promise.race([
-      Promise.allSettled([...pendingStarts]).then(() => true),
+      Promise.allSettled(pendingStarts).then(() => true),
       new Promise<boolean>((resolve) => {
         timeout = setTimeout(() => resolve(false), AUTO_START_STOP_TIMEOUT_MS);
         timeout.unref?.();
