Skip to content

perf(viewer): hydrate session posts#203

Merged
benvinegar merged 1 commit into
mainfrom
perf/hydrate-session-posts
Jul 2, 2026
Merged

perf(viewer): hydrate session posts#203
benvinegar merged 1 commit into
mainfrom
perf/hydrate-session-posts

Conversation

@benvinegar

Copy link
Copy Markdown
Member

Summary

  • Add ?hydrate=1 support to the session posts endpoint so the viewer can load full post details in one request.
  • Update session open and live reconnect resync to use the hydrated endpoint, with a fallback to legacy per-post fetches if needed.
  • Add API and Chromium e2e coverage that verifies session open avoids N+1 GET /api/posts/:id calls.

Review

  • Ran a fresh-context subagent code review; no blockers were found.

Tests

  • npm run typecheck
  • npm run lint
  • npm run format:check
  • npm test
  • npx playwright test e2e/viewer.spec.ts --project=chromium

This PR description was generated by Pi using OpenAI GPT-5

@benvinegar benvinegar merged commit 7698b1f into main Jul 2, 2026
9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant