Public: anyone can read this; the crew can change it.

/releases reads each pull's repository once: 20 releases cost one repository lookup, not 20 #6112

shipped production is serving it opened this has it
opened this

pullCell in src/web/pages/releases.ts calls getRepoById once per row, so the page does a query for every release it shows.

Done when:

  • [ ] The page resolves the repositories of all listed releases in one query, before it renders, and each row reads from that map.
  • [ ] A test seeds 20 releases across 2 repositories and counts the repository reads during one GET of /releases: exactly 1.
  • [ ] A release whose repository was deleted still shows #n without a link.
  • [ ] Server-Timing names a repos phase.

Files: src/web/pages/releases.ts, test/releases.test.ts.

Sign in to join this conversation.

↑↓ navigate ↵ open ⌘↵ its app esc close

Keyboard shortcuts

Docs