Fixing lint issues

Author: LokeshDogga13

plugins/amazonq/chat/jetbrains-community/tst/software/aws/toolkits/jetbrains/services/amazonq/lsp/artifacts/ManifestFetcherTest.kt

@@ -3,7 +3,6 @@
 
 package software.aws.toolkits.jetbrains.services.amazonq.lsp.artifacts
 
-
 import io.mockk.every
 import io.mockk.spyk
 import io.mockk.verify
@@ -23,10 +22,8 @@ class ManifestFetcherTest {
         manifest = ManifestManager.Manifest()
     }
 
-
     @Test
     fun `should return null when both local and remote manifests are null`() {
-
         val fetchLocalManifestMock = spyk<ManifestFetcher>(recordPrivateCalls = true)
 
         every { fetchLocalManifestMock["fetchManifestFromLocal"]() } returns null
@@ -39,7 +36,6 @@ class ManifestFetcherTest {
 
     @Test
     fun `should return valid result from local should not execute remote method`() {
-
         val fetchLocalManifestMock = spyk<ManifestFetcher>(recordPrivateCalls = true)
 
         every { fetchLocalManifestMock["fetchManifestFromLocal"]() } returns manifest
@@ -51,7 +47,6 @@ class ManifestFetcherTest {
 
     @Test
     fun `should return valid result from remote`() {
-
         val fetchLocalManifestMock = spyk<ManifestFetcher>(recordPrivateCalls = true)
 
         every { fetchLocalManifestMock["fetchManifestFromLocal"]() } returns null

plugins/amazonq/shared/jetbrains-community/src/software/aws/toolkits/jetbrains/services/amazonq/lsp/artifacts/ManifestFetcher.kt

@@ -16,7 +16,6 @@ import software.aws.toolkits.jetbrains.services.amazonq.project.manifest.Manifes
 import java.nio.file.Path
 import java.nio.file.Paths
 
-
 class ManifestFetcher {
 
     private val lspManifestUrl = "https://aws-toolkit-language-servers.amazonaws.com/codewhisperer/0/manifest.json"
@@ -30,7 +29,7 @@ class ManifestFetcher {
     /**
      * Method which will be used to fetch latest manifest.
      * */
-    fun fetch() : ManifestManager.Manifest? {
+    fun fetch(): ManifestManager.Manifest? {
         val localManifest = fetchManifestFromLocal()
         if (localManifest != null) {
             return localManifest
@@ -39,13 +38,12 @@ class ManifestFetcher {
         return remoteManifest
     }
 
-    private fun fetchManifestFromRemote() : ManifestManager.Manifest? {
-        val manifest : ManifestManager.Manifest?
+    private fun fetchManifestFromRemote(): ManifestManager.Manifest? {
+        val manifest: ManifestManager.Manifest?
         try {
             val manifestString = getTextFromUrl(lspManifestUrl)
             manifest = manifestManager.readManifestFile(manifestString) ?: return null
-        }
-        catch (e: Exception) {
+        } catch (e: Exception) {
             logger.error("error fetching lsp manifest from remote URL ${e.message}", e)
             return null
         }
@@ -61,23 +59,21 @@ class ManifestFetcher {
     private fun updateManifestCache() {
         try {
             saveFileFromUrl(lspManifestUrl, lspManifestFilePath)
-        }
-        catch (e: Exception) {
+        } catch (e: Exception) {
             logger.error("error occurred while saving lsp manifest to local cache ${e.message}", e)
         }
     }
 
-    private fun fetchManifestFromLocal() : ManifestManager.Manifest? {
-        var manifest : ManifestManager.Manifest? = null
+    private fun fetchManifestFromLocal(): ManifestManager.Manifest? {
+        var manifest: ManifestManager.Manifest? = null
         val localETag = getManifestETagFromLocal()
         val remoteETag = getManifestETagFromUrl()
         // If local and remote have same ETag, we can re-use the manifest file from local to fetch artifacts.
         if (localETag != null && remoteETag != null && localETag == remoteETag) {
             try {
                 val manifestContent = lspManifestFilePath.readText()
                 manifest = manifestManager.readManifestFile(manifestContent) ?: return null
-            }
-            catch (e: Exception) {
+            } catch (e: Exception) {
                 logger.error("error reading lsp manifest file from local ${e.message}", e)
                 return null
             }
@@ -86,21 +82,20 @@ class ManifestFetcher {
         return manifest
     }
 
-    private fun getManifestETagFromLocal() : String? {
-        if(lspManifestFilePath.exists()) {
+    private fun getManifestETagFromLocal(): String? {
+        if (lspManifestFilePath.exists()) {
             val messageDigest = DigestUtil.md5()
             DigestUtil.updateContentHash(messageDigest, lspManifestFilePath)
             return StringUtil.toHexString(messageDigest.digest())
         }
         return null
     }
 
-    private fun getManifestETagFromUrl() : String? {
+    private fun getManifestETagFromUrl(): String? {
         try {
             val actualETag = getETagFromUrl(lspManifestUrl)
             return actualETag.trim('"')
-        }
-        catch (e: Exception) {
+        } catch (e: Exception) {
             logger.error("error fetching ETag of lsp manifest from url.", e)
         }
         return null

plugins/amazonq/shared/jetbrains-community/src/software/aws/toolkits/jetbrains/services/amazonq/project/manifest/ManifestManager.kt

@@ -44,21 +44,21 @@ class ManifestManager {
         @JsonProperty("name")
         val name: String? = null,
         @JsonProperty("version")
-        val version: String? = null
+        val version: String? = null,
     )
 
     data class Capabilities(
         @JsonProperty("name")
         val name: String? = null,
         @JsonProperty("version")
-        val version: String? = null
+        val version: String? = null,
     )
 
     data class Protocol(
         @JsonProperty("name")
-        val name: String?= null,
+        val name: String? = null,
         @JsonProperty("version")
-        val version: String? = null
+        val version: String? = null,
     )
 
     data class Version(

plugins/core/jetbrains-community/src/software/aws/toolkits/jetbrains/core/HttpUtils.kt

@@ -25,7 +25,7 @@ fun writeJsonToUrl(url: String, jsonString: String, indicator: ProgressIndicator
             request.readString(indicator)
         }
 
-fun getETagFromUrl(url: String) : String =
+fun getETagFromUrl(url: String): String =
     HttpRequests.head(url)
         .userAgent("AWS Toolkit for JetBrains")
         .connect { request ->