Unified Renderer

[Raffaello]

Summary by CodeRabbit

• Chores
  • Version bumped to 0.19.3
• Refactor
  • Unified audio renderer into a single, platform-agnostic implementation and removed the legacy platform-specific renderer
  • Simplified renderer API and internal buffer handling
• Tests
  • Consolidated renderer tests to target the unified renderer implementation

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: a07ea57f-db14-4579-8386-30f88021840a

📥 Commits

Reviewing files that changed from the base of the PR and between 1513e91 and ff403ae.

📒 Files selected for processing (1)

• hyper-sonic-drivers/src/HyperSonicDrivers/audio/Renderer.hpp

🚧 Files skipped from review as they are similar to previous changes (1)

• hyper-sonic-drivers/src/HyperSonicDrivers/audio/Renderer.hpp

---

Walkthrough

Consolidates SDL2/SDL3-specific renderers into a single generic HyperSonicDrivers::audio::Renderer, removes mixer dependency from IRenderer, updates build wiring to include the shared renderer, and consolidates renderer tests and namespaces.

Changes

Audio Renderer Backend Unification

Layer / File(s)	Summary
IRenderer interface simplification hyper-sonic-drivers/src/HyperSonicDrivers/audio/IRenderer.hpp	Removed IMixer include and getMixer(); replaced m_mixer with m_buffer_size to decouple buffer sizing from mixer management.
Generic renderer implementation and API change hyper-sonic-drivers/src/HyperSonicDrivers/audio/Renderer.hpp, hyper-sonic-drivers/src/HyperSonicDrivers/audio/Renderer.cpp	Renderer moved from HyperSonicDrivers::audio::sdl3 to HyperSonicDrivers::audio; constructor now takes only buffer_size and methods use m_buffer_size for buffer sizing.
Build wiring and platform include selection hyper-sonic-drivers/CMakeLists.txt, hyper-sonic-drivers/sdl2-hyper-sonic-drivers.cpp	CMake unconditionally includes audio/Renderer.cpp; sdl2-hyper-sonic-drivers.cpp now selects SDL3/SDL2/rtaudio headers and mixer/logger implementations, then includes generic Renderer.hpp.
Test consolidation and updates hyper-sonic-drivers/test/HyperSonicDrivers/CMakeLists.txt, hyper-sonic-drivers/test/HyperSonicDrivers/audio/TestRenderer.cpp	Test CMakeLists now builds audio/TestRenderer.cpp unconditionally; test includes the generic audio/Renderer.hpp, uses HyperSonicDrivers::audio namespace, and constructs Renderer(1024).
Release version bump CMakeLists.txt	Project version updated from 0.19.2 to 0.19.3.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

Suggested labels

enhancement, file

"I nibble at old headers, sew them anew,
SDL branches fold into one bright view.
A buffer kept simple, steady and small,
Tests hop together and answer the call.
Version raised — the rabbit does a joyful sprawl." 🐇

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Docstring Coverage	⚠️ Warning	Docstring coverage is 9.09% which is insufficient. The required threshold is 80.00%.	Write docstrings for the functions missing them to satisfy the coverage threshold.

✅ Passed checks (4 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title 'Unified Renderer' accurately captures the main objective of the pull request, which consolidates the SDL2 and SDL3-specific Renderer implementations into a single, generic Renderer class in the HyperSonicDrivers::audio namespace.
Linked Issues check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check	✅ Passed	Check skipped because no linked issues were found for this pull request.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing Touches

📝 Generate docstrings

• Create stacked PR
• Commit on current branch

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch renderer

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[codacy-production]

Up to standards ✅

🟢 Issues 0 issues

Results:
0 new issues

View in Codacy

🟢 Metrics 23 complexity

Metric	Results
Complexity	23

View in Codacy

_{NEW Get contextual insights on your PRs based on Codacy's metrics, along with PR and Jira context, without leaving GitHub. Enable AI reviewer
TIP This summary will be updated as you push new changes.}

[Raffaello]

@coderabbitai review

[coderabbitai]

✅ Action performed

Review finished.

Note: CodeRabbit is an incremental review system and does not re-review already reviewed commits. This command is applicable only when automatic reviews are paused.

[sonarqubecloud]

Quality Gate failed

Failed conditions
0.0% Coverage on New Code (required ≥ 80%)

See analysis details on SonarQube Cloud

[Raffaello]

@coderabbitai review

[coderabbitai]

✅ Action performed

Review finished.

Note: CodeRabbit is an incremental review system and does not re-review already reviewed commits. This command is applicable only when automatic reviews are paused.