sourcegraph / cody

AI that knows your entire codebase
https://cody.dev
Apache License 2.0
2.22k stars 213 forks source link

Remove redundant loggings #4677

Closed abeatrix closed 5 days ago

abeatrix commented 5 days ago

When running the cody-bench there are lots of events that we are logging when submitting a question:

Cody by Sourcegraph: █ SimpleChatPanelProvider:resolveWebviewViewOrPanel: registering webview view/panel
Cody by Sourcegraph: █ SimpleChatPanelProvider: handleInitialized
Cody by Sourcegraph: █ SimpleChatPanelProvider: updateViewConfig
Cody by Sourcegraph: █ telemetry-v2: recordEvent: cody.chat-question/submitted
Cody by Sourcegraph: █ DefaultPrompter.makePrompt: Ignored context due to context limit: user=0, enhanced=0, previous=0
Cody by Sourcegraph: █ telemetry-v2: recordEvent: cody.chat-question/executed
Cody by Sourcegraph: █ SimpleChatPanelProvider: streamAssistantResponse
Cody by Sourcegraph: █ CompletionLogger:onComplete: {"type":"completion","endpoint":"https://sourcegraph.com/.api/completions/stream?client-name=cody-bench&client-version=0.1.0","status":"success","duration":4207}
Cody by Sourcegraph: █ telemetry-v2: recordEvent: cody.chatResponse/noCode

This PR removes the redundant ones to reduce noises:

Test plan

Verify the events are removed