doing 2.1.89 → 2.1.90

data/lib/doing/completion/fig_completion.rb

@@ -13,15 +13,15 @@ module Doing
       attr_accessor :commands, :global_options
 
       def generate_helpers
-        out=<<~EOFUNCTIONS
-        const completionSpec: Fig.Spec = {
-          name: "doing",
-          description: "A CLI for a What Was I Doing system",
-          subcommands: [
-            #{generate_subcommand_completions.join("\n    ")}
-          ],
-        };
-        export default completionSpec;
+        out = <<~EOFUNCTIONS
+          const completionSpec: Fig.Spec = {
+            name: "doing",
+            description: "A CLI for a What Was I Doing system",
+            subcommands: [
+              #{generate_subcommand_completions.join("\n    ")}
+            ],
+          };
+          export default completionSpec;
         EOFUNCTIONS
         @bar.advance(status: '✅')
         @bar.finish
@@ -62,34 +62,33 @@ module Doing
             arg = ''
 
             if option[:arg]
-              arg =<<~EOARG
-              args: {
-              #{indent}        name: "#{option[:arg]}",
-              #{indent}        description: "#{option[:arg]}",
-              #{indent}  },
+              arg = <<~EOARG
+                args: {
+                #{indent}        name: "#{option[:arg]}",
+                #{indent}        description: "#{option[:arg]}",
+                #{indent}  },
               EOARG
             end
 
-            if option[:short]
-              opt_data =<<~EOOPT
-              {
-              #{indent}  name: ["-#{option[:short]}", "--#{option[:long]}"],
-              #{indent}  description: "#{option[:description].sanitize}",
-              #{indent}  #{arg}
-              #{indent}},
-              EOOPT
-            else
-              opt_data = <<~EOOPT
-              {
-              #{indent}  name: ["--#{option[:long]}"],
-              #{indent}  description: "#{option[:description].sanitize}",
-              #{indent}  #{arg}
-              #{indent}},
-              EOOPT
-            end
+            opt_data = if option[:short]
+                         <<~EOOPT
+                           {
+                           #{indent}  name: ["-#{option[:short]}", "--#{option[:long]}"],
+                           #{indent}  description: "#{option[:description].sanitize}",
+                           #{indent}  #{arg}
+                           #{indent}},
+                         EOOPT
+                       else
+                         <<~EOOPT
+                           {
+                           #{indent}  name: ["--#{option[:long]}"],
+                           #{indent}  description: "#{option[:description].sanitize}",
+                           #{indent}  #{arg}
+                           #{indent}},
+                         EOOPT
+                       end
 
             option_arr << opt_data
-
           end
 
           cmd_opts = <<~EOCMD
@@ -107,7 +106,8 @@ module Doing
         data = Completion.get_help_sections
         @global_options = Completion.parse_options(data[:global_options])
         @commands = Completion.parse_commands(data[:commands])
-        @bar = TTY::ProgressBar.new(" \033[0;0;33mGenerating Fig completions: \033[0;35;40m[:bar] :status\033[0m", total: @commands.count + 1, bar_format: :square, hide_cursor: true, status: 'processing subcommands')
+        @bar = TTY::ProgressBar.new(" \033[0;0;33mGenerating Fig completions: \033[0;35;40m[:bar] :status\033[0m",
+                                    total: @commands.count + 1, bar_format: :square, hide_cursor: true, status: 'processing subcommands')
         width = TTY::Screen.columns - 45
         @bar.resize(width)
       end

data/lib/doing/completion/fish_completion.rb

@@ -4,7 +4,6 @@ module Doing
   module Completion
     # Generate completions for Fish
     class FishCompletions
-
       attr_accessor :commands, :global_options
 
       def generate_helpers
@@ -152,7 +151,6 @@ module Doing
       end
 
       def generate_subcommand_option_completions
-
         out = []
         need_export = []
         need_bool = []
@@ -163,32 +161,32 @@ module Doing
         need_age = []
         need_section = []
 
-        @commands.each_with_index do |cmd, i|
+        @commands.each_with_index do |cmd, _i|
           @bar.advance(status: cmd[:commands].first)
           data = Completion.get_help_sections(cmd[:commands].first)
 
           if data[:synopsis].join(' ').strip.split(/ /).last =~ /(path|file)/i
-            out << "complete -c doing -F -n '__fish_doing_using_command #{cmd[:commands].join(" ")}'"
+            out << "complete -c doing -F -n '__fish_doing_using_command #{cmd[:commands].join(' ')}'"
           end
 
-          if data[:command_options]
-            Completion.parse_options(data[:command_options]).each do |option|
-              next if option.nil?
-
-              arg = option[:arg] ? '-r' : ''
-              short = option[:short] ? "-s #{option[:short]}" : ''
-              long = option[:long] ? "-l #{option[:long]}" : ''
-              out << "complete -c doing #{long} #{short} -f #{arg} -n '__fish_doing_using_command #{cmd[:commands].join(' ')}' -d #{Shellwords.escape(option[:description])}"
-
-              need_export.concat(cmd[:commands]) if option[:long] == 'output'
-              need_bool.concat(cmd[:commands]) if option[:long] == 'bool'
-              need_case.concat(cmd[:commands]) if option[:long] == 'case'
-              need_sort.concat(cmd[:commands]) if option[:long] == 'sort'
-              need_tag_sort.concat(cmd[:commands]) if option[:long] == 'tag_sort'
-              need_tag_order.concat(cmd[:commands]) if option[:long] == 'tag_order'
-              need_age.concat(cmd[:commands]) if option[:long] == 'age'
-              need_section.concat(cmd[:commands]) if option[:long] == 'section'
-            end
+          next unless data[:command_options]
+
+          Completion.parse_options(data[:command_options]).each do |option|
+            next if option.nil?
+
+            arg = option[:arg] ? '-r' : ''
+            short = option[:short] ? "-s #{option[:short]}" : ''
+            long = option[:long] ? "-l #{option[:long]}" : ''
+            out << "complete -c doing #{long} #{short} -f #{arg} -n '__fish_doing_using_command #{cmd[:commands].join(' ')}' -d #{Shellwords.escape(option[:description])}"
+
+            need_export.concat(cmd[:commands]) if option[:long] == 'output'
+            need_bool.concat(cmd[:commands]) if option[:long] == 'bool'
+            need_case.concat(cmd[:commands]) if option[:long] == 'case'
+            need_sort.concat(cmd[:commands]) if option[:long] == 'sort'
+            need_tag_sort.concat(cmd[:commands]) if option[:long] == 'tag_sort'
+            need_tag_order.concat(cmd[:commands]) if option[:long] == 'tag_order'
+            need_age.concat(cmd[:commands]) if option[:long] == 'age'
+            need_section.concat(cmd[:commands]) if option[:long] == 'section'
           end
         end
 
@@ -232,7 +230,8 @@ module Doing
         data = Completion.get_help_sections
         @global_options = Completion.parse_options(data[:global_options])
         @commands = Completion.parse_commands(data[:commands])
-        @bar = TTY::ProgressBar.new("\033[0;0;33mGenerating Fish completions: \033[0;35;40m[:bar] :status\033[0m", total: @commands.count + 1, bar_format: :square, hide_cursor: true, status: 'processing subcommands')
+        @bar = TTY::ProgressBar.new("\033[0;0;33mGenerating Fish completions: \033[0;35;40m[:bar] :status\033[0m",
+                                    total: @commands.count + 1, bar_format: :square, hide_cursor: true, status: 'processing subcommands')
         width = TTY::Screen.columns - 45
         @bar.resize(width)
       end

data/lib/doing/completion/zsh_completion.rb

@@ -13,7 +13,7 @@ module Doing
       attr_accessor :commands, :global_options
 
       def generate_helpers
-        out=<<~EOFUNCTIONS
+        out = <<~EOFUNCTIONS
           compdef _doing doing
 
           function _doing() {
@@ -49,7 +49,7 @@ module Doing
 
       def generate_subcommand_completions
         out = []
-        @commands.each_with_index do |cmd, i|
+        @commands.each_with_index do |cmd, _i|
           cmd[:commands].each do |c|
             out << "'#{c}:#{cmd[:description].gsub(/'/, '\\\'')}'"
           end
@@ -58,10 +58,9 @@ module Doing
       end
 
       def generate_subcommand_option_completions(indent: '        ')
-
         out = []
 
-        @commands.each_with_index do |cmd, i|
+        @commands.each_with_index do |cmd, _i|
           @bar.advance(status: cmd[:commands].first)
 
           data = Completion.get_help_sections(cmd[:commands].first)
@@ -93,7 +92,8 @@ module Doing
         data = Completion.get_help_sections
         @global_options = Completion.parse_options(data[:global_options])
         @commands = Completion.parse_commands(data[:commands])
-        @bar = TTY::ProgressBar.new(" \033[0;0;33mGenerating Zsh completions: \033[0;35;40m[:bar] :status\033[0m", total: @commands.count + 1, bar_format: :square, hide_cursor: true, status: 'processing subcommands')
+        @bar = TTY::ProgressBar.new(" \033[0;0;33mGenerating Zsh completions: \033[0;35;40m[:bar] :status\033[0m",
+                                    total: @commands.count + 1, bar_format: :square, hide_cursor: true, status: 'processing subcommands')
         width = TTY::Screen.columns - 45
         @bar.resize(width)
       end

data/lib/doing/completion.rb

@@ -16,7 +16,7 @@ module Doing
     COMMAND_RX = /^(?<cmd>[^, \t]+)(?<alias>(?:, [^, \t]+)*)?\s+- (?<desc>.*?)$/.freeze
 
     class << self
-      def get_help_sections(command = "")
+      def get_help_sections(command = '')
         res = `doing help #{command}|command cat`.strip
         scanned = res.scan(SECTIONS_RX)
         sections = {}
@@ -97,8 +97,10 @@ module Doing
         FileUtils.mkdir_p(default_dir)
         src = File.expand_path(File.join(File.dirname(__FILE__), '..', 'completion', default_filenames[type]))
 
-        if File.exist?(File.join(default_dir, default_filenames[type]))
-          return unless Doing::Prompt.yn("Update #{type} completion script", default_response: 'n')
+        if File.exist?(File.join(default_dir,
+                                 default_filenames[type])) && !Doing::Prompt.yn("Update #{type} completion script",
+                                                                                default_response: 'n')
+          return
 
         end
 
@@ -203,7 +205,8 @@ module Doing
         case type.to_s
         when /^b/i
           unless dir =~ %r{(\.bash_it/completion|bash_completion/completions)}
-            link_completion(file, ['~/.bash_it/completion/enabled', '/usr/share/bash-completion/completions', '/usr/share/bash_completion/completions'], 'doing.bash')
+            link_completion(file,
+                            ['~/.bash_it/completion/enabled', '/usr/share/bash-completion/completions', '/usr/share/bash_completion/completions'], 'doing.bash')
           end
         when /^fig/i
         when /^f/i

data/lib/doing/configuration.rb

@@ -116,7 +116,7 @@ module Doing
       'interaction' => {
         'confirm_longer_than' => '5h'
       }
-    }
+    }.freeze
 
     def initialize(file = nil, options: {})
       @config_file = file.nil? ? default_config_file : File.expand_path(file)
@@ -237,11 +237,9 @@ module Doing
           if new_cfg.nil?
             return real_path if real_path[-1] == path && real_path.count == element_count
 
-            if distance < 5 && !create
-              return resolve_key_path(keypath, create: false, distance: distance + 1)
-            else
-              return nil unless create
-            end
+            return resolve_key_path(keypath, create: false, distance: distance + 1) if distance < 5 && !create
+
+            return nil unless create
 
             resolved = real_path.count.positive? ? "Resolved #{real_path.join('.')}, but " : ''
             Doing.logger.log_now(:warn, "#{resolved}#{path} is unknown")
@@ -251,13 +249,19 @@ module Doing
             raise InvalidArgument, 'Invalid key path' unless res
 
             real_path.push(path).concat(paths).compact!
-            Doing.logger.debug('Config:', "translated key path #{keypath} to #{real_path.join('.')}") unless keypath == real_path.join('.')
+            unless keypath == real_path.join('.')
+              Doing.logger.debug('Config:',
+                                 "translated key path #{keypath} to #{real_path.join('.')}")
+            end
             return real_path
           end
           cfg = new_cfg
         end
       end
-      Doing.logger.debug('Config:', "translated key path #{keypath} to #{real_path.join('.')}") unless keypath == real_path.join('.')
+      unless keypath == real_path.join('.')
+        Doing.logger.debug('Config:',
+                           "translated key path #{keypath} to #{real_path.join('.')}")
+      end
       real_path
     end
 
@@ -295,7 +299,8 @@ module Doing
     #
     def from(user_config)
       # Util.deep_merge_hashes(DEFAULTS, Configuration[user_config].stringify_keys)
-      Configuration[user_config].stringify_keys.deep_merge(DEFAULTS, { extend_existing_arrays: true, sort_merged_arrays: true })
+      Configuration[user_config].stringify_keys.deep_merge(DEFAULTS,
+                                                           { extend_existing_arrays: true, sort_merged_arrays: true })
     end
 
     ##
@@ -346,7 +351,7 @@ module Doing
 
       load_plugins(plugin_config['plugin_path'])
 
-      Plugins.plugins.each do |_type, plugins|
+      Plugins.plugins.each_value do |plugins|
         plugins.each do |title, plugin|
           plugin_config[title] = plugin[:config] if plugin[:config].good?
           config['export_templates'][title] ||= nil if plugin[:templates] && !plugin[:templates].empty?
@@ -366,7 +371,10 @@ module Doing
 
       Hooks.trigger :post_config, self
 
-      config = local_config.deep_merge(config, { extend_existing_arrays: true, sort_merged_arrays: true }) unless @ignore_local
+      unless @ignore_local
+        config = local_config.deep_merge(config,
+                                         { extend_existing_arrays: true, sort_merged_arrays: true })
+      end
       # config = Util.deep_merge_hashes(config, local_config) unless @ignore_local
 
       Hooks.trigger :post_local_config, self
@@ -483,13 +491,13 @@ module Doing
       end
 
       begin
-
         user_config = Util.safe_load_file(config_file)
         raise StandardError, 'Invalid config file format' unless user_config.is_a?(Hash)
 
         if user_config.key?('html_template')
           user_config['export_templates'] ||= {}
-          user_config['export_templates'].deep_merge(user_config.delete('html_template'), { extend_existing_arrays: true, sort_merged_arrays: true })
+          user_config['export_templates'].deep_merge(user_config.delete('html_template'),
+                                                     { extend_existing_arrays: true, sort_merged_arrays: true })
         end
 
         user_config['include_notes'] = user_config.delete(':include_notes') if user_config.key?(':include_notes')

data/lib/doing/errors.rb

@@ -6,9 +6,7 @@ module Doing
       def initialize(msg = nil, level: nil, topic: 'Error:', exit_code: 1)
         level ||= :error
         Doing.logger.output_results
-        if msg
-          Doing.logger.log_now(level, topic, msg)
-        end
+        Doing.logger.log_now(level, topic, msg) if msg
 
         Process.exit exit_code
       end
@@ -52,7 +50,6 @@ module Doing
     class NoResults < DoingNoTraceError
       def initialize(msg = 'No results', topic = 'Exited:')
         super(msg, level: :warn, topic: topic, exit_code: 0)
-
       end
     end
 

data/lib/doing/good.rb

@@ -18,7 +18,7 @@ module Doing
     ##             has content
     ##
     def good?
-      !nil? && !self&.empty? || false
+      !nil? && !empty? || false
     end
   end
 

data/lib/doing/hash.rb

@@ -153,7 +153,7 @@ module Doing
     ## Remove keys with empty values
     ##
     def remove_empty
-      delete_if { |k, v| !v.is_a?(FalseClass) && !v.good? }
+      delete_if { |_k, v| !v.is_a?(FalseClass) && !v.good? }
     end
 
     def tag_filter_to_options
@@ -193,9 +193,9 @@ module Doing
     ## @param      to_delete  [Array] the keys to delete if key doesn't exist
     ##
     def delete_unless_key(key, to_delete)
-      unless key?(key)
-        to_delete.each { |k| delete(k) }
-      end
+      return if key?(key)
+
+      to_delete.each { |k| delete(k) }
     end
   end
 end

data/lib/doing/help_monkey_patch.rb

@@ -4,7 +4,7 @@ module GLI
   module Commands
     # Help Command Monkeypatch for paginated output
     class Help < Command
-      def show_help(global_options, options, arguments, out, error)
+      def show_help(_global_options, options, arguments, out, error)
         Doing::Pager.paginate = true
 
         command_finder = HelpModules::CommandFinder.new(@app, arguments, error)

data/lib/doing/hooks.rb

@@ -40,10 +40,14 @@ module Doing
     # register a single hook to be called later, internal API
     def self.register_one(event, priority, &block)
       unless @registry[event]
-        raise Doing::Errors::HookUnavailable.new("Invalid hook. Doing only supports #{@registry.keys.inspect}", 'hook', event)
+        raise Doing::Errors::HookUnavailable.new("Invalid hook. Doing only supports #{@registry.keys.inspect}", 'hook',
+                                                 event)
       end
 
-      raise Doing::Errors::PluginUncallable.new('Hooks must respond to :call', 'hook', event) unless block.respond_to? :call
+      unless block.respond_to? :call
+        raise Doing::Errors::PluginUncallable.new('Hooks must respond to :call', 'hook',
+                                                  event)
+      end
 
       Doing.logger.debug('Hook Manager:', "Registered #{event} hook") if ENV['DOING_PLUGIN_DEBUG']
 

data/lib/doing/item/dates.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Doing
   module ItemDates
     # def date=(new_date)
@@ -10,7 +12,7 @@ module Doing
 
       return nil if @title =~ /(?<=^| )@done\b/
 
-      return Time.now - @date
+      Time.now - @date
     end
 
     ##

data/lib/doing/item/query.rb

@@ -448,16 +448,16 @@ module Doing
     end
 
     def value_string_matches?(tag_val, comp, value)
-      case comp
-      when /\^=/
-        tag_val =~ /^#{value.wildcard_to_rx}/i
-      when /\$=/
-        tag_val =~ /#{value.wildcard_to_rx}$/i
-      when %r{==}
-        tag_val =~ /^#{value.wildcard_to_rx}$/i
-      else
-        tag_val =~ /#{value.wildcard_to_rx}/i
-      end
+      tag_val =~ case comp
+                 when /\^=/
+                   /^#{value.wildcard_to_rx}/i
+                 when /\$=/
+                   /#{value.wildcard_to_rx}$/i
+                 when /==/
+                   /^#{value.wildcard_to_rx}$/i
+                 else
+                   /#{value.wildcard_to_rx}/i
+                 end
     end
 
     def value_number_matches?(tag_val, comp, value)

data/lib/doing/item/state.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Doing
   # State queries for a Doing entry
   module ItemState

data/lib/doing/logger.rb

@@ -270,58 +270,126 @@ module Doing
     def benchmark(key, state)
       return unless ENV['DOING_BENCHMARK']
 
+      # Pre-allocate benchmarks hash to avoid repeated allocation
       @benchmarks ||= {}
-      @benchmarks[key] ||= { start: nil, finish: nil }
+
+      # Use direct assignment instead of ||= for better performance
+      @benchmarks[key] = { start: nil, finish: nil } if @benchmarks[key].nil?
       @benchmarks[key][state] = Process.clock_gettime(Process::CLOCK_MONOTONIC)
     end
 
+    # Measure execution time of a block with automatic start/finish
+    def measure(key, &_block)
+      return yield unless ENV['DOING_BENCHMARK']
+
+      benchmark(key, :start)
+      result = yield
+      benchmark(key, :finish)
+      result
+    end
+
+    # Get benchmark statistics for a specific key
+    def benchmark_stats(key)
+      return nil unless @benchmarks.dig(key, :start) && @benchmarks.dig(key, :finish)
+
+      {
+        key: key,
+        duration: (@benchmarks[key][:finish] - @benchmarks[key][:start]).round(4),
+        start_time: @benchmarks[key][:start].round(4),
+        end_time: @benchmarks[key][:finish].round(4)
+      }
+    end
+
+    # Get all benchmark statistics sorted by duration
+    def all_benchmark_stats
+      return [] unless @benchmarks
+
+      @benchmarks.map do |key, timers|
+        next unless timers[:start] && timers[:finish]
+
+        {
+          key: key,
+          duration: (timers[:finish] - timers[:start]).round(4),
+          start_time: timers[:start].round(4),
+          end_time: timers[:finish].round(4)
+        }
+      end.compact.sort_by { |stats| -stats[:duration] }
+    end
+
+    # Generate a compact benchmark summary
+    def benchmark_summary
+      return unless ENV['DOING_BENCHMARK'] && @benchmarks
+
+      stats = all_benchmark_stats
+      return if stats.empty?
+
+      total_duration = stats.find { |s| s[:key] == :total }&.dig(:duration) || 0
+      return if total_duration <= 0
+
+      output = []
+      output << "Benchmark Summary (Total: #{total_duration.round(4)}s):"
+      stats.reject { |s| s[:key] == :total }.each do |stat|
+        percentage = (stat[:duration] / total_duration * 100).round(1)
+        output << "  #{stat[:key]}: #{stat[:duration]}s (#{percentage}%)"
+      end
+
+      output.join("\n")
+    end
+
     def log_benchmarks
-      if ENV['DOING_BENCHMARK']
-
-        output = []
-        beginning = @benchmarks[:total][:start]
-        ending = @benchmarks[:total][:finish]
-        total = ending - beginning
-        factor = TTY::Screen.columns / total
-
-        cols = Array.new(TTY::Screen.columns)
-
-        colors = %w[bgred bggreen bgyellow bgblue bgmagenta bgcyan bgwhite boldbgred boldbggreen boldbgyellow boldbgblue boldbgwhite]
-        idx = 0
-        # @benchmarks.delete(:total)
-
-        @benchmarks.sort_by { |_, timers| [timers[:start], timers[:finish]] }.each do |k, timers|
-          if timers[:finish] && timers[:start]
-            color = colors[idx % colors.count]
-            fg = if idx < 7
-                   Color.boldblack
-                 else
-                   Color.boldwhite
-                 end
-            color = Color.send(color) + fg
-
-            start = ((timers[:start] - beginning) * factor).floor
-            finish = ((timers[:finish] - beginning) * factor).ceil
-
-            cols.fill("#{color}-", start..finish)
-            cols[start] = "#{color}|"
-            cols[finish] = "#{color}|"
-            output << "#{color}#{k}#{Color.default}: #{timers[:finish] - timers[:start]}"
-          else
-            output << "#{k}: error"
-          end
-
-          idx += 1
-        end
+      return unless ENV['DOING_BENCHMARK'] && @benchmarks
 
-        output.each do |msg|
-          $stdout.puts color_message(:debug, 'Benchmark:', msg)
-        end
+      # Cache screen width to avoid repeated calls
+      screen_width = TTY::Screen.columns
+      return if screen_width <= 0
+
+      beginning = @benchmarks[:total]&.dig(:start)
+      ending = @benchmarks[:total]&.dig(:finish)
+      return unless beginning && ending
+
+      total = ending - beginning
+      return if total <= 0
+
+      factor = screen_width.to_f / total
+      cols = Array.new(screen_width, ' ')
+      output = []
 
-        $stdout.puts color_message(:debug, 'Benchmark:', "Total: #{total}")
+      # Pre-allocate colors array to avoid repeated allocation
+      colors = %w[bgred bggreen bgyellow bgblue bgmagenta bgcyan bgwhite
+                  boldbgred boldbggreen boldbgyellow boldbgblue boldbgwhite]
+      color_count = colors.size
 
-        $stdout.puts cols[0..TTY::Screen.columns-1].join + Color.reset
+      # Sort benchmarks once and cache the result
+      sorted_benchmarks = @benchmarks.reject { |k, _| k == :total }
+                                     .select { |_, timers| timers[:finish] && timers[:start] }
+                                     .sort_by { |_, timers| timers[:start] }
+
+      sorted_benchmarks.each_with_index do |(key, timers), idx|
+        color_name = colors[idx % color_count]
+        fg_color = idx < 7 ? Color.boldblack : Color.boldwhite
+        color = Color.send(color_name) + fg_color
+
+        start_pos = ((timers[:start] - beginning) * factor).floor
+        finish_pos = ((timers[:finish] - beginning) * factor).ceil
+
+        # Ensure positions are within bounds
+        start_pos = [start_pos, 0].max
+        finish_pos = [finish_pos, screen_width - 1].min
+
+        if start_pos < finish_pos
+          cols.fill("#{color}-", start_pos..finish_pos)
+          cols[start_pos] = "#{color}|"
+          cols[finish_pos] = "#{color}|"
+        end
+
+        duration = (timers[:finish] - timers[:start]).round(4)
+        output << "#{color}#{key}#{Color.default}: #{duration}"
       end
+
+      # Output all messages at once to reduce I/O overhead
+      output.each { |msg| $stdout.puts color_message(:debug, 'Benchmark:', msg) }
+      $stdout.puts color_message(:debug, 'Benchmark:', "Total: #{total.round(4)}")
+      $stdout.puts cols.join + Color.reset
     end
 
     def log_change(tags_added: [], tags_removed: [], count: 1, item: nil, single: false)
@@ -467,9 +535,9 @@ module Doing
 
       message.sub!(/^(\s*\S.*?): (.*?)$/) do
         m = Regexp.last_match
-        msg = m[2] =~ /(\e\[[\d;]+m)/ ? msg : "#{message_fg}#{m[2]}"
+        msg_content = m[2] =~ /(\e\[[\d;]+m)/ ? m[2] : "#{message_fg}#{m[2]}"
 
-        "#{topic_fg}#{m[1]}#{colors.reset}: #{message_fg}#{m[2]}"
+        "#{topic_fg}#{m[1]}#{colors.reset}: #{msg_content}"
       end
 
       "#{prefix} #{message.highlight_tags}#{colors.reset}"