refactor: use async getApiKey in model services

- Updated getApiKey calls to async
- Improved error logging in git blame

Updated Files 2025-05-22@08:25PM

Author: AhmedOsman101

services/codestralService.ts

@@ -18,7 +18,7 @@ class CodestralService extends ModelService {
     attempt = 1
   ): Promise<CommitMessage> {
     try {
-      const apiKey: string = ConfigService.getApiKey("Codestral");
+      const apiKey: string = await ConfigService.getApiKey("Codestral");
 
       const { ok: model, error: modelError } = await ConfigService.get(
         "codestral",

services/configService.ts

@@ -129,7 +129,7 @@ const ConfigService = {
       message: `Enter your ${service} API Key:`,
       label: "API Key",
       prefix: "",
-      minLength: 32
+      minLength: 32,
     });
 
     const { error: validationErr } = KeyValidationService.baseValidation(key);

services/geminiService.ts

@@ -1,7 +1,6 @@
 import axios, { type AxiosError } from "axios";
 import { errorMessages } from "../lib/constants.ts";
 import type { CommitMessage } from "../lib/index.d.ts";
-import { logError } from "../lib/Logger.ts";
 import { ConfigurationError } from "../models/errors.ts";
 import ConfigService from "./configService.ts";
 import { ModelService } from "./modelService.ts";
@@ -22,7 +21,7 @@ class GeminiService extends ModelService {
     attempt = 1
   ): Promise<CommitMessage> {
     try {
-      const apiKey: string = ConfigService.getApiKey("Gemini");
+      const apiKey: string = await ConfigService.getApiKey("Gemini");
 
       const { ok: model, error: modelError } = await ConfigService.get(
         "gemini",
@@ -58,7 +57,6 @@ class GeminiService extends ModelService {
       const message = GeminiService.extractCommitMessage(response.data);
       return { message, model };
     } catch (error) {
-      logError(error);
       const axiosError = error as AxiosError;
       if (axiosError.response) {
         const { status } = axiosError.response;

services/gitBlameAnalyzer.ts

@@ -32,7 +32,7 @@ const GitBlameAnalyzer = {
       const blameOutput = this.executeGitBlame(filePath);
       return this.parseBlameOutput(blameOutput);
     } catch (error) {
-      void logError("Error getting blame info:", error as Error);
+      void logError("Error getting blame info:", (error as Error).message);
       throw error;
     }
   },
@@ -118,7 +118,7 @@ const GitBlameAnalyzer = {
       );
       return GitBlameAnalyzer.formatAnalysis(authorChanges);
     } catch (error) {
-      void logError("Error analyzing changes:", error as Error);
+      void logError("Error analyzing changes:", (error as Error).message);
       throw error;
     }
   },
@@ -198,7 +198,7 @@ const GitBlameAnalyzer = {
       const blameOutput = GitBlameAnalyzer.executeGitBlame(filePath);
       return GitBlameAnalyzer.parseBlameOutput(blameOutput);
     } catch (error) {
-      void logError("Error getting blame info:", error as Error);
+      void logError("Error getting blame info:", (error as Error).message);
       throw error;
     }
   },

services/gitService.ts

@@ -31,7 +31,7 @@ class GitService {
   static initialize(): string | undefined {
     const { ok: output, error } = this.getRepoPath();
 
-    if (error !== undefined) logError(error);
+    if (error !== undefined) logError(error.message);
     else {
       this.setRepoPath(output);
       return output;

services/openaiService.ts

@@ -30,7 +30,7 @@ class OpenAIService extends ModelService {
     attempt = 1
   ): Promise<CommitMessage> {
     try {
-      const apiKey: string = ConfigService.getApiKey("OpenAI");
+      const apiKey: string = await ConfigService.getApiKey("OpenAI");
 
       const { ok: model, error: modelError } = await ConfigService.get(
         "openai",