feat: split EditorAlgorithms.hpp into 7 cohesive headers (SYS-W3-05)

Decompose the 2,514-line EditorAlgorithms.hpp into EditorCommandAlgorithms
(857 lines), EditorSelectionAlgorithms (326), EditorTransformAlgorithms
(363), EditorPersistenceAlgorithms (479), EditorEventAlgorithms (308),
EditorPreferencesAlgorithms (238) and EditorUtilityAlgorithms (108),
grouped by verified call-site cross-reference rather than the original
proposal's guess (findParentListAlg/removeFromListAlg moved into Commands,
not Selection, since grep showed every caller is a Commands/Transform
mutator). Every function/struct body is byte-identical, confirmed by a
line-range diff against the original before deleting it; the +165 total
lines are entirely the 7x duplicated pragma/comment/include preamble.

Migrated all 30 files the original grep flagged: 5 were comment-only false
positives needing no change, Macro.cpp's include was dead and removed
outright, and of the 25 real consumers 14 need exactly 1 new header, 8
need 2, 1 needs 3, and the largest (Commands.cpp) needs 4 - no consumer
was left including everything, and no facade header was left behind.

MeshCraft and all 13 affected test targets built clean with zero missing-
include errors. ctest -LE render: 181 tests, 163 passed; the only failures
are the two pre-existing, unrelated exceptions (mc3_json_document_budget
Debug-build timeout, 3 Blender-numpy import tests) plus 14 mcb_*/
mc3togltf_* tests that were simply never built in this reused
cmake-build-debug (spot-built 3 to confirm - pre-existing gap, not a
regression; none consume any Editor*Algorithms.hpp header).

Author: robertvokac

NEXT.md

@@ -126,6 +126,27 @@ on `develop` and `plan.md`'s own `[DONE]`/`[IN_PROGRESS]` markers for the
 true current state before assuming
 anything below this point is stale.
 
+`SYS-W11-09` and `SYS-W11-10` landed (see `plan.md`, both `[DONE]`). Then
+**`SYS-W3-05`** (real refactor, not cosmetic): split the 2,514-line
+`EditorAlgorithms.hpp` into 7 cohesive, CNA-free headers
+(`EditorCommandAlgorithms.hpp`/`EditorSelectionAlgorithms.hpp`/
+`EditorTransformAlgorithms.hpp`/`EditorPersistenceAlgorithms.hpp`/
+`EditorEventAlgorithms.hpp`/`EditorPreferencesAlgorithms.hpp`/
+`EditorUtilityAlgorithms.hpp`), moved `findParentListAlg`/`removeFromListAlg`
+into Commands (not Selection as first proposed) once grep proved every call
+site is a Commands/Transform mutator, and migrated all 30 flagged consumer
+files individually — 14 now need exactly 1 new header, 8 need 2, 1 needs 3,
+1 (`Commands.cpp`) needs 4; one (`Macro.cpp`) had a dead include removed
+outright; 5 were comment-only false positives needing no change. No facade
+left behind; the original header is deleted. `MeshCraft` plus all 13
+affected test targets built clean with zero missing-include errors;
+`ctest -LE render` is 181 tests / 163 passed, with only the two
+already-known pre-existing exceptions (`mc3_json_document_budget` timeout,
+3 Blender-`numpy` import tests) plus 14 unrelated `mcb_*`/`mc3togltf_*`
+tests that were simply never built in this reused `cmake-build-debug`
+(spot-built 3 to confirm — pre-existing gap, not a regression). See
+`plan.md`'s `SYS-W3-05` entry for full line-count/fan-out evidence.
+
 ## Known release blockers and decisions
 
 | Area | Live state |

include/MeshCraft/Editor/EventPreviewRunner.hpp

@@ -1,7 +1,8 @@
 #pragma once
 
 #include "MeshCraft/Editor/LuaScriptRunner.hpp"
-#include "MeshCraft/EditorAlgorithms.hpp"
+#include "MeshCraft/EditorCommandAlgorithms.hpp"
+#include "MeshCraft/EditorSelectionAlgorithms.hpp"
 #include "MeshCraft/EventBindingAlgorithms.hpp"
 #include "MeshCraft/Mc3/Mc3Validation.hpp"
 

include/MeshCraft/Editor/ObjectTypeName.hpp

@@ -6,15 +6,17 @@
 // This is the single source of truth. It replaces two divergent copies that
 // both silently mishandled five primitive types:
 //   * objectTypeName()     in src/MeshCraft/MeshCraftPrivate.hpp
-//   * objectTypeNameAlg()  in include/MeshCraft/EditorAlgorithms.hpp
+//   * objectTypeNameAlg()  in include/MeshCraft/EditorCommandAlgorithms.hpp
+//                          (formerly EditorAlgorithms.hpp before its SYS-W3-05
+//                          split into per-concern headers)
 // Both used `default: return "Object"` and omitted Torus, Capsule, Disk, Grid
 // and IcoSphere, so those types displayed as "Object" in the outliner and,
 // critically, the macro recorder wrote `add Object` for them. On replay
 // objectTypeFromName("Object") fell through to Box, so recording "add Torus"
 // and replaying it produced a Box — a silent, lossy round-trip.
 //
-// Depends only on Mc3Object.hpp (the enum), so it is safe to include from the
-// CNA/ImGui-free EditorAlgorithms.hpp and unit-testable standalone.
+// Depends only on Mc3Object.hpp (the enum), so it is safe to include from any
+// CNA/ImGui-free Editor*Algorithms.hpp header and unit-testable standalone.
 
 #include <MeshCraft/Mc3/Mc3Object.hpp>