Skip to content
Merged
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
77 changes: 48 additions & 29 deletions .github/pull_request_template.md
Original file line number Diff line number Diff line change
@@ -1,54 +1,73 @@
## TLDR
<!--
Help reviewers verify this PR quickly.

Maintainers prioritize PRs that include clear proof of work.
If a PR does not include enough validation detail to reproduce and verify the change efficiently, review may be delayed.
-->

<!-- Add a brief description of what this pull request changes and why and any important things for reviewers to look at -->
## Summary

## Screenshots / Video Demo
- What changed:
- Why it changed:
- Reviewer focus:

## Validation

<!--
Please attach a screenshot or short video showing your change in action.
This helps reviewers understand the change quickly and prioritize reviews.
Be concrete. Do not write only "tested locally".
Include the exact commands, prompts, outputs, logs, screenshots, or videos that prove the change was actually run and observed.

- For bug fixes: show the before/after behavior.
- For features: show the new functionality in use.
- For refactors or internal changes with no visible effect: write "N/A — no user-facing change" and briefly explain why.
For user-visible changes, bug fixes, CLI / TUI behavior changes, or interaction changes, include key screenshots or a short video.
When possible, show before/after behavior.

PRs with visual demos typically get reviewed much faster!
If helpful, use the `e2e-testing` skill to gather stronger end-to-end validation evidence.
-->

## Dive Deeper

<!-- more thoughts and in-depth discussion here -->
- Commands run:
```bash
# paste commands here
```
- Prompts / inputs used:
- Expected result:
- Observed result:
- Quickest reviewer verification path:
- Evidence (output, logs, screenshots, video, JSON, before/after, etc.):

## Reviewer Test Plan
## Scope / Risk

<!-- when a person reviews your code they should ideally be pulling and running that code. How would they validate your change works and if relevant what are some good classes of example prompts and ways they can exercise your changes -->
- Main risk or tradeoff:
- Not covered / not validated:
- Breaking changes / migration notes:

## Testing Matrix

<!-- Before submitting please validate your changes on as many of these options as possible -->
<!--
Use:
- ✅ tested
- ⚠️ not tested
- N/A
If anything is ⚠️, explain why briefly below.
-->

| | 🍏 | 🪟 | 🐧 |
| -------- | --- | --- | --- |
| npm run | | | |
| npx | | | |
| Docker | | | |
| Podman | | - | - |
| Seatbelt | | - | - |
| npm run | ⚠️ | ⚠️ | ⚠️ |
| npx | ⚠️ | ⚠️ | ⚠️ |
| Docker | ⚠️ | ⚠️ | ⚠️ |
| Podman | ⚠️ | N/A | N/A |
| Seatbelt | ⚠️ | N/A | N/A |

## Linked issues / bugs
Testing matrix notes:

<!--
Link to any related issues or bugs.
-

**If this PR fully resolves the issue, use one of the following keywords to automatically close the issue when this PR is merged:**
## Linked Issues / Bugs

<!--
If this PR fully resolves an issue, use one of:
- Closes #<issue_number>
- Fixes #<issue_number>
- Resolves #<issue_number>

*Example: `Resolves #123`*

**If this PR is only related to an issue or is a partial fix, simply reference the issue number without a keyword:**

*Example: `This PR makes progress on #456` or `Related to #789`*
Otherwise reference related issues without a closing keyword.
-->
Loading