Skip to content

Commit 339f104

Browse files
Improve documentation for code clarity
Co-authored-by: waldekmastykarz <[email protected]>
1 parent 9c207d6 commit 339f104

File tree

1 file changed

+7
-3
lines changed

1 file changed

+7
-3
lines changed

DevProxy.Abstractions/LanguageModel/OpenAIModels.cs

Lines changed: 7 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -130,8 +130,9 @@ public static bool TryGetOpenAIRequest(string content, ILogger logger, out OpenA
130130
}
131131

132132
/// <summary>
133-
/// Tries to parse completion-like OpenAI requests (completion, chat completion, and responses).
134-
/// Used by plugins that only need to handle text-based completion requests.
133+
/// Tries to parse text generation OpenAI requests (completion, chat completion, and responses API).
134+
/// Used by plugins that only need to handle text-based generation requests, as opposed to
135+
/// embeddings, audio, images, or fine-tuning requests.
135136
/// </summary>
136137
public static bool TryGetCompletionLikeRequest(string content, ILogger logger, out OpenAIRequest? request)
137138
{
@@ -258,7 +259,10 @@ public class OpenAIResponseUsage
258259
[JsonPropertyName("total_tokens")]
259260
public long TotalTokens { get; set; }
260261

261-
// Responses API uses different property names
262+
// Responses API uses different property names (input_tokens, output_tokens)
263+
// These property aliases allow the same class to deserialize both formats.
264+
// When JSON contains "input_tokens", it maps to PromptTokens.
265+
// When JSON contains "output_tokens", it maps to CompletionTokens.
262266
[JsonPropertyName("input_tokens")]
263267
public long InputTokens
264268
{

0 commit comments

Comments
 (0)