Skip to content
This repository was archived by the owner on Aug 1, 2025. It is now read-only.

Commit 1bb8943

Browse files
beyangSteve Yegge
authored andcommitted
log chat and fireworks requests (#3258)
1 parent 7d66676 commit 1bb8943

7 files changed

Lines changed: 27 additions & 18 deletions

File tree

vscode/src/chat/chat-view/SimpleChatPanelProvider.ts

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -846,6 +846,9 @@ export class SimpleChatPanelProvider implements vscode.Disposable, ChatSession {
846846
span: Span,
847847
firstTokenSpan: Span
848848
): void {
849+
logDebug('SimpleChatPanelProvider', 'streamAssistantResponse', {
850+
verbose: { requestID, prompt },
851+
})
849852
let firstTokenMeasured = false
850853
function measureFirstToken() {
851854
if (firstTokenMeasured) {

vscode/src/completions/providers/fireworks.ts

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -29,6 +29,7 @@ import { forkSignal, generatorWithTimeout, zipGenerators } from '../utils'
2929
import { SpanStatusCode } from '@opentelemetry/api'
3030
import { recordErrorToSpan, tracer } from '@sourcegraph/cody-shared/src/tracing'
3131
import type { AuthStatus } from '../../chat/protocol'
32+
import { logDebug } from '../../log'
3233
import { createRateLimitErrorFromResponse, createSSEIterator, logResponseHeadersToSpan } from '../client'
3334
import type { FetchCompletionResult } from './fetch-and-process-completions'
3435
import {
@@ -339,6 +340,7 @@ class FireworksProvider extends Provider {
339340
headers.set('X-Sourcegraph-Feature', 'code_completions')
340341
addTraceparent(headers)
341342

343+
logDebug('FireworksProvider', 'fetch', { verbose: { url, fireworksRequest } })
342344
const response = await fetch(url, {
343345
method: 'POST',
344346
body: JSON.stringify(fireworksRequest),

vscode/src/graph/bfg/spawn-bfg.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -29,7 +29,7 @@ export async function spawnBfg(
2929
},
3030
})
3131
child.stderr.on('data', chunk => {
32-
logDebug('CodyEngine', 'stderr', chunk.toString())
32+
logDebug('CodyEngine', 'spawnBfg:stderr', { verbose: chunk.toString() })
3333
})
3434
child.on('disconnect', () => reject())
3535
child.on('close', () => reject())

vscode/src/local-context/local-embeddings.ts

Lines changed: 6 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -220,14 +220,12 @@ export class LocalEmbeddingsController
220220
codyGatewayEndpoint: this.endpoint,
221221
indexPath: indexPath.fsPath,
222222
})
223-
logDebug(
224-
'LocalEmbeddingsController',
225-
'spawnAndBindService',
226-
'initialized',
227-
initResult,
228-
'token available?',
229-
!!this.accessToken
230-
)
223+
logDebug('LocalEmbeddingsController', 'spawnAndBindService', 'initialized', {
224+
verbose: {
225+
initResult,
226+
tokenAvailable: !!this.accessToken,
227+
},
228+
})
231229

232230
if (this.accessToken) {
233231
// Set the initial access token

vscode/src/log.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -24,7 +24,7 @@ export const outputChannel: vscode.OutputChannel = vscode.window.createOutputCha
2424
* logDebug('label', 'this is a message', 'some', 'args', { verbose: 'verbose info goes here' })
2525
*/
2626
export function logDebug(filterLabel: string, text: string, ...args: unknown[]): void {
27-
log('error', filterLabel, text, ...args)
27+
log('debug', filterLabel, text, ...args)
2828
}
2929

3030
/**

vscode/src/services/telemetry-v2.ts

Lines changed: 7 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -56,12 +56,13 @@ function updateGlobalInstances(updatedProvider: TelemetryRecorderProvider & { no
5656
new CallbackTelemetryProcessor(event => {
5757
logDebug(
5858
debugLogLabel,
59-
`recordEvent${updatedProvider.noOp ? ' (no-op)' : ''}: ${event.feature}/${
60-
event.action
61-
}: ${JSON.stringify({
62-
parameters: event.parameters,
63-
timestamp: event.timestamp,
64-
})}`
59+
`recordEvent${updatedProvider.noOp ? ' (no-op)' : ''}: ${event.feature}/${event.action}`,
60+
{
61+
verbose: {
62+
parameters: event.parameters,
63+
timestamp: event.timestamp,
64+
},
65+
}
6566
)
6667
}),
6768
])

vscode/src/services/telemetry.ts

Lines changed: 7 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -105,8 +105,13 @@ function logEvent(
105105
eventLogger === null || process.env.CODY_TESTING === 'true' ? ' (telemetry disabled)' : ''
106106
}`,
107107
eventName,
108-
getExtensionDetails(getConfiguration(vscode.workspace.getConfiguration())).ide,
109-
JSON.stringify({ properties, opts })
108+
{
109+
verbose: {
110+
details: getExtensionDetails(getConfiguration(vscode.workspace.getConfiguration())).ide,
111+
properties,
112+
opts,
113+
},
114+
}
110115
)
111116
if (!eventLogger || !globalAnonymousUserID) {
112117
return

0 commit comments

Comments
 (0)