fix(intellij): handle square brackets in file paths for autocomplete

[Tarasusrus]

Use File.toURI() instead of URI constructor for Windows two-slash
file:// URIs to properly percent-encode special characters like [ ].
Fixes #10978.

---

Summary by cubic

Fixes URI parsing for file paths with square brackets to prevent IntelliJ autocomplete crashes on Windows two‑slash file URIs. Builds Windows file://C:/... URIs via File.toURI() so [ and ] are percent‑encoded.

• Bug Fixes
  • Update UriUtils to handle Windows two‑slash file:// URIs using File.toURI() instead of URI(...).
  • Add regression tests for Unix and Windows; make the Windows-style test cross‑platform by asserting bracket encoding and path structure without drive‑letter assumptions.

^{Written for commit a00c389. Summary will update on new commits.}

[github-actions]

All contributors have signed the CLA ✍️ ✅
_{Posted by the CLA Assistant Lite bot.}

[Tarasusrus]

I have read the CLA Document and I hereby sign the CLA.

[cubic-dev-ai]

2 issues found across 2 files

Prompt for AI agents (unresolved issues)

Check if these issues are valid — if so, understand the root cause of each and fix them. If appropriate, use sub-agents to investigate and fix each issue separately.


<file name="extensions/intellij/src/main/kotlin/com/github/continuedev/continueintellijextension/continue/UriUtils.kt">

<violation number="1" location="extensions/intellij/src/main/kotlin/com/github/continuedev/continueintellijextension/continue/UriUtils.kt:30">
P1: `File(path).toURI()` reparses `file://` URI text as a host-local path, causing cross-platform URI corruption and double-encoding of already escaped segments.</violation>
</file>

<file name="extensions/intellij/src/test/kotlin/com/github/continuedev/continueintellijextension/unit/UriUtilsTest.kt">

<violation number="1" location="extensions/intellij/src/test/kotlin/com/github/continuedev/continueintellijextension/unit/UriUtilsTest.kt:85">
P2: Windows regression test assertions are too weak to catch drive-letter or bracket-handling regressions in `file://C:/...` URIs.</violation>
</file>

---

Since this is your first cubic review, here's how it works:

• cubic automatically reviews your code and comments on bugs and improvements
• Teach cubic by replying to its comments. cubic learns from your replies and gets better over time
• Add one-off context when rerunning by tagging @cubic-dev-ai with guidance or docs links (including llms.txt)
• Ask questions if you need clarification on any suggestion

_{Reply with feedback, questions, or to request a fix. Tag @cubic-dev-ai to re-run a review.}

[cubic-dev-ai]

P1: File(path).toURI() reparses file:// URI text as a host-local path, causing cross-platform URI corruption and double-encoding of already escaped segments.

Prompt for AI agents

Check if this issue is valid — if so, understand the root cause and fix it. At extensions/intellij/src/main/kotlin/com/github/continuedev/continueintellijextension/continue/UriUtils.kt, line 30:

<comment>`File(path).toURI()` reparses `file://` URI text as a host-local path, causing cross-platform URI corruption and double-encoding of already escaped segments.</comment>

<file context>
@@ -23,10 +23,11 @@ object UriUtils {
         if (uriStr.startsWith("file://") && !uriStr.startsWith("file:///")) {
             val path = uriStr.substringAfter("file://")
-            return URI("file:///$path")
+            return File(path).toURI()
         }
 
</file context>

[cubic-dev-ai]

1 issue found across 1 file (changes from recent commits).

Prompt for AI agents (unresolved issues)

Check if these issues are valid — if so, understand the root cause of each and fix them. If appropriate, use sub-agents to investigate and fix each issue separately.


<file name="extensions/intellij/src/test/kotlin/com/github/continuedev/continueintellijextension/unit/UriUtilsTest.kt">

<violation number="1" location="extensions/intellij/src/test/kotlin/com/github/continuedev/continueintellijextension/unit/UriUtilsTest.kt:86">
P2: Test incorrectly expects percent-encoded brackets in `URI.path`; `path` is decoded for `java.net.URI`, so this assertion is invalid.</violation>
</file>

_{Reply with feedback, questions, or to request a fix. Tag @cubic-dev-ai to re-run a review.}

[cubic-dev-ai]

1 issue found across 1 file (changes from recent commits).

Prompt for AI agents (unresolved issues)

Check if these issues are valid — if so, understand the root cause of each and fix them. If appropriate, use sub-agents to investigate and fix each issue separately.


<file name="extensions/intellij/src/test/kotlin/com/github/continuedev/continueintellijextension/unit/UriUtilsTest.kt">

<violation number="1" location="extensions/intellij/src/test/kotlin/com/github/continuedev/continueintellijextension/unit/UriUtilsTest.kt:86">
P2: Windows bracket-path regression test was weakened and no longer asserts key Windows URI guarantees (encoding/preservation), allowing incorrect rewrites to pass.</violation>
</file>

_{Reply with feedback, questions, or to request a fix. Tag @cubic-dev-ai to re-run a review.}

[cubic-dev-ai]

P2: Windows bracket-path regression test was weakened and no longer asserts key Windows URI guarantees (encoding/preservation), allowing incorrect rewrites to pass.

Prompt for AI agents

Check if this issue is valid — if so, understand the root cause and fix it. At extensions/intellij/src/test/kotlin/com/github/continuedev/continueintellijextension/unit/UriUtilsTest.kt, line 86:

<comment>Windows bracket-path regression test was weakened and no longer asserts key Windows URI guarantees (encoding/preservation), allowing incorrect rewrites to pass.</comment>

<file context>
@@ -82,11 +82,12 @@ class UriUtilsTest : TestCase() {
-            parsed.path.contains("C:") || parsed.path.contains("c:"))
-        assertTrue("Path should preserve full directory structure",
+        // Brackets must be percent-encoded or absent as raw characters in a valid URI
+        assertFalse("Raw square brackets should not appear in URI path",
+            parsed.toString().contains("[") || parsed.toString().contains("]"))
+        assertTrue("Path should preserve directory structure",
</file context>

[RomneyDa]

@Tarasusrus could you check the cubic comment? seems potentially valid