From 30eac026c2daf470d219739ae0f9aa79b96ea76d Mon Sep 17 00:00:00 2001
From: Ralph Khreish <35776126+Crunchyman-ralph@users.noreply.github.com>
Date: Fri, 20 Jun 2025 16:03:48 +0300
Subject: [PATCH] chore: cleanup and format and small refactor

---
 scripts/modules/config-manager.js     | 8 ++++++--
 scripts/modules/supported-models.json | 4 ++--
 src/ai-providers/bedrock.js           | 2 +-
 3 files changed, 9 insertions(+), 5 deletions(-)

diff --git a/scripts/modules/config-manager.js b/scripts/modules/config-manager.js
index 7c08cf4a..6ce73acf 100644
--- a/scripts/modules/config-manager.js
+++ b/scripts/modules/config-manager.js
@@ -497,9 +497,13 @@ function getParametersForRole(role, explicitRoot = null) {
 function isApiKeySet(providerName, session = null, projectRoot = null) {
 	// Define the expected environment variable name for each provider
 
-	const okKeys = ['ollama', 'bedrock'];
+	// Providers that don't require API keys for authentication
+	const providersWithoutApiKeys = [
+		CUSTOM_PROVIDERS.OLLAMA,
+		CUSTOM_PROVIDERS.BEDROCK
+	];
 
-	if (okKeys.includes(providerName?.toLowerCase())) {
+	if (providersWithoutApiKeys.includes(providerName?.toLowerCase())) {
 		return true; // Indicate key status is effectively "OK"
 	}
 
diff --git a/scripts/modules/supported-models.json b/scripts/modules/supported-models.json
index 98bb3758..ef4d78e8 100644
--- a/scripts/modules/supported-models.json
+++ b/scripts/modules/supported-models.json
@@ -4,14 +4,14 @@
 			"id": "us.anthropic.claude-3-7-sonnet-20250219-v1:0",
 			"swe_score": 0.623,
 			"cost_per_1m_tokens": { "input": 3, "output": 15 },
-			"allowed_roles": [ "main", "fallback" ],
+			"allowed_roles": ["main", "fallback"],
 			"max_tokens": 65536
 		},
 		{
 			"id": "us.deepseek.r1-v1:0",
 			"swe_score": 0,
 			"cost_per_1m_tokens": { "input": 1.35, "output": 5.4 },
-			"allowed_roles": [ "research" ],
+			"allowed_roles": ["research"],
 			"max_tokens": 65536
 		}
 	],
diff --git a/src/ai-providers/bedrock.js b/src/ai-providers/bedrock.js
index 24e3e9ea..74912518 100644
--- a/src/ai-providers/bedrock.js
+++ b/src/ai-providers/bedrock.js
@@ -24,7 +24,7 @@ export class BedrockAIProvider extends BaseAIProvider {
 			const credentialProvider = fromNodeProviderChain();
 
 			return createAmazonBedrock({
-				credentialProvider,
+				credentialProvider
 			});
 		} catch (error) {
 			this.handleError('client initialization', error);