From 32c33f4faaac0341d4d5f1974070bdaabe69f772 Mon Sep 17 00:00:00 2001
From: Peter Steinberger <steipete@gmail.com>
Date: Sun, 22 Feb 2026 14:07:46 +0000
Subject: [PATCH] test: isolate doctor allowFrom migration assertions from
 unrelated checks

---
 ...owfrom-channels-whatsapp-allowfrom.test.ts | 50 ++++++++++++++++++-
 1 file changed, 49 insertions(+), 1 deletion(-)

diff --git a/src/commands/doctor.migrates-routing-allowfrom-channels-whatsapp-allowfrom.test.ts b/src/commands/doctor.migrates-routing-allowfrom-channels-whatsapp-allowfrom.test.ts
index 53b12ecd913..ce83587a3cf 100644
--- a/src/commands/doctor.migrates-routing-allowfrom-channels-whatsapp-allowfrom.test.ts
+++ b/src/commands/doctor.migrates-routing-allowfrom-channels-whatsapp-allowfrom.test.ts
@@ -1,4 +1,4 @@
-import { describe, expect, it } from "vitest";
+import { describe, expect, it, vi } from "vitest";
 import {
   createDoctorRuntime,
   findLegacyGatewayServices,
@@ -17,6 +17,54 @@ import {
 
 const DOCTOR_MIGRATION_TIMEOUT_MS = 20_000;
 
+vi.mock("./doctor-completion.js", () => ({
+  doctorShellCompletion: vi.fn().mockResolvedValue(undefined),
+}));
+
+vi.mock("./doctor-gateway-daemon-flow.js", () => ({
+  maybeRepairGatewayDaemon: vi.fn().mockResolvedValue(undefined),
+}));
+
+vi.mock("./doctor-gateway-health.js", () => ({
+  checkGatewayHealth: vi.fn().mockResolvedValue({ healthOk: false }),
+}));
+
+vi.mock("./doctor-memory-search.js", () => ({
+  noteMemorySearchHealth: vi.fn().mockResolvedValue(undefined),
+}));
+
+vi.mock("./doctor-platform-notes.js", () => ({
+  noteDeprecatedLegacyEnvVars: vi.fn(),
+  noteMacLaunchAgentOverrides: vi.fn().mockResolvedValue(undefined),
+  noteMacLaunchctlGatewayEnvOverrides: vi.fn().mockResolvedValue(undefined),
+}));
+
+vi.mock("./doctor-sandbox.js", () => ({
+  maybeRepairSandboxImages: vi.fn(async (cfg: unknown) => cfg),
+  noteSandboxScopeWarnings: vi.fn(),
+}));
+
+vi.mock("./doctor-security.js", () => ({
+  noteSecurityWarnings: vi.fn().mockResolvedValue(undefined),
+}));
+
+vi.mock("./doctor-session-locks.js", () => ({
+  noteSessionLockHealth: vi.fn().mockResolvedValue(undefined),
+}));
+
+vi.mock("./doctor-state-integrity.js", () => ({
+  noteStateIntegrity: vi.fn().mockResolvedValue(undefined),
+  noteWorkspaceBackupTip: vi.fn(),
+}));
+
+vi.mock("./doctor-ui.js", () => ({
+  maybeRepairUiProtocolFreshness: vi.fn().mockResolvedValue(undefined),
+}));
+
+vi.mock("./doctor-workspace-status.js", () => ({
+  noteWorkspaceStatus: vi.fn(),
+}));
+
 describe("doctor command", () => {
   it("does not add a new gateway auth token while fixing legacy issues on invalid config", async () => {
     mockDoctorConfigSnapshot({