@juspay/neurolink 11.6.0 → 11.7.0

package/dist/core/loopEngine.js

@@ -60,6 +60,8 @@ export function runAgenticLoop(adapter, initialConversation, options) {
         let conversation = initialConversation;
         let usage = { inputTokens: 0, outputTokens: 0 };
         let finalText = "";
+        // The most recent step's text, kept only for the step-cap case below.
+        let lastStepText = "";
         let rawStopReason;
         const allToolCalls = [];
         const allToolExecutions = [];
@@ -113,6 +115,7 @@ export function runAgenticLoop(adapter, initialConversation, options) {
                 }
                 usage = sumUsage(usage, stepResult.usage);
                 rawStopReason = stepResult.rawStopReason;
+                lastStepText = stepResult.text || lastStepText;
                 if (adapter.isMalformedStep?.(stepResult) &&
                     !malformedRetryUsed &&
                     !internalAbort.signal.aborted) {
@@ -153,7 +156,16 @@ export function runAgenticLoop(adapter, initialConversation, options) {
                         });
                         continue;
                     }
-                    const tool = options.tools?.[call.name];
+                    // Second lookup path for adapters that discover tools mid-turn.
+                    // The miss is defined as "nothing executable under this name"
+                    // rather than "no key under this name", because the guard directly
+                    // below already treats a present-but-unexecutable entry as absent —
+                    // a deferred-catalog placeholder is exactly that shape, and it is
+                    // precisely what hydration exists to resolve.
+                    const declaredTool = options.tools?.[call.name];
+                    const tool = declaredTool?.execute
+                        ? declaredTool
+                        : (adapter.resolveToolOnMiss?.(call.name) ?? declaredTool);
                     if (!tool?.execute) {
                         const output = breaker
                             ? {
@@ -211,7 +223,14 @@ export function runAgenticLoop(adapter, initialConversation, options) {
             }
             const finishReason = adapter.mapFinishReason(rawStopReason, hadToolCallsAtCap);
             return {
-                text: finalText,
+                // `finalText` is only set by a step that asked for no tools, so a turn
+                // that runs out of steps mid-tool-call would otherwise return "" and
+                // throw away everything the model actually said. An empty result also
+                // reads as a failed generation to callers that retry on empty content,
+                // turning one capped turn into several. Fall back to the last step's
+                // text in that case only — when the loop ended normally, an empty
+                // final step genuinely means the model said nothing.
+                text: finalText || (hadToolCallsAtCap ? lastStepText : ""),
                 toolCalls: allToolCalls,
                 toolExecutions: allToolExecutions,
                 usage,

package/dist/lib/core/loopEngine.js

@@ -60,6 +60,8 @@ export function runAgenticLoop(adapter, initialConversation, options) {
         let conversation = initialConversation;
         let usage = { inputTokens: 0, outputTokens: 0 };
         let finalText = "";
+        // The most recent step's text, kept only for the step-cap case below.
+        let lastStepText = "";
         let rawStopReason;
         const allToolCalls = [];
         const allToolExecutions = [];
@@ -113,6 +115,7 @@ export function runAgenticLoop(adapter, initialConversation, options) {
                 }
                 usage = sumUsage(usage, stepResult.usage);
                 rawStopReason = stepResult.rawStopReason;
+                lastStepText = stepResult.text || lastStepText;
                 if (adapter.isMalformedStep?.(stepResult) &&
                     !malformedRetryUsed &&
                     !internalAbort.signal.aborted) {
@@ -153,7 +156,16 @@ export function runAgenticLoop(adapter, initialConversation, options) {
                         });
                         continue;
                     }
-                    const tool = options.tools?.[call.name];
+                    // Second lookup path for adapters that discover tools mid-turn.
+                    // The miss is defined as "nothing executable under this name"
+                    // rather than "no key under this name", because the guard directly
+                    // below already treats a present-but-unexecutable entry as absent —
+                    // a deferred-catalog placeholder is exactly that shape, and it is
+                    // precisely what hydration exists to resolve.
+                    const declaredTool = options.tools?.[call.name];
+                    const tool = declaredTool?.execute
+                        ? declaredTool
+                        : (adapter.resolveToolOnMiss?.(call.name) ?? declaredTool);
                     if (!tool?.execute) {
                         const output = breaker
                             ? {
@@ -211,7 +223,14 @@ export function runAgenticLoop(adapter, initialConversation, options) {
             }
             const finishReason = adapter.mapFinishReason(rawStopReason, hadToolCallsAtCap);
             return {
-                text: finalText,
+                // `finalText` is only set by a step that asked for no tools, so a turn
+                // that runs out of steps mid-tool-call would otherwise return "" and
+                // throw away everything the model actually said. An empty result also
+                // reads as a failed generation to callers that retry on empty content,
+                // turning one capped turn into several. Fall back to the last step's
+                // text in that case only — when the loop ended normally, an empty
+                // final step genuinely means the model said nothing.
+                text: finalText || (hadToolCallsAtCap ? lastStepText : ""),
                 toolCalls: allToolCalls,
                 toolExecutions: allToolExecutions,
                 usage,

package/dist/lib/providers/amazonBedrock/client.d.ts

@@ -38,21 +38,37 @@ export declare class AmazonBedrockProvider extends BaseProvider {
     protected getDefaultEmbeddingModel(): string;
     generate(optionsOrPrompt: TextGenerationOptions | string): Promise<EnhancedGenerateResult | null>;
     private conversationLoop;
-    private callBedrock;
-    private handleBedrockResponse;
     private convertToAWSMessages;
+    /**
+     * `tools` is passed in rather than re-resolved from `getAllTools()`. That
+     * call returns only the provider's own registry, so resolving here meant a
+     * tool the caller passed to generate/stream could never execute — the
+     * streaming path advertised it to the model and then failed every call to
+     * it with "Tool not found", and the generate path never advertised it at
+     * all. The turn's full merged tool set is resolved once by the caller and
+     * handed down.
+     */
     private executeSingleTool;
+    /**
+     * Resolve the turn's tools once: whatever the caller passed, else the
+     * provider's own registry. `BaseProvider.stream()` has already merged base
+     * tools into `options.tools` by the time it reaches the streaming path;
+     * the generate path has no such pre-merge, so it falls back here.
+     */
+    private resolveTurnTools;
+    /**
+     * Present the resolved tools in the shape `runAgenticLoop` dispatches
+     * through. Execution still goes through `executeSingleTool`, so the tool
+     * span, the parameter defaults and the ToolResult unwrapping are unchanged
+     * — only which tools are reachable changes.
+     */
+    private toEngineTools;
     private convertAISDKToolsToToolDefinitions;
     private formatToolsForBedrock;
     private convertToBedrockMessages;
     getBedrockClient(): BedrockRuntimeClient;
     protected executeStream(options: StreamOptions): Promise<StreamResult>;
     private streamingConversationLoop;
-    private convertToAsyncIterable;
-    private prepareStreamCommand;
-    private processStreamResponse;
-    private handleStreamStopReason;
-    private executeStreamTools;
     /**
      * Health check for Amazon Bedrock service
      * Uses ListFoundationModels API to validate connectivity and permissions