refactor: organize application sources by ownership

Author: robertvokac

NEXT.md

@@ -11,6 +11,11 @@ session activated `.github/workflows/ci.yml` and added a root editor build
 and test job alongside the standalone component matrix. See `plan.md` for
 full evidence; older session history remains below and in `docs/history/`._
 
+_Source-layout note (2026-07-25): the editor application implementation is
+now at `src/MeshCraft/Application/` and `src/MeshCraft/Application/UI/`,
+replacing the former flat `src/MeshCraft/MeshCraftApplication_*.cpp` layout.
+Historical references below intentionally retain their then-current paths._
+
 ## 1. Project summary
 
 **MeshCraft** is a desktop 3D scene editor (C++23, built on the CNA game

include/MeshCraft/Application/MeshCraftApplication.hpp

@@ -0,0 +1,20 @@
+#pragma once
+
+#include <MeshCraft/Mc3/Mc3Validation.hpp>
+
+#include <string>
+
+namespace MeshCraft::Application::UI {
+
+struct ValidationContext {
+    bool& visible;
+    const std::string& source;
+    const Mc3::Mc3Validation& validation;
+};
+
+class Validation final {
+public:
+    static void draw(ValidationContext& context);
+};
+
+} // namespace MeshCraft::Application::UI

include/MeshCraft/Application/UI/Validation.hpp

@@ -557,6 +557,34 @@ _All items in this workstream are DONE — archived to [`docs/history/plan_20260
   invalid count clamping. The focused test and both application source files
   that integrate it compiled with one job and ccache disabled to limit local
   disk writes. **Resolved:** working tree, pending commit.
+  **Phase 13 IN PROGRESS (2026-07-25) — application ownership/layout:**
+  began the structural relocation requested for the remaining application
+  implementation. The concrete class now lives at
+  `MeshCraft::Application::MeshCraftApplication`; its public header is
+  `include/MeshCraft/Application/MeshCraftApplication.hpp`, while the old
+  `MeshCraft/MeshCraftApplication.hpp` remains a forwarding compatibility
+  include with a `MeshCraft::MeshCraftApplication` alias. Its implementation
+  moved from the flat `src/MeshCraft/MeshCraftApplication_*.cpp` naming into
+  `src/MeshCraft/Application/` (lifecycle/input/commands/etc.) and
+  `src/MeshCraft/Application/UI/` (Ai, LeftPanel, MenuBar, Overlays,
+  Properties, Registry, Toolbar, Validation). The application sources now
+  use the owning `MeshCraft::Application` namespace; member methods cannot
+  legally be placed in the child `Application::UI` namespace until they are
+  extracted into real UI components with explicit contexts. The first such
+  component is now `Application::UI::Validation`: it receives only a
+  `ValidationContext` (visibility, source label, immutable validation result),
+  while the application retains ownership and mutation of that state. The
+  historical audit references retain their former paths as time-accurate
+  evidence.
+  Static undo-audit and snapshot-lint path checks pass after their tracked
+  source lists were updated. A serial, ccache-disabled `-fsyntax-only`
+  compilation of all 17 relocated application sources plus `main.cpp`, using
+  the existing Debug configuration's flags, also passes without producing
+  object files. Full link/runtime verification remains intentionally pending:
+  the local build directories were removed to reduce SSD writes; do not
+  recreate a large build tree without explicit approval. The next subphase is
+  another narrow `Application::UI` component extraction using the same
+  context boundary.
   **SYS-W3-01 roadmap status after this session's investigation round:**
   Phases 1–12 done (Keybindings, Preferences, MacroRecorder, UndoManager,
   animation-override computation, WalkController, AudioPreview,

include/MeshCraft/MeshCraftApplication.hpp

@@ -1,4 +1,4 @@
-#include "MeshCraft/MeshCraftApplication.hpp"
+#include "MeshCraft/Application/MeshCraftApplication.hpp"
 #include "MeshCraft/EditorAlgorithms.hpp"
 
 #include <imgui.h>
@@ -11,7 +11,7 @@
 #include <string>
 #include <utility>
 
-namespace MeshCraft {
+namespace MeshCraft::Application {
 
 using namespace Microsoft::Xna::Framework;
 
@@ -936,4 +936,4 @@ void MeshCraftApplication::drawTimelinePanel(int screenW, int screenH) {
     ImGui::PopStyleVar();
 }
 
-} // namespace MeshCraft
+} // namespace MeshCraft::Application

plan.md

@@ -1,7 +1,7 @@
-#include "MeshCraft/MeshCraftApplication.hpp"
+#include "MeshCraft/Application/MeshCraftApplication.hpp"
 #include "MeshCraft/EditorAlgorithms.hpp"
 #include "MeshCraft/GraphicsBackendCheck.hpp"
-#include "MeshCraftPrivate.hpp"
+#include "MeshCraft/MeshCraftPrivate.hpp"
 
 #include <imgui.h>
 #include <imgui_impl_sdl3.h>
@@ -38,12 +38,11 @@
 #include <iostream>
 #include <numbers>
 #include <stdexcept>
-#include "MeshCraftPrivate.hpp"
 #include <stb_image.h>
 
-namespace MeshCraft {
+namespace MeshCraft::Application {
 
-GetTypeNameCPP(MeshCraftApplication, "MeshCraft::MeshCraftApplication")
+GetTypeNameCPP(MeshCraftApplication, "MeshCraft::Application::MeshCraftApplication")
 
 using namespace Microsoft::Xna::Framework;
 using namespace Microsoft::Xna::Framework::Input;
@@ -1174,9 +1173,9 @@ void main() {
 }
 )";
 
-} // anonymous namespace
+} // namespace MeshCraft::Application
 
-namespace MeshCraft {
+namespace MeshCraft::Application {
 
 void MeshCraftApplication::initBloom(int w, int h)
 {
@@ -1508,4 +1507,4 @@ void MeshCraftApplication::renderMatPreview(float r, float g, float b,
         matPreviewTextureToken_ = imguiRenderer_->registerTexture(*matPreviewRt_);
 }
 
-} // namespace MeshCraft
+} // namespace MeshCraft::Application

…/MeshCraft/MeshCraftApplication_Anim.cpp src/MeshCraft/Application/Animation.cppsrc/MeshCraft/MeshCraftApplication_Anim.cpp renamed to src/MeshCraft/Application/Animation.cpp src/MeshCraft/MeshCraftApplication_Anim.cpp renamed to src/MeshCraft/Application/Animation.cpp

@@ -21,14 +21,14 @@
 // hook inside SceneRenderer's mesh/texture loaders); left as a smaller
 // follow-up rather than claiming a false precision this pass doesn't have.
 
-#include "MeshCraft/MeshCraftApplication.hpp"
+#include "MeshCraft/Application/MeshCraftApplication.hpp"
 
 #include <chrono>
 #include <functional>
 #include <iostream>
 #include <numeric>
 
-namespace MeshCraft {
+namespace MeshCraft::Application {
 
 namespace {
 template <typename Fn>
@@ -119,4 +119,4 @@ void MeshCraftApplication::runBenchmarkSuite() {
     }
 }
 
-} // namespace MeshCraft
+} // namespace MeshCraft::Application

src/MeshCraft/MeshCraftApplication.cpp src/MeshCraft/Application/Application.cppsrc/MeshCraft/MeshCraftApplication.cpp renamed to src/MeshCraft/Application/Application.cpp

@@ -1,5 +1,5 @@
-#include "MeshCraft/MeshCraftApplication.hpp"
-#include "MeshCraftPrivate.hpp"
+#include "MeshCraft/Application/MeshCraftApplication.hpp"
+#include "MeshCraft/MeshCraftPrivate.hpp"
 #include "MeshCraft/EditorAlgorithms.hpp"
 
 #include <imgui.h>
@@ -32,7 +32,7 @@
 #include <random>
 #include <string>
 
-namespace MeshCraft {
+namespace MeshCraft::Application {
 
 using namespace Microsoft::Xna::Framework;
 using namespace Microsoft::Xna::Framework::Input;
@@ -881,4 +881,4 @@ void MeshCraftApplication::resetPivot() {
     setStatusMsg("Pivot reset to origin", false, 1.5f);
 }
 
-} // namespace MeshCraft
+} // namespace MeshCraft::Application

…Craft/MeshCraftApplication_Benchmark.cpp src/MeshCraft/Application/Benchmark.cppsrc/MeshCraft/MeshCraftApplication_Benchmark.cpp renamed to src/MeshCraft/Application/Benchmark.cpp src/MeshCraft/MeshCraftApplication_Benchmark.cpp renamed to src/MeshCraft/Application/Benchmark.cpp

@@ -1,5 +1,5 @@
-#include "MeshCraft/MeshCraftApplication.hpp"
-#include "MeshCraftPrivate.hpp"
+#include "MeshCraft/Application/MeshCraftApplication.hpp"
+#include "MeshCraft/MeshCraftPrivate.hpp"
 #include "MeshCraft/EditorAlgorithms.hpp"
 #include "MeshCraft/Mc3/Mc3ImportResolver.hpp"
 #include "MeshCraft/Mcb/McbReader.hpp"
@@ -31,7 +31,7 @@
 #include <string>
 #include <vector>
 
-namespace MeshCraft {
+namespace MeshCraft::Application {
 
 void MeshCraftApplication::newScene() {
     document_ = Mc3::Mc3Document{};
@@ -810,4 +810,4 @@ void MeshCraftApplication::savePrefs() {
     savePrefsAlg(path, p);
 }
 
-} // namespace MeshCraft
+} // namespace MeshCraft::Application