fix: improve message history management for tool use and thinking blocks

[bigboateng]

Important

Improves message history management for tool use and thinking blocks to prevent errors and ensure proper conversation flow.

• Behavior:
  • Updated examples/test-extended-thinking.ts to include multiple tool uses in the task description and improved error handling for tool use and thinking block issues.
• Functions:
  • cleanMessageHistory() in utils/message-processing.ts now ensures tool_result blocks have corresponding tool_use in the immediately previous message and orders assistant message blocks correctly.
  • ensureThinkingBlocksForExtendedThinking() in utils/message-processing.ts removes assistant messages without initial thinking blocks and the following user message to maintain flow.
• Misc:
  • Improved logging in examples/test-extended-thinking.ts for better error diagnosis.

^{This description was created by for 8d9146b. You can customize this summary. It will automatically update as commits are pushed.}

[ellipsis-dev]

Important

Looks good to me! 👍

Reviewed everything up to 8d9146b in 1 minute and 16 seconds. Click for details.

• Reviewed 202 lines of code in 2 files
• Skipped 0 files when reviewing.
• Skipped posting 5 draft comments. View those below.
• Modify your settings and rules to customize what types of comments Ellipsis leaves. And don't forget to react with 👍 or 👎 to teach Ellipsis.

1. examples/test-extended-thinking.ts:44

• Draft comment:
  Updated log message now clarifies that the test uses multiple tool uses. Also, check that the template literal’s indentation doesn’t introduce unintended whitespace in the prompt.
• Reason this comment was not posted:
  Confidence changes required: 33% <= threshold 50% None

2. examples/test-extended-thinking.ts:77

• Draft comment:
  Additional error handling for tool_use/tool_result pairing is a useful diagnostic addition.
• Reason this comment was not posted:
  Confidence changes required: 0% <= threshold 50% None

3. utils/message-processing.ts:185

• Draft comment:
  The revised cleanMessageHistory function now strictly associates each tool_result with a tool_use in the immediately preceding assistant message, aligning with the API requirements.
• Reason this comment was not posted:
  Confidence changes required: 0% <= threshold 50% None

4. utils/message-processing.ts:221

• Draft comment:
  Assistant message reordering now omits tool_result blocks—which matches the API spec. Confirm that excluding tool_result blocks in assistant messages is intended.
• Reason this comment was not posted:
  Confidence changes required: 33% <= threshold 50% None

5. utils/message-processing.ts:263

• Draft comment:
  The updated ensureThinkingBlocksForExtendedThinking function now removes the following user message when an assistant message lacks a thinking block, preserving conversation flow. Consider deduplicating indices in the removal loop to avoid potential double removals.
• Reason this comment was not posted:
  Confidence changes required: 33% <= threshold 50% None

Workflow ID: wflow_FWvHKoNt4dz9r8ig

^{You can customize by changing your verbosity settings, reacting with 👍 or 👎, replying to comments, or adding code review rules.}