toolpack-sdk 0.1.1 → 1.1.0-SNAPSHOT

package/dist/client/index.js

@@ -16,12 +16,12 @@ function newRequestId() {
     return `${Date.now()}-${REQUEST_SEQ}`;
 }
 function logRequestMessages(requestId, messages) {
-    if (!(0, provider_logger_1.isVerbose)())
+    if (!(0, provider_logger_1.shouldLog)('debug'))
         return;
-    (0, provider_logger_1.log)(`[AIClient][${requestId}] Messages (${messages.length}):`);
+    (0, provider_logger_1.logDebug)(`[AIClient][${requestId}] Messages (${messages.length}):`);
     messages.forEach((m, i) => {
         const preview = (0, provider_logger_1.safePreview)(m.content, 300);
-        (0, provider_logger_1.log)(`[AIClient][${requestId}]  #${i} role=${m.role} content=${preview}`);
+        (0, provider_logger_1.logDebug)(`[AIClient][${requestId}]  #${i} role=${m.role} content=${preview}`);
     });
 }
 function extractLastUserText(messages) {
@@ -124,7 +124,7 @@ function extractToolContext(messages) {
 }
 async function inferNeedsToolsWithAI(provider, providerName, messages, fastModel) {
     const requestId = newRequestId();
-    (0, provider_logger_1.log)(`[AIClient][${requestId}] inferNeedsToolsWithAI() provider=${providerName} model=${fastModel}`);
+    (0, provider_logger_1.logDebug)(`[AIClient][${requestId}] inferNeedsToolsWithAI() provider=${providerName} model=${fastModel}`);
     // Extract tool context from recent tool calls
     const toolContext = extractToolContext(messages);
     // Get the last user message
@@ -152,11 +152,11 @@ Answer only: YES or NO`;
         });
         const answer = (response.content || '').trim().toUpperCase();
         const needsTools = answer.startsWith('YES');
-        (0, provider_logger_1.log)(`[AIClient][${requestId}] inferNeedsToolsWithAI() context="${toolContext}" message="${userMessage.substring(0, 50)}" result=${needsTools} (raw: ${answer})`);
+        (0, provider_logger_1.logDebug)(`[AIClient][${requestId}] inferNeedsToolsWithAI() context="${toolContext}" message="${userMessage.substring(0, 50)}" result=${needsTools} (raw: ${answer})`);
         return needsTools;
     }
     catch (error) {
-        (0, provider_logger_1.log)(`[AIClient][${requestId}] inferNeedsToolsWithAI() error=${error} - falling back to false`);
+        (0, provider_logger_1.logWarn)(`[AIClient][${requestId}] inferNeedsToolsWithAI() error=${error} - falling back to false`);
         return false;
     }
 }
@@ -177,11 +177,6 @@ function inferLookupOnly(messages) {
     ];
     return lookupPatterns.some(r => r.test(text)) && !mutatingPatterns.some(r => r.test(text));
 }
-const TOOL_RESULT_MAX_CHARS = (() => {
-    const raw = process.env.TOOLPACK_SDK_TOOL_RESULT_MAX_CHARS;
-    const n = raw ? Number(raw) : NaN;
-    return Number.isFinite(n) && n > 0 ? n : 20_000;
-})();
 class AIClient extends events_1.EventEmitter {
     providers;
     defaultProvider;
@@ -194,6 +189,7 @@ class AIClient extends events_1.EventEmitter {
     activeMode = null;
     overrideSystemPrompt;
     disableBaseContext;
+    toolResultMaxChars;
     constructor(config) {
         super();
         this.providers = new Map(Object.entries(config.providers));
@@ -206,6 +202,8 @@ class AIClient extends events_1.EventEmitter {
         this.toolOrchestrator = new tool_orchestrator_js_1.ToolOrchestrator();
         this.overrideSystemPrompt = config.systemPrompt;
         this.disableBaseContext = config.disableBaseContext || false;
+        const configuredMax = this.toolsConfig.resultMaxChars ?? types_js_1.DEFAULT_TOOLS_CONFIG.resultMaxChars ?? 20_000;
+        this.toolResultMaxChars = Number.isFinite(configuredMax) && configuredMax > 0 ? configuredMax : 20_000;
         // Index tools for BM25 search if registry is provided
         if (this.toolRegistry) {
             this.bm25Engine.index(this.toolRegistry.getAll());
@@ -276,7 +274,7 @@ class AIClient extends events_1.EventEmitter {
      */
     setMode(mode) {
         this.activeMode = mode;
-        (0, provider_logger_1.log)(`[AIClient] Mode set to: ${mode ? mode.displayName : 'none (cleared)'}`);
+        (0, provider_logger_1.logInfo)(`[AIClient] Mode set to: ${mode ? mode.displayName : 'none (cleared)'}`);
     }
     /**
      * Get the currently active mode.
@@ -291,7 +289,7 @@ class AIClient extends events_1.EventEmitter {
     reindexTools() {
         if (this.toolRegistry) {
             this.bm25Engine.index(this.toolRegistry.getAll());
-            (0, provider_logger_1.log)(`[AIClient] Re-indexed ${this.bm25Engine.getIndexedCount()} tools for BM25 search`);
+            (0, provider_logger_1.logInfo)(`[AIClient] Re-indexed ${this.bm25Engine.getIndexedCount()} tools for BM25 search`);
         }
     }
     /**
@@ -326,7 +324,7 @@ class AIClient extends events_1.EventEmitter {
             if (!needsTools && intelligentDetection?.enabled && hasTools) {
                 const afterToolCall = isAfterToolCall(enrichedRequest.messages, intelligentDetection.maxFollowUpMessages);
                 if (afterToolCall) {
-                    (0, provider_logger_1.log)(`[AIClient][${requestId}] Message is after tool call, using AI to infer tool needs`);
+                    (0, provider_logger_1.logInfo)(`[AIClient][${requestId}] Message is after tool call, using AI to infer tool needs`);
                     const fastModel = getFastModelForProvider(resolvedProviderName || 'openai');
                     needsTools = await inferNeedsToolsWithAI(provider, resolvedProviderName || 'openai', enrichedRequest.messages, fastModel);
                     aiInferenceUsed = true;
@@ -340,14 +338,14 @@ class AIClient extends events_1.EventEmitter {
             }
             else if (shouldForceNone) {
                 enrichedRequest.tool_choice = 'none';
-                (0, provider_logger_1.log)(`[AIClient][${requestId}] AI inference determined no tools needed, setting tool_choice=none`);
+                (0, provider_logger_1.logInfo)(`[AIClient][${requestId}] AI inference determined no tools needed, setting tool_choice=none`);
             }
             const providerClass = provider?.constructor?.name || 'UnknownProvider';
             const outboundReq = { ...enrichedRequest, __toolpack_request_id: requestId };
-            (0, provider_logger_1.log)(`[AIClient][${requestId}] generate() start provider=${resolvedProviderName} class=${providerClass} model=${enrichedRequest.model} messages=${enrichedRequest.messages.length} tools=${enrichedRequest.tools?.length || 0} tool_choice=${enrichedRequest.tool_choice ?? 'unset'} policy=${policy} needsTools=${needsTools} autoExecute=${this.toolsConfig.enabled && this.toolsConfig.autoExecute}`);
+            (0, provider_logger_1.logInfo)(`[AIClient][${requestId}] generate() start provider=${resolvedProviderName} class=${providerClass} model=${enrichedRequest.model} messages=${enrichedRequest.messages.length} tools=${enrichedRequest.tools?.length || 0} tool_choice=${enrichedRequest.tool_choice ?? 'unset'} policy=${policy} needsTools=${needsTools} autoExecute=${this.toolsConfig.enabled && this.toolsConfig.autoExecute}`);
             logRequestMessages(requestId, enrichedRequest.messages);
             let response = await provider.generate(outboundReq);
-            (0, provider_logger_1.log)(`[AIClient][${requestId}] generate() initial response finish_reason=${response.finish_reason ?? 'unknown'} tool_calls=${response.tool_calls?.length || 0} content_preview=${(0, provider_logger_1.safePreview)(response.content || '', 200)}`);
+            (0, provider_logger_1.logDebug)(`[AIClient][${requestId}] generate() initial response finish_reason=${response.finish_reason ?? 'unknown'} tool_calls=${response.tool_calls?.length || 0} content_preview=${(0, provider_logger_1.safePreview)(response.content || '', 200)}`);
             // Auto-execute tool call loop
             if (this.toolsConfig.enabled && this.toolsConfig.autoExecute && this.toolRegistry) {
                 // Classify query to adjust maxToolRounds
@@ -356,19 +354,19 @@ class AIClient extends events_1.EventEmitter {
                 const baseMaxRounds = this.toolsConfig.maxToolRounds;
                 const maxRounds = this.queryClassifier.getToolRoundsAdjustment(classification, baseMaxRounds);
                 if (maxRounds !== baseMaxRounds) {
-                    (0, provider_logger_1.log)(`[AIClient][${requestId}] Query classified as ${classification.type} (confidence: ${classification.confidence.toFixed(2)}), adjusted maxToolRounds: ${baseMaxRounds} → ${maxRounds}`);
+                    (0, provider_logger_1.logInfo)(`[AIClient][${requestId}] Query classified as ${classification.type} (confidence: ${classification.confidence.toFixed(2)}), adjusted maxToolRounds: ${baseMaxRounds} → ${maxRounds}`);
                 }
                 else {
-                    (0, provider_logger_1.log)(`[AIClient][${requestId}] Query classified as ${classification.type} (confidence: ${classification.confidence.toFixed(2)}), keeping maxToolRounds: ${maxRounds}`);
+                    (0, provider_logger_1.logDebug)(`[AIClient][${requestId}] Query classified as ${classification.type} (confidence: ${classification.confidence.toFixed(2)}), keeping maxToolRounds: ${maxRounds}`);
                 }
                 let rounds = 0;
                 const messages = [...enrichedRequest.messages];
                 if (response.tool_calls && response.tool_calls.length > 0) {
-                    (0, provider_logger_1.log)(`[AIClient] Received ${response.tool_calls.length} tool call(s): ${response.tool_calls.map(tc => tc.name).join(', ')}`);
+                    (0, provider_logger_1.logInfo)(`[AIClient] Received ${response.tool_calls.length} tool call(s): ${response.tool_calls.map(tc => tc.name).join(', ')}`);
                 }
                 while (response.tool_calls && response.tool_calls.length > 0 && rounds < maxRounds) {
                     rounds++;
-                    (0, provider_logger_1.log)(`[AIClient][${requestId}] generate() tool round ${rounds}/${maxRounds} tool_calls=${response.tool_calls.length}`);
+                    (0, provider_logger_1.logInfo)(`[AIClient][${requestId}] generate() tool round ${rounds}/${maxRounds} tool_calls=${response.tool_calls.length}`);
                     // Add assistant message with tool calls to conversation
                     messages.push({
                         role: 'assistant',
@@ -389,7 +387,7 @@ class AIClient extends events_1.EventEmitter {
                     let toolCallsToExecute = response.tool_calls;
                     const webFetchCalls = response.tool_calls.filter(tc => tc.name === 'web.fetch');
                     if (webFetchCalls.length > MAX_WEB_FETCH_CALLS) {
-                        (0, provider_logger_1.log)(`[AIClient][${requestId}] Limiting web.fetch calls from ${webFetchCalls.length} → ${MAX_WEB_FETCH_CALLS} to prevent context overflow`);
+                        (0, provider_logger_1.logInfo)(`[AIClient][${requestId}] Limiting web.fetch calls from ${webFetchCalls.length} → ${MAX_WEB_FETCH_CALLS} to prevent context overflow`);
                         const limitedWebFetch = webFetchCalls.slice(0, MAX_WEB_FETCH_CALLS);
                         const otherCalls = response.tool_calls.filter(tc => tc.name !== 'web.fetch');
                         toolCallsToExecute = [...otherCalls, ...limitedWebFetch];
@@ -407,7 +405,7 @@ class AIClient extends events_1.EventEmitter {
                     const MAX_TOOL_OUTPUT_PER_ROUND = 50_000;
                     let roundOutputSize = 0;
                     if (useParallel) {
-                        (0, provider_logger_1.log)(`[AIClient][${requestId}] Using parallel execution for ${toolCallsToExecute.length} tools`);
+                        (0, provider_logger_1.logInfo)(`[AIClient][${requestId}] Using parallel execution for ${toolCallsToExecute.length} tools`);
                         const toolResults = await this.toolOrchestrator.executeWithDependencies(toolCallsToExecute, (toolCall) => this.executeTool(toolCall), 5 // maxConcurrent
                         );
                         // Add results in original order with budget tracking
@@ -425,7 +423,7 @@ class AIClient extends events_1.EventEmitter {
                             const resultStr = typeof result === 'string' ? result : JSON.stringify(result);
                             // Check budget before adding
                             if (roundOutputSize + resultStr.length > MAX_TOOL_OUTPUT_PER_ROUND) {
-                                (0, provider_logger_1.log)(`[AIClient][${requestId}] Tool output budget exceeded (${MAX_TOOL_OUTPUT_PER_ROUND} chars), adding placeholder for remaining tools`);
+                                (0, provider_logger_1.logWarn)(`[AIClient][${requestId}] Tool output budget exceeded (${MAX_TOOL_OUTPUT_PER_ROUND} chars), adding placeholder for remaining tools`);
                                 messages.push({
                                     role: 'tool',
                                     tool_call_id: toolCall.id,
@@ -434,8 +432,8 @@ class AIClient extends events_1.EventEmitter {
                                 parallelBudgetExceeded = true;
                                 continue;
                             }
-                            const content = typeof result === 'string' && result.length > TOOL_RESULT_MAX_CHARS
-                                ? `${result.slice(0, TOOL_RESULT_MAX_CHARS)}\n[TRUNCATED tool result: ${result.length} chars]`
+                            const content = typeof result === 'string' && result.length > this.toolResultMaxChars
+                                ? `${result.slice(0, this.toolResultMaxChars)}\n[TRUNCATED tool result: ${result.length} chars]`
                                 : result;
                             const contentStr = typeof content === 'string' ? content : JSON.stringify(content);
                             roundOutputSize += contentStr.length;
@@ -445,10 +443,10 @@ class AIClient extends events_1.EventEmitter {
                                 content,
                             });
                         }
-                        (0, provider_logger_1.log)(`[AIClient][${requestId}] Round tool output size: ${roundOutputSize} chars (budget: ${MAX_TOOL_OUTPUT_PER_ROUND})`);
+                        (0, provider_logger_1.logDebug)(`[AIClient][${requestId}] Round tool output size: ${roundOutputSize} chars (budget: ${MAX_TOOL_OUTPUT_PER_ROUND})`);
                     }
                     else {
-                        (0, provider_logger_1.log)(`[AIClient][${requestId}] Using sequential execution for ${toolCallsToExecute.length} tools`);
+                        (0, provider_logger_1.logInfo)(`[AIClient][${requestId}] Using sequential execution for ${toolCallsToExecute.length} tools`);
                         // Sequential execution with budget tracking
                         let seqBudgetExceeded = false;
                         for (const toolCall of toolCallsToExecute) {
@@ -464,7 +462,7 @@ class AIClient extends events_1.EventEmitter {
                             const resultStr = typeof result === 'string' ? result : JSON.stringify(result);
                             // Check budget before adding
                             if (roundOutputSize + resultStr.length > MAX_TOOL_OUTPUT_PER_ROUND) {
-                                (0, provider_logger_1.log)(`[AIClient][${requestId}] Tool output budget exceeded (${MAX_TOOL_OUTPUT_PER_ROUND} chars), adding placeholder for remaining tools`);
+                                (0, provider_logger_1.logWarn)(`[AIClient][${requestId}] Tool output budget exceeded (${MAX_TOOL_OUTPUT_PER_ROUND} chars), adding placeholder for remaining tools`);
                                 messages.push({
                                     role: 'tool',
                                     tool_call_id: toolCall.id,
@@ -473,8 +471,8 @@ class AIClient extends events_1.EventEmitter {
                                 seqBudgetExceeded = true;
                                 continue;
                             }
-                            const content = typeof result === 'string' && result.length > TOOL_RESULT_MAX_CHARS
-                                ? `${result.slice(0, TOOL_RESULT_MAX_CHARS)}\n[TRUNCATED tool result: ${result.length} chars]`
+                            const content = typeof result === 'string' && result.length > this.toolResultMaxChars
+                                ? `${result.slice(0, this.toolResultMaxChars)}\n[TRUNCATED tool result: ${result.length} chars]`
                                 : result;
                             const contentStr = typeof content === 'string' ? content : JSON.stringify(content);
                             roundOutputSize += contentStr.length;
@@ -484,7 +482,7 @@ class AIClient extends events_1.EventEmitter {
                                 content,
                             });
                         }
-                        (0, provider_logger_1.log)(`[AIClient][${requestId}] Round tool output size: ${roundOutputSize} chars (budget: ${MAX_TOOL_OUTPUT_PER_ROUND})`);
+                        (0, provider_logger_1.logDebug)(`[AIClient][${requestId}] Round tool output size: ${roundOutputSize} chars (budget: ${MAX_TOOL_OUTPUT_PER_ROUND})`);
                     }
                     // Call the model again with updated messages
                     const rawFollowupReq = { ...enrichedRequest, messages, __toolpack_request_id: requestId };
@@ -492,14 +490,14 @@ class AIClient extends events_1.EventEmitter {
                     const followupReq = await this.enrichRequestWithTools(rawFollowupReq);
                     if (followupReq.tool_choice === 'required') {
                         followupReq.tool_choice = lookupOnly ? 'none' : 'auto';
-                        (0, provider_logger_1.log)(`[AIClient][${requestId}] generate() followup tool_choice override required->${followupReq.tool_choice}`);
+                        (0, provider_logger_1.logInfo)(`[AIClient][${requestId}] generate() followup tool_choice override required->${followupReq.tool_choice}`);
                     }
-                    if ((0, provider_logger_1.isVerbose)()) {
-                        (0, provider_logger_1.log)(`[AIClient][${requestId}] generate() followup request messages=${messages.length}`);
+                    if ((0, provider_logger_1.shouldLog)('debug')) {
+                        (0, provider_logger_1.logDebug)(`[AIClient][${requestId}] generate() followup request messages=${messages.length}`);
                         logRequestMessages(requestId, messages);
                     }
                     response = await provider.generate(followupReq);
-                    (0, provider_logger_1.log)(`[AIClient][${requestId}] generate() followup response finish_reason=${response.finish_reason ?? 'unknown'} tool_calls=${response.tool_calls?.length || 0} content_preview=${(0, provider_logger_1.safePreview)(response.content || '', 200)}`);
+                    (0, provider_logger_1.logDebug)(`[AIClient][${requestId}] generate() followup response finish_reason=${response.finish_reason ?? 'unknown'} tool_calls=${response.tool_calls?.length || 0} content_preview=${(0, provider_logger_1.safePreview)(response.content || '', 200)}`);
                 }
             }
             return response;
@@ -533,7 +531,7 @@ class AIClient extends events_1.EventEmitter {
             if (!needsTools && intelligentDetection?.enabled && hasTools) {
                 const afterToolCall = isAfterToolCall(enrichedRequest.messages, intelligentDetection.maxFollowUpMessages);
                 if (afterToolCall) {
-                    (0, provider_logger_1.log)(`[AIClient][${requestId}] Message is after tool call, using AI to infer tool needs`);
+                    (0, provider_logger_1.logInfo)(`[AIClient][${requestId}] Message is after tool call, using AI to infer tool needs`);
                     const fastModel = getFastModelForProvider(resolvedProviderName || 'openai');
                     needsTools = await inferNeedsToolsWithAI(provider, resolvedProviderName || 'openai', enrichedRequest.messages, fastModel);
                     aiInferenceUsed = true;
@@ -547,11 +545,11 @@ class AIClient extends events_1.EventEmitter {
             }
             else if (shouldForceNone) {
                 enrichedRequest.tool_choice = 'none';
-                (0, provider_logger_1.log)(`[AIClient][${requestId}] AI inference determined no tools needed, setting tool_choice=none`);
+                (0, provider_logger_1.logInfo)(`[AIClient][${requestId}] AI inference determined no tools needed, setting tool_choice=none`);
             }
             const providerClass = provider?.constructor?.name || 'UnknownProvider';
             const baseReq = { ...enrichedRequest, __toolpack_request_id: requestId };
-            (0, provider_logger_1.log)(`[AIClient][${requestId}] stream() start provider=${resolvedProviderName} class=${providerClass} model=${enrichedRequest.model} messages=${enrichedRequest.messages.length} tools=${enrichedRequest.tools?.length || 0} tool_choice=${enrichedRequest.tool_choice ?? 'unset'} policy=${policy} needsTools=${needsTools} autoExecute=${this.toolsConfig.enabled && this.toolsConfig.autoExecute}`);
+            (0, provider_logger_1.logInfo)(`[AIClient][${requestId}] stream() start provider=${resolvedProviderName} class=${providerClass} model=${enrichedRequest.model} messages=${enrichedRequest.messages.length} tools=${enrichedRequest.tools?.length || 0} tool_choice=${enrichedRequest.tool_choice ?? 'unset'} policy=${policy} needsTools=${needsTools} autoExecute=${this.toolsConfig.enabled && this.toolsConfig.autoExecute}`);
             logRequestMessages(requestId, enrichedRequest.messages);
             if (!this.toolsConfig.enabled || !this.toolsConfig.autoExecute || !this.toolRegistry) {
                 yield* provider.stream(baseReq);
@@ -565,34 +563,34 @@ class AIClient extends events_1.EventEmitter {
             const baseMaxRounds = this.toolsConfig.maxToolRounds;
             const maxRounds = this.queryClassifier.getToolRoundsAdjustment(classification, baseMaxRounds);
             if (maxRounds !== baseMaxRounds) {
-                (0, provider_logger_1.log)(`[AIClient][${requestId}] stream() Query classified as ${classification.type} (confidence: ${classification.confidence.toFixed(2)}), adjusted maxToolRounds: ${baseMaxRounds} → ${maxRounds}`);
+                (0, provider_logger_1.logInfo)(`[AIClient][${requestId}] stream() Query classified as ${classification.type} (confidence: ${classification.confidence.toFixed(2)}), adjusted maxToolRounds: ${baseMaxRounds} → ${maxRounds}`);
             }
             while (rounds <= maxRounds) {
                 // Check for abort signal at start of each round
                 if (request.signal?.aborted) {
-                    (0, provider_logger_1.log)(`[AIClient][${requestId}] stream() aborted by signal`);
+                    (0, provider_logger_1.logInfo)(`[AIClient][${requestId}] stream() aborted by signal`);
                     return;
                 }
                 let accumulatedContent = '';
                 const pendingToolCalls = [];
-                (0, provider_logger_1.log)(`[AIClient][${requestId}] stream() round_start ${rounds + 1}/${maxRounds}`);
+                (0, provider_logger_1.logInfo)(`[AIClient][${requestId}] stream() round_start ${rounds + 1}/${maxRounds}`);
                 let lastFinishReason = null;
                 const rawRoundReq = { ...baseReq, messages };
                 // Re-enrich to include any newly discovered tools from previous rounds
                 const roundReq = await this.enrichRequestWithTools(rawRoundReq);
                 if (rounds > 0 && roundReq.tool_choice === 'required') {
                     roundReq.tool_choice = lookupOnly ? 'none' : 'auto';
-                    (0, provider_logger_1.log)(`[AIClient][${requestId}] stream() round_${rounds + 1} tool_choice override required->${roundReq.tool_choice}`);
+                    (0, provider_logger_1.logInfo)(`[AIClient][${requestId}] stream() round_${rounds + 1} tool_choice override required->${roundReq.tool_choice}`);
                 }
                 for await (const chunk of provider.stream(roundReq)) {
                     // Check for abort signal during streaming
                     if (request.signal?.aborted) {
-                        (0, provider_logger_1.log)(`[AIClient][${requestId}] stream() aborted by signal during chunk processing`);
+                        (0, provider_logger_1.logInfo)(`[AIClient][${requestId}] stream() aborted by signal during chunk processing`);
                         return;
                     }
                     if (chunk.tool_calls && chunk.tool_calls.length > 0) {
                         pendingToolCalls.push(...chunk.tool_calls);
-                        (0, provider_logger_1.log)(`[AIClient][${requestId}] stream() tool_calls_chunk count=${chunk.tool_calls.length} names=${chunk.tool_calls.map(tc => tc.name).join(', ')}`);
+                        (0, provider_logger_1.logDebug)(`[AIClient][${requestId}] stream() tool_calls_chunk count=${chunk.tool_calls.length} names=${chunk.tool_calls.map(tc => tc.name).join(', ')}`);
                         // Yield tool calls in the chunk so consumers can track them
                         yield chunk;
                     }
@@ -607,18 +605,18 @@ class AIClient extends events_1.EventEmitter {
                         yield chunk;
                     }
                 }
-                (0, provider_logger_1.log)(`[AIClient][${requestId}] stream() round_end finish_reason=${lastFinishReason ?? 'unknown'} accumulated_len=${accumulatedContent.length} tool_calls_total=${pendingToolCalls.length} content_preview=${(0, provider_logger_1.safePreview)(accumulatedContent, 200)}`);
+                (0, provider_logger_1.logDebug)(`[AIClient][${requestId}] stream() round_end finish_reason=${lastFinishReason ?? 'unknown'} accumulated_len=${accumulatedContent.length} tool_calls_total=${pendingToolCalls.length} content_preview=${(0, provider_logger_1.safePreview)(accumulatedContent, 200)}`);
                 // If no tool calls, we're done
                 if (pendingToolCalls.length === 0) {
                     break;
                 }
-                (0, provider_logger_1.log)(`[AIClient][${requestId}] stream() received ${pendingToolCalls.length} tool call(s): ${pendingToolCalls.map(tc => tc.name).join(', ')}`);
+                (0, provider_logger_1.logInfo)(`[AIClient][${requestId}] stream() received ${pendingToolCalls.length} tool call(s): ${pendingToolCalls.map(tc => tc.name).join(', ')}`);
                 rounds++;
                 if (rounds > maxRounds) {
-                    (0, provider_logger_1.log)(`[AIClient][${requestId}] stream() max tool rounds (${maxRounds}) reached`);
+                    (0, provider_logger_1.logInfo)(`[AIClient][${requestId}] stream() max tool rounds (${maxRounds}) reached`);
                     break;
                 }
-                (0, provider_logger_1.log)(`[AIClient][${requestId}] stream() tool round ${rounds}/${maxRounds}`);
+                (0, provider_logger_1.logInfo)(`[AIClient][${requestId}] stream() tool round ${rounds}/${maxRounds}`);
                 // Add assistant message and tool results to conversation
                 messages.push({
                     role: 'assistant',
@@ -637,7 +635,7 @@ class AIClient extends events_1.EventEmitter {
                 let toolCallsToExecute = pendingToolCalls;
                 const webFetchCalls = pendingToolCalls.filter(tc => tc.name === 'web.fetch');
                 if (webFetchCalls.length > MAX_WEB_FETCH_CALLS) {
-                    (0, provider_logger_1.log)(`[AIClient][${requestId}] Limiting web.fetch calls from ${webFetchCalls.length} → ${MAX_WEB_FETCH_CALLS} to prevent context overflow`);
+                    (0, provider_logger_1.logInfo)(`[AIClient][${requestId}] Limiting web.fetch calls from ${webFetchCalls.length} → ${MAX_WEB_FETCH_CALLS} to prevent context overflow`);
                     const limitedWebFetch = webFetchCalls.slice(0, MAX_WEB_FETCH_CALLS);
                     const otherCalls = pendingToolCalls.filter(tc => tc.name !== 'web.fetch');
                     toolCallsToExecute = [...otherCalls, ...limitedWebFetch];
@@ -688,7 +686,7 @@ class AIClient extends events_1.EventEmitter {
                     const resultStr = typeof result === 'string' ? result : JSON.stringify(result);
                     // Check budget before adding
                     if (roundOutputSize + resultStr.length > MAX_TOOL_OUTPUT_PER_ROUND) {
-                        (0, provider_logger_1.log)(`[AIClient][${requestId}] Tool output budget exceeded (${MAX_TOOL_OUTPUT_PER_ROUND} chars), adding placeholder for remaining tools`);
+                        (0, provider_logger_1.logWarn)(`[AIClient][${requestId}] Tool output budget exceeded (${MAX_TOOL_OUTPUT_PER_ROUND} chars), adding placeholder for remaining tools`);
                         messages.push({
                             role: 'tool',
                             tool_call_id: toolCall.id,
@@ -697,8 +695,8 @@ class AIClient extends events_1.EventEmitter {
                         budgetExceeded = true;
                         continue;
                     }
-                    const content = typeof result === 'string' && result.length > TOOL_RESULT_MAX_CHARS
-                        ? `${result.slice(0, TOOL_RESULT_MAX_CHARS)}\n[TRUNCATED tool result: ${result.length} chars]`
+                    const content = typeof result === 'string' && result.length > this.toolResultMaxChars
+                        ? `${result.slice(0, this.toolResultMaxChars)}\n[TRUNCATED tool result: ${result.length} chars]`
                         : result;
                     const contentStr = typeof content === 'string' ? content : JSON.stringify(content);
                     roundOutputSize += contentStr.length;
@@ -717,9 +715,9 @@ class AIClient extends events_1.EventEmitter {
                             }],
                     };
                 }
-                (0, provider_logger_1.log)(`[AIClient][${requestId}] Round tool output size: ${roundOutputSize} chars (budget: ${MAX_TOOL_OUTPUT_PER_ROUND})`);
-                if ((0, provider_logger_1.isVerbose)()) {
-                    (0, provider_logger_1.log)(`[AIClient][${requestId}] stream() after_tools messages=${messages.length}`);
+                (0, provider_logger_1.logDebug)(`[AIClient][${requestId}] Round tool output size: ${roundOutputSize} chars (budget: ${MAX_TOOL_OUTPUT_PER_ROUND})`);
+                if ((0, provider_logger_1.shouldLog)('debug')) {
+                    (0, provider_logger_1.logDebug)(`[AIClient][${requestId}] stream() after_tools messages=${messages.length}`);
                     logRequestMessages(requestId, messages);
                 }
             }
@@ -747,11 +745,11 @@ class AIClient extends events_1.EventEmitter {
     async enrichRequestWithTools(request) {
         // If mode blocks ALL tools, return request with no tools
         if (this.activeMode?.blockAllTools) {
-            (0, provider_logger_1.log)(`[AIClient] Mode "${this.activeMode.displayName}" blocks all tools`);
+            (0, provider_logger_1.logInfo)(`[AIClient] Mode "${this.activeMode.displayName}" blocks all tools`);
             return request;
         }
-        if (!this.toolsConfig.enabled || !this.toolRegistry) {
-            (0, provider_logger_1.log)(`[AIClient] Tools disabled or no registry`);
+        if (!this.toolsConfig.enabled || (!this.toolRegistry && (request.tools?.length || 0) === 0)) {
+            (0, provider_logger_1.logDebug)(`[AIClient] Tools disabled or no registry`);
             return request;
         }
         // Merge mode-specific tool search config with global config
@@ -766,22 +764,22 @@ class AIClient extends events_1.EventEmitter {
                     ...(this.activeMode.toolSearch.alwaysLoadedCategories ? { alwaysLoadedCategories: this.activeMode.toolSearch.alwaysLoadedCategories } : {}),
                 }
             };
-            (0, provider_logger_1.log)(`[AIClient] Merged mode toolSearch config: enabled=${resolvedToolsConfig.toolSearch?.enabled}, alwaysLoadedTools=${resolvedToolsConfig.toolSearch?.alwaysLoadedTools?.length || 0}`);
+            (0, provider_logger_1.logDebug)(`[AIClient] Merged mode toolSearch config: enabled=${resolvedToolsConfig.toolSearch?.enabled}, alwaysLoadedTools=${resolvedToolsConfig.toolSearch?.alwaysLoadedTools?.length || 0}`);
         }
         // If the request already has tools, only add newly discovered tools when tool.search mode is enabled
         if (request.tools && request.tools.length > 0) {
             if (!resolvedToolsConfig.toolSearch?.enabled || !this.toolRegistry) {
-                (0, provider_logger_1.log)(`[AIClient] Request already has ${request.tools.length} tools`);
+                (0, provider_logger_1.logDebug)(`[AIClient] Request already has ${request.tools.length} tools`);
                 return request;
             }
             let schemas = await this.toolRouter.resolve(request.messages, this.toolRegistry, resolvedToolsConfig);
-            (0, provider_logger_1.log)(`[AIClient] Resolved ${schemas.length} tools to send: ${schemas.map(s => s.name).join(', ') || 'none'}`);
+            (0, provider_logger_1.logDebug)(`[AIClient] Resolved ${schemas.length} tools to send: ${schemas.map(s => s.name).join(', ') || 'none'}`);
             if (this.activeMode && schemas.length > 0) {
                 const beforeCount = schemas.length;
                 schemas = this.filterSchemasByMode(schemas, this.activeMode);
                 const filteredCount = beforeCount - schemas.length;
                 if (filteredCount > 0) {
-                    (0, provider_logger_1.log)(`[AIClient] Mode "${this.activeMode.displayName}" filtered out ${filteredCount} tools`);
+                    (0, provider_logger_1.logInfo)(`[AIClient] Mode "${this.activeMode.displayName}" filtered out ${filteredCount} tools`);
                 }
             }
             const existingToolNames = new Set(request.tools.map(tool => tool.function.name));
@@ -796,7 +794,7 @@ class AIClient extends events_1.EventEmitter {
                 },
             }));
             if (newTools.length === 0) {
-                (0, provider_logger_1.log)(`[AIClient] Request already has ${request.tools.length} tools (no new discoveries)`);
+                (0, provider_logger_1.logDebug)(`[AIClient] Request already has ${request.tools.length} tools (no new discoveries)`);
                 return request;
             }
             let enrichedRequest = {
@@ -808,15 +806,20 @@ class AIClient extends events_1.EventEmitter {
             }
             return enrichedRequest;
         }
-        let schemas = await this.toolRouter.resolve(request.messages, this.toolRegistry, resolvedToolsConfig);
-        (0, provider_logger_1.log)(`[AIClient] Resolved ${schemas.length} tools to send: ${schemas.map(s => s.name).join(', ') || 'none'}`);
+        if (!this.toolRegistry) {
+            (0, provider_logger_1.logDebug)('[AIClient] Tool registry not configured, skipping tool resolution');
+            return request;
+        }
+        const activeRegistry = this.toolRegistry;
+        let schemas = await this.toolRouter.resolve(request.messages, activeRegistry, resolvedToolsConfig);
+        (0, provider_logger_1.logDebug)(`[AIClient] Resolved ${schemas.length} tools to send: ${schemas.map(s => s.name).join(', ') || 'none'}`);
         // Apply mode-based filtering
         if (this.activeMode && schemas.length > 0) {
             const beforeCount = schemas.length;
             schemas = this.filterSchemasByMode(schemas, this.activeMode);
             const filteredCount = beforeCount - schemas.length;
             if (filteredCount > 0) {
-                (0, provider_logger_1.log)(`[AIClient] Mode "${this.activeMode.displayName}" filtered out ${filteredCount} tools`);
+                (0, provider_logger_1.logInfo)(`[AIClient] Mode "${this.activeMode.displayName}" filtered out ${filteredCount} tools`);
             }
         }
         if (schemas.length === 0) {
@@ -832,7 +835,7 @@ class AIClient extends events_1.EventEmitter {
         }));
         let enrichedRequest = { ...request, tools };
         // Inject Tool Search system prompt if enabled
-        if (this.toolsConfig.toolSearch?.enabled && this.toolRegistry) {
+        if (this.toolsConfig.toolSearch?.enabled && activeRegistry) {
             enrichedRequest = this.injectToolSearchPrompt(enrichedRequest);
         }
         return enrichedRequest;
@@ -866,11 +869,11 @@ class AIClient extends events_1.EventEmitter {
      */
     injectModeSystemPrompt(request) {
         if (!this.activeMode || !this.activeMode.systemPrompt) {
-            (0, provider_logger_1.log)(`[AIClient] injectModeSystemPrompt: No active mode or empty systemPrompt. activeMode=${this.activeMode?.name}, systemPrompt=${this.activeMode?.systemPrompt?.substring(0, 50)}`);
+            (0, provider_logger_1.logDebug)(`[AIClient] injectModeSystemPrompt: No active mode or empty systemPrompt. activeMode=${this.activeMode?.name}, systemPrompt=${this.activeMode?.systemPrompt?.substring(0, 50)}`);
             return request;
         }
         const modePrompt = this.activeMode.systemPrompt;
-        (0, provider_logger_1.log)(`[AIClient] injectModeSystemPrompt: Injecting mode prompt for ${this.activeMode.name}, length=${modePrompt.length}`);
+        (0, provider_logger_1.logDebug)(`[AIClient] injectModeSystemPrompt: Injecting mode prompt for ${this.activeMode.name}, length=${modePrompt.length}`);
         const hasSystemMessage = request.messages.some(m => m.role === 'system');
         if (hasSystemMessage) {
             // Prepend mode prompt to existing system message
@@ -1036,7 +1039,7 @@ NEVER guess or hallucinate tool names. ALWAYS use tool.search to discover tools
             status: 'started',
             args: toolCall.arguments,
         });
-        (0, provider_logger_1.log)(`[AIClient] Executing tool: ${toolCall.name} with args: ${(0, provider_logger_1.safePreview)(toolCall.arguments, 500)}`);
+        (0, provider_logger_1.logInfo)(`[AIClient] Executing tool: ${toolCall.name} with args: ${(0, provider_logger_1.safePreview)(toolCall.arguments, 500)}`);
         if (!this.toolRegistry) {
             const error = 'No tool registry configured';
             this.emit('tool:failed', {
@@ -1073,7 +1076,7 @@ NEVER guess or hallucinate tool names. ALWAYS use tool.search to discover tools
         }
         const tool = this.toolRegistry.get(toolCall.name);
         if (!tool) {
-            (0, provider_logger_1.log)(`[AIClient] Tool '${toolCall.name}' not found in registry`);
+            (0, provider_logger_1.logWarn)(`[AIClient] Tool '${toolCall.name}' not found in registry`);
             // Fuzzy match: detect common hallucination patterns
             const suggestion = this.findSimilarToolName(toolCall.name);
             const errorMsg = suggestion
@@ -1092,7 +1095,7 @@ NEVER guess or hallucinate tool names. ALWAYS use tool.search to discover tools
             const ctx = {
                 workspaceRoot: process.cwd(),
                 config: this.toolsConfig?.additionalConfigurations ?? {},
-                log: (msg) => (0, provider_logger_1.log)(`[Tool] ${msg}`),
+                log: (msg) => (0, provider_logger_1.logInfo)(`[Tool] ${msg}`),
             };
             const result = await tool.execute(toolCall.arguments, ctx);
             const duration = Date.now() - startTime;
@@ -1114,9 +1117,9 @@ NEVER guess or hallucinate tool names. ALWAYS use tool.search to discover tools
                 status: 'success',
                 timestamp: Date.now(),
             });
-            (0, provider_logger_1.log)(`[AIClient] Tool ${toolCall.name} executed successfully in ${duration}ms result_len=${result?.length ?? 0}`);
-            if ((0, provider_logger_1.isVerbose)()) {
-                (0, provider_logger_1.log)(`[AIClient] Tool ${toolCall.name} result_preview=${(0, provider_logger_1.safePreview)(result, 400)}`);
+            (0, provider_logger_1.logInfo)(`[AIClient] Tool ${toolCall.name} executed successfully in ${duration}ms result_len=${result?.length ?? 0}`);
+            if ((0, provider_logger_1.shouldLog)('debug')) {
+                (0, provider_logger_1.logDebug)(`[AIClient] Tool ${toolCall.name} result_preview=${(0, provider_logger_1.safePreview)(result, 400)}`);
             }
             return result;
         }
@@ -1141,7 +1144,7 @@ NEVER guess or hallucinate tool names. ALWAYS use tool.search to discover tools
                 status: 'error',
                 timestamp: Date.now(),
             });
-            (0, provider_logger_1.log)(`[AIClient] Tool ${toolCall.name} failed: ${(0, provider_logger_1.safePreview)(errorMsg, 300)}`);
+            (0, provider_logger_1.logError)(`[AIClient] Tool ${toolCall.name} failed: ${(0, provider_logger_1.safePreview)(errorMsg, 300)}`);
             return JSON.stringify({ error: errorMsg });
         }
     }
@@ -1213,12 +1216,12 @@ NEVER guess or hallucinate tool names. ALWAYS use tool.search to discover tools
     executeToolSearch(args) {
         const { query, category } = args;
         const limit = this.toolsConfig.toolSearch?.searchResultLimit ?? 5;
-        (0, provider_logger_1.log)(`[AIClient] Executing tool.search: query="${query}" category=${category || 'all'} limit=${limit}`);
+        (0, provider_logger_1.logInfo)(`[AIClient] Executing tool.search: query="${query}" category=${category || 'all'} limit=${limit}`);
         const results = this.bm25Engine.search(query, { limit, category });
         // Record discovered tools in the cache
         const toolNames = results.map(r => r.toolName);
         this.toolRouter.getDiscoveryCache().recordDiscovery(query, toolNames);
-        (0, provider_logger_1.log)(`[AIClient] tool.search found ${results.length} tools: ${toolNames.join(', ') || 'none'}`);
+        (0, provider_logger_1.logDebug)(`[AIClient] tool.search found ${results.length} tools: ${toolNames.join(', ') || 'none'}`);
         return JSON.stringify({
             query,
             found: results.length,