From e401e00e38dd27044655b435376dd7d3a08d3cff Mon Sep 17 00:00:00 2001 From: nighca Date: Thu, 23 Jan 2025 14:49:48 +0800 Subject: [PATCH] remove unnecessary logs for LS --- tools/spxls/client.ts | 16 ++++++---------- 1 file changed, 6 insertions(+), 10 deletions(-) diff --git a/tools/spxls/client.ts b/tools/spxls/client.ts index 6bd9bb014..1f7fbf829 100644 --- a/tools/spxls/client.ts +++ b/tools/spxls/client.ts @@ -75,10 +75,7 @@ export class Spxlc { */ request(method: string, params?: any): Promise { const id = this.nextRequestId++ - /* eslint-disable no-console */ - // TODO: remove debug logs https://github.com/goplus/builder/issues/1255 - console.debug(`[${id}][${method}] params:`, params) - const sendAt = Date.now() + const sendAt = performance.now() return new Promise((resolve, reject) => { const message: RequestMessage = { jsonrpc: '2.0', @@ -94,15 +91,14 @@ export class Spxlc { } }).then( result => { - const time = Date.now() - sendAt - console.debug(`[${id}][${method}] took ${time}ms`) - console.debug(`[${id}][${method}] result:`, result) + if (process.env.NODE_ENV === 'development') { + const time = performance.now() - sendAt + if (time > 50) console.warn(`[LSP] ${method} took ${Math.round(time)}ms, params:`, params) + } return result }, err => { - const time = Date.now() - sendAt - console.debug(`[${id}][${method}] took ${time}ms`) - console.debug(`[${id}][${method}] error:`, err) + console.warn(`[LSP] ${method} error:`, err, ', params:', params) throw err } )