pretty-git 0.1.3 → 0.1.5

data/lib/pretty_git/render/markdown_renderer.rb

@@ -2,7 +2,8 @@
 
 module PrettyGit
   module Render
-    # Renders Markdown tables and sections per specs/output_formats.md
+    # Renders Markdown tables and sections per docs/output_formats.md
+    # rubocop:disable Metrics/ClassLength
     class MarkdownRenderer
       TITLES = {
         'activity' => 'Activity',
@@ -33,7 +34,8 @@ module PrettyGit
 
         headers = headers_for(report, result)
         title = title_for(report)
-        render_table(title, headers, result[:items])
+        rows = sort_rows(report, result[:items], result)
+        render_table(title, headers, rows)
       end
 
       private
@@ -82,6 +84,53 @@ module PrettyGit
         end
         @io.puts 'No data' if rows.empty?
       end
+
+      # Deterministic ordering per docs/determinism.md
+      # NOTE: High branching is intentional to keep tie-breakers explicit and stable across formats.
+      # rubocop:disable Metrics/AbcSize, Metrics/CyclomaticComplexity, Metrics/MethodLength, Metrics/PerceivedComplexity
+      def sort_rows(report, rows, ctx = nil)
+        return rows unless rows.is_a?(Array)
+
+        case report
+        when 'hotspots'
+          rows.sort_by { |r| [-to_f(r[:score]), -to_i(r[:commits]), -to_i(r[:changes]), to_s(r[:path])] }
+        when 'churn'
+          rows.sort_by { |r| [-to_i(r[:churn]), -to_i(r[:commits]), to_s(r[:path])] }
+        when 'ownership'
+          rows.sort_by { |r| [-to_f(r[:owner_share]), -to_i(r[:authors]), to_s(r[:path])] }
+        when 'files'
+          rows.sort_by { |r| [-to_i(r[:changes]), -to_i(r[:commits]), to_s(r[:path])] }
+        when 'authors'
+          rows.sort_by { |r| [-to_i(r[:commits]), -to_i(r[:additions]), -to_i(r[:deletions]), to_s(r[:author_email])] }
+        when 'languages'
+          metric = ctx && ctx[:metric] ? ctx[:metric].to_sym : :bytes
+          rows.sort_by { |r| [-to_i(r[metric]), to_s(r[:language])] }
+        when 'activity'
+          rows.sort_by { |r| [to_s(r[:timestamp])] }
+        when 'heatmap'
+          rows.sort_by { |r| [to_i(r[:dow] || r[:day] || r[:weekday]), to_i(r[:hour])] }
+        else
+          rows
+        end
+      end
+      # rubocop:enable Metrics/AbcSize, Metrics/CyclomaticComplexity, Metrics/MethodLength, Metrics/PerceivedComplexity
+
+      def to_i(val)
+        Integer(val || 0)
+      rescue StandardError
+        0
+      end
+
+      def to_f(val)
+        Float(val || 0.0)
+      rescue StandardError
+        0.0
+      end
+
+      def to_s(val)
+        (val || '').to_s
+      end
     end
+    # rubocop:enable Metrics/ClassLength
   end
 end

data/lib/pretty_git/render/xml_renderer.rb

@@ -10,15 +10,29 @@ module PrettyGit
         @io = io
       end
 
-      def call(_report, result, _filters)
+      # rubocop:disable Metrics/CyclomaticComplexity
+      def call(report, result, _filters)
+        ordered = apply_order(report, result)
         doc = REXML::Document.new
         doc << REXML::XMLDecl.new('1.0', 'UTF-8')
-        root = doc.add_element('report')
-        hash_to_xml(root, result)
+        root_name = case report
+                    when 'hotspots' then 'hotspotsReport'
+                    when 'churn' then 'churnReport'
+                    when 'ownership' then 'ownershipReport'
+                    when 'languages' then 'languagesReport'
+                    when 'files' then 'filesReport'
+                    when 'authors' then 'authorsReport'
+                    when 'activity' then 'activityReport'
+                    when 'heatmap' then 'heatmapReport'
+                    else 'report'
+                    end
+        root = doc.add_element(root_name)
+        hash_to_xml(root, ordered)
         formatter = REXML::Formatters::Pretty.new(2)
         formatter.compact = true
         formatter.write(doc, @io)
       end
+      # rubocop:enable Metrics/CyclomaticComplexity
 
       private
 
@@ -38,6 +52,60 @@ module PrettyGit
           parent.text = obj.nil? ? '' : obj.to_s
         end
       end
+
+      # Deterministic ordering per docs/determinism.md
+      # NOTE: High branching is intentional to keep tie-breakers explicit and stable across formats.
+      # rubocop:disable Metrics/AbcSize, Metrics/CyclomaticComplexity, Metrics/MethodLength, Metrics/PerceivedComplexity
+      def apply_order(report, result)
+        dup = Marshal.load(Marshal.dump(result))
+        items = dup[:items]
+        return dup unless items.is_a?(Array)
+
+        dup[:items] =
+          case report
+          when 'hotspots'
+            items.sort_by do |r|
+              [-to_f(r[:score]), -to_i(r[:commits]), -to_i(r[:changes]), to_s(r[:path])]
+            end
+          when 'churn'
+            items.sort_by { |r| [-to_i(r[:churn]), -to_i(r[:commits]), to_s(r[:path])] }
+          when 'ownership'
+            items.sort_by { |r| [-to_f(r[:owner_share]), -to_i(r[:authors]), to_s(r[:path])] }
+          when 'files'
+            items.sort_by { |r| [-to_i(r[:changes]), -to_i(r[:commits]), to_s(r[:path])] }
+          when 'authors'
+            items.sort_by do |r|
+              [-to_i(r[:commits]), -to_i(r[:additions]), -to_i(r[:deletions]), to_s(r[:author_email])]
+            end
+          when 'languages'
+            metric = (dup[:metric] || 'bytes').to_sym
+            items.sort_by { |r| [-to_i(r[metric]), to_s(r[:language])] }
+          when 'activity'
+            items.sort_by { |r| [to_s(r[:timestamp])] }
+          when 'heatmap'
+            items.sort_by { |r| [to_i(r[:dow] || r[:day] || r[:weekday]), to_i(r[:hour])] }
+          else
+            items
+          end
+        dup
+      end
+      # rubocop:enable Metrics/AbcSize, Metrics/CyclomaticComplexity, Metrics/MethodLength, Metrics/PerceivedComplexity
+
+      def to_i(val)
+        Integer(val || 0)
+      rescue StandardError
+        0
+      end
+
+      def to_f(val)
+        Float(val || 0.0)
+      rescue StandardError
+        0.0
+      end
+
+      def to_s(val)
+        (val || '').to_s
+      end
     end
   end
 end

data/lib/pretty_git/render/yaml_renderer.rb

@@ -10,9 +10,11 @@ module PrettyGit
         @io = io
       end
 
-      def call(_report, result, _filters)
+      def call(report, result, _filters)
+        # Apply deterministic ordering for items where applicable
+        ordered = apply_order(report, result)
         # Dump the entire result structure to YAML with string keys for safe parsing
-        @io.write(stringify_keys(result).to_yaml)
+        @io.write(stringify_keys(ordered).to_yaml)
       end
 
       private
@@ -29,6 +31,60 @@ module PrettyGit
           obj
         end
       end
+
+      # Deterministic ordering per docs/determinism.md
+      # NOTE: High branching is intentional to keep tie-breakers explicit and stable across formats.
+      # rubocop:disable Metrics/AbcSize, Metrics/CyclomaticComplexity, Metrics/MethodLength, Metrics/PerceivedComplexity
+      def apply_order(report, result)
+        dup = Marshal.load(Marshal.dump(result)) # deep dup
+        items = dup[:items]
+        return dup unless items.is_a?(Array)
+
+        dup[:items] =
+          case report
+          when 'hotspots'
+            items.sort_by do |r|
+              [-to_f(r[:score]), -to_i(r[:commits]), -to_i(r[:changes]), to_s(r[:path])]
+            end
+          when 'churn'
+            items.sort_by { |r| [-to_i(r[:churn]), -to_i(r[:commits]), to_s(r[:path])] }
+          when 'ownership'
+            items.sort_by { |r| [-to_f(r[:owner_share]), -to_i(r[:authors]), to_s(r[:path])] }
+          when 'files'
+            items.sort_by { |r| [-to_i(r[:changes]), -to_i(r[:commits]), to_s(r[:path])] }
+          when 'authors'
+            items.sort_by do |r|
+              [-to_i(r[:commits]), -to_i(r[:additions]), -to_i(r[:deletions]), to_s(r[:author_email])]
+            end
+          when 'languages'
+            metric = (dup[:metric] || 'bytes').to_sym
+            items.sort_by { |r| [-to_i(r[metric]), to_s(r[:language])] }
+          when 'activity'
+            items.sort_by { |r| [to_s(r[:timestamp])] }
+          when 'heatmap'
+            items.sort_by { |r| [to_i(r[:dow] || r[:day] || r[:weekday]), to_i(r[:hour])] }
+          else
+            items
+          end
+        dup
+      end
+      # rubocop:enable Metrics/AbcSize, Metrics/CyclomaticComplexity, Metrics/MethodLength, Metrics/PerceivedComplexity
+
+      def to_i(val)
+        Integer(val || 0)
+      rescue StandardError
+        0
+      end
+
+      def to_f(val)
+        Float(val || 0.0)
+      rescue StandardError
+        0.0
+      end
+
+      def to_s(val)
+        (val || '').to_s
+      end
     end
   end
 end

data/lib/pretty_git/utils/path_utils.rb

@@ -0,0 +1,30 @@
+# frozen_string_literal: true
+
+module PrettyGit
+  module Utils
+    # Utilities for path/glob normalization and handling cross-platform quirks.
+    module PathUtils
+      module_function
+
+      # Normalize a string path or glob to Unicode NFC form.
+      # Returns nil if input is nil.
+      def normalize_nfc(str)
+        return nil if str.nil?
+
+        s = str.to_s
+        # Only normalize if supported in this Ruby build; otherwise return as-is
+        if s.respond_to?(:unicode_normalize)
+          s.unicode_normalize(:nfc)
+        else
+          s
+        end
+      end
+
+      # Normalize each entry in an Array-like collection to NFC and compact
+      # nils. Returns an Array.
+      def normalize_globs(collection)
+        Array(collection).compact.map { |p| normalize_nfc(p) }
+      end
+    end
+  end
+end

data/lib/pretty_git/utils/time_utils.rb

@@ -0,0 +1,39 @@
+# frozen_string_literal: true
+
+require 'time'
+
+module PrettyGit
+  module Utils
+    # Utilities for time parsing and ISO8601 normalization used by filters.
+    module TimeUtils
+      module_function
+
+      # Converts various time inputs to ISO8601 in UTC.
+      # Accepts Time, String(ISO8601), or String(YYYY-MM-DD) treated as UTC midnight.
+      # Returns nil for nil/blank input. Raises ArgumentError for invalid values.
+      def to_utc_iso8601(val)
+        return nil if val.nil? || val.to_s.strip.empty?
+
+        parse_to_time(val).utc.iso8601
+      rescue ArgumentError
+        raise ArgumentError, "Invalid datetime: #{val} (expected ISO8601 or YYYY-MM-DD)"
+      end
+
+      def parse_to_time(val)
+        return val if val.is_a?(Time)
+        return parse_date_only(val) if val.is_a?(String) && date_only?(val)
+
+        Time.parse(val.to_s)
+      end
+
+      def parse_date_only(str)
+        y, m, d = str.split('-').map(&:to_i)
+        Time.new(y, m, d, 0, 0, 0, '+00:00')
+      end
+
+      def date_only?(str)
+        !!(str =~ /^\d{4}-\d{2}-\d{2}$/)
+      end
+    end
+  end
+end

data/lib/pretty_git/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module PrettyGit
-  VERSION = '0.1.3'
+  VERSION = '0.1.5'
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: pretty-git
 version: !ruby/object:Gem::Version
-  version: 0.1.3
+  version: 0.1.5
 platform: ruby
 authors:
 - Pretty Git Authors
@@ -51,8 +51,6 @@ dependencies:
         version: '4.0'
 description: Generates structured analytics from local Git repositories with multiple
   export formats.
-email:
-- ''
 executables:
 - pretty-git
 extensions: []
@@ -76,8 +74,10 @@ files:
 - lib/pretty_git/app.rb
 - lib/pretty_git/cli.rb
 - lib/pretty_git/cli_helpers.rb
+- lib/pretty_git/constants.rb
 - lib/pretty_git/filters.rb
 - lib/pretty_git/git/provider.rb
+- lib/pretty_git/logger.rb
 - lib/pretty_git/render/console_renderer.rb
 - lib/pretty_git/render/csv_renderer.rb
 - lib/pretty_git/render/json_renderer.rb
@@ -87,14 +87,18 @@ files:
 - lib/pretty_git/render/xml_renderer.rb
 - lib/pretty_git/render/yaml_renderer.rb
 - lib/pretty_git/types.rb
+- lib/pretty_git/utils/path_utils.rb
+- lib/pretty_git/utils/time_utils.rb
 - lib/pretty_git/version.rb
 homepage: https://github.com/MikoMikocchi/pretty-git
 licenses:
 - MIT
 metadata:
+  homepage_uri: https://github.com/MikoMikocchi/pretty-git
   source_code_uri: https://github.com/MikoMikocchi/pretty-git
   changelog_uri: https://github.com/MikoMikocchi/pretty-git/blob/main/CHANGELOG.md
   bug_tracker_uri: https://github.com/MikoMikocchi/pretty-git/issues
+  documentation_uri: https://github.com/MikoMikocchi/pretty-git#readme
   rubygems_mfa_required: 'true'
 rdoc_options: []
 require_paths: