refactor: reuse constant from processUtils

Author: Hweinstock

packages/amazonq/src/lsp/client.ts

@@ -33,6 +33,7 @@ import {
     undefinedIfEmpty,
     getOptOutPreference,
 } from 'aws-core-vscode/shared'
+import { processUtils } from 'aws-core-vscode/shared'
 import { activate } from './chat/activation'
 import { AmazonQResourcePaths } from './lspInstaller'
 import { ConfigSection, isValidConfigSection, toAmazonQLSPLogLevel } from './config'
@@ -55,7 +56,7 @@ export async function startLanguageServer(
         '--pre-init-encryption',
         '--set-credentials-encryption-key',
     ]
-    const memoryWarnThreshold = 200 * 1024 * 1024 // 200 MB
+    const memoryWarnThreshold = 200 * processUtils.oneMB // 200 MB
     const serverOptions = createServerOptions({
         encryptionKey,
         executable: resourcePaths.node,

packages/core/src/amazonq/lsp/lspClient.ts

@@ -8,6 +8,7 @@
  *  Licensed under the MIT License. See License.txt in the project root for license information.
  */
 import * as vscode from 'vscode'
+import { oneMB } from '../../shared/utilities/processUtils'
 import * as path from 'path'
 import * as nls from 'vscode-nls'
 import * as crypto from 'crypto'
@@ -252,7 +253,7 @@ export async function activate(extensionContext: ExtensionContext, resourcePaths
     }
 
     const serverModule = resourcePaths.lsp
-    const memoryWarnThreshold = 600 * 1024 * 1024 // 600 MB
+    const memoryWarnThreshold = 600 * oneMB // 600 MB
 
     const serverOptions = createServerOptions({
         encryptionKey: key,

packages/core/src/shared/utilities/processUtils.ts

@@ -89,7 +89,7 @@ export class ChildProcessTracker {
         this.#pids = new PollingSet(ChildProcessTracker.pollingInterval, () => this.monitor())
     }
 
-    private getThreshold(pid: number) {
+    private getThreshold(pid: number): ProcessStats {
         if (!this.#processByPid.has(pid)) {
             ChildProcessTracker.logOnce(pid, `Missing process with id ${pid}, returning default threshold`)
             return defaultProcessWarnThresholds
@@ -126,10 +126,13 @@ export class ChildProcessTracker {
         if (stats) {
             ChildProcessTracker.logger.debug(`Process ${pid} usage: %O`, stats)
             if (stats.memory > threshold.memory) {
-                ChildProcessTracker.logOnce(pid, `Process ${pid} exceeded memory threshold: ${stats.memory / oneMB} MB`)
+                ChildProcessTracker.logOnce(
+                    pid,
+                    `Process ${pid} exceeded memory threshold: ${(stats.memory / oneMB).toFixed(2)} MB`
+                )
             }
             if (stats.cpu > threshold.cpu) {
-                ChildProcessTracker.logOnce(pid, `Process ${pid} exceeded cpu threshold: %${stats.cpu}`)
+                ChildProcessTracker.logOnce(pid, `Process ${pid} exceeded cpu threshold: ${stats.cpu}%`)
             }
         }
     }