fix(api): correct safetySettings payload structure (#61)

* fix(api): correct safetySettings payload structure

The Gemini API expects the 'safetySettings' object to be a top-level property within the main request body, not nested inside 'generationConfig'.

This commit moves the 'safetySettings' object to the correct location in the stream request payload to resolve the 400 Bad Request error.

* refactor(api): optimize safetySettings creation

* fix: added default KV namespace ID

Author: jd-pr

src/gemini-client.ts

@@ -349,8 +349,7 @@ export class GeminiApiClient {
 			modelId,
 			req,
 			isRealThinkingEnabled,
-			includeReasoning,
-			this.env
+			includeReasoning
 		);
 
 		const { tools, toolConfig } = GenerationConfigValidator.createValidateTools(req);
@@ -362,7 +361,17 @@ export class GeminiApiClient {
 			needsThinkingClose = streamThinkingAsContent; // Only need to close if we streamed as content
 		}
 
-		const streamRequest = {
+		const streamRequest: {
+			model: string;
+			project: string;
+			request: {
+				contents: unknown;
+				generationConfig: unknown;
+				tools: unknown;
+				toolConfig: unknown;
+				safetySettings?: unknown;
+			};
+		} = {
 			model: modelId,
 			project: projectId,
 			request: {
@@ -373,6 +382,11 @@ export class GeminiApiClient {
 			}
 		};
 
+		const safetySettings = GenerationConfigValidator.createSafetySettings(this.env);
+		if (safetySettings.length > 0) {
+			streamRequest.request.safetySettings = safetySettings;
+		}
+
 		yield* this.performStreamRequest(
 			streamRequest,
 			needsThinkingClose,

src/helpers/generation-config-validator.ts

@@ -126,8 +126,7 @@ export class GenerationConfigValidator {
 		modelId: string,
 		options: Partial<ChatCompletionRequest> = {},
 		isRealThinkingEnabled: boolean,
-		includeReasoning: boolean,
-		env?: Env
+		includeReasoning: boolean
 	): Record<string, unknown> {
 		const generationConfig: Record<string, unknown> = {
 			temperature: options.temperature ?? DEFAULT_TEMPERATURE,
@@ -143,14 +142,6 @@ export class GenerationConfigValidator {
 			generationConfig.responseMimeType = "application/json";
 		}
 
-		// Add safety settings if environment variables are provided
-		if (env) {
-			const safetySettings = this.createSafetySettings(env);
-			if (safetySettings.length > 0) {
-				generationConfig.safetySettings = safetySettings;
-			}
-		}
-
 		const modelInfo = geminiCliModels[modelId];
 		const isThinkingModel = modelInfo?.thinking || false;