From f4cd08d700403f02e26758834d51c055502d7c64 Mon Sep 17 00:00:00 2001 From: omigamedev Date: Sun, 14 Jun 2026 05:29:13 +0200 Subject: [PATCH] Record STR-056 closeout hash --- docs/modernization/debt.md | 4 ++++ docs/modernization/tasks.md | 21 +++++++++++++++++++++ 2 files changed, 25 insertions(+) diff --git a/docs/modernization/debt.md b/docs/modernization/debt.md index d8b80da..28b5f61 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::stroke_commit()` now + routes the retained sequence plan assembly through + `make_canvas_stroke_commit_sequence_plan(...)`; the wrapper still owns the + request dispatch and commit sequencing. - 2026-06-14: `DEBT-0036` was narrowed again. `Canvas::stroke_commit()` now routes the retained request build and execution through `execute_canvas_stroke_commit_request(...)`; the wrapper still owns the diff --git a/docs/modernization/tasks.md b/docs/modernization/tasks.md index d307697..f84e327 100644 --- a/docs/modernization/tasks.md +++ b/docs/modernization/tasks.md @@ -2304,6 +2304,27 @@ Done Checks: - `docs/modernization/debt.md` records the reduced stroke-commit wrapper surface. +### STR-056 - Extract Stroke Commit Plan Assembly + +Status: Done +Score: no score movement +Debt: `DEBT-0036` +Scope: `src/canvas.cpp`, `tests/paint_renderer/stroke_execution_tests.cpp` +Closeout: `7baf377c` + +Goal: + +Move the retained `Canvas::stroke_commit()` plan construction into a helper so +the callsite only supplies current state and dispatches the result. + +Done Checks: + +- `Canvas::stroke_commit()` no longer builds the stroke-commit plan inline. +- Regression coverage proves the helper preserves the plan fields used by the + retained commit request. +- `docs/modernization/debt.md` records the reduced stroke-commit planning + surface. + Validation: ```powershell