# Pull Request
## Description
## Type of Change
- [ ] ๐ Bug fix (non-breaking change that fixes an issue)
- [ ] โจ New feature (non-breaking change that adds functionality)
- [ ] ๐ฅ Breaking change (fix or feature that would cause existing functionality to not work as expected)
- [ ] ๐ Documentation update
- [ ] ๐งน Code cleanup/refactoring
- [ ] โก Performance improvement
- [ ] ๐ Security fix
- [ ] ๐งช Test improvement
## Related Issues
Fixes #(issue number)
Relates to #(issue number)
## Changes Made
-
-
-
## Quality Checklist
### Code Quality
- [ ] Code follows the project's style guidelines
- [ ] Self-review of code has been performed
- [ ] Code is self-documenting with clear variable/function names
- [ ] Complex logic includes explanatory comments
### Testing
- [ ] All quality gates pass (`./scripts/pre_quality.sh`)
- [ ] New tests added for new functionality
- [ ] All existing tests pass
- [ ] Test coverage is maintained or improved
- [ ] Edge cases and error conditions are tested
### Documentation
- [ ] Documentation updated for public API changes
- [ ] Rust doc comments added/updated for new public functions
- [ ] README updated if needed
- [ ] Examples updated if applicable
### Security
- [ ] No sensitive data exposed in logs or error messages
- [ ] Input validation added where appropriate
- [ ] Error messages use `create_safe_error_message` utility
- [ ] No hardcoded secrets or API keys
## API Impact
### Breaking Changes
- [ ] No breaking changes
- [ ] Breaking changes documented with migration guide
### New Public APIs
```rust
// Example:
pub fn new_function() -> Result<()> {
// ...
}
```
## Testing Strategy
### Manual Testing
1.
2.
3.
### Automated Testing
- [ ] Unit tests: `cargo test test_name`
- [ ] Integration tests: `cargo test --test integration_tests`
- [ ] Documentation tests: `cargo test --doc`
## Performance Impact
- [ ] No performance impact
- [ ] Performance improved
- [ ] Performance impact acceptable for the feature
- [ ] Performance benchmarks included
## Security Considerations
- [ ] No security implications
- [ ] Security review completed
- [ ] Follows security best practices
- [ ] Potential security concerns documented
## Dependencies
- [ ] No new dependencies
- [ ] New dependencies justified and minimal
- [ ] Dependencies updated with security patches
## Deployment Notes
- [ ] No special deployment considerations
- [ ] Migration steps documented
- [ ] Environment variables or configuration changes needed
## Additional Context
## Reviewer Notes
- Please pay special attention to:
- Areas where feedback is particularly welcome:
- Known limitations or trade-offs:
---