Potential Fix 1

sourcegraph · Aug 9, 2024 · 12b71b6 · 12b71b6
1 parent 1897c8b
commit 12b71b6
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 2 deletions.
diff --git a/src/main/kotlin/com/sourcegraph/cody/agent/protocol/ProtocolTextDocument.kt b/src/main/kotlin/com/sourcegraph/cody/agent/protocol/ProtocolTextDocument.kt
@@ -198,6 +198,7 @@ private constructor(
       return normalizeUriOrPath(uriString)
     }
 
+    //TODO: CODY-3223 - This should be moved to the protocol extension layer so that ALL uri's & paths are properly converted. Probably also means we want to alter the generated bindings to provide a hook or automatically generate the serializer.
     @JvmStatic
     fun normalizeUriOrPath(uriString: String): String {
       val hasScheme = uriString.startsWith("file://")

diff --git a/src/main/kotlin/com/sourcegraph/cody/inspections/CodyFixHighlightPass.kt b/src/main/kotlin/com/sourcegraph/cody/inspections/CodyFixHighlightPass.kt
@@ -88,7 +88,7 @@ class CodyFixHighlightPass(val file: PsiFile, val editor: Editor) :
                     location =
                         // TODO: Rik Nauta -- Got incorrect range; see QA report Aug 6 2024.
                         ProtocolLocation(
-                            uri = protocolTextDocument.uri,
+                            uri = ProtocolTextDocument.normalizeUriOrPath(protocolTextDocument.uri),
                             range = document.codyRange(it.startOffset, it.endOffset)),
                     code = it.problemGroup?.problemName)
               } catch (x: Exception) {
@@ -113,7 +113,7 @@ class CodyFixHighlightPass(val file: PsiFile, val editor: Editor) :
           }
           val location =
               ProtocolLocation(
-                  uri = protocolTextDocument.uri,
+                  uri = ProtocolTextDocument.normalizeUriOrPath(protocolTextDocument.uri),
                   range = document.codyRange(highlight.startOffset, highlight.endOffset))
           val provideResponse =
               agent.server