Skip to content
This repository was archived by the owner on Jun 3, 2026. It is now read-only.
Strands Command Handler

Task 3.3: Implement Tool Result Status Auto-Detection for BedrockModelProvider #38

Sign in to view logs

Task 3.3: Implement Tool Result Status Auto-Detection for BedrockModelProvider

Task 3.3: Implement Tool Result Status Auto-Detection for BedrockModelProvider #38

Triggered via issue October 29, 2025 16:15
@afarntrogafarntrog
commented on #24 bef74f8
Status Success
Total duration 19m 57s
Artifacts –

strands-command.yml

on: issue_comment
authorization-check
5s
authorization-check
execute
19m 42s
execute
Fit to window
Zoom out
Zoom in

Annotations

10 errors
src/models/__tests__/bedrock.test.ts > BedrockModel > includeToolResultStatus configuration > when includeToolResultStatus is auto > excludes status field for non-Claude models: src/models/__tests__/bedrock.test.ts#L1025
AssertionError: expected last "spy" call to have been called with [ { messages: [ { …(2) } ], …(1) } ] - Expected + Received @@ -8,10 +8,11 @@ "content": [ { "text": "Result", }, ], + "status": "success", "toolUseId": "tool-123", }, }, ], "role": "user", ❯ src/models/__tests__/bedrock.test.ts:1025:43
src/models/__tests__/bedrock.test.ts > BedrockModel > includeToolResultStatus configuration > when includeToolResultStatus is false > never includes status field in tool results: src/models/__tests__/bedrock.test.ts#L944
AssertionError: expected last "spy" call to have been called with [ { messages: [ { …(2) } ], …(1) } ] - Expected + Received @@ -8,10 +8,11 @@ "content": [ { "text": "Result", }, ], + "status": "success", "toolUseId": "tool-123", }, }, ], "role": "user", ❯ src/models/__tests__/bedrock.test.ts:944:43
src/models/__tests__/bedrock.test.ts > BedrockModel > includeToolResultStatus configuration > debug logging > logs auto-detection decision for undefined (default auto mode): src/models/__tests__/bedrock.test.ts#L1171
AssertionError: expected "log" to be called with arguments: [ Array(1) ] Number of calls: 0 ❯ src/models/__tests__/bedrock.test.ts:1171:31
src/models/__tests__/bedrock.test.ts > BedrockModel > includeToolResultStatus configuration > debug logging > logs auto-detection decision for auto mode: src/models/__tests__/bedrock.test.ts#L1143
AssertionError: expected "log" to be called with arguments: [ Array(1) ] Number of calls: 0 ❯ src/models/__tests__/bedrock.test.ts:1143:31
src/models/__tests__/bedrock.test.ts > BedrockModel > includeToolResultStatus configuration > when includeToolResultStatus is undefined (default) > follows auto logic for non-Claude models: src/models/__tests__/bedrock.test.ts#L1101
ReferenceError: mockConverseStreamCommand is not defined ❯ src/models/__tests__/bedrock.test.ts:1101:16
src/models/__tests__/bedrock.test.ts > BedrockModel > includeToolResultStatus configuration > when includeToolResultStatus is undefined (default) > follows auto logic for Claude models: src/models/__tests__/bedrock.test.ts#L1062
ReferenceError: mockConverseStreamCommand is not defined ❯ src/models/__tests__/bedrock.test.ts:1062:16
src/models/__tests__/bedrock.test.ts > BedrockModel > includeToolResultStatus configuration > when includeToolResultStatus is auto > excludes status field for non-Claude models: src/models/__tests__/bedrock.test.ts#L1022
ReferenceError: mockConverseStreamCommand is not defined ❯ src/models/__tests__/bedrock.test.ts:1022:16
src/models/__tests__/bedrock.test.ts > BedrockModel > includeToolResultStatus configuration > when includeToolResultStatus is auto > includes status field for Claude models: src/models/__tests__/bedrock.test.ts#L982
ReferenceError: mockConverseStreamCommand is not defined ❯ src/models/__tests__/bedrock.test.ts:982:16
src/models/__tests__/bedrock.test.ts > BedrockModel > includeToolResultStatus configuration > when includeToolResultStatus is false > never includes status field in tool results: src/models/__tests__/bedrock.test.ts#L941
ReferenceError: mockConverseStreamCommand is not defined ❯ src/models/__tests__/bedrock.test.ts:941:16
src/models/__tests__/bedrock.test.ts > BedrockModel > includeToolResultStatus configuration > when includeToolResultStatus is true > always includes status field in tool results: src/models/__tests__/bedrock.test.ts#L902
ReferenceError: mockConverseStreamCommand is not defined ❯ src/models/__tests__/bedrock.test.ts:902:16