fix: add optional chaining on providerConfig.value

alenkacz · claude · alenkacz · commit e1bc754be5a4 · 2026-04-24T16:34:31.000+02:00
Addresses review comment — use ?. for type safety.

Co-Authored-By: Claude Opus 4.6 (1M context) &lt;noreply@anthropic.com&gt;
diff --git a/frontend/src/components/pages/agents/details/ai-agent-configuration-tab.tsx b/frontend/src/components/pages/agents/details/ai-agent-configuration-tab.tsx
@@ -1261,7 +1261,7 @@ export const AIAgentConfigurationTab = () => {
                           if (formTypeId === 'bedrock' && providersData?.llmProviders) {
                             const fullProvider = providersData.llmProviders.find((p) => p.name === value);
                             if (fullProvider?.providerConfig?.case === 'bedrockConfig') {
-                              baseUrl = fullProvider.providerConfig.value.region || '';
+                              baseUrl = fullProvider.providerConfig.value?.region || '';
                             }
                           }
 

Original file line number	Diff line number	Diff line change
`@@ -1261,7 +1261,7 @@ export const AIAgentConfigurationTab = () => {`
`1261`	`1261`	`if (formTypeId === 'bedrock' && providersData?.llmProviders) {`
`1262`	`1262`	`const fullProvider = providersData.llmProviders.find((p) => p.name === value);`
`1263`	`1263`	`if (fullProvider?.providerConfig?.case === 'bedrockConfig') {`
`1264`		`- baseUrl = fullProvider.providerConfig.value.region \|\| '';`
	`1264`	`+ baseUrl = fullProvider.providerConfig.value?.region \|\| '';`
`1265`	`1265`	`}`
`1266`	`1266`	`}`
`1267`	`1267`