Add some TODOs and remove some unneeded keys

My React brain added some keys in places that I now know did not need
them.

src/lib/server/opengrok-compat.ts

@@ -1,6 +1,10 @@
 import * as v from "@badrap/valita";
 import fs from "node:fs";
 
+// TODO we should have one environment variable pointing at a single
+// configuration file, so that we can extend the configuration later without
+// introducing new env vars.
+
 /**
  * Users may provide a simple JSON file that maps from OpenGrok project names to
  * Zoekt repo names, in case project names changed.

src/routes/(search-page)/search-results-file.svelte

@@ -64,16 +64,23 @@
   <SearchResultsFileHeader {file} {rank} />
   {#if lineGroups.length > 0}
     <div class="font-mono text-sm divide-y">
-      {#each lineGroups as lines (lines[0].lineNumber)}
+      {#each lineGroups as lines}
         <!--
           minmax because we don't want the line number column to slide left and
           right as you scroll down through sections with different `min-content`s'
           worth of line numbers. 2rem is enough for 4 digits.
+
+          TODO grid isn't doing a whole lot for us here, and browsers will
+          reject grids with more than a certain number of rows:
+          https://stackoverflow.com/a/63247486. We can probably simulate the
+          grid minmax calculation ourselves, as we trivially know the number of
+          digits in the maximum line number. Alternatively, a simple small fixed
+          value will probably be acceptable.
         -->
         <div
           class="py-1 grid grid-cols-[minmax(2rem,_min-content)_1fr] gap-x-2 whitespace-pre overflow-x-auto"
         >
-          {#each lines as { lineNumber, lineTokens } (lineNumber)}
+          {#each lines as { lineNumber, lineTokens }}
             <span class="select-none text-gray-600 text-right pr-1">
               {#if file.fileUrl && file.lineNumberTemplate}
                 <a

src/routes/(search-page)/search-results.svelte

@@ -28,12 +28,14 @@
     {Math.floor(duration / 1e4) / 1e2}
     ms
   </span>
+  <!-- TODO indicate when frontend files/matches are limited by the parameters in the search form -->
   <span class="ml-auto">
     Frontend: {files.length}
     {files.length === 1 ? "file" : "files"} / {frontendMatchCount}
     {frontendMatchCount === 1 ? "match" : "matches"} / {requestDuration}ms
   </span>
 </h1>
+<!-- TODO enable removing this key by making SearchResultsFile reset its internal state when props change. -->
 {#each files as file, i (`${file.repository}/${file.fileName.tokens
   .map(({ text }) => text)
   .join()}@${file.branches.join(";")}`)}

src/routes/repositories/repositories-list.svelte

@@ -33,7 +33,7 @@ but that's not understandable from the UI -->
       </tr>
     </thead>
     <tbody>
-      {#each repositories as repository (repository.name)}
+      {#each repositories as repository}
         <Repository {repository} />
       {/each}
     </tbody>