From e59db45f5b824200cd2ec75e9443d363a0c26636 Mon Sep 17 00:00:00 2001
From: tombii <github@tombii.com>
Date: Wed, 1 Apr 2026 23:18:30 +0200
Subject: [PATCH] refactor(model-sync): move empty-list guard to early return

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
---
 src/lib/db/models.ts | 6 ------
 1 file changed, 6 deletions(-)

diff --git a/src/lib/db/models.ts b/src/lib/db/models.ts
index 6f062af580..74234a7f12 100644
--- a/src/lib/db/models.ts
+++ b/src/lib/db/models.ts
@@ -429,12 +429,6 @@ export async function replaceCustomModels(
   });
 
   if (merged.length === 0) {
-    // Guard: skip destructive clear when the caller hasn't explicitly opted in.
-    // This prevents auto-sync from wiping manually-imported models when the
-    // upstream /models endpoint fails, times out, or returns an empty list.
-    if (!allowEmpty) {
-      return Array.isArray(existing) ? existing : [];
-    }
     db.prepare("DELETE FROM key_value WHERE namespace = 'customModels' AND key = ?").run(
       providerId
     );