Simplify error styling, add human http status

spec/lucky/pretty_log_formatter_spec.cr

@@ -15,7 +15,7 @@ describe Lucky::PrettyLogFormatter do
       io = IO::Memory.new
       format(io, {status: 200, duration: "1.4ms"})
 
-      io.to_s.chomp.should start_with(" #{"▸".colorize.dim} Sent #{"200".colorize.green} (1.4ms)")
+      io.to_s.chomp.should start_with(" #{"▸".colorize.dim} Sent #{"200 Ok".colorize.bold} (1.4ms)")
     end
   end
 
@@ -64,8 +64,8 @@ describe Lucky::PrettyLogFormatter do
 
     format(io, severity: Log::Severity::Error, data: nil, exception: ex)
 
-    io.to_s.should start_with(" #{"▸".colorize.red} #{ex.class.name.colorize.bold.red}")
-    io.to_s.should contain("Details:")
+    io.to_s.should start_with(" #{"▸".colorize.red}")
+    io.to_s.should contain(" #{ex.class.name} ".colorize.bold.on_red.to_s)
   end
 end
 

src/lucky/error_handler.cr

@@ -17,7 +17,7 @@ class Lucky::ErrorHandler
   end
 
   private def call_error_action(context : HTTP::Server::Context, error : Exception) : HTTP::Server::Context
-    Lucky::Log.dexter.error { {exception: error.inspect_with_backtrace} }
+    Lucky::Log.error(exception: error) { "" }
     action.new(context).perform_action(error)
     context
   end

src/lucky/logger_helpers.cr

@@ -1,15 +1,16 @@
 module Lucky::LoggerHelpers
-  def self.colored_status_code(status_code : Int32) : String
+  def self.colored_http_status(status_code : Int32) : String
+    status_name = Wordsmith::Inflector.humanize(HTTP::Status.from_value?(status_code) || "")
+    message = "#{status_code} #{status_name}".colorize.bold
+
     case status_code
-    when 200..399
-      "#{status_code.colorize.green}"
     when 400..499
-      "#{status_code.colorize(:yellow)}"
+      message.yellow
     when 500..599
-      "#{status_code.colorize(:red)}"
+      message.red
     else
-      "#{status_code}"
-    end
+      message
+    end.to_s
   end
 
   def self.elapsed_text(elapsed : Time::Span) : String

src/lucky/pretty_log_formatter.cr

@@ -67,8 +67,8 @@ struct Lucky::PrettyLogFormatter < Dexter::BaseFormatter
 
     def write : Nil
       add_arrow
-      colored_status_code = Lucky::LoggerHelpers.colored_status_code(local_context["status"].as_i)
-      io << "Sent #{colored_status_code} (#{local_context["duration"]})"
+      http_status = Lucky::LoggerHelpers.colored_http_status(local_context["status"].as_i)
+      io << "Sent #{http_status} (#{local_context["duration"]})"
     end
   end
 
@@ -80,19 +80,23 @@ struct Lucky::PrettyLogFormatter < Dexter::BaseFormatter
     def write : Nil
       add_arrow
       entry.exception.try do |ex|
-        io << ex.class.name.colorize.bold.red
-        io << " "
+        io << " #{ex.class.name} ".colorize.bold.on_red
         if ex.message.try(&.lines)
-          io << "\n\n   Details:\n".colorize.bold
+          io << "\n"
           ex.message.try(&.lines).try(&.each do |line|
             io << "\n     "
             io << line
           end)
         end
-        io << "\n\n   Backtrace:\n".colorize.bold if ex.backtrace?
-        (ex.backtrace? || [] of String).each do |trace_line|
-          trace_line = trace_line.colorize.dim unless trace_line.starts_with?(/src|spec/)
-          io << "\n     #{trace_line}"
+        if backtrace = ex.backtrace?
+          io << "\n\n   "
+          io << " Backtrace ".colorize.bold.black.on_white
+          io << "\n"
+          backtrace.each do |trace_line|
+            trace_line = trace_line.colorize.dim unless trace_line.starts_with?(/src|spec/)
+            io << "\n     #{trace_line}"
+          end
+          io << "\n"
         end
       end
     end