fix: remove todos

Author: olafurpg

src/commands/goToRepositoryCommand.ts

@@ -6,6 +6,7 @@ import { openSourcegraphUriCommand } from './openSourcegraphUriCommand'
 import { BrowseQuickPickItem, SourcegraphQuickPick } from './SourcegraphQuickPick'
 import { recentlyOpenRepositoriesSetting } from '../settings/recentlyOpenRepositoriesSetting'
 import { endpointSetting } from '../settings/endpointSetting'
+import { log } from '../log'
 
 export async function goToRepositoryCommand(fs: SourcegraphFileSystemProvider): Promise<void> {
     const quick = new SourcegraphQuickPick(fs)
@@ -31,8 +32,8 @@ export async function goToRepositoryCommand(fs: SourcegraphFileSystemProvider):
                     detail: query.text,
                 }
                 quick.pick.items = [item]
-            } catch {
-                // TODO: report helpful error message
+            } catch (error) {
+                log.error(`goToRepositoryCommand(${query.text})`, error)
             }
             return
         }

src/file-system/FileTree.ts

@@ -12,9 +12,6 @@ export class FileTree {
         return `FileTree(${this.uri.uri}, files.length=${this.files.length})`
     }
 
-    // TODO: optimize this for very large repos like chromium/chromium. It's
-    // usable in its current state but could be much faster if we use binary
-    // search to skip unrelated paths.
     public directChildren(directory: string): string[] {
         return this.directChildrenInternal(directory, true)
     }

src/search/SourcegraphLocationRenderer.js

@@ -1,4 +1,3 @@
-// TODO: try to use file extension `.mts` or see if we can add types.
 /**
  * Renders the 'application/sourcegraph-location' MIME type for Sourcegraph Notebooks
  *