Refactor logger error messages for consistency. (#1257)

src/lib/assistantStats/refresh-assistants-counts.ts

@@ -59,8 +59,7 @@ async function refreshAssistantsCountsHelper() {
 				})
 			);
 	} catch (e) {
-		logger.error("Refresh assistants counter failed!");
-		logger.error(e);
+		logger.error(e, "Refresh assistants counter failed!");
 	}
 }
 

src/lib/server/endpoints/cloudflare/endpointCloudflare.ts

@@ -113,7 +113,7 @@ export async function endpointCloudflare(
 							data = JSON.parse(jsonString);
 						} catch (e) {
 							logger.error(e, "Failed to parse JSON");
-							logger.error("Problematic JSON string:", jsonString);
+							logger.error(jsonString, "Problematic JSON string:");
 							continue; // Skip this iteration and try the next chunk
 						}
 

src/lib/server/endpoints/langserve/endpointLangserve.ts

@@ -101,7 +101,7 @@ export function endpointLangserve(
 							data = JSON.parse(jsonString);
 						} catch (e) {
 							logger.error(e, "Failed to parse JSON");
-							logger.error("Problematic JSON string:", jsonString);
+							logger.error(jsonString, "Problematic JSON string:");
 							continue; // Skip this iteration and try the next chunk
 						}
 						// Assuming content within data is a plain string

src/lib/server/endpoints/llamacpp/endpointLlamacpp.ts

@@ -95,7 +95,7 @@ export function endpointLlamacpp(
 							data = JSON.parse(jsonString);
 						} catch (e) {
 							logger.error(e, "Failed to parse JSON");
-							logger.error("Problematic JSON string:", jsonString);
+							logger.error(jsonString, "Problematic JSON string:");
 							continue; // Skip this iteration and try the next chunk
 						}
 

src/lib/server/models.ts

@@ -88,10 +88,8 @@ async function getChatPromptRender(
 		tokenizer = await getTokenizer(m.tokenizer);
 	} catch (e) {
 		logger.error(
-			"Failed to load tokenizer for model " +
-				m.name +
-				" consider setting chatPromptTemplate manually or making sure the model is available on the hub. Error: " +
-				(e as Error).message
+			e,
+			`Failed to load tokenizer for model ${m.name} consider setting chatPromptTemplate manually or making sure the model is available on the hub.`
 		);
 		process.exit();
 	}

src/lib/server/websearch/scrape/playwright.ts

@@ -17,7 +17,7 @@ const blocker = await PlaywrightBlocker.fromPrebuiltAdsAndTracking(fetch)
 		return mostBlocked;
 	})
 	.catch((err) => {
-		logger.error("Failed to initialize PlaywrightBlocker from prebuilt lists", err);
+		logger.error(err, "Failed to initialize PlaywrightBlocker from prebuilt lists");
 		return PlaywrightBlocker.empty();
 	});
 

src/lib/server/websearch/scrape/scrape.ts

@@ -28,7 +28,7 @@ export const scrape = (maxCharsPerElem: number) =>
 			return { ...source, page };
 		} catch (e) {
 			MetricsServer.getMetrics().webSearch.pageFetchCountError.inc();
-			logger.debug(`Error scraping webpage: ${source.link}`, { error: e });
+			logger.error(e, `Error scraping webpage: ${source.link}`);
 		}
 	};