commiti 1.3.0 → 1.3.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 751893cc1501fbd4723682605f17bcc6f0617fafe28997b398affd8e33a20d1e
-  data.tar.gz: 6b05cf37e1617655547c5e2de646dd75d07cde47d0883b70ee8eb7114242719b
+  metadata.gz: 026566db469248936e47f40faa446650926b7bb893089246a6c23a45dfaa432a
+  data.tar.gz: 251995106b1883d61cb12c00f911a4e3158d6d47e9b9d1b434d2026234dd3c2f
 SHA512:
-  metadata.gz: 777e2d6ddc6e40f0599abbe0ee191b819993926c5e0ca5bed79abb81407071356fa97807de031ba21dc3221a96bfa8b526765ee35a80aeee95cf9410c952214f
-  data.tar.gz: 739b300910794916477e8222b1ba2ea4f08698c337b1af10273c126a8fe8f3c619c6ed8171134cf545550e10767746b3b8d0da29f4cd203627ac7e26108edabf
+  metadata.gz: c0d857c40f016686ee37b6b7f2d1109c5d7f79a7037c0ed41d0e49e1d3556ff57ce0db9f1da3f0f26e4956840e5a7ab0376a39799f359673068542e89a11480c
+  data.tar.gz: 677c08e9d0d5ae62723f7bb3fdea97c3c3799dd5a3775b133ca5520af32a102d924863c5f733794e005b3ff82c79a22d30a7aaee578985118ec0af31cd13c8ce

data/lib/commiti.rb

@@ -10,6 +10,7 @@ require_relative 'services/helpers/prompt_builder'
 require_relative 'services/helpers/interactive_prompt'
 require_relative 'services/git/pr/pr_opener'
 require_relative 'services/helpers/clipboard'
+require_relative 'services/helpers/terminal_ui'
 require_relative 'services/helpers/spinner'
 require_relative 'services/flow_context_builder'
 require_relative 'services/message_generator'

data/lib/flows/commit_flow.rb

@@ -51,7 +51,8 @@ module Commiti
       end
 
       def run_single_group_context(context:, client:, model:)
-        puts "\nAuto-split found a single connected change group. Falling back to single commit flow."
+        message = 'Auto-split found a single connected change group. Falling back to single commit flow.'
+        puts "\n#{Commiti::TerminalUI.status(:info, message)}"
         Commiti::MessagePresenter.print_summarization_notice(context[:summarized_result])
 
         message = generate_message_for_context(context:, client:, model:)
@@ -63,7 +64,7 @@ module Commiti
         groups = context[:change_groups]
         run_stage('Unstaging current index for grouped commit execution') { Commiti::GitWriter.unstage_all! }
 
-        puts "\nAuto-split detected #{groups.length} connected change groups."
+        puts "\n#{Commiti::TerminalUI.status(:info, "Auto-split detected #{groups.length} connected change groups.")}"
 
         groups.each_with_index do |group, index|
           break if process_group(group:, index:, total: groups.length, client:, model:) == :stop
@@ -74,7 +75,7 @@ module Commiti
         run_stage("Staging files for group #{index + 1}/#{total}") { Commiti::GitWriter.stage_files!(group[:files]) }
         return :continue unless run_stage('Checking staged changes') { Commiti::GitWriter.staged_changes? }
 
-        puts "\nGroup #{index + 1}/#{total} files:"
+        puts "\n#{Commiti::TerminalUI.header("Group #{index + 1}/#{total} files")}:"
         group[:files].each { |path| puts "- #{path}" }
 
         group_context = build_context(diff: group_diff(group), client:, model:)
@@ -84,7 +85,8 @@ module Commiti
         maybe_copy_to_clipboard(message)
         return :continue if finalize(message) == :committed
 
-        puts "Stopping auto-split flow at group #{index + 1} because commit was skipped."
+        stop_message = "Stopping auto-split flow at group #{index + 1} because commit was skipped."
+        puts Commiti::TerminalUI.status(:warn, stop_message)
         run_stage('Restaging remaining uncommitted changes') { Commiti::GitWriter.stage_all! }
         :stop
       end

data/lib/services/git/commit/commit_execution.rb

@@ -12,13 +12,13 @@ module Commiti
         when :yes
           errors = Commiti::InteractivePrompt.commit_message_errors(working_message)
           unless errors.empty?
-            puts "\nCurrent message needs fixes before commit:"
+            puts "\n#{Commiti::TerminalUI.status(:warn, 'Current message needs fixes before commit:')}"
             errors.each { |error| puts "- #{error}" }
 
             if Commiti::InteractivePrompt.ask_yes_no('Open editor to fix now?', default: :yes)
               edited = edit_message_until_valid(working_message)
               if edited.nil?
-                puts "\nEditor did not exit successfully. Commit skipped.\n\n"
+                puts "\n#{Commiti::TerminalUI.status(:fail, 'Editor did not exit successfully. Commit skipped.')}\n\n"
                 return :skipped
               end
 
@@ -27,25 +27,25 @@ module Commiti
               next
             end
 
-            puts "\nCommit skipped.\n\n"
+            puts "\n#{Commiti::TerminalUI.status(:warn, 'Commit skipped.')}\n\n"
             return :skipped
           end
 
           output = run_stage.call('Writing commit') { Commiti::GitWriter.commit_with_message_file(working_message) }
           puts output unless output.to_s.strip.empty?
-          puts "\nCommit created.\n\n"
+          puts "\n#{Commiti::TerminalUI.status(:success, 'Commit created.')}\n\n"
           return :committed
         when :edit
           edited = edit_message_until_valid(working_message)
           if edited.nil?
-            puts "\nEditor did not exit successfully.\n\n"
+            puts "\n#{Commiti::TerminalUI.status(:fail, 'Editor did not exit successfully.')}\n\n"
             next
           end
 
           working_message = edited
           print_message.call(working_message)
         else
-          puts "\nCommit skipped.\n\n"
+          puts "\n#{Commiti::TerminalUI.status(:warn, 'Commit skipped.')}\n\n"
           return :skipped
         end
       end
@@ -59,7 +59,7 @@ module Commiti
         return nil if edited.nil?
 
         if edited == working.to_s.strip
-          puts "\nNo changes detected in editor."
+          puts "\n#{Commiti::TerminalUI.status(:info, 'No changes detected in editor.')}"
           return edited unless Commiti::InteractivePrompt.ask_yes_no('Re-open editor now?', default: :yes)
 
           next
@@ -68,7 +68,7 @@ module Commiti
         errors = Commiti::InteractivePrompt.commit_message_errors(edited)
         return edited if errors.empty?
 
-        puts "\nEdited message needs fixes:"
+        puts "\n#{Commiti::TerminalUI.status(:warn, 'Edited message needs fixes:')}"
         errors.each { |error| puts "- #{error}" }
         return edited unless Commiti::InteractivePrompt.ask_yes_no('Re-open editor now?', default: :yes)
 

data/lib/services/git/commit/commit_staging.rb

@@ -11,11 +11,11 @@ module Commiti
       status = run_stage.call('Reading git status') { Commiti::GitWriter.status_short }
       raise 'No changes found in working tree.' if status.strip.empty?
 
-      puts "\nCurrent git status:\n\n#{status}"
+      puts "\n#{Commiti::TerminalUI.header('Current git status')}\n\n#{status}"
       return unless Commiti::InteractivePrompt.ask_yes_no('Run git add -A now?', default: :no)
 
       run_stage.call('Staging changes (git add -A)') { Commiti::GitWriter.stage_all! }
-      puts "\nStaged changes with git add -A.\n"
+      puts "\n#{Commiti::TerminalUI.status(:success, 'Staged changes with git add -A.')}\n"
     end
     private_class_method :maybe_stage_changes
 
@@ -26,7 +26,7 @@ module Commiti
       if Commiti::InteractivePrompt.ask_yes_no('No staged changes found. Stage all changes now with git add -A?',
                                                default: :yes)
         run_stage.call('Staging changes (git add -A)') { Commiti::GitWriter.stage_all! }
-        puts "\nStaged changes with git add -A.\n"
+        puts "\n#{Commiti::TerminalUI.status(:success, 'Staged changes with git add -A.')}\n"
       end
 
       staged = run_stage.call('Checking staged changes') { Commiti::GitWriter.staged_changes? }

data/lib/services/helpers/spinner.rb

@@ -5,13 +5,8 @@ module Commiti
     FRAMES = ['|', '/', '-', '\\'].freeze
     INTERVAL_SECONDS = 0.1
 
-    def self.run(message)
-      unless $stdout.tty?
-        puts "#{message}..."
-        result = yield
-        puts "[done] #{message}"
-        return result
-      end
+    def self.run(message, &block)
+      return run_without_spinner(message, &block) unless $stdout.tty?
 
       done = false
       error = nil
@@ -20,7 +15,7 @@ module Commiti
       spinner_thread = Thread.new do
         index = 0
         until done
-          frame = FRAMES[index % FRAMES.length]
+          frame = Commiti::TerminalUI.color(FRAMES[index % FRAMES.length], :cyan)
           print "\r#{frame} #{message}"
           $stdout.flush
           index += 1
@@ -29,15 +24,14 @@ module Commiti
       end
 
       begin
-        result = yield
+        result = block.call
       rescue StandardError => e
         error = e
       ensure
         done = true
         spinner_thread.join
 
-        status = error.nil? ? '[done]' : '[fail]'
-        print "\r#{status} #{message}\n"
+        print "\r#{final_status_line(error, message)}\n"
         $stdout.flush
       end
 
@@ -45,5 +39,19 @@ module Commiti
 
       result
     end
+
+    def self.run_without_spinner(message, &block)
+      puts Commiti::TerminalUI.status(:info, "#{message}...")
+      result = block.call
+      puts Commiti::TerminalUI.status(:success, message)
+      result
+    end
+    private_class_method :run_without_spinner
+
+    def self.final_status_line(error, message)
+      kind = error.nil? ? :success : :fail
+      Commiti::TerminalUI.status(kind, message)
+    end
+    private_class_method :final_status_line
   end
 end

data/lib/services/helpers/terminal_ui.rb

@@ -0,0 +1,58 @@
+# frozen_string_literal: true
+
+module Commiti
+  module TerminalUI
+    COLORS = {
+      green: 32,
+      red: 31,
+      yellow: 33,
+      blue: 34,
+      cyan: 36,
+      gray: 90,
+      bold: 1
+    }.freeze
+
+    SYMBOLS = {
+      success: '✅',
+      fail: '❌',
+      info: 'ℹ',
+      warn: '⚠'
+    }.freeze
+
+    def self.supports_ansi?
+      return false unless $stdout.tty?
+      return false if ENV.key?('NO_COLOR')
+
+      term = ENV.fetch('TERM', '').downcase
+      term != 'dumb'
+    end
+
+    def self.color(text, *styles)
+      return text unless supports_ansi?
+
+      codes = styles.filter_map { |style| COLORS[style] }
+      return text if codes.empty?
+
+      "\e[#{codes.join(';')}m#{text}\e[0m"
+    end
+
+    def self.status(kind, text)
+      symbol = SYMBOLS.fetch(kind, '*')
+      color_style = case kind
+                    when :success then :green
+                    when :fail then :red
+                    when :warn then :yellow
+                    else :blue
+                    end
+      "#{color(symbol, color_style)} #{text}"
+    end
+
+    def self.separator(length = 60)
+      color('─' * length, :gray)
+    end
+
+    def self.header(text)
+      color(text, :bold, :cyan)
+    end
+  end
+end

data/lib/services/message_presenter.rb

@@ -4,9 +4,9 @@ module Commiti
   module MessagePresenter
     def self.print_summarization_notice(summarized_result)
       if summarized_result[:fallback_reason]
-        puts "\n#{summarized_result[:fallback_reason]}\n"
+        puts "\n#{Commiti::TerminalUI.status(:warn, summarized_result[:fallback_reason])}\n"
       elsif summarized_result[:summarized]
-        puts "\nDiff is large - summarizing first to preserve system prompt focus...\n"
+        puts "\n#{Commiti::TerminalUI.status(:info, 'Diff is large — summarizing first to preserve prompt focus.')}\n"
       end
     end
 
@@ -19,7 +19,7 @@ module Commiti
       print_candidates(candidates)
       selected_index = Commiti::InteractivePrompt.ask_candidate_selection(candidates.length)
       selected_message = candidates[selected_index]
-      puts "\nUsing candidate #{selected_index + 1}."
+      puts "\n#{Commiti::TerminalUI.status(:info, "Using candidate #{selected_index + 1}.")}"
       print_message(selected_message)
       selected_message
     end
@@ -29,21 +29,23 @@ module Commiti
 
       copied = run_stage.call('Copying output to clipboard') { Commiti::Clipboard.copy(message) }
       if copied
-        puts "Copied to clipboard!\n\n"
+        puts "#{Commiti::TerminalUI.status(:success, 'Copied output to clipboard!')}\n\n"
       else
-        puts "Clipboard not available. Install xclip: sudo apt install xclip\n\n"
+        puts "#{Commiti::TerminalUI.status(:warn, 'Clipboard unavailable. Install xclip: sudo apt install xclip')}\n\n"
       end
     end
 
     def self.print_message(message)
-      puts "\n#{'─' * 60}"
+      puts "\n#{Commiti::TerminalUI.separator}"
+      puts Commiti::TerminalUI.header('Generated output')
+      puts Commiti::TerminalUI.separator
       puts message
-      puts "#{'─' * 60}\n"
+      puts "#{Commiti::TerminalUI.separator}\n"
     end
 
     def self.print_candidates(candidates)
       candidates.each_with_index do |candidate, index|
-        puts "\nCandidate #{index + 1}:"
+        puts "\n#{Commiti::TerminalUI.header("Candidate #{index + 1}")}"
         print_message(candidate)
       end
     end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: commiti
 version: !ruby/object:Gem::Version
-  version: 1.3.0
+  version: 1.3.1
 platform: ruby
 authors:
 - Setoju
@@ -71,6 +71,7 @@ files:
 - lib/services/helpers/interactive_prompt.rb
 - lib/services/helpers/prompt_builder.rb
 - lib/services/helpers/spinner.rb
+- lib/services/helpers/terminal_ui.rb
 - lib/services/message_generator.rb
 - lib/services/message_presenter.rb
 homepage: https://github.com/setoju/commiti