From 5d5e0e7f21aae27ca17bcf711237a8f1b53dc54c Mon Sep 17 00:00:00 2001 From: omigamedev Date: Sun, 14 Jun 2026 00:14:24 +0200 Subject: [PATCH] Record STR-045 closeout hash --- docs/modernization/debt.md | 5 +++++ docs/modernization/tasks.md | 31 +++++++++++++++++++++++++++++++ 2 files changed, 36 insertions(+) diff --git a/docs/modernization/debt.md b/docs/modernization/debt.md index 5230cee..07b4733 100644 --- a/docs/modernization/debt.md +++ b/docs/modernization/debt.md @@ -18,6 +18,11 @@ agent or engineer to remove them without reconstructing context from chat. ## Recent Reductions +- 2026-06-14: `DEBT-0036` was narrowed again. `execute_stroke_preview_final_composite_pass()` + now routes the retained final-composite request construction through + `make_stroke_preview_final_composite_request(...)`; the preview path still + owns the concrete shader setup, sampler binding, input binding, and draw + callback. - 2026-06-14: `DEBT-0036` was narrowed again. `NodeStrokePreview::stroke_draw_mix()` now routes the retained preview mix request construction through `make_stroke_preview_mix_execution_request(...)`; the preview path still diff --git a/docs/modernization/tasks.md b/docs/modernization/tasks.md index a8f2ce6..ea2e4c6 100644 --- a/docs/modernization/tasks.md +++ b/docs/modernization/tasks.md @@ -1990,6 +1990,37 @@ ctest --preset desktop-fast --build-config Debug -R "pp_paint_renderer_composito cmake --build --preset windows-msvc-default --config Debug --target PanoPainter ``` +### STR-045 - Extract Preview Final Composite Request Construction + +Status: Done +Score: no score movement +Debt: `DEBT-0036` +Scope: `src/node_stroke_preview.cpp`, `tests/paint_renderer/compositor_tests.cpp` + +Goal: + +Move the remaining `execute_stroke_preview_final_composite_pass()` request +construction into a retained helper so the final preview composite wrapper +keeps only dispatch and copy handling. + +Done Checks: + +- `execute_stroke_preview_final_composite_pass()` no longer owns the + final-composite request construction inline. +- Existing preview final-composite executor coverage still proves the helper + preserves shader setup, sampler binding, input binding, and draw ordering. +- `docs/modernization/debt.md` records the reduced preview final-composite + surface. + +Closeout: `538441a5` + +Validation: + +```powershell +ctest --preset desktop-fast --build-config Debug -R "pp_paint_renderer_compositor|pp_paint_renderer_stroke_execution" --output-onfailure +cmake --build --preset windows-msvc-default --config Debug --target PanoPainter +``` + ### STR-010 - Extract Remaining Draw Merge Composite Orchestration ### STR-016 - Extract Draw Merge Layer Composite Execution