refactor: Simplify effect render API and fix uniform initialization

Root cause: Uniform buffers created but not initialized before bind group
creation, causing undefined UV coordinates in circle_mask_compute.wgsl.

Changes:
- Add get_common_uniforms() helper to Effect base class
- Refactor render()/compute() signatures: 5 params → CommonPostProcessUniforms&
- Fix uninitialized uniforms in CircleMaskEffect and CNNEffect
- Update all 19 effect implementations and headers
- Fix WGSL syntax error in FlashEffect (u.audio_intensity → audio_intensity)
- Update test files (test_sequence.cc)

Benefits:
- Cleaner API: construct uniforms once per frame, reuse across effects
- More maintainable: CommonPostProcessUniforms changes need no call site updates
- Fixes UV coordinate bug in circle_mask_compute.wgsl

All 36 tests passing (100%)

handoff(Claude): Effect API refactor complete

1 files changed, 2 insertions, 2 deletions

diff --git a/src/gpu/effects/scene1_effect.h b/src/gpu/effects/scene1_effect.h
index dc5c747..190ffa9 100644
--- a/src/gpu/effects/scene1_effect.h
+++ b/src/gpu/effects/scene1_effect.h
@@ -9,8 +9,8 @@
 class Scene1Effect : public Effect {
  public:
   Scene1Effect(const GpuContext& ctx);
-  void render(WGPURenderPassEncoder pass, float time, float beat,
-              float intensity, float aspect_ratio) override;
+  void render(WGPURenderPassEncoder pass,
+              const CommonPostProcessUniforms& uniforms) override;
 
  private:
   RenderPass pass_;