From 549a4a2744799ac82291d8fe433dc23b8fd746be Mon Sep 17 00:00:00 2001 From: hkobew Date: Wed, 7 May 2025 18:50:03 -0400 Subject: [PATCH 1/3] feat: add error code to invalid node error --- packages/core/src/shared/lsp/utils/platform.ts | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/packages/core/src/shared/lsp/utils/platform.ts b/packages/core/src/shared/lsp/utils/platform.ts index 8b775433277..65dd91e1ddd 100644 --- a/packages/core/src/shared/lsp/utils/platform.ts +++ b/packages/core/src/shared/lsp/utils/platform.ts @@ -40,7 +40,7 @@ export async function validateNodeExe(nodePath: string[], lsp: string, args: str if (!ok) { const msg = `failed to run basic "node -e" test (exitcode=${r.exitCode}): ${proc.toString(false, true)}` logger.error(msg) - throw new ToolkitError(`amazonqLsp: ${msg}`) + throw new ToolkitError(`amazonqLsp: ${msg}`, { code: 'InvalidNode' }) } // Check that we can start `node …/lsp.js --stdio …`. From 2ccac57889753d3e13f536655096a584ec51663e Mon Sep 17 00:00:00 2001 From: hkobew Date: Wed, 7 May 2025 18:54:20 -0400 Subject: [PATCH 2/3] feat: add error code when lsp check fails --- packages/core/src/shared/lsp/utils/platform.ts | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/packages/core/src/shared/lsp/utils/platform.ts b/packages/core/src/shared/lsp/utils/platform.ts index 65dd91e1ddd..11290a38305 100644 --- a/packages/core/src/shared/lsp/utils/platform.ts +++ b/packages/core/src/shared/lsp/utils/platform.ts @@ -68,7 +68,8 @@ export async function validateNodeExe(nodePath: string[], lsp: string, args: str }) if (!ok2 || selfExit) { throw new ToolkitError( - `amazonqLsp: failed to run (exitcode=${lspProc.exitCode()}): ${lspProc.toString(false, true)}` + `amazonqLsp: failed to run (exitcode=${lspProc.exitCode()}): ${lspProc.toString(false, true)}`, + { code: 'InvalidLSP' } ) } } finally { From 2992fcb369d0ee6ff41714cda063547664b85e0e Mon Sep 17 00:00:00 2001 From: hkobew Date: Wed, 7 May 2025 18:56:24 -0400 Subject: [PATCH 3/3] refactor: change error codes --- packages/core/src/shared/lsp/utils/platform.ts | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/packages/core/src/shared/lsp/utils/platform.ts b/packages/core/src/shared/lsp/utils/platform.ts index 11290a38305..2555793ceb5 100644 --- a/packages/core/src/shared/lsp/utils/platform.ts +++ b/packages/core/src/shared/lsp/utils/platform.ts @@ -40,7 +40,7 @@ export async function validateNodeExe(nodePath: string[], lsp: string, args: str if (!ok) { const msg = `failed to run basic "node -e" test (exitcode=${r.exitCode}): ${proc.toString(false, true)}` logger.error(msg) - throw new ToolkitError(`amazonqLsp: ${msg}`, { code: 'InvalidNode' }) + throw new ToolkitError(`amazonqLsp: ${msg}`, { code: 'FailedToRunNode' }) } // Check that we can start `node …/lsp.js --stdio …`. @@ -69,7 +69,7 @@ export async function validateNodeExe(nodePath: string[], lsp: string, args: str if (!ok2 || selfExit) { throw new ToolkitError( `amazonqLsp: failed to run (exitcode=${lspProc.exitCode()}): ${lspProc.toString(false, true)}`, - { code: 'InvalidLSP' } + { code: 'FailedToStartLanguageServer' } ) } } finally {