fix(openai): Reorder model matching logic to prioritize tool calls

The change swaps the order of the two model matching cases to give
    priority to processing the tool call message. This is because
    in some cases, the tool call message may be more important and
    should be processed first. The updated code first matches the
    tool call message, and then the content message. This ensures
    that the tool call message is processed correctly and promptly.

rig-core/src/providers/openai.rs

@@ -375,17 +375,6 @@ impl TryFrom<CompletionResponse> for completion::CompletionResponse<CompletionRe
 
     fn try_from(value: CompletionResponse) -> std::prelude::v1::Result<Self, Self::Error> {
         match value.choices.as_slice() {
-            [Choice {
-                message:
-                    Message {
-                        content: Some(content),
-                        ..
-                    },
-                ..
-            }, ..] => Ok(completion::CompletionResponse {
-                choice: completion::ModelChoice::Message(content.to_string()),
-                raw_response: value,
-            }),
             [Choice {
                 message:
                     Message {
@@ -406,6 +395,17 @@ impl TryFrom<CompletionResponse> for completion::CompletionResponse<CompletionRe
                     raw_response: value,
                 })
             }
+            [Choice {
+                message:
+                    Message {
+                        content: Some(content),
+                        ..
+                    },
+                ..
+            }, ..] => Ok(completion::CompletionResponse {
+                choice: completion::ModelChoice::Message(content.to_string()),
+                raw_response: value,
+            }),
             _ => Err(CompletionError::ResponseError(
                 "Response did not contain a message or tool call".into(),
             )),