From 9c5eab471c42aa03ea2eb66cc1d32b2bf86960de Mon Sep 17 00:00:00 2001
From: Peter Steinberger <steipete@gmail.com>
Date: Thu, 16 Jul 2026 01:26:04 +0100
Subject: [PATCH] chore(doctor): format migration tests, keep cron migration
 helpers module-local

---
 src/commands/doctor/cron/payload-migration.ts           | 4 ++--
 src/commands/doctor/shared/codex-route-warnings.test.ts | 2 +-
 src/commands/doctor/shared/preview-warnings.test.ts     | 5 +++--
 3 files changed, 6 insertions(+), 5 deletions(-)

diff --git a/src/commands/doctor/cron/payload-migration.ts b/src/commands/doctor/cron/payload-migration.ts
index a120999b3b01..7b2484d04cfe 100644
--- a/src/commands/doctor/cron/payload-migration.ts
+++ b/src/commands/doctor/cron/payload-migration.ts
@@ -42,7 +42,7 @@ function hasShellToolAccess(toolsAllow: unknown): boolean {
   });
 }
 
-export type LegacyOpenAICodexCronModelRoute = {
+type LegacyOpenAICodexCronModelRoute = {
   legacyModelRef: string;
   canonicalModelRef: string;
 };
@@ -76,7 +76,7 @@ export function collectLegacyOpenAICodexCronModelRoutes(
 }
 
 /** Canonical OpenAI refs whose legacy cron shape implied the Codex runtime. */
-export function collectLegacyOpenAICodexCronModelRefs(payload: UnknownRecord): string[] {
+function collectLegacyOpenAICodexCronModelRefs(payload: UnknownRecord): string[] {
   return [
     ...new Set(
       collectLegacyOpenAICodexCronModelRoutes(payload).map((route) => route.canonicalModelRef),
diff --git a/src/commands/doctor/shared/codex-route-warnings.test.ts b/src/commands/doctor/shared/codex-route-warnings.test.ts
index 2f71d18b481d..ad2326642f7e 100644
--- a/src/commands/doctor/shared/codex-route-warnings.test.ts
+++ b/src/commands/doctor/shared/codex-route-warnings.test.ts
@@ -34,9 +34,9 @@ vi.mock("../../../plugins/installed-plugin-index.js", async (importOriginal) =>
 }));
 
 import { legacyCodexProviderIdentityKey } from "./codex-route-model-ref.js";
-import { collectBlockedLegacyOpenAICodexProviderPlan } from "./legacy-config-migrations.runtime.models.js";
 import { repairCodexSessionStoreRoutes } from "./codex-route-session-repair.test-support.js";
 import { collectCodexRouteWarnings, maybeRepairCodexRoutes } from "./codex-route-warnings.js";
+import { collectBlockedLegacyOpenAICodexProviderPlan } from "./legacy-config-migrations.runtime.models.js";
 
 describe("collectCodexRouteWarnings", () => {
   beforeEach(() => {
diff --git a/src/commands/doctor/shared/preview-warnings.test.ts b/src/commands/doctor/shared/preview-warnings.test.ts
index 451c577e9462..8ab7dfbf88aa 100644
--- a/src/commands/doctor/shared/preview-warnings.test.ts
+++ b/src/commands/doctor/shared/preview-warnings.test.ts
@@ -324,8 +324,9 @@ vi.mock("./codex-route-warnings.js", () => ({
 
 async function useRealCodexRouteWarningsOnce(): Promise<void> {
   const mocked = await import("./codex-route-warnings.js");
-  const actual =
-    await vi.importActual<typeof import("./codex-route-warnings.js")>("./codex-route-warnings.js");
+  const actual = await vi.importActual<typeof import("./codex-route-warnings.js")>(
+    "./codex-route-warnings.js",
+  );
   vi.mocked(mocked.collectCodexRouteWarnings).mockImplementationOnce(
     actual.collectCodexRouteWarnings,
   );