pr — pre-merge checklist
SkillSecurityThis adds a pull request self-review checklist based on how the FlexPrice team reviews code before merging. Once added, your AI can walk through the checklist with you and flag issues in security, code layering, migrations, and tests before a change is merged.
Available today. Use it from your connected AI after setup.
No other account needed.
After adding it, say pr check, ship review, or pre-PR when you have a change ready. Your AI will run through the checklist and report what needs attention.
Then ask your AI: use the pr — pre-merge checklist skill
What your AI can do with it
- Check code for security issues before merging
- Spot problems with how code is layered
- Review migrations as part of the pre-merge pass
- Confirm tests are in place for a change
- Start a review by saying pr check, ship review, or pre-PR
What this skill tells your AI
The instructions your AI receives, as published by flexprice/flexprice in .cursor/skills/pr/SKILL.md and read by ahel’s review.
Run mentally or out loud through this list; fix gaps before requesting review.
Scope & layering
- Change lives in correct layer (handler vs service vs domain vs repo per
docs/ARCHITECTURE.md). - No new hidden globals unless unavoidable and documented (
HOTSPOTS.mdpatterns). - Tenant/environment scoping preserved for multi-tenant data paths.
Security & config
- No secrets, tokens, or production URLs committed (
.env, keys in tests). - Auth/RBAC-sensitive routes still behind correct middleware (
internal/api/router.gopatterns).
Data & migrations
- Ent schema edits accompanied by
make generate-entand migration plan (make migrate-ent/make generate-migrationper deployment). - ClickHouse changes have matching files under
migrations/clickhouse/when applicable.
API & clients
- Swagger annotations updated for new/changed endpoints; run
make swagger. - If public SDK contract changes:
make sdk-all(or agreed subset) andapi/custommerged.
Quality
-
gofmt,go veton affected scope. -
go test -raceon touched packages ormake testfor broad changes. - Prefer table-driven tests for multiple cases (
internal/ee/service/*_test.goconventions).
Docs & graph
- Structural change updates
docs/REPO_MAP.md/DEPENDENCY_GRAPH.md/FLOWS/*when behavior or topology changes. - If Graphify is used:
graphify update .at repo root (seerepo-architecture-intelligencepersonal skill).
Tone for reviewers
Summarize intent, risk, and test evidence in the PR body (not “fixed stuff”).
Signals
- GitHub stars
- 5k
- Forks
- 495
- Last commit
- Sep 2026
Advanced
- Catalog kind
- skill
- Gateway key
pr-flexprice- Source
- github.com/flexprice/flexprice