From cf77a8ee5d85e0d084548f06f4382f26809c7b93 Mon Sep 17 00:00:00 2001
From: Satyaki Ghosh <satyakig@amazon.com>
Date: Thu, 11 Dec 2025 16:29:12 -0500
Subject: [PATCH] feat(cloudformation): Make lsp artifact hash algorithm
 configurable

---
 .../awsService/cloudformation/lsp-server/lspInstaller.ts   | 3 ++-
 packages/core/src/shared/lsp/baseLspInstaller.ts           | 6 ++++--
 packages/core/src/shared/lsp/lspResolver.ts                | 7 +++----
 3 files changed, 9 insertions(+), 7 deletions(-)

diff --git a/packages/core/src/awsService/cloudformation/lsp-server/lspInstaller.ts b/packages/core/src/awsService/cloudformation/lsp-server/lspInstaller.ts
index 2d9c811af55..fa54beba44b 100644
--- a/packages/core/src/awsService/cloudformation/lsp-server/lspInstaller.ts
+++ b/packages/core/src/awsService/cloudformation/lsp-server/lspInstaller.ts
@@ -73,7 +73,8 @@ export class CfnLspInstaller extends BaseLspInstaller {
                         throw error
                     }
                 },
-            } as any
+            } as any,
+            'sha256'
         )
     }
 
diff --git a/packages/core/src/shared/lsp/baseLspInstaller.ts b/packages/core/src/shared/lsp/baseLspInstaller.ts
index 8ba85ebe1e1..d2a335abe24 100644
--- a/packages/core/src/shared/lsp/baseLspInstaller.ts
+++ b/packages/core/src/shared/lsp/baseLspInstaller.ts
@@ -27,7 +27,8 @@ export abstract class BaseLspInstaller<T extends ResourcePaths = ResourcePaths,
     constructor(
         protected config: Config,
         loggerName: Extract<LogTopic, 'amazonqLsp' | 'amazonqWorkspaceLsp' | 'awsCfnLsp'>,
-        private manifestResolver?: ManifestResolver
+        private manifestResolver?: ManifestResolver,
+        private readonly hashAlgorithm: string = 'sha384'
     ) {
         this.logger = getLogger(loggerName)
     }
@@ -56,7 +57,8 @@ export abstract class BaseLspInstaller<T extends ResourcePaths = ResourcePaths,
                 includePrerelease: true,
             }),
             manifestUrl,
-            this.downloadMessageOverride
+            this.downloadMessageOverride,
+            this.hashAlgorithm
         ).resolve()
 
         const assetDirectory = installationResult.assetDirectory
diff --git a/packages/core/src/shared/lsp/lspResolver.ts b/packages/core/src/shared/lsp/lspResolver.ts
index f5c6e5a340c..f4086333ecd 100644
--- a/packages/core/src/shared/lsp/lspResolver.ts
+++ b/packages/core/src/shared/lsp/lspResolver.ts
@@ -34,7 +34,7 @@ export class LanguageServerResolver {
          * Custom message to show user when downloading, if undefined it will use the default.
          */
         downloadMessage?: string,
-        private readonly _defaultDownloadFolder?: string
+        private readonly hashAlgorithm: string = 'sha384'
     ) {
         this.downloadMessage = downloadMessage ?? `Updating '${this.lsName}' language server`
     }
@@ -294,7 +294,7 @@ export class LanguageServerResolver {
                     return [{ filename: fetchResult.filename, data }]
                 }
 
-                const hash = createHash('sha384', data)
+                const hash = createHash(this.hashAlgorithm, data)
                 if (hash === fetchResult.hash) {
                     return [{ filename: fetchResult.filename, data }]
                 }
@@ -501,7 +501,6 @@ export class LanguageServerResolver {
     }
 
     private getDownloadDirectory(version: string) {
-        const directory = this._defaultDownloadFolder ?? this.defaultDownloadFolder()
-        return `${directory}/${version}`
+        return `${this.defaultDownloadFolder()}/${version}`
     }
 }