Skip to content

examples: use tool call arguments in the realtime sample #1126

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
Rohan5commit wants to merge 1 commit into from
Closed

examples: use tool call arguments in the realtime sample #1126

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
13 changes: 11 additions & 2 deletions examples/Realtime/Example01_AudioFromFileWithToolsAsync.cs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -4,6 +4,7 @@
using System.Collections.Generic;
using System.IO;
using System.Linq;
using System.Text.Json;
using System.Threading.Tasks;

namespace OpenAI.Examples;
Expand Down Expand Up @@ -209,9 +210,17 @@ public async Task Example01_AudioFromFileWithToolsAsync()
{
hasToolCalls = true;

Console.WriteLine($">> Calling {functionCallItem.FunctionName} function...");
using JsonDocument argumentsJson = JsonDocument.Parse(functionCallItem.FunctionArguments.ToString());

@chatgpt-codex-connector chatgpt-codex-connector Bot Apr 27, 2026

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

P2 Badge Handle empty arguments before JSON parsing

functionCallItem.FunctionArguments is a string payload and can be empty in realtime function-call events; parsing it unconditionally with JsonDocument.Parse(...) will throw JsonException and stop the sample before it can submit tool output. This is a regression from the previous behavior (which continued with defaults) and can surface whenever the model returns an empty/malformed arguments string, so the example should defensively fall back (for example, treat empty input as {}) before parsing.

Useful? React with 👍 / 👎.

string location = argumentsJson.RootElement.TryGetProperty("location", out JsonElement locationElement)
? locationElement.GetString() ?? "San Francisco, CA"
: "San Francisco, CA";
string unit = argumentsJson.RootElement.TryGetProperty("unit", out JsonElement unitElement)
? unitElement.GetString() ?? "celsius"
: "celsius";
Comment on lines +213 to +219

string output = GetCurrentWeather(location: "San Francisco, CA");
Console.WriteLine($">> Calling {functionCallItem.FunctionName} function for {location}...");

string output = GetCurrentWeather(location: location, unit: unit);

RealtimeItem functionCallOutputItem = RealtimeItem.CreateFunctionCallOutputItem(
callId: functionCallItem.CallId,
Expand Down
Loading