feat(amazonQFlareChat): Set up Flare chat connection #5545
Conversation
Qodana Community for JVM3 new problems were found
💡 Qodana analysis was run in the pull request mode: only the changed files were checked Contact Qodana teamContact us at [email protected]
|
![github-advanced-security[bot]](https://avatars.githubusercontent.com/in/57789?s=60&v=4){kind=small}
.../jetbrains-community/src/software/aws/toolkits/jetbrains/services/amazonq/webview/Browser.kt
Fixed
Show fixed
Hide fixed
...s-community/src/software/aws/toolkits/jetbrains/services/amazonq/webview/BrowserConnector.kt
Fixed
Show fixed
Hide fixed
...s-community/src/software/aws/toolkits/jetbrains/services/amazonq/webview/BrowserConnector.kt
Fixed
Show fixed
Hide fixed
...c/software/aws/toolkits/jetbrains/services/amazonq/lsp/flareChat/ChatCommunicationManager.kt
Fixed
Show fixed
Hide fixed
|
|
||
| package software.aws.toolkits.jetbrains.services.amazonq.lsp.model.aws.chat | ||
|
|
||
| data class ChatResult( |
Check warning
Code scanning / QDJVMC
Unused symbol Warning
| val end: Int, | ||
| ) | ||
|
|
||
| data class EncryptedChatResult( |
Check warning
Code scanning / QDJVMC
Unused symbol Warning
…Flare' into manodnyb/setupChatComponentsWithFlare
manodnyab
changed the title
| Range( | ||
| Position( | ||
| 0, | ||
| 0 | ||
| ), | ||
| Position( | ||
| 1, | ||
| 1 | ||
| ) |
There was a problem hiding this comment.
Adding this as a follow-up
| project.messageBus.connect().subscribe( | ||
| AsyncChatUiListener.TOPIC, | ||
| object : AsyncChatUiListener { | ||
| override fun onChange(message: String) { | ||
| runInEdt { | ||
| chatPanel.browser?.postChat(message) | ||
| } | ||
| } | ||
| } | ||
| ) |
There was a problem hiding this comment.
Discussed offline. This will involve moving the toolwindow to shared. Will address this as a part of a new PR
| companion object { | ||
| private const val WEB_SCRIPT_URI = "http://mynah/js/mynah-ui.js" | ||
| // TODO: Switch this to respect the overriden paths too | ||
| private val WEB_SCRIPT_URI = ArtifactHelper().getLatestLocalLspArtifact().resolve("amazonq-ui.js").toUri() |
There was a problem hiding this comment.
this will block class loading until we download the lsp artifacts, which is not what we want
There was a problem hiding this comment.
Converted this to a loading screen while the resource is being fetched. Will happen on the creation of a new browser
...s-community/src/software/aws/toolkits/jetbrains/services/amazonq/webview/BrowserConnector.kt
Outdated
Show resolved
Hide resolved
...s-community/src/software/aws/toolkits/jetbrains/services/amazonq/webview/BrowserConnector.kt
Outdated
Show resolved
Hide resolved
...munity/src/software/aws/toolkits/jetbrains/services/amazonq/lsp/AmazonQLanguageClientImpl.kt
Fixed
Show fixed
Hide fixed
manodnyab
changed the title
| import com.fasterxml.jackson.databind.SerializationFeature | ||
| import com.fasterxml.jackson.datatype.jsr310.JavaTimeModule | ||
| import com.fasterxml.jackson.module.kotlin.jacksonObjectMapper | ||
| import com.fasterxml.jackson.module.kotlin.treeToValue |
Check warning
Code scanning / QDJVMC
Unused import directive Warning
…Flare' into manodnyb/setupChatComponentsWithFlare
|
/retryBuilds |
2 participants
Types of changes
Description
Checklist
License
I confirm that my contribution is made under the terms of the Apache 2.0 license.