test(core): trim redundant test resets and use mockClear

src/agents/pi-embedded-subscribe.subscribe-embedded-pi-session.filters-final-suppresses-output-without-start-tag.e2e.test.ts

@@ -30,7 +30,7 @@ describe("subscribeEmbeddedPiSession", () => {
     const firstPayload = onPartialReply.mock.calls[0][0];
     expect(firstPayload.text).toBe("Hi there");
 
-    onPartialReply.mockReset();
+    onPartialReply.mockClear();
 
     emit({ type: "message_start", message: { role: "assistant" } });
     emitAssistantTextDelta({ emit, delta: "</final>Oops no start" });

src/auto-reply/reply.triggers.trigger-handling.stages-inbound-media-into-sandbox-workspace.test.ts

@@ -22,7 +22,7 @@ import { stageSandboxMedia } from "./reply/stage-sandbox-media.js";
 
 afterEach(() => {
   vi.restoreAllMocks();
-  childProcessMocks.spawn.mockReset();
+  childProcessMocks.spawn.mockClear();
 });
 
 describe("stageSandboxMedia", () => {

src/auto-reply/reply/get-reply-inline-actions.skip-when-config-empty.test.ts

@@ -17,8 +17,6 @@ const { handleInlineActions } = await import("./get-reply-inline-actions.js");
 
 describe("handleInlineActions", () => {
   it("skips whatsapp replies when config is empty and From !== To", async () => {
-    handleCommandsMock.mockReset();
-
     const typing: TypingController = {
       onReplyStart: async () => {},
       startTypingLoop: async () => {},

src/commands/doctor-session-locks.test.ts

@@ -17,7 +17,7 @@ describe("noteSessionLockHealth", () => {
   let envSnapshot: ReturnType<typeof captureEnv>;
 
   beforeEach(async () => {
-    note.mockReset();
+    note.mockClear();
     envSnapshot = captureEnv(["OPENCLAW_STATE_DIR"]);
     root = await fs.mkdtemp(path.join(os.tmpdir(), "openclaw-doctor-locks-"));
     process.env.OPENCLAW_STATE_DIR = root;

src/commands/doctor-state-integrity.test.ts

@@ -77,7 +77,7 @@ describe("doctor state integrity oauth dir checks", () => {
     process.env.OPENCLAW_STATE_DIR = path.join(tempHome, ".openclaw");
     delete process.env.OPENCLAW_OAUTH_DIR;
     fs.mkdirSync(process.env.OPENCLAW_STATE_DIR, { recursive: true, mode: 0o700 });
-    vi.mocked(note).mockReset();
+    vi.mocked(note).mockClear();
   });
 
   afterEach(() => {

src/discord/send.webhook-activity.test.ts

@@ -13,7 +13,7 @@ vi.mock("../infra/channel-activity.js", async (importOriginal) => {
 
 describe("sendWebhookMessageDiscord activity", () => {
   beforeEach(() => {
-    recordChannelActivityMock.mockReset();
+    recordChannelActivityMock.mockClear();
     vi.stubGlobal(
       "fetch",
       vi.fn(async () => {

src/gateway/startup-auth.test.ts

@@ -36,7 +36,7 @@ describe("ensureGatewayStartupAuth", () => {
 
   beforeEach(() => {
     vi.restoreAllMocks();
-    mocks.writeConfigFile.mockReset();
+    mocks.writeConfigFile.mockClear();
   });
 
   async function expectNoTokenGeneration(cfg: OpenClawConfig, mode: string) {

src/imessage/targets.test.ts

@@ -87,7 +87,7 @@ describe("imessage targets", () => {
 
 describe("createIMessageRpcClient", () => {
   beforeEach(() => {
-    spawnMock.mockReset();
+    spawnMock.mockClear();
     vi.stubEnv("VITEST", "true");
   });
 

src/infra/ports.test.ts

@@ -91,7 +91,7 @@ describe("ports helpers", () => {
 
 describeUnix("inspectPortUsage", () => {
   beforeEach(() => {
-    runCommandWithTimeoutMock.mockReset();
+    runCommandWithTimeoutMock.mockClear();
   });
 
   it("reports busy when lsof is missing but loopback listener exists", async () => {

src/infra/process-respawn.test.ts

@@ -17,7 +17,7 @@ afterEach(() => {
   envSnapshot.restore();
   process.argv = [...originalArgv];
   process.execArgv = [...originalExecArgv];
-  spawnMock.mockReset();
+  spawnMock.mockClear();
 });
 
 function clearSupervisorHints() {

src/line/probe.test.ts

@@ -15,7 +15,7 @@ let probeLineBot: typeof import("./probe.js").probeLineBot;
 
 afterEach(() => {
   vi.useRealTimers();
-  getBotInfoMock.mockReset();
+  getBotInfoMock.mockClear();
 });
 
 describe("probeLineBot", () => {

src/plugins/tools.optional.test.ts

@@ -54,7 +54,7 @@ function setRegistry(entries: MockRegistryToolEntry[]) {
 
 describe("resolvePluginTools optional tools", () => {
   beforeEach(() => {
-    loadOpenClawPluginsMock.mockReset();
+    loadOpenClawPluginsMock.mockClear();
   });
 
   it("skips optional tools without explicit allowlist", () => {

src/process/kill-tree.test.ts

@@ -25,7 +25,7 @@ describe("killProcessTree", () => {
   let killSpy: ReturnType<typeof vi.spyOn>;
 
   beforeEach(() => {
-    spawnMock.mockReset();
+    spawnMock.mockClear();
     killSpy = vi.spyOn(process, "kill");
     vi.useFakeTimers();
   });

src/process/supervisor/supervisor.pty-command.test.ts

@@ -40,7 +40,7 @@ describe("process supervisor PTY command contract", () => {
   });
 
   beforeEach(() => {
-    createPtyAdapterMock.mockReset();
+    createPtyAdapterMock.mockClear();
   });
 
   it("passes PTY command verbatim to shell args", async () => {

src/telegram/accounts.test.ts

@@ -19,7 +19,7 @@ vi.mock("../logging/subsystem.js", () => ({
 
 describe("resolveTelegramAccount", () => {
   afterEach(() => {
-    warnMock.mockReset();
+    warnMock.mockClear();
   });
 
   it("falls back to the first configured account when accountId is omitted", () => {