Refactor Bedrock API Configuration Check and Improve ChatActions Logging

- Removed redundant Bedrock configuration check in the API handler for cleaner code.
- Enhanced logging in `ChatActions` to provide clearer feedback when an unknown provider ID is encountered, defaulting to OpenAI when necessary.
This commit is contained in:
AC 2025-04-06 21:50:33 +08:00
parent 9a865fd76f
commit 3aae552167
2 changed files with 2 additions and 8 deletions

View File

@ -59,13 +59,6 @@ export async function handle(
try { try {
const config = getServerSideConfig(); const config = getServerSideConfig();
if (!config.isBedrock) {
// This check might be redundant due to getAwsCredentials, but good practice
return NextResponse.json(
{ error: true, msg: "AWS Bedrock is not configured properly" },
{ status: 500 },
);
}
const bedrockRegion = config.bedrockRegion as string; const bedrockRegion = config.bedrockRegion as string;
const bedrockEndpoint = config.bedrockEndpoint; const bedrockEndpoint = config.bedrockEndpoint;

View File

@ -720,9 +720,10 @@ export function ChatActions(props: {
targetProvider = upperProvider as ServiceProvider; targetProvider = upperProvider as ServiceProvider;
} else { } else {
console.error( console.error(
`[ChatActions] Unknown provider ID: ${providerId}`, `[ChatActions] Unknown provider ID: ${providerId}. Falling back to OpenAI.`,
); );
// Handle error or fallback if needed // Handle error or fallback if needed
targetProvider = ServiceProvider.OpenAI; // Fallback for unrecognized provider
} }
} else { } else {
// Handle case where providerId is missing, maybe default to OpenAI? // Handle case where providerId is missing, maybe default to OpenAI?