Skip to content

Pull Request

📝 Description

🎯 Type of Change

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Performance improvement (change that improves performance)
  • Refactoring (code change that neither fixes a bug nor adds a feature)
  • Documentation (changes to documentation only)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)

✅ Testing

  • Unit tests added/updated
  • Integration tests pass
  • Manual testing performed
  • Performance validated (if applicable)
  • Real-time safety verified (if audio thread code)

Test Coverage

  • Before: ____%
  • After: ____%

📋 Checklist

  • Code follows style guide
  • Tests included for new functionality
  • Documentation updated (inline comments, README, etc.)
  • No performance regression
  • Breaking changes documented
  • Commit messages follow convention
  • All CI checks pass

Closes # Relates to #

🏗️ Architecture Impact

  • No architectural changes
  • Adds new abstraction/pattern:
  • Modifies existing architecture:
  • Introduces new dependency:

⚡ Performance Impact

  • No performance impact
  • Performance improved:
  • Performance regression:

Benchmarks

Before: X ms
After:  Y ms
Change: Z%

🔒 Security Considerations

  • No security impact
  • Security review required:
  • Input validation added
  • Secrets/credentials handled properly

📸 Screenshots / Demos

Before

After

📚 Additional Notes

👥 Reviewers

@ -

🚀 Deployment Notes

  • Database migration required
  • Configuration changes needed
  • Feature flag required
  • Backward compatibility maintained

Note for Reviewers: Please use the Review Checklist and follow our Review Guidelines.