Skip to content

Commit 0aa98bd

Browse files
committed
Open Telemetry. Add index for ChoiceEvent in OpenTelemetry
1 parent e6b54c4 commit 0aa98bd

File tree

3 files changed

+14
-3
lines changed

3 files changed

+14
-3
lines changed

agents/agents-features/agents-features-opentelemetry/src/jvmMain/kotlin/ai/koog/agents/features/opentelemetry/event/ChoiceEvent.kt

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -10,6 +10,7 @@ internal class ChoiceEvent(
1010
provider: LLMProvider,
1111
private val message: Message.Response,
1212
private val arguments: JsonObject? = null,
13+
val index: Int,
1314
override val verbose: Boolean = false,
1415
) : GenAIAgentEvent {
1516

@@ -20,7 +21,7 @@ internal class ChoiceEvent(
2021
}
2122

2223
override val bodyFields: List<EventBodyField> = buildList {
23-
add(EventBodyFields.Index(0))
24+
add(EventBodyFields.Index(index))
2425

2526
when (message) {
2627
is Message.Assistant -> {

agents/agents-features/agents-features-opentelemetry/src/jvmMain/kotlin/ai/koog/agents/features/opentelemetry/feature/OpenTelemetry.kt

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -279,7 +279,6 @@ public class OpenTelemetry {
279279
when (message) {
280280
is Message.User -> UserMessageEvent(provider, message, verbose = config.isVerbose)
281281
is Message.System -> SystemMessageEvent(provider, message, verbose = config.isVerbose)
282-
is Message.Assistant -> AssistantMessageEvent(provider, message, verbose = config.isVerbose)
283282
is Message.Tool.Result -> {
284283
ToolMessageEvent(
285284
provider = provider,
@@ -325,7 +324,7 @@ public class OpenTelemetry {
325324
when (message) {
326325
is Message.Assistant -> add(AssistantMessageEvent(provider, message, verbose = config.isVerbose))
327326
is Message.Tool.Call -> add(
328-
ChoiceEvent(provider, message, arguments = message.contentJson, verbose = config.isVerbose)
327+
ChoiceEvent(provider, message, arguments = message.contentJson, index = 0, verbose = config.isVerbose)
329328
)
330329
}
331330
}

agents/agents-features/agents-features-opentelemetry/src/jvmTest/kotlin/ai/koog/agents/features/opentelemetry/event/ChoiceEventTest.kt

Lines changed: 11 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -24,6 +24,7 @@ class ChoiceEventTest {
2424
val choiceEvent = ChoiceEvent(
2525
provider = llmProvider,
2626
message = expectedMessage,
27+
index = 0,
2728
verbose = false,
2829
)
2930

@@ -44,6 +45,7 @@ class ChoiceEventTest {
4445
val choiceEvent = ChoiceEvent(
4546
provider = llmProvider,
4647
message = expectedMessage,
48+
index = 0,
4749
verbose = true,
4850
)
4951

@@ -67,6 +69,7 @@ class ChoiceEventTest {
6769
val choiceEvent = ChoiceEvent(
6870
provider = MockLLMProvider(),
6971
message = expectedMessage,
72+
index = 0,
7073
verbose = false,
7174
)
7275

@@ -86,6 +89,7 @@ class ChoiceEventTest {
8689
val choiceEvent = ChoiceEvent(
8790
provider = MockLLMProvider(),
8891
message = expectedMessage,
92+
index = 0,
8993
verbose = false,
9094
)
9195

@@ -106,6 +110,7 @@ class ChoiceEventTest {
106110
val choiceEvent = ChoiceEvent(
107111
provider = MockLLMProvider(),
108112
message = expectedMessage,
113+
index = 0,
109114
verbose = false,
110115
)
111116

@@ -125,6 +130,7 @@ class ChoiceEventTest {
125130
val choiceEvent = ChoiceEvent(
126131
provider = MockLLMProvider(),
127132
message = expectedMessage,
133+
index = 0,
128134
verbose = true,
129135
)
130136

@@ -148,6 +154,7 @@ class ChoiceEventTest {
148154
val choiceEvent = ChoiceEvent(
149155
provider = MockLLMProvider(),
150156
message = expectedMessage,
157+
index = 0,
151158
verbose = true,
152159
)
153160

@@ -177,6 +184,7 @@ class ChoiceEventTest {
177184
provider = MockLLMProvider(),
178185
message = expectedMessage,
179186
arguments = args,
187+
index = 0,
180188
verbose = true,
181189
)
182190

@@ -203,6 +211,7 @@ class ChoiceEventTest {
203211
provider = MockLLMProvider(),
204212
message = expectedMessage,
205213
arguments = args,
214+
index = 0,
206215
verbose = false,
207216
)
208217

@@ -224,6 +233,7 @@ class ChoiceEventTest {
224233
provider = MockLLMProvider(),
225234
message = expectedMessage,
226235
arguments = args,
236+
index = 0,
227237
verbose = true,
228238
)
229239

@@ -246,6 +256,7 @@ class ChoiceEventTest {
246256
provider = MockLLMProvider(),
247257
message = expectedMessage,
248258
arguments = args,
259+
index = 0,
249260
verbose = false,
250261
)
251262

0 commit comments

Comments
 (0)