Enable strictNullChecks in tsconfig.json

Author: daviwil

lib/adapters/apply-edit-adapter.ts

@@ -40,7 +40,7 @@ export default class ApplyEditAdapter {
       }),
     );
 
-    const checkpoints = [];
+    const checkpoints: Array<{ buffer: TextBuffer, checkpoint: number}> = [];
     try {
       for (let i = 0; i < editors.length; i++) {
         const editor = editors[i] as TextEditor;
@@ -52,7 +52,7 @@ export default class ApplyEditAdapter {
         const buffer = editor.getBuffer();
         const checkpoint = buffer.createCheckpoint();
         checkpoints.push({buffer, checkpoint});
-        let prevEdit = null;
+        let prevEdit: atomIde.TextEdit | null = null;
         for (const edit of edits) {
           ApplyEditAdapter.validateEdit(buffer, edit, prevEdit);
           buffer.setTextInRange(edit.oldRange, edit.newText);
@@ -74,9 +74,13 @@ export default class ApplyEditAdapter {
   }
 
   // Private: Do some basic sanity checking on the edit ranges.
-  private static validateEdit(buffer: TextBuffer, edit: atomIde.TextEdit, prevEdit: atomIde.TextEdit | null): void {
+  private static validateEdit(
+    buffer: TextBuffer,
+    edit: atomIde.TextEdit,
+    prevEdit: atomIde.TextEdit | null,
+  ): void {
     const path = buffer.getPath() || '';
-    if (prevEdit != null && edit.oldRange.end.compare(prevEdit.oldRange.start) > 0) {
+    if (prevEdit && edit.oldRange.end.compare(prevEdit.oldRange.start) > 0) {
       throw Error(`Found overlapping edit ranges in ${path}`);
     }
     const startRow = edit.oldRange.start.row;

lib/adapters/autocomplete-adapter.ts

@@ -267,11 +267,11 @@ export default class AutocompleteAdapter {
   // * `editor` An Atom {TextEditor} used to obtain the necessary text replacement.
   // * `suggestion` An {atom$AutocompleteSuggestion} to set the replacementPrefix and text properties of.
   public static applyTextEditToSuggestion(
-    textEdit: TextEdit | null,
+    textEdit: TextEdit | undefined,
     editor: TextEditor,
     suggestion: AutocompleteSuggestion,
   ): void {
-    if (textEdit != null) {
+    if (textEdit) {
       suggestion.replacementPrefix = editor.getTextInBufferRange(Convert.lsRangeToAtomRange(textEdit.range));
       suggestion.text = textEdit.newText;
     }
@@ -296,7 +296,7 @@ export default class AutocompleteAdapter {
   //
   // Returns a {String} containing the AutoComplete+ suggestion type equivalent
   // to the given completion kind.
-  public static completionKindToSuggestionType(kind: number | null): string {
+  public static completionKindToSuggestionType(kind: number | undefined): string {
     switch (kind) {
       case CompletionItemKind.Constant:
         return 'constant';

lib/adapters/code-action-adapter.ts

@@ -27,7 +27,7 @@ export default class CodeActionAdapter {
   public static async getCodeActions(
     connection: LanguageClientConnection,
     serverCapabilities: ServerCapabilities,
-    linterAdapter: LinterPushV2Adapter | null,
+    linterAdapter: LinterPushV2Adapter | undefined ,
     editor: TextEditor,
     range: Range,
     diagnostics: atomIde.Diagnostic[],

lib/adapters/datatip-adapter.ts

@@ -88,5 +88,8 @@ export default class DatatipAdapter {
         value: markedString.value,
       };
     }
+
+    // Catch-all case
+    return { type: 'markdown', value: markedString.toString() };
   }
 }

lib/adapters/document-sync-adapter.ts

@@ -64,7 +64,7 @@ export default class DocumentSyncAdapter {
   //                    indicating whether this adapter should care about the contents of the editor.
   constructor(
     connection: LanguageClientConnection,
-    documentSyncKind: TextDocumentSyncOptions | number | null,
+    documentSyncKind: TextDocumentSyncOptions | number | undefined,
     editorSelector: (editor: TextEditor) => boolean,
   ) {
     this._connection = connection;
@@ -129,7 +129,7 @@ export default class DocumentSyncAdapter {
     );
   }
 
-  public getEditorSyncAdapter(editor: TextEditor): TextEditorSyncAdapter | null {
+  public getEditorSyncAdapter(editor: TextEditor): TextEditorSyncAdapter | undefined {
     return this._editors.get(editor);
   }
 }

lib/adapters/linter-push-v2-adapter.ts

@@ -113,13 +113,13 @@ export default class LinterPushV2Adapter {
     if (path != null) {
       const diagnosticCodes = this._diagnosticCodes.get(path);
       if (diagnosticCodes != null) {
-        return diagnosticCodes.get(getCodeKey(range, text));
+        return diagnosticCodes.get(getCodeKey(range, text)) || null;
       }
     }
     return null;
   }
 }
 
 function getCodeKey(range: atom.Range, text: string): string {
-  return [].concat(...range.serialize(), text).join(',');
+  return ([] as any[]).concat(...range.serialize(), text).join(',');
 }

lib/adapters/outline-view-adapter.ts

@@ -115,24 +115,27 @@ export default class OutlineViewAdapter {
       return null;
     }
 
-    let parent = null;
+    let parent: atomIde.OutlineTree | undefined;
     for (const candidate of candidates) {
       if (
         candidate !== child &&
         candidate.startPosition.isLessThanOrEqual(child.startPosition) &&
-        (candidate.endPosition == null || candidate.endPosition.isGreaterThanOrEqual(child.endPosition))
+        (candidate.endPosition === undefined ||
+          (child.endPosition && candidate.endPosition.isGreaterThanOrEqual(child.endPosition)))
       ) {
         if (
-          parent == null ||
+          parent === undefined ||
           (parent.startPosition.isLessThanOrEqual(candidate.startPosition) ||
-            (parent.endPosition != null && parent.endPosition.isGreaterThanOrEqual(candidate.endPosition)))
+            (parent.endPosition != null &&
+              candidate.endPosition &&
+              parent.endPosition.isGreaterThanOrEqual(candidate.endPosition)))
         ) {
           parent = candidate;
         }
       }
     }
 
-    return parent;
+    return parent || null;
   }
 
   // Public: Convert an individual {SymbolInformation} from the language server

lib/adapters/signature-help-adapter.ts

@@ -31,8 +31,8 @@ export default class SignatureHelpAdapter {
     const {signatureHelpProvider} = this._capabilities;
     assert(signatureHelpProvider != null);
 
-    let triggerCharacters = null;
-    if (Array.isArray(signatureHelpProvider.triggerCharacters)) {
+    let triggerCharacters: Set<string> | undefined;
+    if (signatureHelpProvider && Array.isArray(signatureHelpProvider.triggerCharacters)) {
       triggerCharacters = new Set(signatureHelpProvider.triggerCharacters);
     }
 

lib/auto-languageclient.ts

@@ -62,7 +62,7 @@ export default class AutoLanguageClient {
   private _disposable = new CompositeDisposable();
   private _serverManager: ServerManager;
   private _linterDelegate: linter.V2IndieDelegate;
-  private _signatureHelpRegistry: atomIde.SignatureHelpRegistry;
+  private _signatureHelpRegistry: atomIde.SignatureHelpRegistry | null;
   private _lastAutocompleteRequest: AutocompleteRequest;
   private _isDeactivating: boolean;
 

lib/convert.ts

@@ -159,7 +159,7 @@ export default class Convert {
       case 'deleted':
         return [{uri: Convert.pathToUri(fileEvent.path), type: ls.FileChangeType.Deleted}];
       case 'renamed': {
-        const results = [];
+        const results: Array<{ uri: string, type: ls.FileChangeType }> = [];
         if (fileEvent.oldPath) {
           results.push({uri: Convert.pathToUri(fileEvent.oldPath || ''), type: ls.FileChangeType.Deleted});
         }

lib/download-file.ts

@@ -31,7 +31,7 @@ export default (async function downloadFile(
     throw Error('No response body');
   }
 
-  const finalLength = length || parseInt(response.headers.get('Content-Length' || '0'), 10);
+  const finalLength = length || parseInt(response.headers.get('Content-Length') || '0', 10);
   const reader = body.getReader();
   const writer = fs.createWriteStream(targetFile);
 
@@ -72,7 +72,7 @@ async function streamWithProgress(
       writer.write(Buffer.from(chunk));
       if (progressCallback != null) {
         bytesDone += chunk.byteLength;
-        const percent: number = length === 0 ? null : Math.floor(bytesDone / length * 100);
+        const percent: number | undefined = length === 0 ? undefined : Math.floor(bytesDone / length * 100);
         progressCallback(bytesDone, percent);
       }
     }

lib/server-manager.ts

@@ -269,7 +269,7 @@ export class ServerManager {
     if (filePath == null) {
       return null;
     }
-    return this._normalizedProjectPaths.find((d) => filePath.startsWith(d));
+    return this._normalizedProjectPaths.find((d) => filePath.startsWith(d)) || null;
   }
 
   public updateNormalizedProjectPaths(): void {
@@ -293,7 +293,7 @@ export class ServerManager {
     }
 
     for (const activeServer of this._activeServers) {
-      const changes = [];
+      const changes: ls.FileEvent[] = [];
       for (const fileEvent of fileEvents) {
         if (fileEvent.path.startsWith(activeServer.projectPath) && this._changeWatchedFileFilter(fileEvent.path)) {
           changes.push(Convert.atomFileEventToLSFileEvents(fileEvent)[0]);

test/adapters/autocomplete-adapter.test.ts

@@ -20,7 +20,7 @@ describe('AutoCompleteAdapter', () => {
       capabilities: {completionProvider: { }},
       connection: new ls.LanguageClientConnection(createSpyConnection()),
       disposable: new CompositeDisposable(),
-      process: undefined,
+      process: undefined as any,
       projectPath: '/',
     };
   }
@@ -279,7 +279,7 @@ describe('AutoCompleteAdapter', () => {
 
     it('does not do anything if there is no textEdit', () => {
       const completionItem: AutocompleteSuggestion = {};
-      AutoCompleteAdapter.applyTextEditToSuggestion(null, new TextEditor(), completionItem);
+      AutoCompleteAdapter.applyTextEditToSuggestion(undefined, new TextEditor(), completionItem);
       expect(completionItem).deep.equals({});
     });
 
@@ -315,7 +315,7 @@ describe('AutoCompleteAdapter', () => {
     });
 
     it('defaults to "value"', () => {
-      const result = AutoCompleteAdapter.completionKindToSuggestionType(null);
+      const result = AutoCompleteAdapter.completionKindToSuggestionType(undefined);
       expect(result).equals('value');
     });
   });

test/adapters/code-highlight-adapter.test.ts

@@ -50,8 +50,10 @@ describe('CodeHighlightAdapter', () => {
       expect(highlightStub.called).to.be.true;
 
       invariant(result != null);
-      expect(result.length).to.equal(1);
-      expect(result[0].isEqual(new Range([0, 1], [0, 2]))).to.be.true;
+      if (result) {
+        expect(result.length).to.equal(1);
+        expect(result[0].isEqual(new Range([0, 1], [0, 2]))).to.be.true;
+      }
     });
 
     it('throws if document highlights are not supported', async () => {

test/adapters/datatip-adapter.test.ts

@@ -48,21 +48,23 @@ describe('DatatipAdapter', () => {
       expect(datatip).to.be.ok;
       invariant(datatip != null);
 
-      expect(datatip.range.start.row).equal(0);
-      expect(datatip.range.start.column).equal(1);
-      expect(datatip.range.end.row).equal(0);
-      expect(datatip.range.end.column).equal(2);
+      if (datatip) {
+        expect(datatip.range.start.row).equal(0);
+        expect(datatip.range.start.column).equal(1);
+        expect(datatip.range.end.row).equal(0);
+        expect(datatip.range.end.column).equal(2);
 
-      expect(datatip.markedStrings).to.have.lengthOf(2);
-      expect(datatip.markedStrings[0]).eql({type: 'markdown', value: 'test'});
+        expect(datatip.markedStrings).to.have.lengthOf(2);
+        expect(datatip.markedStrings[0]).eql({type: 'markdown', value: 'test'});
 
-      const snippet = datatip.markedStrings[1];
-      expect(snippet.type).equal('snippet');
-      invariant(snippet.type === 'snippet');
-      expect((snippet as any).grammar.scopeName).equal('text.plain.null-grammar');
-      expect(snippet.value).equal('test snippet');
+        const snippet = datatip.markedStrings[1];
+        expect(snippet.type).equal('snippet');
+        invariant(snippet.type === 'snippet');
+        expect((snippet as any).grammar.scopeName).equal('text.plain.null-grammar');
+        expect(snippet.value).equal('test snippet');
 
-      expect(grammarSpy.calledWith('source.testlang')).to.be.true;
+        expect(grammarSpy.calledWith('source.testlang')).to.be.true;
+      }
     });
   });
 });

test/adapters/document-sync-adapter.test.ts

@@ -49,7 +49,7 @@ describe('DocumentSyncAdapter', () => {
 
   describe('constructor', () => {
     function create(textDocumentSync) {
-      return new DocumentSyncAdapter(null, textDocumentSync, () => false);
+      return new DocumentSyncAdapter(null as any, textDocumentSync, () => false);
     }
 
     it('sets _documentSyncKind correctly Incremental for v2 capabilities', () => {

test/adapters/outline-view-adapter.test.ts

@@ -121,11 +121,22 @@ describe('OutlineViewAdapter', () => {
           containerName: 'duplicate',
         },
       ];
+
       const result = OutlineViewAdapter.createOutlineTrees(sourceItems);
       expect(result.length).to.equal(1);
-      expect(result[0].endPosition.row).to.equal(10);
-      expect(result[0].children.length).to.equal(1);
-      expect(result[0].children[0].endPosition.row).to.equal(7);
+
+      const outline = result[0];
+      expect(outline.endPosition).to.not.be.undefined;
+      if (outline.endPosition) {
+        expect(outline.endPosition.row).to.equal(10);
+        expect(outline.children.length).to.equal(1);
+
+        const outlineChild = outline.children[0];
+        expect(outlineChild.endPosition).to.not.be.undefined;
+        if (outlineChild.endPosition) {
+          expect(outlineChild.endPosition.row).to.equal(7);
+        }
+      }
     });
 
     it('parents to the innnermost named container', () => {
@@ -141,11 +152,29 @@ describe('OutlineViewAdapter', () => {
       ];
       const result = OutlineViewAdapter.createOutlineTrees(sourceItems);
       expect(result.length).to.equal(1);
-      expect(result[0].endPosition.row).to.equal(10);
-      expect(result[0].children.length).to.equal(1);
-      expect(result[0].children[0].endPosition.row).to.equal(8);
-      expect(result[0].children[0].children.length).to.equal(1);
-      expect(result[0].children[0].children[0].endPosition.row).to.equal(5);
+
+      const outline = result[0];
+      expect(outline).to.not.be.undefined;
+      if (outline) {
+        expect(outline.endPosition).to.not.be.undefined;
+        if (outline.endPosition) {
+          expect(outline.endPosition.row).to.equal(10);
+          expect(outline.children.length).to.equal(1);
+
+          const outlineChild = outline.children[0];
+          expect(outlineChild.endPosition).to.not.be.undefined;
+          if (outlineChild.endPosition) {
+            expect(outlineChild.endPosition.row).to.equal(8);
+            expect(outlineChild.children.length).to.equal(1);
+
+            const outlineGrandChild = outlineChild.children[0];
+            expect(outlineGrandChild.endPosition).to.not.be.undefined;
+            if (outlineGrandChild.endPosition) {
+              expect(outlineGrandChild.endPosition.row).to.equal(5);
+            }
+          }
+        }
+      }
     });
   });
 
@@ -156,12 +185,19 @@ describe('OutlineViewAdapter', () => {
       expect(result.icon).to.equal('type-class');
       expect(result.representativeName).to.equal('Program');
       expect(result.children).to.deep.equal([]);
-      expect(result.tokenizedText[0].kind).to.equal('type');
-      expect(result.tokenizedText[0].value).to.equal('Program');
-      expect(result.startPosition.row).to.equal(1);
-      expect(result.startPosition.column).to.equal(2);
-      expect(result.endPosition.row).to.equal(3);
-      expect(result.endPosition.column).to.equal(4);
+      expect(result.tokenizedText).to.not.be.undefined;
+      if (result.tokenizedText) {
+        const resultTokenixedText = result.tokenizedText[0];
+        expect(result.tokenizedText[0].kind).to.equal('type');
+        expect(result.tokenizedText[0].value).to.equal('Program');
+        expect(result.startPosition.row).to.equal(1);
+        expect(result.startPosition.column).to.equal(2);
+        expect(result.endPosition).to.not.be.undefined;
+        if (result.endPosition) {
+          expect(result.endPosition.row).to.equal(3);
+          expect(result.endPosition.column).to.equal(4);
+        }
+      }
     });
   });
 });

tsconfig.json

@@ -7,6 +7,7 @@
         "declaration": true,
         "inlineSources": true,
         "inlineSourceMap": true,
+        "strictNullChecks": true,
         "baseUrl": "./",
         "paths": {
             "atom-ide": [

typings/atom-ide/index.d.ts

@@ -178,7 +178,7 @@ export interface CodeActionProvider {
     editor: TextEditor,
     range: Range,
     diagnostics: Diagnostic[],
-  ): Promise<CodeAction[]>;
+  ): Promise<CodeAction[] | null>;
 }
 
 export interface BusySignalOptions {