bugfix(completion): fix bug with choosing wrong server for processing requests and small improvement log system

Author: gespispace

src/common/completion/index.ts

@@ -87,9 +87,7 @@ export const getInlineCompletionProvider = (
           parameters,
           abortController,
           loggerCompletion.uuid(),
-          triggerAuto
-            ? servers["base-small"].serverUrl
-            : servers["base-small"].serverUrl
+          servers[modelType].serverUrl
         );
 
         if (completion === null) {

src/common/download/index.ts

@@ -264,7 +264,7 @@ export const downloadModel = async (typeModel: TypeModel) => {
     throw new Error("Server file info not found");
   }
 
-  Logger.info("Got model file info", {
+  Logger.info(`Got model ${typeModel} file info`, {
     component: "download>model",
     sendTelemetry: true,
   });
@@ -275,13 +275,16 @@ export const downloadModel = async (typeModel: TypeModel) => {
     const checksum = await getChecksum(modelPath);
 
     if (checksum === modelFileInfo.checksum) {
-      Logger.info("Checksum model is correct, just return model path", {
-        component: "download>model",
-        sendTelemetry: true,
-      });
+      Logger.info(
+        `Checksum model ${typeModel} is correct, just return model path`,
+        {
+          component: "download>model",
+          sendTelemetry: true,
+        }
+      );
       return modelPath;
     }
-    Logger.info("Checksum model mismatch", {
+    Logger.info(`Checksum model ${typeModel} mismatch`, {
       component: "download>model",
       sendTelemetry: true,
     });
@@ -291,7 +294,7 @@ export const downloadModel = async (typeModel: TypeModel) => {
     await fsPromise.unlink(modelPath);
   }
 
-  Logger.info("Started download model", {
+  Logger.info(`Started download model ${typeModel}`, {
     component: "download>model",
     sendTelemetry: true,
   });
@@ -304,7 +307,7 @@ export const downloadModel = async (typeModel: TypeModel) => {
       `Downloading model ${typeModel}: ${downloaded} / ${total}`
   );
 
-  Logger.info("Finish download model", {
+  Logger.info(`Finish download model ${typeModel}`, {
     component: "download>model",
     sendTelemetry: true,
   });

src/common/server/index.ts

@@ -82,17 +82,20 @@ class Server {
         sendTelemetry: true,
       });
       FirecoderTelemetrySenderInstance.sendErrorData(error as Error, {
-        step: "Error while downloading server or model",
+        step: `Error while downloading server or model ${this.typeModel}`,
       });
       stopTask();
       throw error;
     }
 
     if (!serverPath || !modelPath) {
-      Logger.error("Server is not started. Don't have server or model path.", {
-        component: "server",
-        sendTelemetry: true,
-      });
+      Logger.error(
+        `Server ${this.typeModel} is not started. Don't have server or model path.`,
+        {
+          component: "server",
+          sendTelemetry: true,
+        }
+      );
 
       stopTask();
       return false;