From fd19130b3360d17b44247ec26533b20e051b7f8c Mon Sep 17 00:00:00 2001 From: skal Date: Mon, 9 Feb 2026 11:17:53 +0100 Subject: feat: WGSL Uniform Buffer Validation & Consolidation (Task #75) - Added to validate WGSL/C++ struct alignment. - Integrated validation into . - Standardized uniform usage in , , , . - Renamed generic to specific names in WGSL and C++ to avoid collisions. - Added and updated . - handoff(Gemini): Completed Task #75. --- TODO.md | 7 +++++++ 1 file changed, 7 insertions(+) (limited to 'TODO.md') diff --git a/TODO.md b/TODO.md index 4b5819b..ca755ec 100644 --- a/TODO.md +++ b/TODO.md @@ -6,6 +6,13 @@ This file tracks prioritized tasks with detailed attack plans. ## Recently Completed (February 9, 2026) +- [x] **WGSL Uniform Buffer Validation & Consolidation (Task #75)**: + - **Standardization**: Refactored `DistortEffect` and others to use `CommonPostProcessUniforms` (binding 2) + `EffectParams` (binding 3). + - **Validation Tool**: Created `tools/validate_uniforms.py` to parse C++ and WGSL (including embedded strings) and verify size/alignment. + - **Integration**: Added validation step to CMake build system. + - **Cleanup**: Renamed generic `EffectParams` to specific names (`FadeParams`, `CircleMaskParams`, etc.) in WGSL and C++. + - **Documentation**: Added `doc/UNIFORM_BUFFER_GUIDELINES.md` and updated `CONTRIBUTING.md`. + - [x] **Uniform Buffer Alignment (Task #74)**: Fixed WGSL struct alignment issues across multiple shaders: - `circle_mask_compute.wgsl`: Changed `_pad: vec3` to three `f32` fields - `fade_effect.cc`: Changed EffectParams padding from `vec3` to `_pad0/1/2: f32` -- cgit v1.2.3