@agi-cli/sdk 0.1.110 → 0.1.112

package/src/core/src/tools/builtin/grep.ts

@@ -5,6 +5,7 @@ import { promisify } from 'node:util';
 import { join } from 'node:path';
 import DESCRIPTION from './grep.txt' with { type: 'text' };
 import { defaultIgnoreGlobs } from './ignore.ts';
+import { createToolError, type ToolResponse } from '../error.ts';
 
 const execAsync = promisify(exec);
 
@@ -39,9 +40,24 @@ export function buildGrepTool(projectRoot: string): {
 				.optional()
 				.describe('Glob patterns to exclude from search'),
 		}),
-		async execute(params) {
+		async execute(params: {
+			pattern: string;
+			path?: string;
+			include?: string;
+			ignore?: string[];
+		}): Promise<
+			ToolResponse<{
+				count: number;
+				matches: Array<{ file: string; line: number; text: string }>;
+			}>
+		> {
 			const pattern = String(params.pattern || '');
-			if (!pattern) throw new Error('pattern is required');
+			if (!pattern) {
+				return createToolError('pattern is required', 'validation', {
+					parameter: 'pattern',
+					suggestion: 'Provide a regex pattern to search for',
+				});
+			}
 
 			const p = expandTilde(String(params.path || '')).trim();
 			const isAbs = p.startsWith('/') || /^[A-Za-z]:[\\/]/.test(p);
@@ -63,10 +79,19 @@ export function buildGrepTool(projectRoot: string): {
 			} catch (error: unknown) {
 				const err = error as { code?: number; stderr?: string };
 				if (err.code === 1) {
-					return { count: 0, matches: [] };
+					return { ok: true, count: 0, matches: [] };
 				}
 				const err2 = error as { stderr?: string; message?: string };
-				throw new Error(`ripgrep failed: ${err2.stderr || err2.message}`);
+				return createToolError(
+					`ripgrep failed: ${err2.stderr || err2.message}`,
+					'execution',
+					{
+						parameter: 'pattern',
+						value: pattern,
+						suggestion:
+							'Check if ripgrep (rg) is installed and the pattern is valid',
+					},
+				);
 			}
 
 			const lines = output.trim().split('\n');
@@ -94,6 +119,7 @@ export function buildGrepTool(projectRoot: string): {
 			const finalMatches = truncated ? matches.slice(0, limit) : matches;
 
 			return {
+				ok: true,
 				count: finalMatches.length,
 				matches: finalMatches,
 			};

package/src/core/src/tools/builtin/ripgrep.ts

@@ -3,6 +3,7 @@ import { z } from 'zod';
 import { spawn } from 'node:child_process';
 import { join } from 'node:path';
 import DESCRIPTION from './ripgrep.txt' with { type: 'text' };
+import { createToolError, type ToolResponse } from '../error.ts';
 
 export function buildRipgrepTool(projectRoot: string): {
 	name: string;
@@ -36,7 +37,12 @@ export function buildRipgrepTool(projectRoot: string): {
 			ignoreCase?: boolean;
 			glob?: string[];
 			maxResults?: number;
-		}) {
+		}): Promise<
+			ToolResponse<{
+				count: number;
+				matches: Array<{ file: string; line: number; text: string }>;
+			}>
+		> {
 			function expandTilde(p: string) {
 				const home = process.env.HOME || process.env.USERPROFILE || '';
 				if (!home) return p;
@@ -70,7 +76,16 @@ export function buildRipgrepTool(projectRoot: string): {
 
 					proc.on('close', (code) => {
 						if (code !== 0 && code !== 1) {
-							resolve({ count: 0, matches: [], error: stderr.trim() });
+							resolve(
+								createToolError(
+									stderr.trim() || 'ripgrep failed',
+									'execution',
+									{
+										suggestion:
+											'Check if ripgrep (rg) is installed and the query is valid',
+									},
+								),
+							);
 							return;
 						}
 
@@ -86,15 +101,21 @@ export function buildRipgrepTool(projectRoot: string): {
 							const text = parts.slice(2).join(':');
 							return { file, line, text };
 						});
-						resolve({ count: matches.length, matches });
+						resolve({ ok: true, count: matches.length, matches });
 					});
 
 					proc.on('error', (err) => {
-						resolve({ count: 0, matches: [], error: String(err) });
+						resolve(
+							createToolError(String(err), 'execution', {
+								suggestion: 'Ensure ripgrep (rg) is installed',
+							}),
+						);
 					});
 				});
 			} catch (err) {
-				return { count: 0, matches: [], error: String(err) };
+				return createToolError(String(err), 'execution', {
+					suggestion: 'Ensure ripgrep (rg) is installed',
+				});
 			}
 		},
 	});

package/src/core/src/tools/builtin/websearch.ts

@@ -1,6 +1,7 @@
 import { tool, type Tool } from 'ai';
 import { z } from 'zod';
 import DESCRIPTION from './websearch.txt' with { type: 'text' };
+import { createToolError, type ToolResponse } from '../error.ts';
 
 export function buildWebSearchTool(): {
 	name: string;
@@ -42,7 +43,22 @@ export function buildWebSearchTool(): {
 			url?: string;
 			query?: string;
 			maxLength?: number;
-		}) {
+		}): Promise<
+			ToolResponse<
+				| {
+						url: string;
+						content: string;
+						contentLength: number;
+						truncated: boolean;
+						contentType: string;
+				  }
+				| {
+						query: string;
+						results: Array<{ title: string; url: string; snippet: string }>;
+						count: number;
+				  }
+			>
+		> {
 			const maxLen = maxLength ?? 50000;
 
 			if (url) {
@@ -76,9 +92,11 @@ export function buildWebSearchTool(): {
 					) {
 						content = await response.text();
 					} else {
-						return {
-							error: `Unsupported content type: ${contentType}. Only text-based content can be fetched.`,
-						};
+						return createToolError(
+							`Unsupported content type: ${contentType}. Only text-based content can be fetched.`,
+							'unsupported',
+							{ contentType },
+						);
 					}
 
 					// Strip HTML tags for better readability (basic cleaning)
@@ -93,6 +111,7 @@ export function buildWebSearchTool(): {
 					const wasTruncated = cleanContent.length > maxLen;
 
 					return {
+						ok: true,
 						url,
 						content: truncated,
 						contentLength: cleanContent.length,
@@ -102,9 +121,11 @@ export function buildWebSearchTool(): {
 				} catch (error) {
 					const errorMessage =
 						error instanceof Error ? error.message : String(error);
-					return {
-						error: `Failed to fetch URL: ${errorMessage}`,
-					};
+					return createToolError(
+						`Failed to fetch URL: ${errorMessage}`,
+						'execution',
+						{ url },
+					);
 				}
 			}
 
@@ -183,16 +204,19 @@ export function buildWebSearchTool(): {
 					}
 
 					if (results.length === 0) {
-						return {
-							error:
-								'No search results found. The search service may have changed its format or blocked the request.',
-							query,
-							suggestion:
-								'Try using the url parameter to fetch a specific webpage instead.',
-						};
+						return createToolError(
+							'No search results found. The search service may have changed its format or blocked the request.',
+							'execution',
+							{
+								query,
+								suggestion:
+									'Try using the url parameter to fetch a specific webpage instead.',
+							},
+						);
 					}
 
 					return {
+						ok: true,
 						query,
 						results,
 						count: results.length,
@@ -200,18 +224,25 @@ export function buildWebSearchTool(): {
 				} catch (error) {
 					const errorMessage =
 						error instanceof Error ? error.message : String(error);
-					return {
-						error: `Search failed: ${errorMessage}`,
-						query,
-						suggestion:
-							'Search services may be temporarily unavailable. Try using the url parameter to fetch a specific webpage instead.',
-					};
+					return createToolError(
+						`Search failed: ${errorMessage}`,
+						'execution',
+						{
+							query,
+							suggestion:
+								'Search services may be temporarily unavailable. Try using the url parameter to fetch a specific webpage instead.',
+						},
+					);
 				}
 			}
 
-			return {
-				error: 'Must provide either url or query parameter',
-			};
+			return createToolError(
+				'Must provide either url or query parameter',
+				'validation',
+				{
+					suggestion: 'Provide either a url to fetch or a query to search',
+				},
+			);
 		},
 	});
 

package/src/index.ts

@@ -40,6 +40,7 @@ export {
 	providerIds,
 	defaultModelFor,
 	hasModel,
+	getFastModel,
 } from './providers/src/index.ts';
 export {
 	isProviderAuthorized,
@@ -60,6 +61,11 @@ export type {
 	SolforgeAuth,
 	SolforgeProviderOptions,
 } from './providers/src/index.ts';
+export {
+	createOpenAIOAuthFetch,
+	createOpenAIOAuthModel,
+} from './providers/src/index.ts';
+export type { OpenAIOAuthConfig } from './providers/src/index.ts';
 
 // =======================
 // Authentication (from internal auth module)
@@ -75,6 +81,14 @@ export {
 	openAuthUrl,
 	createApiKey,
 } from './auth/src/index.ts';
+export {
+	authorizeOpenAI,
+	exchangeOpenAI,
+	refreshOpenAIToken,
+	openOpenAIAuthUrl,
+	obtainOpenAIApiKey,
+} from './auth/src/index.ts';
+export type { OpenAIOAuthResult } from './auth/src/index.ts';
 
 // =======================
 // Configuration (from internal config module)