Polish modules command output

Author: GaspardKirira

src/commands/ModulesCommand.cpp

@@ -113,7 +113,6 @@ namespace vix::commands::ModulesCommand
 
     if (opt.subcmd == "init")
     {
-      ui::section(std::cout, "Modules");
       return cmds::cmd_init(root, opt.patchRoot) ? 0 : 1;
     }
 
@@ -125,13 +124,12 @@ namespace vix::commands::ModulesCommand
         ui::warn_line(std::cout, "Usage: vix modules add <name>");
         return 1;
       }
-      ui::section(std::cout, "Modules");
+
       return cmds::cmd_add(root, project, opt.module, opt.patchLink) ? 0 : 1;
     }
 
     if (opt.subcmd == "check")
     {
-      ui::section(std::cout, "Modules");
       return cmds::cmd_check(root, project) ? 0 : 1;
     }
 

src/commands/NewCommand.cpp

@@ -349,8 +349,8 @@ namespace vix::commands::NewCommand
         << "  • Generates a ready-to-run Vix project\n"
         << "  • Sets up CMake, source structure, and config files\n"
         << "  • Creates a vix.json manifest\n"
-        << "  • For apps, includes default tasks and an empty deps list\n"
-        << "  • For libraries, includes package metadata and an empty deps list\n"
+        << "  • For apps, creates an executable target matching the project name\n"
+        << "  • For libraries, creates a header-only CMake interface target\n"
         << "  • Applies the selected template (app or library)\n\n"
 
         << "Options\n"
@@ -359,6 +359,17 @@ namespace vix::commands::NewCommand
         << "  -d, --dir   Base directory for project creation\n"
         << "  --force     Overwrite existing directory\n\n"
 
+        << "Application workflow\n"
+        << "  cd api/\n"
+        << "  vix build\n"
+        << "  vix run\n\n"
+
+        << "Library workflow\n"
+        << "  cd tree/\n"
+        << "  vix build --build-target all\n"
+        << "  vix build --build-target all -- -Dtree_BUILD_TESTS=ON\n"
+        << "  vix tests\n\n"
+
         << "Environment\n"
         << "  VIX_NONINTERACTIVE=1   Disable prompts\n"
         << "  CI=1                   Disable prompts\n\n"
@@ -368,6 +379,8 @@ namespace vix::commands::NewCommand
         << "  • Use 'vix add <pkg>' to add dependencies\n"
         << "  • Use 'vix install' to install from vix.lock\n"
         << "  • Use 'vix task <name>' to run generated tasks\n"
+        << "  • For header-only libraries, use '--build-target all' to build the generated project\n"
+        << "  • Tests for header-only libraries are disabled by default and must be enabled with CMake args\n"
         << "  • Designed for fast start with zero setup\n";
 
     return 0;

src/commands/modules/ModulesCommands.cpp

@@ -28,6 +28,44 @@ namespace vix::commands::modules_cmd::commands
   namespace ui = vix::cli::util;
   using namespace vix::cli::style;
 
+  static constexpr const char *TEAL = "\033[38;5;35m";
+
+  static void sep()
+  {
+    std::cout << PAD << GRAY << "─────────────────────────────────────" << RESET << "\n";
+  }
+
+  static void section(const std::string &title)
+  {
+    std::cout << PAD << GRAY << title << RESET << "\n";
+  }
+
+  static void print_command_step(
+      int index,
+      const std::string &cmd,
+      const std::string &hint = "")
+  {
+    std::cout << PAD
+              << GRAY << index << RESET
+              << "  "
+              << CYAN << BOLD << cmd << RESET;
+
+    if (!hint.empty())
+      std::cout << "  " << GRAY << hint << RESET;
+
+    std::cout << "\n";
+  }
+
+  static void print_modules_banner(
+      const std::string &name,
+      const std::string &kind)
+  {
+    std::cout << PAD << TEAL << BOLD << "✔" << RESET
+              << "  " << TEAL << BOLD << name << RESET
+              << "  " << GRAY << kind << RESET
+              << "\n";
+  }
+
   bool cmd_init(const fs::path &root, bool patchRoot)
   {
     const fs::path modulesDir = root / "modules";
@@ -58,12 +96,19 @@ namespace vix::commands::modules_cmd::commands
       return false;
     }
 
-    ui::ok_line(std::cout, "Modules mode initialized");
-    ui::kv(std::cout, "modules", (root / "modules").string(), 10);
-    ui::kv(std::cout, "cmake", (root / "cmake" / "vix_modules.cmake").string(), 10);
+    print_modules_banner("modules", "initialized");
+    sep();
+
+    section("files");
+    print_command_step(1, "modules/", "module directory");
+    print_command_step(2, "cmake/vix_modules.cmake", "module loader");
+
     if (patchRoot)
-      ui::kv(std::cout, "patch", "root CMakeLists.txt updated (idempotent markers)", 10);
-    ui::warn_line(std::cout, "Next: vix modules add <name>");
+      print_command_step(3, "CMakeLists.txt", "patched");
+
+    sep();
+    section("next");
+    print_command_step(1, "vix modules add <name>", "create module");
 
     return true;
   }
@@ -148,20 +193,47 @@ namespace vix::commands::modules_cmd::commands
       return false;
     }
 
-    ui::ok_line(std::cout, "Module created");
-    ui::kv(std::cout, "name", normalized, 10);
-    ui::kv(std::cout, "target", cnt::module_alias_name(project, module), 10);
-    ui::kv(std::cout, "public", "modules/" + normalized + "/include/" + normalized + "/api.hpp", 10);
-    ui::kv(std::cout, "impl", "modules/" + normalized + "/src/" + normalized + ".cpp", 10);
+    print_modules_banner(normalized, "module");
+    sep();
+
+    section("files");
+    print_command_step(
+        1,
+        "modules/" + normalized + "/include/" + normalized + "/api.hpp",
+        "public header");
+
+    print_command_step(
+        2,
+        "modules/" + normalized + "/src/" + normalized + ".cpp",
+        "implementation");
+
+    print_command_step(
+        3,
+        "modules/" + normalized + "/CMakeLists.txt",
+        "target");
+
+    sep();
+    section("target");
+    print_command_step(1, cnt::module_alias_name(project, module), "CMake alias");
+
+    sep();
+    section("next");
+    print_command_step(
+        1,
+        "#include <" + normalized + "/api.hpp>",
+        "include");
 
-    ui::warn_line(std::cout, "Next steps (CMake):");
-    std::cout << "    " << GRAY << "• " << RESET
-              << "Include: " << YELLOW << BOLD << "#include <" << normalized << "/api.hpp>" << RESET << "\n";
     if (patchRootLink)
-      std::cout << "    " << GRAY << "• " << RESET
-                << "Root: " << GRAY
-                << "(auto-linked if main target is named like project(" << project << "))"
-                << RESET << "\n";
+    {
+      print_command_step(2, "vix build", "compile");
+    }
+    else
+    {
+      print_command_step(
+          2,
+          "target_link_libraries(" + project + " PRIVATE " + cnt::module_alias_name(project, module) + ")",
+          "link manually");
+    }
 
     return true;
   }

src/commands/new/NewOutput.cpp

@@ -73,19 +73,19 @@ namespace vix::commands::new_cmd::output
     print_command_step(1, "cd " + projName + "/", "enter project");
     print_command_step(2, "vix build", "compile");
     print_command_step(3, "vix run", "start app");
-
-    std::cout << "\n";
   }
 
   static void print_steps_lib(const std::string &projName)
   {
     section("next");
 
     print_command_step(1, "cd " + projName + "/", "enter project");
-    print_command_step(2, "vix build", "compile");
-    print_command_step(3, "vix tests", "run tests");
-
-    std::cout << "\n";
+    print_command_step(2, "vix build --build-target all", "compile");
+    print_command_step(
+        3,
+        "vix build --build-target all -- -D" + projName + "_BUILD_TESTS=ON",
+        "enable tests");
+    print_command_step(4, "vix tests", "run tests");
   }
 
   // ------------------------------------------------------------------

src/commands/new/NewTemplates.cpp

@@ -251,6 +251,12 @@ namespace vix::commands::new_cmd::templates
     readme += "```\n\n";
 
     readme += "## Tests\n\n";
+    readme += "Tests are disabled by default for header-only libraries.\n\n";
+    readme += "Enable them first:\n\n";
+    readme += "```bash\n";
+    readme += "vix build --build-target all -- -D" + name + "_BUILD_TESTS=ON\n";
+    readme += "```\n\n";
+    readme += "Then run:\n\n";
     readme += "```bash\n";
     readme += "vix tests\n";
     readme += "```\n\n";

src/errors/build/CMakeBuildErrors.cpp

@@ -43,6 +43,48 @@ namespace vix::cli::errors::build
       return {};
     }
 
+    static bool handle_unknown_ninja_target(std::string_view log)
+    {
+      constexpr std::string_view marker = "ninja: error: unknown target '";
+
+      const std::size_t pos = log.find(marker);
+
+      if (pos == std::string_view::npos)
+        return false;
+
+      const std::size_t start = pos + marker.size();
+      const std::size_t end = log.find('\'', start);
+
+      std::string target = "unknown";
+
+      if (end != std::string_view::npos && end > start)
+        target = std::string(log.substr(start, end - start));
+
+      std::cerr << PAD
+                << RED << BOLD << "✖ Build target not found" << RESET
+                << "\n";
+
+      std::cerr << PAD
+                << CYAN << "target:" << RESET
+                << " "
+                << target
+                << "\n";
+
+      std::cerr << PAD
+                << YELLOW << "hint:" << RESET
+                << " "
+                << "This project does not define a CMake target named '"
+                << target
+                << "'.\n";
+
+      std::cerr << PAD
+                << "      "
+                << "Check your CMakeLists.txt or use --build-target with an existing target."
+                << "\n";
+
+      return true;
+    }
+
     bool contains(std::string_view log, std::string_view needle)
     {
       return log.find(needle) != std::string_view::npos;
@@ -570,6 +612,9 @@ namespace vix::cli::errors::build
 
   bool handleCMakeBuildError(std::string_view log)
   {
+    if (handle_unknown_ninja_target(log))
+      return true;
+
     if (handleCacheMismatch(log))
       return true;