| true |
permissions |
tools |
network |
safe-outputs |
| pull_request |
| types |
opened |
reopened |
synchronize |
ready_for_review |
|
|
|
| contents |
pull-requests |
read |
read |
|
|
defaults |
|
Flags likely performance regressions or expensive new code paths.
Run when a pull request changes code in hot paths, large loops, IO boundaries, caching, data processing, or query behavior.
- Inspect the changed logic for operations that may scale poorly: nested loops, repeated remote calls, N+1 access patterns, large allocations, redundant parsing, or cache bypasses.
- Focus on performance regressions that matter at realistic load, not theoretical micro-optimizations.
- Explain the likely cost multiplier: more rows, more requests, more allocations, slower startup, or higher p95 latency.
- If a likely regression exists, use safe outputs to leave one concise comment pointing to the hotspot and the most practical improvement or validation path.
- If the code is performance-sensitive but the evidence is incomplete, recommend measurement or profiling rather than making a false precision claim.
- Avoid nitpicking low-impact style changes that do not alter runtime behavior meaningfully.
- Prioritize the few issues most likely to affect users or infrastructure cost.
- A focused performance review note when the change likely introduces real cost or latency risk.
- Actionable guidance that helps reviewers decide whether deeper benchmarking is required.
- Add the paths or modules your team treats as performance-critical.
- If your project has known scale thresholds, mention them so the workflow can prioritize realistic bottlenecks.
- Keep comment count low and focus on the most impactful regression risks.