## Description
## Type of Change
- [ ] 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 not work as expected)
- [ ] Documentation update
- [ ] Refactoring (no functional changes)
- [ ] Performance improvement
- [ ] Test coverage improvement
- [ ] CI/CD changes
- [ ] Dependency update
## Related Issue
Fixes #(issue_number)
Relates to #(issue_number)
Implements RFC/Design: [design-doc-url]
## Changes Made
**Core Changes:**
-
-
-
**Documentation:**
-
-
**Testing:**
-
-
## Testing
**Automated Testing:**
- [ ] All existing tests pass
- [ ] New tests added for new functionality
- [ ] Integration tests updated/added
- [ ] Performance tests (if applicable)
**Manual Testing:**
- [ ] Tested happy path scenarios
- [ ] Tested error/edge cases
- [ ] Cross-platform testing (if applicable)
- [ ] Backward compatibility verified
**Test Coverage:**
- New code coverage: __%
- Overall project coverage: __%
### Test Commands
```bash
cargo test
cargo clippy -- -D warnings
cargo fmt --check
./scripts/build.sh
```
## Screenshots/Recordings
## Review Focus Areas
**Please pay special attention to:**
- [ ] Security implications
- [ ] Performance impact
- [ ] Architecture changes
- [ ] Error handling
- [ ] User experience
- [ ] Backward compatibility
## Checklist
- [ ] My code follows the project's style guidelines
- [ ] I have performed a self-review of my own 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
- [ ] I have read and followed the [PR Guidelines](../.omni-dev/pr-guidelines.md)
## Performance Impact
- [ ] No performance impact
- [ ] Performance improvement (describe below)
- [ ] Potential performance impact (describe below)
## Security Considerations
- [ ] No security implications
- [ ] Security improvement (describe below)
- [ ] Potential security impact (describe below)
## Breaking Changes
**Migration Required:**
1.
2.
3.
**Backward Compatibility:**
-
## Deployment Notes
- [ ] No special deployment requirements
- [ ] Database migration required
- [ ] Environment variable changes required
- [ ] Configuration file updates required
- [ ] Service restart required
## Additional Notes
**Future Enhancements:**
-
**Known Limitations:**
-
**Alternatives Considered:**
-