mirror of
https://github.com/MBombeck/HealthLog.git
synced 2026-11-08 12:32:43 +00:00
Marc-André Bombeck
83b6ce4b15
Neither read is a bug a behavioural test can see: both shapes return the right rows, and only the plan differs. So the guards split by what each can actually answer. The unit guard asserts on the SQL the readers build, with comments stripped so it matches the query text and never the prose describing it: no DISTINCT ON on this path, a LATERAL driven from an unnested type list with a LIMIT on the per-type branch, a bound type predicate on the rollup read sitting alongside the user and granularity filters, and the route still calling the shared readers rather than growing a second copy. Four of its five cases fail against the pre-change SQL. The integration companion runs the shipped readers and the pre-change SQL side by side over one fixture on the real Postgres and requires identical rows. The fixture carries the cases the rewrite could have broken: a type whose only rows are tombstoned, a type with no rows, a type whose newest row is a tombstone over a surviving one, and coarser rollup granularities the DAY read must ignore. Dropping the soft-delete filter or shortening the type list makes both cases fail. Plan assertions are deliberately not in the integration file: a testcontainer fixture is too small for the planner to choose the access paths a real account produces, so an EXPLAIN there would pin nothing.
src/
Application source. HealthLog is a Next.js 16 App Router project (output: "standalone", React 19, TypeScript strict). The path alias @/* resolves to this directory. Everything a request touches lives under here; the layout below follows the request flow rather than alphabetical order.
Request flow
client → src/proxy.ts → src/app/<route>/route.ts → src/lib/api-handler.ts → handler body → src/lib/api-response.ts → client
proxy.ts— the HTTP edge (Next.js 16's renamedmiddleware). Public-path allowlist, demo-mode mutation block, server-side onboarding redirect, worker-only refusal,x-request-id+ CSP nonce generation, every security header.app/— App Router pages (RSC by default) and API routes. Every API route wraps its handler inapiHandler. Seeapp/README.md.lib/— the server/runtime toolkit:apiHandler, auth, crypto, rollups, AI, notifications, pg-boss jobs, integrations, rate limiting. Seelib/README.md.components/— React components (shadcn/ui new-york, kebab-case files). Seecomponents/README.md.
Other directories
hooks/— shared React hooks (use-prefix, kebab-case).types/— shared TypeScript declarations.generated/prisma/— the generated Prisma client (do not edit; regenerated byprisma generate). Imported as@/generated/prisma.instrumentation.ts— Next.js startup hook; boots the pg-boss worker registry and boot-time backfill queues.__tests__/— cross-cutting unit tests; most tests live in__tests__/folders next to the code they cover.
Conventions
See ../CLAUDE.md for the full set. The load-bearing ones: RSC by default ("use client" only when a hook/state/browser API needs it); every API route wraps apiHandler and parses its body with Zod safeParse; userId is always narrowed from session or Bearer, never accepted as a body field; TanStack Query keys come only from src/lib/query-keys/; outbound HTTP goes through src/lib/safe-fetch.ts.