From 147aa4befd5fd775c139109da708ce66d6578685 Mon Sep 17 00:00:00 2001
From: Han Xiao <han.xiao@jina.ai>
Date: Tue, 10 Jun 2025 19:48:27 -0700
Subject: [PATCH] refactor: update logging imports and enhance error messages

---
 src/tools/build-ref.ts    | 2 +-
 src/tools/md-fixer.ts     | 7 +++++--
 src/utils/axios-client.ts | 1 -
 src/utils/url-tools.ts    | 2 +-
 4 files changed, 7 insertions(+), 5 deletions(-)

diff --git a/src/tools/build-ref.ts b/src/tools/build-ref.ts
index ac2a511..9392043 100644
--- a/src/tools/build-ref.ts
+++ b/src/tools/build-ref.ts
@@ -5,7 +5,7 @@ import { cosineSimilarity, jaccardRank } from "./cosine";
 import { getEmbeddings } from "./embeddings";
 import { dedupImagesWithEmbeddings } from '../utils/image-tools';
 import { normalizeHostName } from '../utils/url-tools';
-import { logInfo, logError, logDebug, logWarning } from '../logging';
+import { logError, logDebug } from '../logging';
 
 export async function buildReferences(
   answer: string,
diff --git a/src/tools/md-fixer.ts b/src/tools/md-fixer.ts
index 3daf0ef..50df7c3 100644
--- a/src/tools/md-fixer.ts
+++ b/src/tools/md-fixer.ts
@@ -3,7 +3,7 @@ import { getKnowledgeStr } from "../utils/text-tools";
 import { getModel } from "../config";
 import { generateText } from "ai";
 import { Schemas } from "../utils/schemas";
-import { logInfo, logError, logDebug } from '../logging';
+import { logInfo, logError, logDebug, logWarning } from '../logging';
 
 
 function getPrompt(mdContent: string, allKnowledge: KnowledgeItem[], schema: Schemas): PromptPair {
@@ -86,7 +86,10 @@ export async function reviseAnswer(
     logDebug(`repaired before/after: ${mdContent.length} -> ${result.text.length}`);
 
     if (result.text.length < mdContent.length * 0.85) {
-      logError(`repaired content length ${result.text.length} is significantly shorter than original content ${mdContent.length}, return original content instead.`);
+      logWarning(`repaired content length ${result.text.length} is significantly shorter than original content ${mdContent.length}, return original content instead.`, {
+        originalContent: mdContent,
+        repairedContent: result.text
+      });
       return mdContent;
     }
 
diff --git a/src/utils/axios-client.ts b/src/utils/axios-client.ts
index 6124da9..cdbd74e 100644
--- a/src/utils/axios-client.ts
+++ b/src/utils/axios-client.ts
@@ -56,7 +56,6 @@ axiosClient.interceptors.response.use(
   (response) => response,
   (error) => {
     if (error.code === 'ECONNABORTED') {
-      logError('Request timed out:', { error: error.message });
       error.request?.destroy?.();
     }
     if (axios.isAxiosError(error)) {
diff --git a/src/utils/url-tools.ts b/src/utils/url-tools.ts
index 824817e..f736cdc 100644
--- a/src/utils/url-tools.ts
+++ b/src/utils/url-tools.ts
@@ -430,7 +430,7 @@ export async function getLastModified(url: string): Promise<string | undefined>
 
     return undefined;
   } catch (error) {
-    logError('Failed to fetch last modified date:');
+    logError('Failed to fetch last modified date', { error });
     return undefined;
   }
 }