Document links should be URIs, not file paths

see #14
grahambates · Jun 28, 2024 · da90df8 · da90df8
1 parent ac93b14
commit da90df8
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 1 deletion.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -7,6 +7,10 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ## [Unreleased]
 
+### Fixed
+
+- Document links should be URIs, not file paths [#14](https://github.com/grahambates/m68k-lsp/issues/14)
+
 ## [0.11.0] - 2024-06-28
 
 ### Added

diff --git a/server/src/providers/DocumentLinkProvider.ts b/server/src/providers/DocumentLinkProvider.ts
@@ -1,4 +1,5 @@
 import * as lsp from "vscode-languageserver";
+import { URI } from "vscode-uri";
 import { Provider } from ".";
 import { resolveInclude } from "../files";
 import { Context } from "../context";
@@ -22,7 +23,7 @@ export default class DocumentLinkProvider implements Provider {
     const { path, uri } = item.data;
     const resolved = await resolveInclude(uri, path, this.ctx);
     if (resolved) {
-      item.target = resolved;
+      item.target = URI.file(resolved).toString();
     }
     return item;
   }