Skip to content
Merged
Show file tree
Hide file tree
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
101 changes: 8 additions & 93 deletions .github/pull_request_template.md
Original file line number Diff line number Diff line change
@@ -1,102 +1,17 @@
## Description

Provide a brief description of the changes made in this pull request.
<!-- Explain the problem, resulting behavior, and linked issue (Closes #...). -->

## Type of Change
## Validation

- [ ] Bug fix (non-breaking change which fixes an issue)
- [ ] New feature (non-breaking change which adds functionality)
- [ ] Breaking change (fix or feature that would cause existing functionality to change)
- [ ] Documentation update
- [ ] Code refactoring
- [ ] Performance improvement
- [ ] Test coverage improvement
<!-- List commands/results and the revision checked. Distinguish local tests,
CI results, intercepted dependencies, manual checks, and checks not run. -->

## Related Issues
## Risks and rollback

- Closes #(issue number)
- Fixes #(issue number)
- Relates to #(issue number)

## Changes Made

### Core Changes
- [ ] List specific changes made to core functionality
- [ ] Include any architectural changes
- [ ] Note any new dependencies added

### UI/UX Changes
- [ ] Describe any user interface changes
- [ ] Include screenshots if applicable
- [ ] Note any accessibility improvements

### API Changes
- [ ] Document any API changes
- [ ] Note breaking changes
- [ ] Update version numbers if needed

## Testing

### Automated Testing
- [ ] Unit tests pass (`npm test`)
- [ ] E2E tests pass (`npm run test:e2e`)
- [ ] Coverage threshold met (`npm run test:coverage`)
- [ ] Linting passes (`npm run lint`)
- [ ] Type checking passes (`npm run typecheck`)

### Manual Testing
- [ ] Extension loads without errors in Chrome
- [ ] Extension loads without errors in Edge
- [ ] Context menus function correctly
- [ ] Popup opens and functions correctly
- [ ] Background service worker initializes
- [ ] Content scripts inject properly
- [ ] AI providers respond correctly
- [ ] Error handling works as expected

### Browser Compatibility
- [ ] Chrome (version 88+)
- [ ] Edge (version 88+)
- [ ] Tested in incognito/private mode

## Security Considerations

- [ ] Input validation implemented
- [ ] XSS prevention measures in place
- [ ] API keys handled securely
- [ ] Content Security Policy updated if needed
- [ ] No sensitive data exposed in logs

## Performance Impact

- [ ] No significant performance degradation
- [ ] Memory usage is reasonable
- [ ] API calls are optimized
- [ ] Caching implemented where appropriate
- [ ] Bundle size impact is minimal

## Documentation

- [ ] README.md updated if needed
- [ ] CONTRIBUTING.md updated if needed
- [ ] Code comments added for complex logic
- [ ] JSDoc comments added for new functions
- [ ] CHANGELOG.md updated
<!-- Describe material compatibility, security, privacy, or packaging risks,
remaining evidence gaps, and how to revert if needed. -->

## Screenshots

<!-- Include screenshots for UI changes -->

## Additional Notes

<!-- Add any additional context, concerns, or notes for reviewers -->

## Checklist

- [ ] I have performed a self-review of my code
- [ ] I have commented my code, particularly in hard-to-understand areas
- [ ] I have made corresponding changes to the documentation
- [ ] My changes generate no new warnings
- [ ] I have added tests that prove my fix is effective or that my feature works
- [ ] New and existing unit tests pass locally with my changes
- [ ] Any dependent changes have been merged and published
<!-- Include before/after images for UI changes; remove this section otherwise. -->
2 changes: 2 additions & 0 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -9,6 +9,8 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0

### Fixed

- Replace fictional contribution examples, consolidate setup and validation
guidance, and exclude development configs from the npm package (#35).
- Pin verified current CI action releases, select Ubuntu 26.04 explicitly, and
avoid retaining checkout credentials while preserving the Node test matrix (#32).
- Preserve saved choices when settings reads fail: suppress optional notices and
Expand Down
Loading