Pull Request

Pre-flight checklist

• Scope is focused; no unrelated refactors.
• Code is formatted (gofmt, goimports).
• No secrets checked in (env files, credentials, tokens).
• Behavior changes are tested (at least targeted tests; full suite when appropriate).
• For review mutations: aggregates update in the same DB transaction and outbox events are emitted (if relevant).
• Product responses still do not include reviews (if touching product endpoints).

Testing guidance

Default (unit tests, containerized toolchain):

bash
1docker run --rm -v "$PWD":/src -w /src golang:1.25.6-alpine sh -c "go test ./..."

Optional integration concurrency test (when relevant):

bash
1RUN_INTEGRATION=1 docker run --rm -v "$PWD":/src -w /src golang:1.25.6-alpine sh -c "go test ./... -run TestConcurrentReviewCreates_UpdateAggregates"

PR description template

Use this structure:

markdown
1## Summary
2- <What changed and why>
3- <Any user-facing/API impact>
4
5## Test plan
6- [ ] `go test ./...` (or targeted packages)
7- [ ] Manual check: <endpoint / scenario> (if applicable)
8- [ ] Integration test (if applicable)
9
10## Notes
11- <Risks, migrations, rollout considerations>

Review-ready final pass

1. Re-read the diff like a reviewer.
2. Ensure errors are handled and logs are not leaking sensitive info.
3. Ensure docs/config are updated if behavior or env vars changed.