fix: Fix input message attribute issues + toolcalling from dogfooding

[21ShisodeParth]

No description provided.

[axiomofjoy]

Can we add sync and async tests for incorporating back in tool call results in subsequent calls to the API? This is a good starting place.

[axiomofjoy]

Looks like we have the correct information showing in the UI 👍

[


    {
        "role": "user",
        "content": "What is the weather like in San Francisco in Fahrenheit?"
    },
    {
        "content": [
            {
                "text": "Certainly! I can help you get the current weather information for San Francisco in Fahrenheit. To do this, I'll use the `get_weather` function. Let me fetch that information for you right away.",
                "type": "text"
            },
            {
                "id": "toolu_01Frt3tu9xCxmPXJ1qtigiae",
                "input": {
                    "location": "San Francisco, CA",
                    "unit": "fahrenheit"
                },
                "name": "get_weather",
                "type": "tool_use"
            }
        ],
        "role": "assistant"
    },
    {
        "content": [
            {
                "tool_use_id": "toolu_01Frt3tu9xCxmPXJ1qtigiae",
                "content": "{\"weather\": \"sunny\", \"temperature\": \"75\"}",
                "type": "tool_result",
                "is_error": false
            }
        ],
        "role": "user"
    }
]

[axiomofjoy]

Nice job getting this one right:

  {
    "message": {
      "content": "Certainly! I can help you get the current weather information for San Francisco in Fahrenheit. To do this, I'll use the `get_weather` function. Let me fetch that information for you right away.",
      "tool_calls": [
        {
          "tool_call": {
            "function": {
              "arguments": "{\"location\": \"San Francisco, CA\", \"unit\": \"fahrenheit\"}",
              "name": "get_weather"
            }
          }
        }
      ],
      "role": "assistant"
    }
  },

It's cool that that shows up correctly in the UI, because I don't think OpenAI does the same thing where it sends content and tool calls in the same message. Can we do this same thing in the output messages?

[vercel]

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name	Status	Preview	Comments	Updated (UTC)
phoenix-vercel	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Aug 28, 2024 0:19am