refactor: enhance type definitions and error handling in Snowflake provider

- Updated the return type of `executeCortexCli` to include `exitCode` as nullable and `signal`.
- Improved the `grepTool` to accurately reflect truncation status based on match limits.
- Refactored the `normalizeModelId` function to return an empty string for invalid inputs.
- Enhanced the handling of deeply nested Zod objects in the conversion process.
- Removed deprecated integration test file and updated related documentation.

Author: sfc-gh-dflippo

packages/ai-sdk-provider-snowflake/src/cli/language-model.ts

@@ -131,7 +131,7 @@ export class CliLanguageModel implements LanguageModelV2 {
 	private async executeCortexCli(
 		args: string[],
 		options: { timeout?: number } = {}
-	): Promise<{ stdout: string; stderr: string; exitCode: number }> {
+	): Promise<{ stdout: string; stderr: string; exitCode: number | null; signal: NodeJS.Signals | null }> {
 		const timeout = options.timeout ?? this.settings.timeout ?? 60000;
 
 		return new Promise((resolve, reject) => {
@@ -176,7 +176,7 @@ export class CliLanguageModel implements LanguageModelV2 {
 			});
 
 			// Handle process completion
-			child.on('close', (code) => {
+			child.on('close', (code, signal) => {
 				if (timeoutId) {
 					clearTimeout(timeoutId);
 				}
@@ -189,7 +189,8 @@ export class CliLanguageModel implements LanguageModelV2 {
 				resolve({
 					stdout,
 					stderr,
-					exitCode: code ?? 0
+					exitCode: code,
+					signal
 				});
 			});
 

packages/ai-sdk-provider-snowflake/src/tools/file-operations.ts

@@ -326,15 +326,22 @@ export const grepTool: ToolDefinition<GrepInput, GrepResult> = {
 
 		const files = await findFiles(targetDir, filePattern);
 		const allMatches: GrepMatch[] = [];
+		let hitLimit = false;
 
 		for (const filePath of files) {
-			if (allMatches.length >= maxMatches) break;
+			if (allMatches.length >= maxMatches) {
+				hitLimit = true;
+				break;
+			}
 
 			const matches = await grepFile(filePath, regex, contextLines);
 			for (const match of matches) {
 				match.file = path.relative(targetDir, filePath);
 				allMatches.push(match);
-				if (allMatches.length >= maxMatches) break;
+				if (allMatches.length >= maxMatches) {
+					hitLimit = true;
+					break;
+				}
 			}
 		}
 
@@ -343,7 +350,7 @@ export const grepTool: ToolDefinition<GrepInput, GrepResult> = {
 			directory: path.relative(projectRoot, targetDir) || '.',
 			matches: allMatches.slice(0, maxMatches),
 			totalMatches: allMatches.length,
-			truncated: allMatches.length > maxMatches
+			truncated: hitLimit
 		};
 	}
 };

packages/ai-sdk-provider-snowflake/src/utils/models.ts

@@ -296,7 +296,7 @@ export const STREAMING_PREFIXED_MODEL_IDS = STREAMING_MODEL_IDS.map(
  */
 export function normalizeModelId(modelId: SnowflakeModelId): string {
 	if (!modelId || typeof modelId !== 'string') {
-		return modelId;
+		return '';
 	}
 	return modelId.replace(/^cortex\//, '').toLowerCase();
 }

packages/ai-sdk-provider-snowflake/src/utils/tool-helpers.ts

@@ -118,8 +118,39 @@ function convertZodType(def: Record<string, unknown>): Record<string, unknown> {
 			};
 		}
 
-		case 'ZodObject':
-			return zodToJsonSchema(def as Record<string, unknown>);
+		case 'ZodObject': {
+			// def is already _def, so handle shape directly
+			if (def.shape) {
+				const shape = def.shape as
+					| (() => Record<string, unknown>)
+					| Record<string, unknown>;
+				const properties: Record<string, unknown> = {};
+				const required: string[] = [];
+				const shapeObj = typeof shape === 'function' ? shape() : shape;
+
+				for (const [key, value] of Object.entries(shapeObj)) {
+					const propDef =
+						(value as { _def?: Record<string, unknown> })?._def || {};
+					properties[key] = convertZodType(propDef);
+
+					// Check if required (not optional)
+					if (
+						propDef.typeName !== 'ZodOptional' &&
+						propDef.typeName !== 'ZodDefault'
+					) {
+						required.push(key);
+					}
+				}
+
+				return {
+					type: 'object',
+					properties,
+					description: def.description as string | undefined,
+					required: required.length > 0 ? required : undefined
+				};
+			}
+			return { type: 'object', properties: {} };
+		}
 
 		default:
 			return { type: 'string' };