Skip to content

Commit

Permalink
Make RequiredActionUpdate a RunUpdate (#128)
Browse files Browse the repository at this point in the history
  • Loading branch information
annelo-msft authored Jul 17, 2024
1 parent 39941ca commit 3e12a62
Showing 1 changed file with 3 additions and 5 deletions.
8 changes: 3 additions & 5 deletions src/Custom/Assistants/Streaming/RequiredActionUpdate.cs
Original file line number Diff line number Diff line change
Expand Up @@ -11,7 +11,7 @@ namespace OpenAI.Assistants;
/// Distinct <see cref="RequiredActionUpdate"/> instances will generated for each required action, meaning that
/// parallel function calling will present multiple updates even if the tool calls arrive at the same time.
/// </remarks>
public class RequiredActionUpdate : StreamingUpdate
public class RequiredActionUpdate : RunUpdate
{
/// <inheritdoc cref="InternalRequiredFunctionToolCall.InternalName"/>
public string FunctionName => AsFunctionCall?.FunctionName;
Expand All @@ -24,13 +24,11 @@ public class RequiredActionUpdate : StreamingUpdate

private InternalRequiredFunctionToolCall AsFunctionCall => _requiredAction as InternalRequiredFunctionToolCall;

private readonly ThreadRun _run;
private readonly RequiredAction _requiredAction;

internal RequiredActionUpdate(ThreadRun run, RequiredAction action)
: base(StreamingUpdateReason.RunRequiresAction)
: base(run, StreamingUpdateReason.RunRequiresAction)
{
_run = run;
_requiredAction = action;
}

Expand All @@ -39,7 +37,7 @@ internal RequiredActionUpdate(ThreadRun run, RequiredAction action)
/// update.
/// </summary>
/// <returns></returns>
public ThreadRun GetThreadRun() => _run;
public ThreadRun GetThreadRun() => Value;

internal static IEnumerable<RequiredActionUpdate> DeserializeRequiredActionUpdates(JsonElement element)
{
Expand Down

0 comments on commit 3e12a62

Please sign in to comment.