shiyue/PinchChat
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

refactor: replace console.log with debug logger in gateway client

Browse Source 
WebSocket debug logs are now silent by default. Enable with:
  localStorage.setItem('pinchchat:debug', '1')

Reduces console noise in production while keeping full debug
visibility available for developers.
This commit is contained in:
Nicolas Varrot
2026-02-13 06:41:26 +00:00
parent 6b0261f9c1
commit f05db6aa6d
1 changed files with 14 additions and 8 deletions
Download Patch File Download Diff File Expand all files Collapse all files

22
src/lib/gateway.ts
View File

@@ -1,5 +1,11 @@
import { genId } from './utils'; import { genId } from './utils';
/** Debug logger — enable with localStorage.setItem('pinchchat:debug', '1') */
const isDebug = () => {
try { return localStorage.getItem('pinchchat:debug') === '1'; } catch { return false; }
};
const log = (...args: unknown[]) => { if (isDebug()) console.log('[GW]', ...args); };
/** JSON-safe payload type used for gateway messages. */ /** JSON-safe payload type used for gateway messages. */
export type JsonPayload = Record<string, unknown>; export type JsonPayload = Record<string, unknown>;
@@ -57,12 +63,12 @@ export class GatewayClient {
this._onStatus('connecting'); this._onStatus('connecting');
this.ws = new WebSocket(this.wsUrl); this.ws = new WebSocket(this.wsUrl);
this.ws.onopen = () => { console.log('[GW] WS open'); }; this.ws.onopen = () => { log('WS open'); };
this.ws.onmessage = (ev) => { this.ws.onmessage = (ev) => {
let msg: GatewayMessage; let msg: GatewayMessage;
try { msg = JSON.parse(ev.data as string) as GatewayMessage; } catch { console.log('[GW] parse error', ev.data); return; } try { msg = JSON.parse(ev.data as string) as GatewayMessage; } catch { log('parse error', ev.data); return; }
console.log('[GW] msg:', msg.type, msg.event || msg.id || '', msg.ok); log('msg:', msg.type, msg.event || msg.id || '', msg.ok);
if (msg.type === 'event') { if (msg.type === 'event') {
if (msg.event === 'connect.challenge') { if (msg.event === 'connect.challenge') {
@@ -81,7 +87,7 @@ export class GatewayClient {
}; };
this.ws.onclose = (ev) => { this.ws.onclose = (ev) => {
console.log('[GW] WS close:', ev.code, ev.reason); log('WS close:', ev.code, ev.reason);
this.ws = null; this.ws = null;
this.connected = false; this.connected = false;
this._onStatus('disconnected'); this._onStatus('disconnected');
@@ -90,7 +96,7 @@ export class GatewayClient {
if (this.autoReconnect) this.scheduleReconnect(); if (this.autoReconnect) this.scheduleReconnect();
}; };
this.ws.onerror = (e) => { console.log('[GW] WS error', e); }; this.ws.onerror = (e) => { log('WS error', e); };
} }
private handleChallenge() { private handleChallenge() {
@@ -108,12 +114,12 @@ export class GatewayClient {
locale: navigator.language || 'en', locale: navigator.language || 'en',
userAgent: `pinchchat/${__APP_VERSION__}`, userAgent: `pinchchat/${__APP_VERSION__}`,
}).then((res) => { }).then((res) => {
console.log('[GW] connected!', res); log('connected!', res);
this.connected = true; this.connected = true;
this.reconnectAttempts = 0; this.reconnectAttempts = 0;
this._onStatus('connected'); this._onStatus('connected');
}).catch((err) => { }).catch((err) => {
console.log('[GW] connect failed:', err); log('connect failed:', err);
this.autoReconnect = false; this.autoReconnect = false;
this.disconnect(); this.disconnect();
}); });
@@ -125,7 +131,7 @@ export class GatewayClient {
const jitter = Math.random() * base * 0.3; const jitter = Math.random() * base * 0.3;
const delay = base + jitter; const delay = base + jitter;
this.reconnectAttempts++; this.reconnectAttempts++;
console.log(`[GW] reconnecting in ${Math.round(delay)}ms (attempt ${this.reconnectAttempts})`); log(`reconnecting in ${Math.round(delay)}ms (attempt ${this.reconnectAttempts})`);
this.reconnectTimer = setTimeout(() => { this.reconnectTimer = setTimeout(() => {
this.reconnectTimer = null; this.reconnectTimer = null;
this.connect(); this.connect();