fix: handle null cases and enable typescript strict mode

Author: Mayank1791989

src/@types/playlyfe__gql-language-server.d.ts

@@ -0,0 +1,3 @@
+declare module '@playlyfe/gql-language-server' {
+ export function findConfigFile(path: string): string;
+}

src/ClientStatusBarItem.ts

@@ -69,8 +69,6 @@ export default class ClientStatusBarItem {
  this._disposables.forEach(item => {
  item.dispose();
  });
- this._item = null;
- this._client = null;
  }
 
  private _registerStatusChangeListeners() {
@@ -101,11 +99,15 @@ export default class ClientStatusBarItem {
  return disposable;
  }
 
- private _updateVisibility = (textEditor: TextEditor) => {
+ private _updateVisibility = (textEditor: TextEditor | undefined) => {
  let hide = true;
 
  if (textEditor) {
- if (this._client.initializeResult) {
+ const workspaceFolder = workspace.getWorkspaceFolder(
+ textEditor.document.uri,
+ );
+
+ if (this._client.initializeResult && workspaceFolder) {
  // if client is initialized then show only for file extensions
  // defined in .gqlconfig
  // @TODO: if possible, match against patterns defined in .gqlconfig
@@ -115,7 +117,7 @@ export default class ClientStatusBarItem {
  {
  scheme: 'file',
  pattern: new RelativePattern(
- workspace.getWorkspaceFolder(textEditor.document.uri),
+ workspaceFolder,
  `**/*.{${extensions.join(',')}}`,
  ),
  },

src/extension.ts

@@ -39,14 +39,16 @@ export function activate(context: ExtensionContext) {
 export function deactivate(): Thenable<void> {
  const promises: Array<Thenable<void>> = [];
  clients.forEach(client => {
- promises.push(client.dispose());
+ if (client) {
+ promises.push(client.dispose());
+ }
  });
  return Promise.all(promises).then(() => undefined);
 }
 
 function createClientForWorkspaces() {
  const workspaceFolders = Workspace.workspaceFolders || [];
- const workspaceFoldersIndex = {};
+ const workspaceFoldersIndex: { [key: string]: boolean } = {};
 
  workspaceFolders.forEach(folder => {
  const key = folder.uri.toString();

src/tsconfig.json

@@ -5,6 +5,8 @@
  "moduleResolution": "node",
  "lib": [ "es6" ],
  "outDir": "../out",
- "sourceMap": true
+ "sourceMap": true,
+ "noImplicitAny": true,
+ "strictNullChecks": true
  }
 }