Skip to content

Commit 19f51a9

Browse files
fix(amazonq): general code quality fixes for CodeReview and DisplayFindings tools
1 parent ec2d4fd commit 19f51a9

File tree

5 files changed

+9
-12
lines changed

5 files changed

+9
-12
lines changed

plugins/amazonq/chat/jetbrains-community/src/software/aws/toolkits/jetbrains/services/amazonq/webview/BrowserConnector.kt

Lines changed: 3 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -113,10 +113,8 @@ import software.aws.toolkits.jetbrains.services.amazonqCodeTest.auth.isCodeTestA
113113
import software.aws.toolkits.jetbrains.services.amazonqDoc.auth.isDocAvailable
114114
import software.aws.toolkits.jetbrains.services.amazonqFeatureDev.auth.isFeatureDevAvailable
115115
import software.aws.toolkits.jetbrains.services.codemodernizer.utils.isCodeTransformAvailable
116-
import software.aws.toolkits.jetbrains.services.codewhisperer.codescan.CodeLine
117116
import software.aws.toolkits.jetbrains.services.codewhisperer.codescan.CodeWhispererCodeScanIssue
118117
import software.aws.toolkits.jetbrains.services.codewhisperer.codescan.CodeWhispererCodeScanManager
119-
import software.aws.toolkits.jetbrains.services.codewhisperer.codescan.CodeWhispererCodeScanSession
120118
import software.aws.toolkits.jetbrains.services.codewhisperer.codescan.Description
121119
import software.aws.toolkits.jetbrains.services.codewhisperer.codescan.Recommendation
122120
import software.aws.toolkits.jetbrains.services.codewhisperer.codescan.SuggestedFix
@@ -585,7 +583,7 @@ class BrowserConnector(
585583
}
586584
}
587585

588-
data class flareCodeScanIssue(
586+
data class FlareCodeScanIssue(
589587
val startLine: Int,
590588
val endLine: Int,
591589
val comment: String?,
@@ -609,7 +607,7 @@ class BrowserConnector(
609607

610608
data class AggregatedCodeScanIssue(
611609
val filePath: String,
612-
val issues: List<flareCodeScanIssue>
610+
val issues: List<FlareCodeScanIssue>
613611
)
614612

615613
private fun showResult(
@@ -631,7 +629,7 @@ class BrowserConnector(
631629
val findingsMessage = additionalMessages?.find {message ->
632630
(message["messageId"] as String).endsWith(CODE_REVIEW_FINDINGS_SUFFIX)
633631
|| (message["messageId"] as String).endsWith(DISPLAY_FINDINGS_SUFFIX)}
634-
val scannedFiles = mutableListOf<VirtualFile>();
632+
val scannedFiles = mutableListOf<VirtualFile>()
635633
if (findingsMessage != null) {
636634
additionalMessages.remove(findingsMessage)
637635
val gson = Gson()

plugins/amazonq/chat/jetbrains-community/src/software/aws/toolkits/jetbrains/services/cwc/commands/EditorContextCommand.kt

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -34,8 +34,8 @@ enum class EditorContextCommand(
3434
verb = "sendToPrompt",
3535
actionId = "aws.amazonq.sendToPrompt",
3636
),
37-
ExplainCodeScanIssue(
38-
verb = "ExplainIssue",
39-
actionId = "aws.amazonq.explainCodeScanIssue",
37+
HandleCodeScanIssue(
38+
verb = "HandleCodeScanIssue",
39+
actionId = "aws.amazonq.handleCodeScanIssueCommand",
4040
),
4141
}

plugins/amazonq/chat/jetbrains-community/src/software/aws/toolkits/jetbrains/services/cwc/controller/chat/userIntent/UserIntentRecognizer.kt

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -15,7 +15,7 @@ class UserIntentRecognizer {
1515
EditorContextCommand.Refactor -> UserIntent.SUGGEST_ALTERNATE_IMPLEMENTATION
1616
EditorContextCommand.Fix -> UserIntent.APPLY_COMMON_BEST_PRACTICES
1717
EditorContextCommand.Optimize -> UserIntent.IMPROVE_CODE
18-
EditorContextCommand.ExplainCodeScanIssue -> UserIntent.EXPLAIN_CODE_SELECTION
18+
EditorContextCommand.HandleCodeScanIssue -> UserIntent.EXPLAIN_CODE_SELECTION
1919
EditorContextCommand.GenerateUnitTests -> UserIntent.GENERATE_UNIT_TESTS
2020
EditorContextCommand.SendToPrompt -> null
2121
}

plugins/amazonq/codewhisperer/jetbrains-community/src/software/aws/toolkits/jetbrains/services/codewhisperer/codescan/utils/CodeWhispererCodeScanIssueUtils.kt

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -8,7 +8,6 @@ import com.intellij.diff.DiffContentFactory
88
import com.intellij.diff.DiffManager
99
import com.intellij.diff.requests.SimpleDiffRequest
1010
import com.intellij.diff.util.DiffUserDataKeys
11-
import com.intellij.docker.agent.util.toJson
1211
import com.intellij.openapi.actionSystem.ActionManager
1312
import com.intellij.openapi.actionSystem.AnActionEvent
1413
import com.intellij.openapi.actionSystem.CommonDataKeys

plugins/amazonq/shared/jetbrains-community/src/software/aws/toolkits/jetbrains/services/amazonq/lsp/model/aws/chat/ChatMessage.kt

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -110,5 +110,5 @@ enum class MessageType(@JsonValue val repr: String) {
110110
TOOL("tool"),
111111
}
112112

113-
val CODE_REVIEW_FINDINGS_SUFFIX = "_codeReviewFindings"
114-
val DISPLAY_FINDINGS_SUFFIX = "_displayFindings"
113+
const val CODE_REVIEW_FINDINGS_SUFFIX = "_codeReviewFindings"
114+
const val DISPLAY_FINDINGS_SUFFIX = "_displayFindings"

0 commit comments

Comments
 (0)