diff options
| author | jerop <[email protected]> | 2025-06-11 16:50:24 +0000 |
|---|---|---|
| committer | Jerop Kipruto <[email protected]> | 2025-06-11 13:24:41 -0400 |
| commit | d96af8bacd84d065310dddb49abb7561c4a7b059 (patch) | |
| tree | 7315ef5f5475ad38f1435aaea6864ee362fb2143 /packages/cli/src | |
| parent | 9c5b5ff82337cccdae59c0f8c11cf47793e7b37d (diff) | |
refactor(telemetry): pass config object to telemetry functions
This commit refactors the telemetry system to pass a object to various logging and metrics functions. This change centralizes configuration management within the telemetry system, making it more modular and easier to maintain.
The constructor and various tool execution functions have been updated to accept the object, which is then passed down to the telemetry functions. This eliminates the need to pass individual configuration values, such as , through multiple layers of the application.
Diffstat (limited to 'packages/cli/src')
| -rw-r--r-- | packages/cli/src/nonInteractiveCli.test.ts | 1 | ||||
| -rw-r--r-- | packages/cli/src/nonInteractiveCli.ts | 1 | ||||
| -rw-r--r-- | packages/cli/src/ui/hooks/useReactToolScheduler.ts | 2 |
3 files changed, 3 insertions, 1 deletions
diff --git a/packages/cli/src/nonInteractiveCli.test.ts b/packages/cli/src/nonInteractiveCli.test.ts index 4e7fd727..959cf03f 100644 --- a/packages/cli/src/nonInteractiveCli.test.ts +++ b/packages/cli/src/nonInteractiveCli.test.ts @@ -134,6 +134,7 @@ describe('runNonInteractive', () => { expect(mockChat.sendMessageStream).toHaveBeenCalledTimes(2); expect(mockCoreExecuteToolCall).toHaveBeenCalledWith( + mockConfig, expect.objectContaining({ callId: 'fc1', name: 'testTool' }), mockToolRegistry, expect.any(AbortSignal), diff --git a/packages/cli/src/nonInteractiveCli.ts b/packages/cli/src/nonInteractiveCli.ts index 6b828dc8..069a7be8 100644 --- a/packages/cli/src/nonInteractiveCli.ts +++ b/packages/cli/src/nonInteractiveCli.ts @@ -85,6 +85,7 @@ export async function runNonInteractive( }; const toolResponse = await executeToolCall( + config, requestInfo, toolRegistry, abortController.signal, diff --git a/packages/cli/src/ui/hooks/useReactToolScheduler.ts b/packages/cli/src/ui/hooks/useReactToolScheduler.ts index 8dbcfb87..4e55cba4 100644 --- a/packages/cli/src/ui/hooks/useReactToolScheduler.ts +++ b/packages/cli/src/ui/hooks/useReactToolScheduler.ts @@ -122,7 +122,7 @@ export function useReactToolScheduler( } duration = call.durationMs || 0; - logToolCall({ + logToolCall(config, { function_name: call.request.name, function_args: call.request.args, duration_ms: duration, |