fix: lsp server paths for windows, special chars

using the standard `vscode-uri` package, this should fix the parsing of file URIs by `graphql-language-service-server` in cases such as:

- windows without WSL
- special/unicode characters in filenames
- likely other cases

previously we were using the old approach of `URL(uri).pathname` which was not working!

this should fix issues with object and fragment type completion as well I think

packages/graphql-language-service-server/package.json

@@ -40,7 +40,9 @@
     "nullthrows": "^1.0.0",
     "vscode-jsonrpc": "^5.0.1",
     "vscode-languageserver": "^6.1.1",
-    "fast-glob": "^3.2.7"
+    "fast-glob": "^3.2.7",
+    "vscode-uri": "^3.0.2",
+    "glob": "^7.2.0"
   },
   "devDependencies": {
     "@types/mkdirp": "^1.0.1",

packages/graphql-language-service-server/src/GraphQLCache.ts

@@ -31,7 +31,8 @@ import { parseDocument } from './parseDocument';
 import stringToHash from './stringToHash';
 import glob from 'glob';
 import { LoadConfigOptions } from './types';
-import { fileURLToPath, pathToFileURL } from 'url';
+import { pathToFileURL } from 'url';
+import { URI } from 'vscode-uri';
 
 // Maximum files to read when processing GraphQL files.
 const MAX_READS = 200;
@@ -106,7 +107,7 @@ export class GraphQLCache implements GraphQLCacheInterface {
   getGraphQLConfig = (): GraphQLConfig => this._graphQLConfig;
 
   getProjectForFile = (uri: string): GraphQLProjectConfig => {
-    return this._graphQLConfig.getProjectForFile(new URL(uri).pathname);
+    return this._graphQLConfig.getProjectForFile(URI.parse(uri).fsPath);
   };
 
   getFragmentDependencies = async (
@@ -776,7 +777,7 @@ export class GraphQLCache implements GraphQLCacheInterface {
    */
   promiseToReadGraphQLFile = (filePath: Uri): Promise<GraphQLFileInfo> => {
     return new Promise((resolve, reject) =>
-      fs.readFile(fileURLToPath(filePath), 'utf8', (error, content) => {
+      fs.readFile(URI.parse(filePath).fsPath, 'utf8', (error, content) => {
         if (error) {
           reject(error);
           return;

packages/graphql-language-service-server/src/MessageProcessor.ts

@@ -12,6 +12,7 @@ import { readFileSync, existsSync, writeFileSync, writeFile } from 'fs';
 import { pathToFileURL } from 'url';
 import * as path from 'path';
 import glob from 'fast-glob';
+import { URI } from 'vscode-uri';
 import {
   CachedContent,
   Uri,
@@ -67,6 +68,7 @@ import { tmpdir } from 'os';
 import { GraphQLExtensionDeclaration } from 'graphql-config';
 import type { LoadConfigOptions } from './types';
 import { promisify } from 'util';
+import { uriToFilePath } from 'vscode-languageserver/lib/files';
 
 const writeFileAsync = promisify(writeFile);
 
@@ -225,11 +227,10 @@ export class MessageProcessor {
       }, this._settings.load),
       rootDir,
     };
-
+    // reload the graphql cache
     this._graphQLCache = await getGraphQLCache({
       parser: this._parser,
-      loadConfigOptions: this._loadConfigOptions,
-      config: this._graphQLConfig,
+      loadConfigOptions: this._loadConfigOptions
     });
     this._languageService = new GraphQLLanguageService(this._graphQLCache);
     if (this._graphQLCache?.getGraphQLConfig) {
@@ -248,6 +249,7 @@ export class MessageProcessor {
     try {
       if (!this._isInitialized || !this._graphQLCache) {
         if (!this._settings) {
+          // then initial call to update graphql config
           await this._updateGraphQLConfig();
           this._isInitialized = true;
         } else {
@@ -283,6 +285,20 @@ export class MessageProcessor {
 
       await this._invalidateCache(textDocument, uri, contents);
     } else {
+      const configMatchers = [
+        'graphql.config',
+        'graphqlrc',
+        'package.json',
+        this._settings.load.fileName,
+      ].filter(Boolean);
+      if (
+        configMatchers.some(v => uri.match(v)?.length)
+      ) {
+        this._logger.info('updating graphql config');
+        this._updateGraphQLConfig();
+        return { uri, diagnostics: [] }
+      }
+      // update graphql config only when graphql config is saved!
       const cachedDocument = this._getCachedDocument(textDocument.uri);
       if (cachedDocument) {
         contents = cachedDocument.contents;
@@ -567,25 +583,13 @@ export class MessageProcessor {
       params.changes.map(async (change: FileEvent) => {
         if (!this._isInitialized || !this._graphQLCache) {
           throw Error('No cache available for handleWatchedFilesChanged');
-        }
-        // update when graphql config changes!
-        const configMatchers = [
-          'graphql.config',
-          'graphqlrc',
-          this._settings.load.fileName,
-        ].filter(Boolean);
-        if (configMatchers.some(v => change.uri.match(v)?.length)) {
-          this._logger.info('updating graphql config');
-          this._updateGraphQLConfig();
-        }
-
-        if (
+        } else if (
           change.type === FileChangeTypeKind.Created ||
           change.type === FileChangeTypeKind.Changed
         ) {
           const uri = change.uri;
 
-          const text = readFileSync(new URL(uri).pathname).toString();
+          const text = readFileSync(URI.parse(uri).fsPath, 'utf-8');
           const contents = this._parser(text, uri);
 
           await this._updateFragmentDefinition(uri, contents);

yarn.lock

@@ -5102,10 +5102,10 @@
   resolved "https://registry.yarnpkg.com/@types/minimist/-/minimist-1.2.1.tgz#283f669ff76d7b8260df8ab7a4262cc83d988256"
   integrity sha512-fZQQafSREFyuZcdWFAExYjBiCL7AUCdgsk80iO0q4yihYYdcIiH28CcuPTGFgLOCC8RlW49GSQxdHwZP+I7CNg==
 
-"@types/mkdirp@^1.0.1":
-  version "1.0.1"
-  resolved "https://registry.yarnpkg.com/@types/mkdirp/-/mkdirp-1.0.1.tgz#0930b948914a78587de35458b86c907b6e98bbf6"
-  integrity sha512-HkGSK7CGAXncr8Qn/0VqNtExEE+PHMWb+qlR1faHMao7ng6P3tAaoWWBMdva0gL5h4zprjIO89GJOLXsMcDm1Q==
+"@types/mkdirp@^1.0.`1":
+  version "1.0.2"
+  resolved "https://registry.yarnpkg.com/@types/mkdirp/-/mkdirp-1.0.2.tgz#8d0bad7aa793abe551860be1f7ae7f3198c16666"
+  integrity sha512-o0K1tSO0Dx5X6xlU5F1D6625FawhC3dU3iqr25lluNv/+/QIVH8RLNEiVokgIZo+mz+87w/3Mkg/VvQS+J51fQ==
   dependencies:
     "@types/node" "*"
 
@@ -10784,6 +10784,18 @@ glob@^7.0.0, glob@^7.0.3, glob@^7.0.5, glob@^7.1.1, glob@^7.1.2, glob@^7.1.3, gl
     once "^1.3.0"
     path-is-absolute "^1.0.0"
 
+glob@^7.2.0:
+  version "7.2.0"
+  resolved "https://registry.yarnpkg.com/glob/-/glob-7.2.0.tgz#d15535af7732e02e948f4c41628bd910293f6023"
+  integrity sha512-lmLf6gtyrPq8tTjSmrO94wBeQbFR3HbLHbuyD69wuyQkImp2hWqMGB47OX65FBkPffO641IP9jWa1z4ivqG26Q==
+  dependencies:
+    fs.realpath "^1.0.0"
+    inflight "^1.0.4"
+    inherits "2"
+    minimatch "^3.0.4"
+    once "^1.3.0"
+    path-is-absolute "^1.0.0"
+
 global-dirs@^2.0.1:
   version "2.1.0"
   resolved "https://registry.yarnpkg.com/global-dirs/-/global-dirs-2.1.0.tgz#e9046a49c806ff04d6c1825e196c8f0091e8df4d"
@@ -11003,16 +11015,16 @@ grapheme-splitter@^1.0.4:
   integrity sha512-bzh50DW9kTPM00T8y4o8vQg89Di9oLJVLW/KaOGIXJWP/iqCN6WKYkbNOF04vFLJhwcpYUh9ydh/+5vpOqV4YQ==
 
 "graphiql@file:packages/graphiql":
-  version "1.5.5"
+  version "1.5.7"
   dependencies:
     "@graphiql/toolkit" "^0.4.2"
     codemirror "^5.58.2"
-    codemirror-graphql "^1.2.3"
+    codemirror-graphql "^1.2.4"
     copy-to-clipboard "^3.2.0"
     dset "^3.1.0"
     entities "^2.0.0"
     escape-html "^1.0.3"
-    graphql-language-service "^3.2.3"
+    graphql-language-service "^3.2.4"
     markdown-it "^12.2.0"
 
 graphql-config@^4.1.0:
@@ -19949,6 +19961,11 @@ vscode-languageserver@^6.1.1:
   dependencies:
     vscode-languageserver-protocol "^3.15.3"
 
+vscode-uri@^3.0.2:
+  version "3.0.2"
+  resolved "https://registry.yarnpkg.com/vscode-uri/-/vscode-uri-3.0.2.tgz#ecfd1d066cb8ef4c3a208decdbab9a8c23d055d0"
+  integrity sha512-jkjy6pjU1fxUvI51P+gCsxg1u2n8LSt0W6KrCNQceaziKzff74GoWmjVG46KieVzybO1sttPQmYfrwSHey7GUA==
+
 w3c-hr-time@^1.0.1, w3c-hr-time@^1.0.2:
   version "1.0.2"
   resolved "https://registry.yarnpkg.com/w3c-hr-time/-/w3c-hr-time-1.0.2.tgz#0a89cdf5cc15822df9c360543676963e0cc308cd"