Skip to content

Commit 555359c

Browse files
committed
fix: crossplatform include URI check
1 parent 1c0b0fe commit 555359c

File tree

4 files changed

+19
-6
lines changed

4 files changed

+19
-6
lines changed

CHANGELOG.md

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,9 @@
11
# Change Log
22

3+
## [Unreleased]
4+
### Fixed
5+
- Windows url detection
6+
37
## [1.1.1] - 2023-01-03
48

59
### Fixed

server/package.json

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -8,6 +8,7 @@
88
"dependencies": {
99
"vscode-languageserver": "^8.0.2",
1010
"vscode-languageserver-textdocument": "^1.0.7",
11+
"vscode-uri": "^3.0.7",
1112
"web-tree-sitter": "^0.20.5"
1213
},
1314
"devDependencies": {

server/src/features/depsIndex.ts

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -3,12 +3,12 @@ import * as Parser from 'web-tree-sitter';
33
import { DocumentStore } from '../documentStore';
44
import { queryDirectives } from '../queries/directives';
55
import { Trees } from '../trees';
6-
import { normalize as normalizePath } from 'path';
6+
import { Utils, URI } from 'vscode-uri';
77

88
function resolvePathSegment(documentUri: string, path: string) {
99
let result = documentUri.split('/').slice(0, -1).join('/').substring('file://'.length);
1010

11-
return 'file://' + normalizePath(result + '/' + path);
11+
return Utils.joinPath(URI.parse(result), '/', path).toString();
1212
}
1313

1414
export class DepsIndex {

server/yarn.lock

Lines changed: 12 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1761,7 +1761,7 @@ __metadata:
17611761

17621762
"fsevents@patch:fsevents@^2.3.2#~builtin<compat/fsevents>":
17631763
version: 2.3.2
1764-
resolution: "fsevents@patch:fsevents@npm%3A2.3.2#~builtin<compat/fsevents>::version=2.3.2&hash=18f3a7"
1764+
resolution: "fsevents@patch:fsevents@npm%3A2.3.2#~builtin<compat/fsevents>::version=2.3.2&hash=df0bf1"
17651765
dependencies:
17661766
node-gyp: latest
17671767
conditions: os=darwin
@@ -3376,7 +3376,7 @@ __metadata:
33763376

33773377
"resolve@patch:resolve@^1.20.0#~builtin<compat/resolve>":
33783378
version: 1.22.1
3379-
resolution: "resolve@patch:resolve@npm%3A1.22.1#~builtin<compat/resolve>::version=1.22.1&hash=07638b"
3379+
resolution: "resolve@patch:resolve@npm%3A1.22.1#~builtin<compat/resolve>::version=1.22.1&hash=c3c19d"
33803380
dependencies:
33813381
is-core-module: ^2.9.0
33823382
path-parse: ^1.0.7
@@ -3466,6 +3466,7 @@ __metadata:
34663466
typescript: ^4.6.3
34673467
vscode-languageserver: ^8.0.2
34683468
vscode-languageserver-textdocument: ^1.0.7
3469+
vscode-uri: ^3.0.7
34693470
web-tree-sitter: ^0.20.5
34703471
languageName: unknown
34713472
linkType: soft
@@ -3865,11 +3866,11 @@ __metadata:
38653866

38663867
"typescript@patch:typescript@^4.6.3#~builtin<compat/typescript>":
38673868
version: 4.8.2
3868-
resolution: "typescript@patch:typescript@npm%3A4.8.2#~builtin<compat/typescript>::version=4.8.2&hash=f456af"
3869+
resolution: "typescript@patch:typescript@npm%3A4.8.2#~builtin<compat/typescript>::version=4.8.2&hash=3b564f"
38693870
bin:
38703871
tsc: bin/tsc
38713872
tsserver: bin/tsserver
3872-
checksum: 6f49363af8af2fe480da1d5fa68712644438785208b06690a3cbe5e7365fd652c3a0f1e587bc8684d78fb69de3dde4de185c0bad7bb4f3664ddfc813ce8caad6
3873+
checksum: 28e130a49d9330b64b8229ad336247bf980e9a705c5850f30a859a68c051e8e68e367b3275f119da4a825dca71ad0fe315964a1315d4a381c0021bca5eeeb62e
38733874
languageName: node
38743875
linkType: hard
38753876

@@ -3982,6 +3983,13 @@ __metadata:
39823983
languageName: node
39833984
linkType: hard
39843985

3986+
"vscode-uri@npm:^3.0.7":
3987+
version: 3.0.7
3988+
resolution: "vscode-uri@npm:3.0.7"
3989+
checksum: c899a0334f9f6ba53021328e083f6307978c09b94407d7e5fe86fcd8fcb8f1da0cb344123a335e55769055007a46d51aff83f9ee1dfc0296ee54b78f34ef0e4f
3990+
languageName: node
3991+
linkType: hard
3992+
39853993
"w3c-hr-time@npm:^1.0.2":
39863994
version: 1.0.2
39873995
resolution: "w3c-hr-time@npm:1.0.2"

0 commit comments

Comments
 (0)