Add some TODOs so we don't forget to do this.

Author: corbob

src/features/GetCommands.ts

@@ -5,6 +5,8 @@ import * as vscode from "vscode";
 import { LanguageClient, RequestType } from "vscode-languageclient";
 import { IFeature } from "../feature";
 
+// TODO: Document this export: https://github.com/PowerShell/vscode-powershell/pull/1406#discussion_r209325655
+// TODO: Also use something other than any if possible... We may have already addressed this with a previous attempt.
 export const GetAllCommandsRequestType = new RequestType<any, any, void, void>("powerShell/getAllCommands");
 
 export class GetCommandsFeature implements IFeature {
@@ -18,6 +20,7 @@ export class GetCommandsFeature implements IFeature {
                 // TODO: Log error message
                 return;
             }
+            // TODO: Refactor network code out of constructor
             this.languageClient.sendRequest(GetAllCommandsRequestType, "").then((result) => {
                 this.commandsExplorerProvider.powerShellCommands = result.map(toCommand);
                 this.commandsExplorerProvider.refresh();
@@ -69,6 +72,7 @@ class CommandsExplorerProvider implements vscode.TreeDataProvider<Command> {
 
 }
 
+// TODO: Define and export an ICommand interface to describe the properties we require.
 function toCommand(command: any): Command {
     return new Command(
         command.name,
@@ -100,5 +104,6 @@ class Command extends vscode.TreeItem {
 
     public async getChildren(element): Promise<Command[]> {
         return [];
+        // TODO: Determine why we're returning an empty array... I think it's because we have to return something and we're not actually using the tree view part just yet...
     }
 }