From 15cccd6747353e6707548af133ea1c971224f62d Mon Sep 17 00:00:00 2001
From: joshavant <830519+joshavant@users.noreply.github.com>
Date: Tue, 17 Mar 2026 17:04:57 -0500
Subject: [PATCH] Tests: remove resetModules crash from extension lanes

---
 extensions/discord/src/monitor/message-utils.test.ts          | 4 ----
 extensions/discord/src/monitor/monitor.test.ts                | 4 ----
 .../discord/src/monitor/thread-bindings.discord-api.test.ts   | 4 ----
 extensions/slack/src/monitor.test-helpers.ts                  | 4 ----
 extensions/slack/src/monitor/slash.test-harness.ts            | 4 ----
 src/line/bot-handlers.test.ts                                 | 4 ----
 src/line/download.test.ts                                     | 4 ----
 src/line/monitor.lifecycle.test.ts                            | 4 ----
 src/line/probe.test.ts                                        | 4 ----
 src/line/send.test.ts                                         | 4 ----
 10 files changed, 40 deletions(-)

diff --git a/extensions/discord/src/monitor/message-utils.test.ts b/extensions/discord/src/monitor/message-utils.test.ts
index 92d4250bac3..e337807d622 100644
--- a/extensions/discord/src/monitor/message-utils.test.ts
+++ b/extensions/discord/src/monitor/message-utils.test.ts
@@ -2,10 +2,6 @@ import { ChannelType, type Client, type Message } from "@buape/carbon";
 import { StickerFormatType } from "discord-api-types/v10";
 import { beforeEach, describe, expect, it, vi } from "vitest";
 
-vi.hoisted(() => {
-  vi.resetModules();
-});
-
 const fetchRemoteMedia = vi.fn();
 const saveMediaBuffer = vi.fn();
 
diff --git a/extensions/discord/src/monitor/monitor.test.ts b/extensions/discord/src/monitor/monitor.test.ts
index 5d7c2e2139e..8778560535b 100644
--- a/extensions/discord/src/monitor/monitor.test.ts
+++ b/extensions/discord/src/monitor/monitor.test.ts
@@ -46,10 +46,6 @@ import {
   resolveDiscordReplyDeliveryPlan,
 } from "./threading.js";
 
-vi.hoisted(() => {
-  vi.resetModules();
-});
-
 const readAllowFromStoreMock = vi.hoisted(() => vi.fn());
 const upsertPairingRequestMock = vi.hoisted(() => vi.fn());
 const enqueueSystemEventMock = vi.hoisted(() => vi.fn());
diff --git a/extensions/discord/src/monitor/thread-bindings.discord-api.test.ts b/extensions/discord/src/monitor/thread-bindings.discord-api.test.ts
index bb95b642e26..eb085235da7 100644
--- a/extensions/discord/src/monitor/thread-bindings.discord-api.test.ts
+++ b/extensions/discord/src/monitor/thread-bindings.discord-api.test.ts
@@ -3,10 +3,6 @@ import { beforeEach, describe, expect, it, vi } from "vitest";
 import type { OpenClawConfig } from "../../../../src/config/config.js";
 import type { ThreadBindingRecord } from "./thread-bindings.types.js";
 
-vi.hoisted(() => {
-  vi.resetModules();
-});
-
 const hoisted = vi.hoisted(() => {
   const restGet = vi.fn();
   const sendMessageDiscord = vi.fn();
diff --git a/extensions/slack/src/monitor.test-helpers.ts b/extensions/slack/src/monitor.test-helpers.ts
index 301531361d5..4f64d74abda 100644
--- a/extensions/slack/src/monitor.test-helpers.ts
+++ b/extensions/slack/src/monitor.test-helpers.ts
@@ -1,9 +1,5 @@
 import { Mock, vi } from "vitest";
 
-vi.hoisted(() => {
-  vi.resetModules();
-});
-
 type SlackHandler = (args: unknown) => Promise<void>;
 type SlackProviderMonitor = (params: {
   botToken: string;
diff --git a/extensions/slack/src/monitor/slash.test-harness.ts b/extensions/slack/src/monitor/slash.test-harness.ts
index 593b1443362..6f26562dd5b 100644
--- a/extensions/slack/src/monitor/slash.test-harness.ts
+++ b/extensions/slack/src/monitor/slash.test-harness.ts
@@ -1,9 +1,5 @@
 import { vi } from "vitest";
 
-vi.hoisted(() => {
-  vi.resetModules();
-});
-
 const mocks = vi.hoisted(() => ({
   dispatchMock: vi.fn(),
   readAllowFromStoreMock: vi.fn(),
diff --git a/src/line/bot-handlers.test.ts b/src/line/bot-handlers.test.ts
index 4d625c2b18b..b52a9f13b54 100644
--- a/src/line/bot-handlers.test.ts
+++ b/src/line/bot-handlers.test.ts
@@ -2,10 +2,6 @@ import type { MessageEvent, PostbackEvent } from "@line/bot-sdk";
 import { beforeAll, beforeEach, describe, expect, it, vi } from "vitest";
 import type { LineAccountConfig } from "./types.js";
 
-vi.hoisted(() => {
-  vi.resetModules();
-});
-
 // Avoid pulling in globals/pairing/media dependencies; this suite only asserts
 // allowlist/groupPolicy gating and message-context wiring.
 vi.mock("../globals.js", () => ({
diff --git a/src/line/download.test.ts b/src/line/download.test.ts
index 9c239f760cc..1a0d12b2a3b 100644
--- a/src/line/download.test.ts
+++ b/src/line/download.test.ts
@@ -3,10 +3,6 @@ import path from "node:path";
 import { beforeEach, describe, expect, it, vi } from "vitest";
 import { resolvePreferredOpenClawTmpDir } from "../infra/tmp-openclaw-dir.js";
 
-vi.hoisted(() => {
-  vi.resetModules();
-});
-
 const getMessageContentMock = vi.hoisted(() => vi.fn());
 
 vi.mock("@line/bot-sdk", () => ({
diff --git a/src/line/monitor.lifecycle.test.ts b/src/line/monitor.lifecycle.test.ts
index 66317e7f90e..d1ad3194096 100644
--- a/src/line/monitor.lifecycle.test.ts
+++ b/src/line/monitor.lifecycle.test.ts
@@ -2,10 +2,6 @@ import { beforeEach, describe, expect, it, vi } from "vitest";
 import type { OpenClawConfig } from "../config/config.js";
 import type { RuntimeEnv } from "../runtime.js";
 
-vi.hoisted(() => {
-  vi.resetModules();
-});
-
 const { createLineBotMock, registerPluginHttpRouteMock, unregisterHttpMock } = vi.hoisted(() => ({
   createLineBotMock: vi.fn(() => ({
     account: { accountId: "default" },
diff --git a/src/line/probe.test.ts b/src/line/probe.test.ts
index 690dd5974e5..0b9251238aa 100644
--- a/src/line/probe.test.ts
+++ b/src/line/probe.test.ts
@@ -1,9 +1,5 @@
 import { afterEach, beforeAll, describe, expect, it, vi } from "vitest";
 
-vi.hoisted(() => {
-  vi.resetModules();
-});
-
 const { getBotInfoMock, MessagingApiClientMock } = vi.hoisted(() => {
   const getBotInfoMock = vi.fn();
   const MessagingApiClientMock = vi.fn(function () {
diff --git a/src/line/send.test.ts b/src/line/send.test.ts
index 76dbc6da8f1..01695925932 100644
--- a/src/line/send.test.ts
+++ b/src/line/send.test.ts
@@ -1,9 +1,5 @@
 import { afterEach, beforeAll, beforeEach, describe, expect, it, vi } from "vitest";
 
-vi.hoisted(() => {
-  vi.resetModules();
-});
-
 const {
   pushMessageMock,
   replyMessageMock,