From b10b2788a3547e8be0681e89ac3ab58f626e6217 Mon Sep 17 00:00:00 2001 From: omigamedev Date: Sun, 14 Jun 2026 00:22:56 +0200 Subject: [PATCH] Record STR-049 closeout hash --- docs/modernization/debt.md | 4 ++++ docs/modernization/tasks.md | 28 ++++++++++++++++++++++++++++ 2 files changed, 32 insertions(+) diff --git a/docs/modernization/debt.md b/docs/modernization/debt.md index d01bc33..2958172 100644 --- a/docs/modernization/debt.md +++ b/docs/modernization/debt.md @@ -18,6 +18,10 @@ agent or engineer to remove them without reconstructing context from chat. ## Recent Reductions +- 2026-06-14: `DEBT-0036` was narrowed again. `Canvas::draw_merge_temporary_paint_branch()` + now routes the retained temporary-paint request construction through + `make_canvas_draw_merge_temporary_paint_request(...)`; the temporary-paint + wrapper still owns the branch-selection guard and execution dispatch. - 2026-06-14: `DEBT-0036` was narrowed again. `Canvas::draw_merge_branch_orchestration()` now routes the retained branch execution body through `execute_canvas_draw_merge_branch_body(...)`; the branch wrapper still owns diff --git a/docs/modernization/tasks.md b/docs/modernization/tasks.md index 29ac267..abc0047 100644 --- a/docs/modernization/tasks.md +++ b/docs/modernization/tasks.md @@ -2110,6 +2110,34 @@ Validation: ctest --preset desktop-fast --build-config Debug -R "pp_paint_renderer_compositor|pp_paint_renderer_stroke_execution" --output-on-failure ``` +### STR-049 - Extract Draw Merge Temporary Paint Request Construction + +Status: Ready +Score: no score movement +Debt: `DEBT-0036` +Scope: `src/canvas.cpp`, `src/legacy_canvas_draw_merge_services.h`, `tests/paint_renderer/compositor_tests.cpp` + +Goal: + +Move the remaining `Canvas::draw_merge_temporary_paint_branch()` request +construction into a retained helper so the temporary-paint wrapper keeps only +dispatch and execution handling. + +Done Checks: + +- `Canvas::draw_merge_temporary_paint_branch()` no longer owns the + temporary-paint request construction inline. +- Regression coverage proves the helper preserves temporary-paint sampler, + texture, and draw ordering. +- `docs/modernization/debt.md` records the reduced draw-merge temporary-paint + surface. + +Validation: + +```powershell +ctest --preset desktop-fast --build-config Debug -R "pp_paint_renderer_compositor|pp_paint_renderer_stroke_execution" --output-on-failure +``` + ### STR-010 - Extract Remaining Draw Merge Composite Orchestration ### STR-016 - Extract Draw Merge Layer Composite Execution