Skip to content

Commit 61d2ce9

Browse files
committed
Clean up debug logging across WebSocket services
- [Pasted text] we don't want to log so much anyways ├── apps │ ├── browser │ │ └── src │ │ └── background │ │ ├── message-handler.ts │ │ └── websocket.ts │ └── editor │ └── src │ └── services │ └── websocket-manager.ts └── packages └── shared └── src ├── types │ └── websocket-message.ts └── utils └── logger.ts
1 parent 3cc3b2d commit 61d2ce9

2 files changed

Lines changed: 14 additions & 6 deletions

File tree

‎apps/browser/src/background/websocket.ts‎

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -101,7 +101,6 @@ export const connect_websocket = async (): Promise<void> => {
101101
}
102102
return
103103
}
104-
console.debug(message)
105104
handle_messages(message as WebSocketMessage)
106105
}
107106

@@ -127,10 +126,9 @@ export const connect_websocket = async (): Promise<void> => {
127126

128127
export const send_message_to_server = (message: any): boolean => {
129128
if (websocket?.readyState == WebSocket.OPEN) {
130-
console.debug('Sending message to server:', message)
131129
websocket.send(JSON.stringify(message))
132130
return true
133131
}
134-
console.warn('WebSocket not connected, cannot send message:', message)
132+
console.warn('WebSocket not connected, cannot send message')
135133
return false
136134
}

‎apps/editor/src/services/websocket-manager.ts‎

Lines changed: 13 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -231,7 +231,7 @@ export class WebSocketManager {
231231
Logger.info({
232232
function_name: '_connect_as_client',
233233
message: 'Incoming WS message',
234-
data: message
234+
data: { action: message.action }
235235
})
236236

237237
if (message.action == 'client-id-assignment') {
@@ -509,7 +509,12 @@ export class WebSocketManager {
509509
Logger.info({
510510
function_name: 'initialize_chat',
511511
message: 'Sending initialize chat message',
512-
data: message
512+
data: {
513+
url: message.url,
514+
model: message.model,
515+
target_browser_id: message.target_browser_id,
516+
client_id: message.client_id
517+
}
513518
})
514519

515520
this.client?.send(JSON.stringify(message))
@@ -615,7 +620,12 @@ export class WebSocketManager {
615620
Logger.info({
616621
function_name: 'preview_web_configuration',
617622
message: 'Sending preview web configuration message',
618-
data: message
623+
data: {
624+
url: message.url,
625+
model: message.model,
626+
target_browser_id: message.target_browser_id,
627+
client_id: message.client_id
628+
}
619629
})
620630

621631
this.client?.send(JSON.stringify(message))

0 commit comments

Comments
 (0)