git-fit 0.25.0 → 0.25.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 0b5ac295fe113517b660831a4cdaf826ac0905781ccdc591174117e0ec91973b
-  data.tar.gz: ecafcdedbdb2469570e946bd2b885d2605d92bf43a65fed7668f0074709a9a09
+  metadata.gz: e7cac4c94c7c256f2385670c9b386a73edf9d25c54cd0a2b4def3e69e93be273
+  data.tar.gz: dcf8048d51b7249ec2dd8097058cf4cbc1b592c19d71bb75eb1d283eaf22f6b1
 SHA512:
-  metadata.gz: a588657ec9decf802ec4b2dc68bce7eb9e3c149282fd67e10a8e80a07c69c22cb418aef44513ba62e4de59c153475ba37dbf66da596b330d88d64601ea89f5a1
-  data.tar.gz: 2f1cdbe55b0f3ddd59406bb3b01c8d43ea09d491ec9a47a3fb316356e35cf6834633a1aa0315e495f5f26e623b214052b6934bd025a3c620d6f869c50d707a1e
+  metadata.gz: 2e5dbe464f60f71675e256e639a0515acec48b69e1a92d9b64170b8de22d90a6727a2cc6692f273f8742a5164ae279dc94dacce5ea4d6c30ed71a82311c4f597
+  data.tar.gz: f6af70eda9113085f911f2126eef03e112b578994327ef9f825239be36e91ac329eb4f4c03bc49dee21b2445c6a671b1f33e00a81731560972f9ed4a40bdcf25

data/lib/git_fit/ability/computer.rb

@@ -44,13 +44,11 @@ module GitFit
       def normalize_as_of(as_of)
         return nil if as_of.nil? || as_of.to_s.empty?
 
-        valid = as_of.to_s.match?(/\A\d{4}-\d{2}-\d{2}\z/)
-        unless valid
-          raise ArgumentError, "Ability: invalid --as-of '#{as_of}' (expected YYYY-MM-DD)\n" \
-                               '  Fix: git fit ability compute --as-of 2025-03-15'
-        end
-
+        Date.iso8601(as_of.to_s)
         as_of
+      rescue Date::Error
+        raise ArgumentError, "Ability: invalid --as-of '#{as_of}' (expected YYYY-MM-DD)\n" \
+                             '  Fix: git fit ability compute --as-of 2025-03-15'
       end
 
       def load_rows
@@ -99,21 +97,19 @@ module GitFit
       # --- fitness layer ---
 
       def fitness_snapshots(rows)
-        base = {}
-        Snapshot::PERIOD_TYPES.each do |type|
-          buckets = bucketize(rows, type)
-          buckets.each { |key, period_rows| base[[type, key]] = fitness_fields(type, key, period_rows) }
-        end
-        out = []
-        Snapshot::PERIOD_TYPES.each do |type|
-          bucketize(rows, type).keys.sort.each do |key|
+        buckets = Snapshot::PERIOD_TYPES.to_h { |type| [type, bucketize(rows, type)] }
+        base = buckets.flat_map do |type, per|
+          per.map do |key, period_rows|
+            [[type, key], fitness_fields(type, key, period_rows)]
+          end
+        end.to_h
+        Snapshot::PERIOD_TYPES.flat_map do |type|
+          buckets[type].keys.sort.map do |key|
             fields = base[[type, key]]
-            prev_key = Snapshot.previous_key(type, key)
-            prev = prev_key && base[[type, prev_key]]
-            out << fields.merge('trend_vs_prev' => prev ? trend_fields(fields, prev) : nil)
+            prev = Snapshot.previous_key(type, key)&.then { |pk| base[[type, pk]] }
+            fields.merge('trend_vs_prev' => prev ? trend_fields(fields, prev) : nil)
           end
         end
-        out
       end
 
       def bucketize(rows, type)

data/lib/git_fit/ability/snapshot.rb

@@ -84,21 +84,18 @@ module GitFit
           format('%<year>04d-W%<week>02d', year: date.cwyear, week: date.cweek)
         end
 
-        # Distinct ISO weeks overlapping [start, end].
+        # Distinct ISO weeks overlapping [start, end] (cwyear*53+cweek is monotonic;
+        # cweek never exceeds 53).
         def week_count(start_date, end_date)
-          keys = {}
-          (start_date..end_date).each { |d| keys[week_key_of_date(d)] = true }
-          keys.size
+          (end_date.cwyear * 53 + end_date.cweek) - (start_date.cwyear * 53 + start_date.cweek) + 1
         end
 
         # Sample-count confidence: <3 low (prompt-excluded), 3-9 medium, >=10 high.
         def confidence_for(count)
           if count < 3
             'low'
-          elsif count < 10
-            'medium'
           else
-            'high'
+            count < 10 ? 'medium' : 'high'
           end
         end
 

data/lib/git_fit/cli/ability.rb

@@ -29,33 +29,5 @@ module GitFit
     rescue StandardError => e
       say_status :error, "Ability compute failed: #{e.message}", :red
     end
-
-    desc 'show', 'Print a single ability snapshot by period key'
-    option :period, type: :string, required: true, desc: 'Period key, e.g. 2025-03 / 2025-Q1 / 2025-W12 / 2025 / all'
-    option :output, type: :string, aliases: '-o', desc: 'ability.json path to read'
-
-    def show
-      path = options[:output] || git_fit_config.export_config.dig('ability', 'path') || 'site/ability.json'
-      doc = JSON.parse(File.read(path))
-      snapshot = find_snapshot(doc, options[:period])
-      if snapshot
-        puts JSON.pretty_generate(snapshot)
-      else
-        say_status :warn, "Ability: no snapshot for period '#{options[:period]}' in #{path}", :yellow
-      end
-    rescue StandardError => e
-      say_status :error, "Ability show failed: #{e.message}", :red
-    end
-
-    private
-
-    def find_snapshot(doc, period_key)
-      candidates = (doc.dig('fitness', 'snapshots') || []) +
-                   (doc['sports'] || {}).values.flat_map { |s| s['snapshots'] || [] }
-      matches = candidates.select { |s| s['period_key'] == period_key }
-      return matches.first if matches.one?
-
-      matches # both fitness and sport layers may carry the key — print all
-    end
   end
 end

data/lib/git_fit/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module GitFit
-  VERSION = '0.25.0'
+  VERSION = '0.25.1'
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: git-fit
 version: !ruby/object:Gem::Version
-  version: 0.25.0
+  version: 0.25.1
 platform: ruby
 authors:
 - Lax