Add color support for output in diff, show, and history commands

Author: andrew

CHANGELOG.md

@@ -4,6 +4,7 @@
 - Consistent error handling across all commands (JSON errors when `--format=json`)
 - `git pkgs update` now uses a transaction for atomicity and better performance
 - `GIT_PKGS_DB` environment variable to customize database location
+- Colored output for diff, show, and history commands (respects `NO_COLOR`)
 - Renamed `git pkgs outdated` to `git pkgs stale` (outdated remains as alias)
 - `git pkgs log` command to list commits with dependency changes
 - `git pkgs schema` command to output database schema in text, SQL, JSON, or markdown

lib/git/pkgs.rb

@@ -2,6 +2,7 @@
 
 require_relative "pkgs/version"
 require_relative "pkgs/output"
+require_relative "pkgs/color"
 require_relative "pkgs/cli"
 require_relative "pkgs/database"
 require_relative "pkgs/repository"

lib/git/pkgs/color.rb

@@ -0,0 +1,51 @@
+# frozen_string_literal: true
+
+module Git
+  module Pkgs
+    module Color
+      CODES = {
+        red: 31,
+        green: 32,
+        yellow: 33,
+        blue: 34,
+        magenta: 35,
+        cyan: 36,
+        bold: 1,
+        dim: 2
+      }.freeze
+
+      def self.enabled?
+        return @enabled if defined?(@enabled)
+
+        @enabled = determine_color_support
+      end
+
+      def self.enabled=(value)
+        @enabled = value
+      end
+
+      def self.determine_color_support
+        return false unless $stdout.respond_to?(:tty?) && $stdout.tty?
+        return false if ENV["NO_COLOR"] && !ENV["NO_COLOR"].empty?
+        return false if ENV["TERM"] == "dumb"
+
+        true
+      end
+
+      def self.colorize(text, *codes)
+        return text unless enabled?
+
+        code_str = codes.map { |c| CODES[c] || c }.join(";")
+        "\e[#{code_str}m#{text}\e[0m"
+      end
+
+      def self.red(text)     = colorize(text, :red)
+      def self.green(text)   = colorize(text, :green)
+      def self.yellow(text)  = colorize(text, :yellow)
+      def self.blue(text)    = colorize(text, :blue)
+      def self.cyan(text)    = colorize(text, :cyan)
+      def self.bold(text)    = colorize(text, :bold)
+      def self.dim(text)     = colorize(text, :dim)
+    end
+  end
+end

lib/git/pkgs/commands/diff.rb

@@ -60,35 +60,38 @@ def run
           removed = changes.select { |c| c.change_type == "removed" }
 
           if added.any?
-            puts "Added:"
+            puts Color.green("Added:")
             added.group_by(&:name).each do |name, pkg_changes|
               latest = pkg_changes.last
-              puts "  + #{name} #{latest.requirement} (#{latest.manifest.path})"
+              puts Color.green("  + #{name} #{latest.requirement} (#{latest.manifest.path})")
             end
             puts
           end
 
           if modified.any?
-            puts "Modified:"
+            puts Color.yellow("Modified:")
             modified.group_by(&:name).each do |name, pkg_changes|
               first = pkg_changes.first
               latest = pkg_changes.last
-              puts "  ~ #{name} #{first.previous_requirement} -> #{latest.requirement}"
+              puts Color.yellow("  ~ #{name} #{first.previous_requirement} -> #{latest.requirement}")
             end
             puts
           end
 
           if removed.any?
-            puts "Removed:"
+            puts Color.red("Removed:")
             removed.group_by(&:name).each do |name, pkg_changes|
               latest = pkg_changes.last
-              puts "  - #{name} (was #{latest.requirement})"
+              puts Color.red("  - #{name} (was #{latest.requirement})")
             end
             puts
           end
 
           # Summary
-          puts "Summary: +#{added.map(&:name).uniq.count} -#{removed.map(&:name).uniq.count} ~#{modified.map(&:name).uniq.count}"
+          added_count = Color.green("+#{added.map(&:name).uniq.count}")
+          removed_count = Color.red("-#{removed.map(&:name).uniq.count}")
+          modified_count = Color.yellow("~#{modified.map(&:name).uniq.count}")
+          puts "Summary: #{added_count} #{removed_count} #{modified_count}"
         end
 
         def find_or_create_commit(repo, sha)

lib/git/pkgs/commands/history.rb

@@ -76,13 +76,13 @@ def output_text(changes, package_name)
 
             case change.change_type
             when "added"
-              action = "Added"
+              action = Color.green("Added")
               version_info = change.requirement
             when "modified"
-              action = "Updated"
+              action = Color.yellow("Updated")
               version_info = "#{change.previous_requirement} -> #{change.requirement}"
             when "removed"
-              action = "Removed"
+              action = Color.red("Removed")
               version_info = change.requirement
             end
 

lib/git/pkgs/commands/show.rb

@@ -56,25 +56,25 @@ def output_text(commit, changes)
           removed = changes.select { |c| c.change_type == "removed" }
 
           if added.any?
-            puts "Added:"
+            puts Color.green("Added:")
             added.each do |change|
-              puts "  #{change.name} #{change.requirement} (#{change.ecosystem}, #{change.manifest.path})"
+              puts Color.green("  + #{change.name} #{change.requirement} (#{change.ecosystem}, #{change.manifest.path})")
             end
             puts
           end
 
           if modified.any?
-            puts "Modified:"
+            puts Color.yellow("Modified:")
             modified.each do |change|
-              puts "  #{change.name} #{change.previous_requirement} -> #{change.requirement} (#{change.ecosystem}, #{change.manifest.path})"
+              puts Color.yellow("  ~ #{change.name} #{change.previous_requirement} -> #{change.requirement} (#{change.ecosystem}, #{change.manifest.path})")
             end
             puts
           end
 
           if removed.any?
-            puts "Removed:"
+            puts Color.red("Removed:")
             removed.each do |change|
-              puts "  #{change.name} #{change.requirement} (#{change.ecosystem}, #{change.manifest.path})"
+              puts Color.red("  - #{change.name} #{change.requirement} (#{change.ecosystem}, #{change.manifest.path})")
             end
             puts
           end