fix: address greptile review feedback on PR #3708

Use BedrockRuntimeClientConfig from SDK instead of inline type.
Add default return value for prepareToolsWithUsageControl mock.

Signed-off-by: majiayu000 <1835304752@qq.com>

Author: majiayu000

apps/sim/providers/bedrock/index.test.ts

@@ -30,7 +30,11 @@ vi.mock('@/providers/models', () => ({
 vi.mock('@/providers/utils', () => ({
   calculateCost: vi.fn().mockReturnValue({ input: 0, output: 0, total: 0, pricing: null }),
   prepareToolExecution: vi.fn(),
-  prepareToolsWithUsageControl: vi.fn(),
+  prepareToolsWithUsageControl: vi.fn().mockReturnValue({
+    tools: [],
+    toolChoice: 'auto',
+    forcedTools: [],
+  }),
   sumToolCosts: vi.fn().mockReturnValue(0),
 }))
 

apps/sim/providers/bedrock/index.ts

@@ -1,6 +1,7 @@
 import {
   type Message as BedrockMessage,
   BedrockRuntimeClient,
+  type BedrockRuntimeClientConfig,
   type ContentBlock,
   type ConversationRole,
   ConverseCommand,
@@ -68,10 +69,7 @@ export const bedrockProvider: ProviderConfig = {
       )
     }
 
-    const clientConfig: {
-      region: string
-      credentials?: { accessKeyId: string; secretAccessKey: string }
-    } = { region }
+    const clientConfig: BedrockRuntimeClientConfig = { region }
     if (request.bedrockAccessKeyId && request.bedrockSecretKey) {
       clientConfig.credentials = {
         accessKeyId: request.bedrockAccessKeyId,