Fixed Uri length

[armanio123]

documentUriToFileName was returning an incorrect when the Uri contained a drive letter. ie::

file:///C:/Folder1/file.ts --> C:/lder1/file.ts

[Copilot]

Pull Request Overview

This PR fixes the issue where documentUriToFileName mis-calculates the file name when the URI contains a drive letter. The change adjusts the offset calculation to correctly handle drive-letter URIs.

• Adjusted the substring offset from len(possibleDrive)+3 to len(possibleDrive)+1.

Comments suppressed due to low confidence (1)

internal/lsp/converters.go:156

• Consider adding unit tests for documentUriToFileName to cover various drive letter scenarios, ensuring the offset computation works correctly for all expected URI formats.

return possibleDrive + path[len(possibleDrive)+1:]

Tip: Copilot only keeps its highest confidence comments to reduce noise and keep you focused. Learn more

[jakebailey]

We should really add some tests for these converters 😄

[Mustafiz04]

Agree with @jakebailey to have a test for this.

[armanio123]

@jakebailey Agree, I'll see to adding some tests.