In job-results/reader.ts, decode(), extractFile(), and entries() each call this.zipReader.getEntries() separately, re-walking the central directory on every method. Minor for small result zips, but wasteful for large ones.
Fix: cache the entries (alongside the existing decoded/manifest state) and reuse. Low priority / opportunistic.
Surfaced during a tech-debt audit.
In
job-results/reader.ts,decode(),extractFile(), andentries()each callthis.zipReader.getEntries()separately, re-walking the central directory on every method. Minor for small result zips, but wasteful for large ones.Fix: cache the entries (alongside the existing
decoded/manifeststate) and reuse. Low priority / opportunistic.Surfaced during a tech-debt audit.