git-fit 0.16.1 → 0.17.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: b6d2e554e3fbd61794daf4d15fe4a934d20e1bf8f988fbe13778e925eb26bb91
-  data.tar.gz: 83550dc8b0bb9ada27501fab66a5d9fd0e8f5dd5aadb846de9dfdbb46e5ccedb
+  metadata.gz: 1091af70dde8e2b640208f49633b3572ddf64e41c5063ec4541349a873829988
+  data.tar.gz: aa450db53cfe1eacc076d32f35c78da9bb0bfe86ab014d73554127d9467db876
 SHA512:
-  metadata.gz: 5f288c5a834bff0877a6949d2b79a642f6599b51188f653b9fed9f6699d5d5fab63a48a725c5088935d26d6fb2a90ae1984032add551cb303cacbd3ef6851aae
-  data.tar.gz: 54494ec90b395bf6417c9a437beb128661522b4ebed0421d1129c0a3e9b9ad17e5acda8ec1524643b66dae70028bc6ba22e5686ef51ab57b8f839b4aab528a39
+  metadata.gz: 122b1ecaac183df2215020f86fdd9f534aed6f356e9bee3e0e53c281f88a6d0c519007aa9809f89c99cfbaeb2e563c712b597065373280160f655188c633749a
+  data.tar.gz: 5c4e2a6f3832390138c1d6ea4d96ede15e8119fb85d67829da6801b81ff0c810d1e42c072bfcd647586b67c5482ec7209168015d0c97bc870326a5b6302916e2

data/lib/git-fit.rb

@@ -87,6 +87,7 @@ require_relative 'git_fit/export'
 require_relative 'git_fit/db/cli'
 require_relative 'git_fit/install/actions'
 require_relative 'git_fit/cli/install_cli'
+require_relative 'git_fit/cli/purge_cli'
 require_relative 'git_fit/cli/gh_cli'
 require_relative 'git_fit/cli/export'
 require_relative 'git_fit/cli/import_cli'

data/lib/git_fit/cli/purge_cli.rb

@@ -0,0 +1,118 @@
+# frozen_string_literal: true
+
+module GitFit
+  class PurgeCLI < Thor
+    include Cli::Checkpointable
+
+    class RecordNotFound < StandardError; end
+    class PurgeFailed < StandardError; end
+
+    def self.exit_on_failure?
+      false
+    end
+
+    no_commands do
+      def git_fit_config
+        @git_fit_config ||= GitFit::Config.new(options[:config])
+      end
+    end
+
+    desc 'activity RUN_ID', 'Delete activity by run_id (e.g. keep_9223370467120855807)'
+    option :dry_run, type: :boolean, default: false, desc: 'Preview without deleting'
+    option :checkpoint, type: :boolean, default: true, desc: 'Run WAL checkpoint after delete'
+    option :delete_raw, type: :boolean, default: false, desc: 'Also delete raw source file(s)'
+    option :delete_std, type: :boolean, default: false, desc: 'Also delete standardized JSON file'
+
+    def activity(run_id)
+      db = connect_db
+
+      record = db[:activities].where(run_id: run_id).first
+
+      raise RecordNotFound, "Not found: #{run_id}" unless record
+
+      if options[:dry_run]
+        say_status :dry_run, 'Would delete activity:', :yellow
+        say "  run_id:          #{record[:run_id]}"
+        say "  name:            #{record[:name]}"
+        say "  source:          #{record[:source]}"
+        say "  sport_category:  #{record[:sport_category]}"
+        say "  start_date:      #{record[:start_date]}"
+        say "  distance:        #{record[:distance]}m" if record[:distance]
+        say "  moving_time:     #{record[:moving_time]}s" if record[:moving_time]
+        list_raw_files(run_id) if options[:delete_raw]
+        list_std_file(run_id) if options[:delete_std]
+        return
+      end
+
+      db[:dedup_group_members].where(run_id: run_id).delete
+      deleted = db[:activities].where(run_id: run_id).delete
+
+      say_status :deleted, "Removed #{deleted} activity: #{run_id}", :green
+
+      delete_raw_files(run_id) if options[:delete_raw]
+      delete_std_file(run_id) if options[:delete_std]
+
+      checkpoint_db(db) if options[:checkpoint]
+    rescue RecordNotFound
+      raise
+    rescue StandardError => e
+      raise PurgeFailed, "Purge failed: #{e.message}"
+    end
+
+    private
+
+    def connect_db
+      conn = GitFit::DB::Connection.new(git_fit_config.db_path)
+      conn.migrate!
+      conn.db
+    end
+
+    def delete_raw_files(run_id)
+      source, natural_id = GitFit::Std::Resolver.decompose(run_id)
+      return unless source && natural_id
+
+      raw_base = File.join('data', 'raw', source, natural_id)
+      files = Dir.glob("#{raw_base}.*")
+      if files.any?
+        files.each { |f| File.delete(f) }
+      elsif File.directory?(raw_base)
+        FileUtils.rm_rf(raw_base)
+      end
+    end
+
+    def list_raw_files(run_id)
+      source, natural_id = GitFit::Std::Resolver.decompose(run_id)
+      return unless source && natural_id
+
+      raw_base = File.join('data', 'raw', source, natural_id)
+      files = Dir.glob("#{raw_base}.*")
+      if files.any?
+        files.each { |f| say "Would delete: #{f}" }
+      elsif File.directory?(raw_base)
+        say "Would delete: #{raw_base}/ (directory)"
+      else
+        say "No raw files found for #{run_id}"
+      end
+    end
+
+    def delete_std_file(run_id)
+      source, natural_id = GitFit::Std::Resolver.decompose(run_id)
+      return unless source && natural_id
+
+      std_path = File.join('data', 'std', source, "#{natural_id}.json")
+      File.delete(std_path) if File.exist?(std_path)
+    end
+
+    def list_std_file(run_id)
+      source, natural_id = GitFit::Std::Resolver.decompose(run_id)
+      return unless source && natural_id
+
+      std_path = File.join('data', 'std', source, "#{natural_id}.json")
+      if File.exist?(std_path)
+        say "Would delete: #{std_path}"
+      else
+        say "No std file found for #{run_id}"
+      end
+    end
+  end
+end

data/lib/git_fit/cli.rb

@@ -120,6 +120,9 @@ module GitFit
     desc 'strava SUBCOMMAND', 'Strava web operations'
     subcommand 'strava', StravaCLI
 
+    desc 'purge SUBCOMMAND', 'Purge activities from database'
+    subcommand 'purge', PurgeCLI
+
     desc 'install SUBCOMMAND', 'Install project assets (actions)'
     subcommand 'install', InstallCLI
 

data/lib/git_fit/export/calculator.rb

@@ -100,46 +100,41 @@ module GitFit
                    max_power_peak: 0.0, total_max_power: 0.0, max_power_count: 0 }
         end
 
-        rows.each do |r|
-          next unless r[:start_date_local] || r[:start_date]
+        rows.each { |row| accumulate_monthly_row(monthly, row) }
 
-          ym = (r[:start_date_local] || r[:start_date])[0..6]
-          d = r[:distance] || 0
-          t = r[:moving_time] || 0
-          hr = r[:average_heartrate]
-          e = r[:elevation_gain] || 0
-          l = r[:elevation_loss] || 0
-          s = r[:steps] || 0
+        format_monthly(monthly.sort)
+      end
 
-          monthly[ym][:count] += 1
-          monthly[ym][:distance_km] += d / 1000.0
-          monthly[ym][:moving_time_h] += t / 3600.0
-          monthly[ym][:total_elev] += e
-          monthly[ym][:total_loss] += l
-          monthly[ym][:total_steps] += s
-          if hr && hr > 0
-            monthly[ym][:total_hr] += hr
-            monthly[ym][:hr_count] += 1
-          end
-          cad = r[:average_cadence]
-          pow = r[:average_power]
-          if cad && cad > 0
-            monthly[ym][:total_cadence] += cad
-            monthly[ym][:cadence_count] += 1
-          end
-          if pow && pow > 0
-            monthly[ym][:total_power] += pow
-            monthly[ym][:power_count] += 1
-          end
-          max_pow = r[:max_power]
-          if max_pow && max_pow > 0
-            monthly[ym][:max_power_peak] = max_pow if max_pow > monthly[ym][:max_power_peak]
-            monthly[ym][:total_max_power] += max_pow
-            monthly[ym][:max_power_count] += 1
-          end
-        end
+      def accumulate_monthly_row(monthly, row)
+        return unless row[:start_date_local] || row[:start_date]
+
+        ym = (row[:start_date_local] || row[:start_date])[0..6]
+        acc = monthly[ym]
+        acc[:count] += 1
+        acc[:distance_km] += (row[:distance] || 0) / 1000.0
+        acc[:moving_time_h] += (row[:moving_time] || 0) / 3600.0
+        acc[:total_elev] += row[:elevation_gain] || 0
+        acc[:total_loss] += row[:elevation_loss] || 0
+        acc[:total_steps] += row[:steps] || 0
+        add_if_positive(acc, :total_hr, :hr_count, row[:average_heartrate])
+        add_if_positive(acc, :total_cadence, :cadence_count, row[:average_cadence])
+        add_if_positive(acc, :total_power, :power_count, row[:average_power])
+        max_pow = row[:max_power]
+        return unless max_pow && max_pow > 0
+        acc[:max_power_peak] = max_pow if max_pow > acc[:max_power_peak]
+        acc[:total_max_power] += max_pow
+        acc[:max_power_count] += 1
+      end
+
+      def add_if_positive(acc, sum_key, count_key, value)
+        return unless value && value > 0
+
+        acc[sum_key] += value
+        acc[count_key] += 1
+      end
 
-        monthly.sort.map do |ym, v|
+      def format_monthly(sorted)
+        sorted.map do |ym, v|
           pace = if v[:moving_time_h] > 0 && v[:distance_km] > 0
                    format_pace(v[:moving_time_h] * 3600 / v[:distance_km])
                  end

data/lib/git_fit/sync/xoss.rb

@@ -73,7 +73,9 @@ module GitFit
             next if skip_type?(type)
             seen[act_id] = seen.fetch(act_id, 0) + 1
             if seen[act_id] > 3
-              raise SyncError, "XOSS: activity list returned #{seen[act_id]} duplicate records for id #{act_id} — server-side degradation.  Run: git fit sync --source xoss"
+              raise SyncError,
+                    "XOSS: activity list returned #{seen[act_id]} duplicate records for id #{act_id} " \
+                    '— server-side degradation.  Run: git fit sync --source xoss'
             end
             next if seen[act_id] > 1
             ids << { id: act_id, type: type, summary: act }

data/lib/git_fit/version.rb

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

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: git-fit
 version: !ruby/object:Gem::Version
-  version: 0.16.1
+  version: 0.17.1
 platform: ruby
 authors:
 - Lax
@@ -262,6 +262,7 @@ files:
 - lib/git_fit/cli/gh_cli.rb
 - lib/git_fit/cli/import_cli.rb
 - lib/git_fit/cli/install_cli.rb
+- lib/git_fit/cli/purge_cli.rb
 - lib/git_fit/cli/strava.rb
 - lib/git_fit/cli/sync.rb
 - lib/git_fit/config.rb