fix tool use bug

[comhar]

In PR #34 the tool_choice check in Client.__call__ was changed from

if tool_choice and pr: kwargs['tool_choice'] = mk_tool_choice(tool_choice)

to

if tool_choice: kwargs['tool_choice'] = mk_tool_choice(tool_choice)

As shown in this gist tool use (for the calculator example at least) is silently failing when pr is removed. The detailed sections below show the chat history with/without pr. The opening 3 chat messages match but things get out of sync after that.

`tool_choice` with `pr`

[
  {'role': 'user', 'content': [{'type': 'text', 'text': 'What is 604542+6458932?'}]},
  {'role': 'assistant', 'content': [ToolUseBlock(id='toolu_018ErvHSzm6FwTj3VcqWYMDY', input={'a': 604542, 'b': 6458932}, name='sums', type='tool_use')]},
  {'role': 'user', 'content': [{'type': 'tool_result', 'tool_use_id': 'toolu_018ErvHSzm6FwTj3VcqWYMDY', 'content': '7063474'}]},
  {'role': 'assistant', 'content': [TextBlock(text='The sum of 604542 and 6458932 is 7063474.', type='text')]}
]

`tool_choice` without `pr`

[
 {'role': 'user', 'content': [{'type': 'text', 'text': 'What is 604542+6458932?'}]},
 {'role': 'assistant', 'content': [ToolUseBlock(id='toolu_01T6nhnbdKmvbifDaXrmaDVJ', input={'a': 604542, 'b': 6458932}, name='sums', type='tool_use')]},
 {'role': 'user', 'content': [{'type': 'tool_result', 'tool_use_id': 'toolu_01T6nhnbdKmvbifDaXrmaDVJ', 'content': '7063474'}]},
 {'role': 'assistant','content': [ToolUseBlock(id='toolu_01GzsM6QBFVrkdqWX8ZuGh7T', input={'a': 604542, 'b': 6458932}, name='sums', type='tool_use')]},
 {'role': 'user', 'content': [{'type': 'tool_result', 'tool_use_id': 'toolu_01GzsM6QBFVrkdqWX8ZuGh7T', 'content': '7063474'}]}
]

Question

Why was pr originally removed from the tool_choice check?

[gitnotebooks]

Found 1 changed notebook. Review the changes at https://app.gitnotebooks.com/AnswerDotAI/claudette/pull/35