feat(cli): show rebuild timing in dev mode

Author: GaspardKirira

include/vix/cli/build/BuildStyle.hpp

@@ -177,6 +177,14 @@ namespace vix::cli::build
       const std::string &message,
       long long milliseconds);
 
+  /** Format a build duration using the build/watch convention. */
+  std::string format_build_duration(long long milliseconds);
+
+  /** Write only a duration, colored when the destination is a TTY. */
+  void write_build_duration(
+      std::ostream &out,
+      long long milliseconds);
+
   void print_task_header_full(
       std::ostream &out,
       const std::string &action,

include/vix/cli/commands/run/dev/DevSession.hpp

@@ -126,7 +126,8 @@ namespace vix::commands::RunCommand::dev
     vix::async::core::task<DevChildRunResult> run_child_once_async(
         vix::async::core::io_context &ctx,
         const fs::path &exePath,
-        vix::async::core::cancel_token ct);
+        vix::async::core::cancel_token ct,
+        std::optional<long long> rebuildDurationMs = std::nullopt);
 
     [[noreturn]] void exec_child_process(const fs::path &exePath) const;
 

src/build/BuildStyle.cpp

@@ -26,11 +26,41 @@
 #include <iomanip>
 
 #include <vix/cli/Style.hpp>
+#include <vix/cli/util/Strings.hpp>
+
+#ifdef _WIN32
+#include <io.h>
+#else
+#include <unistd.h>
+#endif
 
 namespace vix::cli::build
 {
   namespace style = vix::cli::style;
 
+  namespace
+  {
+    bool output_is_tty(std::ostream &out) noexcept
+    {
+      if (&out != &std::cout)
+        return false;
+#ifdef _WIN32
+      return _isatty(_fileno(stdout)) != 0;
+#else
+      return ::isatty(STDOUT_FILENO) != 0;
+#endif
+    }
+
+    const char *duration_color(long long milliseconds) noexcept
+    {
+      if (milliseconds >= 10000)
+        return style::RED;
+      if (milliseconds >= 3000)
+        return style::YELLOW;
+      return style::GREEN;
+    }
+  } // namespace
+
   static std::string colorize(
       const char *color,
       const std::string &value)
@@ -320,6 +350,26 @@ namespace vix::cli::build
     out << "\n";
   }
 
+  std::string format_build_duration(long long milliseconds)
+  {
+    if (milliseconds > 0 && milliseconds < 1000)
+      return std::to_string(milliseconds) + "ms";
+
+    return util::format_seconds(milliseconds);
+  }
+
+  void write_build_duration(
+      std::ostream &out,
+      long long milliseconds)
+  {
+    const std::string duration = format_build_duration(milliseconds);
+    if (output_is_tty(out))
+      out << duration_color(milliseconds) << style::BOLD
+          << duration << style::RESET;
+    else
+      out << duration;
+  }
+
   bool BuildLocation::valid() const
   {
     return !file.empty();

src/commands/run/RunScript.cpp

@@ -22,6 +22,7 @@
 #include <vix/cli/commands/replay/ReplayCapture.hpp>
 #include <vix/cli/commands/replay/ReplayRecorder.hpp>
 #include <vix/cli/errors/RawLogDetectors.hpp>
+#include <vix/cli/build/BuildStyle.hpp>
 #include <vix/cli/Style.hpp>
 #include <vix/cli/util/Ui.hpp>
 #include <vix/utils/Env.hpp>
@@ -37,7 +38,6 @@
 #include <cstring>
 #include <filesystem>
 #include <fstream>
-#include <iomanip>
 #include <iostream>
 #include <mutex>
 #include <nlohmann/json.hpp>
@@ -173,21 +173,6 @@ namespace vix::commands::RunCommand::detail
       std::thread worker_;
     };
 
-    std::string format_single_file_build_duration(std::chrono::milliseconds duration)
-    {
-      const long long milliseconds = duration.count();
-      if (milliseconds < 1000)
-        return std::to_string(milliseconds) + "ms";
-
-      std::ostringstream out;
-      out << std::fixed << std::setprecision(1)
-          << static_cast<double>(milliseconds) / 1000.0;
-      std::string seconds = out.str();
-      if (seconds.size() >= 2 &&
-          seconds.compare(seconds.size() - 2, 2, ".0") == 0)
-        seconds.resize(seconds.size() - 2);
-      return seconds + "s";
-    }
 #endif
 
     void print_double_dash_warning_if_needed(const Options &opt);
@@ -2342,8 +2327,11 @@ namespace vix::commands::RunCommand::detail
       if (isRebuild)
       {
         std::cout << "Rebuilt " << script.filename().string()
-                  << " in " << format_single_file_build_duration(buildDuration)
-                  << "\n" << std::flush;
+                  << " in ";
+        vix::cli::build::write_build_duration(
+            std::cout,
+            buildDuration.count());
+        std::cout << "\n" << std::flush;
       }
 
       const auto childStart = Clock::now();

src/commands/run/dev/DevSession.cpp

@@ -15,6 +15,7 @@
  */
 
 #include <vix/cli/commands/run/dev/DevSession.hpp>
+#include <vix/cli/build/BuildStyle.hpp>
 #include <vix/cli/Style.hpp>
 #include <vix/cli/commands/run/RunScriptHelpers.hpp>
 #include <vix/cli/errors/RawLogDetectors.hpp>
@@ -112,13 +113,21 @@ namespace vix::commands::RunCommand::dev
                 << "\n\n";
     }
 
-    void print_dev_started(int pid)
+    void print_dev_started(
+        int pid,
+        std::optional<long long> rebuildDurationMs)
     {
       std::cout << "  "
-                << GREEN << "✔" << RESET
-                << " Started"
-                << GRAY << " pid=" << pid << RESET
-                << "\n";
+                << GREEN << "✔" << RESET << " ";
+      if (rebuildDurationMs)
+      {
+        std::cout << "Rebuilt in ";
+        vix::cli::build::write_build_duration(
+            std::cout,
+            *rebuildDurationMs);
+        std::cout << GRAY << " · " << RESET;
+      }
+      std::cout << "Started" << GRAY << " pid=" << pid << RESET << "\n";
     }
 
     void print_dev_app_exited_cleanly()
@@ -284,9 +293,18 @@ namespace vix::commands::RunCommand::dev
               << GRAY << " (dev)" << RESET
               << "\n";
 
+    fs::path changedPath = change.path;
+    if (!dev_verbose_ui(options_))
+    {
+      std::error_code ec;
+      const fs::path relative = fs::relative(change.path, options_.projectDir, ec);
+      if (!ec && !relative.empty())
+        changedPath = relative;
+    }
+
     std::cout << "  "
               << GRAY << "changed: " << RESET
-              << change.path.string()
+              << changedPath.generic_string()
               << "\n";
   }
 
@@ -378,10 +396,19 @@ namespace vix::commands::RunCommand::dev
     {
       const DevChangeKind rebuildKind = pendingChangeKind_;
       pendingChangeKind_ = DevChangeKind::Ignore;
+      const bool isRebuild = rebuildKind != DevChangeKind::Ignore;
+      const auto rebuildStart = std::chrono::steady_clock::now();
 
       DevRebuilderResult rebuildResult =
           co_await rebuild_async(ctx, rebuildKind, ct);
 
+      const std::optional<long long> rebuildDurationMs = isRebuild && rebuildResult.ok
+                                                              ? std::optional<long long>(
+                                                                    std::chrono::duration_cast<std::chrono::milliseconds>(
+                                                                        std::chrono::steady_clock::now() - rebuildStart)
+                                                                        .count())
+                                                              : std::nullopt;
+
       if (ct.is_cancelled())
       {
         result.exitCode = 130;
@@ -448,7 +475,7 @@ namespace vix::commands::RunCommand::dev
       }
 
       const DevChildRunResult childResult =
-          co_await run_child_once_async(ctx, *exePath, ct);
+          co_await run_child_once_async(ctx, *exePath, ct, rebuildDurationMs);
 
       if (childResult.reason == DevChildExitReason::RestartRequested)
         continue;
@@ -859,7 +886,8 @@ namespace vix::commands::RunCommand::dev
   vix::async::core::task<DevChildRunResult> DevSession::run_child_once_async(
       vix::async::core::io_context &ctx,
       const fs::path &exePath,
-      vix::async::core::cancel_token ct)
+      vix::async::core::cancel_token ct,
+      std::optional<long long> rebuildDurationMs)
   {
     using Clock = std::chrono::steady_clock;
 
@@ -907,7 +935,7 @@ namespace vix::commands::RunCommand::dev
     std::string runtimeLog;
 
     if (!options_.quiet)
-      print_dev_started(static_cast<int>(pid));
+      print_dev_started(static_cast<int>(pid), rebuildDurationMs);
 
     // Do not make the first observable child output wait for a complete file
     // polling period. File indexing remains at pollInterval; this short,