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):
docker run --rm -v "$PWD":/src -w /src golang:1.25.6-alpine sh -c "go test ./..."
Optional integration concurrency test (when relevant):
RUN_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:
## Summary
- <What changed and why>
- <Any user-facing/API impact>
## Test plan
- [ ] `go test ./...` (or targeted packages)
- [ ] Manual check: <endpoint / scenario> (if applicable)
- [ ] Integration test (if applicable)
## Notes
- <Risks, migrations, rollout considerations>
Review-ready final pass
Re-read the diff like a reviewer.
Ensure errors are handled and logs are not leaking sensitive info.
Ensure docs/config are updated if behavior or env vars changed.
Converted and distributed by TomeVault — claim your Tome and manage your conversions.
1---2name: entazis-product-reviews-pull-request3description: Pull Request4---56# Pull Request78## Pre-flight checklist910- [ ] Scope is focused; no unrelated refactors.11- [ ] Code is formatted (`gofmt`, `goimports`).12- [ ] No secrets checked in (env files, credentials, tokens).13- [ ] Behavior changes are tested (at least targeted tests; full suite when appropriate).14- [ ] For review mutations: aggregates update in the same DB transaction and outbox events are emitted (if relevant).15- [ ] Product responses still do **not** include reviews (if touching product endpoints).1617## Testing guidance1819Default (unit tests, containerized toolchain):2021```bash22docker run --rm -v "$PWD":/src -w /src golang:1.25.6-alpine sh -c "go test ./..."23```2425Optional integration concurrency test (when relevant):2627```bash28RUN_INTEGRATION=1 docker run --rm -v "$PWD":/src -w /src golang:1.25.6-alpine sh -c "go test ./... -run TestConcurrentReviewCreates_UpdateAggregates"29```3031## PR description template3233Use this structure:3435```markdown36## Summary37- <What changed and why>38- <Any user-facing/API impact>3940## Test plan41- [ ] `go test ./...` (or targeted packages)42- [ ] Manual check: <endpoint / scenario> (if applicable)43- [ ] Integration test (if applicable)4445## Notes46- <Risks, migrations, rollout considerations>47```4849## Review-ready final pass50511. Re-read the diff like a reviewer.522. Ensure errors are handled and logs are not leaking sensitive info.533. Ensure docs/config are updated if behavior or env vars changed.5455---56> Converted and distributed by [TomeVault](https://tomevault.io/claim/entazis) — claim your Tome and manage your conversions.57<!-- tomevault:4.0:skill_md:2026-04-15 -->
Run npx skillmds@latest add tomevault-io/entazis-product-reviews-pull-request in your terminal (requires Node.js), paste this page's agent-chat prompt into Claude, Cursor, or any MCP-connected agent, or download the SKILL.md file and copy it into your agent's skills directory.
Pull Request It is listed under Coding & Dev Tools on SkillMD.
This skill has not completed SkillMD's automated safety review yet. Independent scanners report: SkillSpector: PASS, Skill Scanner: PASS. SkillMD never runs a skill's scripts for you; review the SKILL.md before installing.
This skill is tagged as working with Claude Code, Claude.ai, OpenAI Codex. SKILL.md is an open format, so most agents that read a skills directory can load it too.
Yes. Installing skills from SkillMD is free, and the skill stays under its author's original license.
tomevault-io (@tomevault-io) published this skill. Their other Agent Skills are listed on their SkillMD profile.
tomevault-io