From 03d97ba6175dd348dfe9e4d437c26b4bac755c4b Mon Sep 17 00:00:00 2001
From: "R.D." <rogerproself@gmail.com>
Date: Tue, 24 Mar 2026 21:40:13 -0400
Subject: [PATCH] fix: show readable provider name in model-sync logs

Use connection.name instead of the raw provider node ID
(e.g. "BltCy API" instead of "openai-compatible-chat-09fdb807-...")
in call logs and scheduler console output.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
---
 src/app/api/providers/[id]/sync-models/route.ts | 7 +++++--
 src/shared/services/modelSyncScheduler.ts       | 4 +++-
 2 files changed, 8 insertions(+), 3 deletions(-)

diff --git a/src/app/api/providers/[id]/sync-models/route.ts b/src/app/api/providers/[id]/sync-models/route.ts
index e5ebdfd51e..8a10038538 100644
--- a/src/app/api/providers/[id]/sync-models/route.ts
+++ b/src/app/api/providers/[id]/sync-models/route.ts
@@ -31,6 +31,9 @@ export async function POST(request: Request, { params }: { params: Promise<{ id:
       return NextResponse.json({ error: "Connection not found" }, { status: 404 });
     }
 
+    // Use a human-readable provider name for logs
+    const providerLabel = connection.name || connection.provider || "unknown";
+
     // Fetch models from the existing /api/providers/[id]/models endpoint
     const origin = new URL(request.url).origin;
     const modelsUrl = `${origin}/api/providers/${id}/models`;
@@ -52,7 +55,7 @@ export async function POST(request: Request, { params }: { params: Promise<{ id:
         path: `/api/providers/${id}/models`,
         status: modelsRes.status,
         model: "model-sync",
-        provider: connection.provider || "unknown",
+        provider: providerLabel,
         connectionId: id,
         duration,
         error: modelsData.error || `HTTP ${modelsRes.status}`,
@@ -84,7 +87,7 @@ export async function POST(request: Request, { params }: { params: Promise<{ id:
       path: `/api/providers/${id}/models`,
       status: 200,
       model: "model-sync",
-      provider: connection.provider || "unknown",
+      provider: providerLabel,
       connectionId: id,
       duration: Date.now() - start,
       requestType: "model-sync",
diff --git a/src/shared/services/modelSyncScheduler.ts b/src/shared/services/modelSyncScheduler.ts
index 1fd37359ee..45601209bf 100644
--- a/src/shared/services/modelSyncScheduler.ts
+++ b/src/shared/services/modelSyncScheduler.ts
@@ -94,7 +94,9 @@ async function runSyncCycle(apiBaseUrl: string): Promise<void> {
     console.log(`[ModelSync] Starting model sync cycle — ${connections.length} connection(s)`);
 
     const results = await Promise.allSettled(
-      connections.map((conn) => syncConnectionModels(conn.id, conn.provider, apiBaseUrl))
+      connections.map((conn) =>
+        syncConnectionModels(conn.id, conn.name || conn.provider, apiBaseUrl)
+      )
     );
 
     const succeeded = results.filter((r) => r.status === "fulfilled" && r.value === true).length;