Delete TODO comments that didn't prove to have the performance benfits I thought they would

c84aa1cdb1bd98261f435ff8809b7b893464053b

Tucker McKnight <tmcknight@instructure.com> | Sat Jun 20 2026

Delete TODO comments that didn't prove to have the performance benfits I thought they would
src/repos.ts:243
Before
242
243
244
245
246
247
248
              break
            }
            if (fileMap.has(`${filename}-${currentCommit.hash}`)) {
              // TODO: maybe cache here as well, so that we don't have to loop as many
              // times next time?
              return fileMap.get(`${filename}-${currentCommit.hash}`)
            }
After
242
243
244


245
246
              break
            }
            if (fileMap.has(`${filename}-${currentCommit.hash}`)) {
⁣
⁣
              return fileMap.get(`${filename}-${currentCommit.hash}`)
            }
src/repos.ts:259
Before
258
259
260
261
262
263
264
          blameLines,
        }

// TODO: cache it as deeply in the git history as it can be, so that as many other commits
// benefit from this as possible.
        fileMap.set(key, commit)
        return commit
      },
After
258
259
260


261
262
          blameLines,
        }

⁣
⁣
        fileMap.set(key, commit)
        return commit
      },