fix cli stderr bug

graphql · Jul 10, 2022 · 9f8cedc · 9f8cedc
1 parent 2074061
commit 9f8cedc
Show file tree

Hide file tree

Showing 3 changed files with 4 additions and 5 deletions.
diff --git a/packages/graphql-language-service-cli/src/cli.ts b/packages/graphql-language-service-cli/src/cli.ts
@@ -110,10 +110,11 @@ if (!command) {
 switch (command) {
   case 'server':
     process.on('uncaughtException', error => {
-      process.stdout.write(
+      process.stderr.write(
         'An error was thrown from GraphQL language service: ' + String(error),
       );
-      process.exit(0);
+      // don't exit at all if there is an uncaughtException
+      // process.exit(0);
     });
 
     const options: { [key: string]: any } = {};

diff --git a/packages/graphql-language-service/src/interface/__tests__/getAutocompleteSuggestions-test.ts b/packages/graphql-language-service/src/interface/__tests__/getAutocompleteSuggestions-test.ts
@@ -126,11 +126,8 @@ describe('getAutocompleteSuggestions', () => {
       { label: 'skip' },
     ];
 
-    console.log({ graphQLVersion });
-
     // TODO: remove this once defer and stream are merged to `graphql`
     if (graphQLVersion.startsWith('16.0.0-experimental-stream-defer')) {
-      console.log('expect stream');
       expectedDirectiveSuggestions.push({ label: 'stream' }, { label: 'test' });
     } else {
       expectedDirectiveSuggestions.push({ label: 'test' });

diff --git a/packages/vscode-graphql/src/apis/statusBar.ts b/packages/vscode-graphql/src/apis/statusBar.ts
@@ -6,6 +6,7 @@ import {
   ThemeColor,
   version,
 } from 'vscode';
+
 import { LanguageClient, State } from 'vscode-languageclient/node';
 
 enum Status {