git-fit 0.7.9 → 0.8.0

data/lib/git_fit/import/apple_health.rb

@@ -1,10 +1,12 @@
-require "digest"
-require "fileutils"
-require "json"
-require "nokogiri"
-require "time"
-require "tzinfo"
-require "zip"
+# frozen_string_literal: true
+
+require 'digest'
+require 'fileutils'
+require 'json'
+require 'nokogiri'
+require 'time'
+require 'tzinfo'
+require 'zip'
 
 module GitFit
   module Import
@@ -16,7 +18,7 @@ module GitFit
 
       APPLE_WATCH_PATTERN = /Apple Watch/
 
-      DEFAULT_ZIP_PATH = "data/import/apple_health/export.zip"
+      DEFAULT_ZIP_PATH = 'data/import/apple_health/export.zip'
 
       HeartRateRecord = Struct.new(:time, :value, keyword_init: true)
 
@@ -24,7 +26,7 @@ module GitFit
 
       def initialize(config:, db:, activity_filter: nil, privacy: nil, time_budget: nil)
         super
-        @zip_path = @config["zip_path"] || DEFAULT_ZIP_PATH
+        @zip_path = @config['zip_path'] || DEFAULT_ZIP_PATH
         @heart_rate_index = nil
         @heart_rate_by_date = nil
         @skip_patterns = load_skip_sources
@@ -64,40 +66,40 @@ module GitFit
         hr_records = []
         @route_index = []
         zip = Zip::File.new(@zip_path)
-        xml_entry = zip.find_entry("apple_health_export/export.xml")
+        xml_entry = zip.find_entry('apple_health_export/export.xml')
         return results unless xml_entry
 
         io = xml_entry.get_input_stream
-        buffer = ""
+        buffer = ''
         in_workout = false
         route_buf = nil
 
         io.each_line do |line|
-          if line.include?("HKQuantityTypeIdentifierHeartRate") && APPLE_WATCH_PATTERN.match?(line)
+          if line.include?('HKQuantityTypeIdentifierHeartRate') && APPLE_WATCH_PATTERN.match?(line)
             hr = parse_hr_line(line)
             hr_records << hr if hr
           end
 
-          if line.include?("<WorkoutRoute ")
+          if line.include?('<WorkoutRoute ')
             route_buf = line
           elsif route_buf
             route_buf << line
-            if line.include?("</WorkoutRoute>")
+            if line.include?('</WorkoutRoute>')
               process_route_entry(route_buf)
               route_buf = nil
             end
           end
 
           if !in_workout
-            if line.include?("<Workout ")
+            if line.include?('<Workout ')
               in_workout = true
               buffer = line
             end
           else
             buffer << line
-            if line.include?("</Workout>")
+            if line.include?('</Workout>')
               process_workout_chunk(buffer, existing, results)
-              buffer = ""
+              buffer = ''
               in_workout = false
             end
           end
@@ -107,7 +109,7 @@ module GitFit
 
         @heart_rate_by_date = {}
         @heart_rate_index.each do |hr|
-          date_key = hr.time.strftime("%Y-%m-%d")
+          date_key = hr.time.strftime('%Y-%m-%d')
           @heart_rate_by_date[date_key] ||= []
           @heart_rate_by_date[date_key] << hr
         end
@@ -153,7 +155,7 @@ module GitFit
             begin
               zip = Zip::File.new(@zip_path)
               route_paths.each do |gpx_path|
-                normalized = gpx_path.sub(%r{^/?}, "apple_health_export/")
+                normalized = gpx_path.sub(%r{^/?}, 'apple_health_export/')
                 entry = zip.find_entry(normalized)
                 next unless entry
                 data = entry.get_input_stream.read
@@ -162,7 +164,7 @@ module GitFit
                 File.rename(tmpp, gpx_l1_path)
                 break
               end
-            rescue => e
+            rescue StandardError => e
               $stderr.puts "AppleHealth: GPX extract error for #{run_id}: #{e.message}"
             ensure
               zip&.close
@@ -210,15 +212,15 @@ module GitFit
       end
 
       def source_label
-        "AppleHealth"
+        'AppleHealth'
       end
 
       def raw_dir
-        File.join("data", "raw", "apple_health")
+        File.join('data', 'raw', 'apple_health')
       end
 
       def std_dir
-        File.join("data", "std", "apple_health")
+        File.join('data', 'std', 'apple_health')
       end
 
       def raw_path(platform_id)
@@ -233,10 +235,10 @@ module GitFit
       end
 
       def load_skip_sources
-        configured = @config["skip_sources"]
+        configured = @config['skip_sources']
         return [] unless configured
 
-        list = configured.is_a?(String) ? configured.split(",").map(&:strip) : configured
+        list = configured.is_a?(String) ? configured.split(',').map(&:strip) : configured
         validated = list.select do |name|
           SKIPPED_SOURCE_PATTERNS.any? { |p| p.match?(name) }
         end
@@ -247,7 +249,7 @@ module GitFit
       end
 
       def existing_run_ids_set
-        @db[:activities].where(source: "apple_health").select_map(:run_id).map(&:to_s).to_set
+        @db[:activities].where(source: 'apple_health').select_map(:run_id).map(&:to_s).to_set
       end
 
       def parse_hr_line(line)
@@ -277,7 +279,7 @@ module GitFit
         if @heart_rate_by_date
           results = []
           (start_t.to_date..end_t.to_date).each do |date|
-            date_key = date.strftime("%Y-%m-%d")
+            date_key = date.strftime('%Y-%m-%d')
             bucket = @heart_rate_by_date[date_key]
             if bucket&.any?
               lo = bucket.bsearch_index { |hr| hr.time >= start_t }
@@ -301,24 +303,24 @@ module GitFit
 
       def process_workout_chunk(raw_xml, existing_ids, results)
         doc = Nokogiri::XML("<root>#{raw_xml}</root>")
-        workout = doc.at_xpath("//Workout")
+        workout = doc.at_xpath('//Workout')
         return unless workout
 
-        source_name = workout["sourceName"]
+        source_name = workout['sourceName']
         return if known_source?(source_name)
 
-        workout_type = workout["workoutActivityType"]
+        workout_type = workout['workoutActivityType']
         return unless workout_type
 
-        start_date = workout["startDate"]
-        end_date = workout["endDate"]
+        start_date = workout['startDate']
+        end_date = workout['endDate']
         return if start_date.nil? || end_date.nil?
 
-        duration = workout["duration"]
-        duration_unit = workout["durationUnit"] || "min"
+        duration = workout['duration']
+        duration_unit = workout['durationUnit'] || 'min'
         return if duration.nil?
 
-        creation_date = workout["creationDate"] || start_date
+        creation_date = workout['creationDate'] || start_date
         run_id = generate_run_id(source_name, creation_date, start_date, end_date)
         return if existing_ids.include?("apple_health_#{run_id}")
 
@@ -329,7 +331,7 @@ module GitFit
         meta = extract_metadata(workout)
         route_paths = extract_route_paths(workout)
         route_paths = match_routes_by_time(start_date, end_date) if route_paths.empty?
-        source_version = workout["sourceVersion"]
+        source_version = workout['sourceVersion']
 
         results << {
           id: run_id,
@@ -346,26 +348,26 @@ module GitFit
           stats: stats,
           meta: meta,
           route_paths: route_paths,
-          creation_date: creation_date
+          creation_date: creation_date,
         }
       end
 
       def process_route_entry(xml)
         doc = Nokogiri::XML("<root>#{xml}</root>")
-        route = doc.at_xpath("//WorkoutRoute")
+        route = doc.at_xpath('//WorkoutRoute')
         return unless route
 
-        start_date = route["startDate"]
-        end_date = route["endDate"]
+        start_date = route['startDate']
+        end_date = route['endDate']
         return unless start_date && end_date
 
-        paths = route.xpath("FileReference").filter_map { |ref| ref["path"] }
+        paths = route.xpath('FileReference').filter_map { |ref| ref['path'] }
         return if paths.empty?
 
         @route_index << {
           route_start: (Time.parse(start_date) rescue nil),
           route_end: (Time.parse(end_date) rescue nil),
-          route_paths: paths
+          route_paths: paths,
         }
       end
 
@@ -386,47 +388,47 @@ module GitFit
 
       def extract_statistics(workout)
         stats = {}
-        workout.xpath("WorkoutStatistics").each do |stat|
-          type = stat["type"]
-          sum = stat["sum"]
-          unit = stat["unit"]
+        workout.xpath('WorkoutStatistics').each do |stat|
+          type = stat['type']
+          sum = stat['sum']
+          unit = stat['unit']
           next unless type && sum
-          stats[type] = { "sum" => sum, "unit" => unit }
+          stats[type] = { 'sum' => sum, 'unit' => unit }
         end
         stats
       end
 
       def extract_metadata(workout)
         meta = {}
-        workout.xpath("MetadataEntry").each do |entry|
-          key = entry["key"]
-          value = entry["value"]
+        workout.xpath('MetadataEntry').each do |entry|
+          key = entry['key']
+          value = entry['value']
           meta[key] = value if key && value
         end
         meta
       end
 
       def extract_route_paths(workout)
-        workout.xpath("WorkoutRoute/FileReference").filter_map do |ref|
-          ref["path"]
+        workout.xpath('WorkoutRoute/FileReference').filter_map do |ref|
+          ref['path']
         end
       end
 
       def parse_gpx_file(file_path)
         doc = Nokogiri::XML(File.read(file_path))
         doc.remove_namespaces!
-        doc.xpath("//trkpt").filter_map do |pt|
-          lat = pt["lat"]&.to_f
-          lng = pt["lon"]&.to_f
+        doc.xpath('//trkpt').filter_map do |pt|
+          lat = pt['lat']&.to_f
+          lng = pt['lon']&.to_f
           next unless lat && lng
-          ele = pt.at_xpath("ele")&.text&.to_f
-          time_str = pt.at_xpath("time")&.text
-          ext = pt.at_xpath("extensions")
-          speed = ext&.at_xpath("speed")&.text&.to_f
+          ele = pt.at_xpath('ele')&.text&.to_f
+          time_str = pt.at_xpath('time')&.text
+          ext = pt.at_xpath('extensions')
+          speed = ext&.at_xpath('speed')&.text&.to_f
           speed = nil if speed && speed < MIN_GPX_SPEED
           { lat: lat, lng: lng, ele: ele, time: time_str, speed: speed }
         end
-      rescue => e
+      rescue StandardError => e
         $stderr.puts "AppleHealth: GPX parse error for #{file_path}: #{e.message}"
         nil
       end
@@ -439,10 +441,10 @@ module GitFit
 
         cals_sum = 0.0
         stats.each do |type_str, data|
-          if type_str.include?("Distance")
-            distance = parse_distance(data["sum"], data["unit"])
-          elsif type_str.include?("ActiveEnergyBurned") || type_str.include?("BasalEnergyBurned")
-            cals_sum += data["sum"].to_f
+          if type_str.include?('Distance')
+            distance = parse_distance(data['sum'], data['unit'])
+          elsif type_str.include?('ActiveEnergyBurned') || type_str.include?('BasalEnergyBurned')
+            cals_sum += data['sum'].to_f
           end
         end
         calories = cals_sum > 0 ? cals_sum.round : nil
@@ -462,16 +464,16 @@ module GitFit
           sport_category: map_sport_type(workout_type),
           sport_type: workout_type,
           start_date: format_datetime(start_date),
-          start_date_local: meta["HKTimeZone"] ? format_datetime(start_date, utc: false) : nil,
+          start_date_local: meta['HKTimeZone'] ? format_datetime(start_date, utc: false) : nil,
           summary_polyline: polyline,
           average_heartrate: hr_avg,
           max_heartrate: hr_max,
           average_speed: avg_speed,
           calories: calories,
           elevation_gain: elevation_gain&.round(1),
-          average_temperature: parse_temperature(meta["HKWeatherTemperature"]),
-          source: "apple_health",
-          external_id: meta["HKExternalUUID"]
+          average_temperature: parse_temperature(meta['HKWeatherTemperature']),
+          source: 'apple_health',
+          external_id: meta['HKExternalUUID'],
         }
       end
 
@@ -493,9 +495,9 @@ module GitFit
       def parse_duration(value, unit)
         v = value.to_f
         case unit.to_s.downcase
-        when "min" then (v * 60).round
-        when "sec" then v.round
-        when "hour" then (v * 3600).round
+        when 'min' then (v * 60).round
+        when 'sec' then v.round
+        when 'hour' then (v * 3600).round
         else (v * 60).round
         end
       end
@@ -503,10 +505,10 @@ module GitFit
       def parse_distance(value, unit)
         v = value.to_f
         case unit.to_s.downcase
-        when "km" then (v * 1000).round(2)
-        when "mi" then (v * 1609.344).round(2)
-        when "m" then v.round(2)
-        when "ft" then (v * 0.3048).round(2)
+        when 'km' then (v * 1000).round(2)
+        when 'mi' then (v * 1609.344).round(2)
+        when 'm' then v.round(2)
+        when 'ft' then (v * 0.3048).round(2)
         else v.round(2)
         end
       end
@@ -517,7 +519,7 @@ module GitFit
         return nil unless m
         num = m[1].to_f
         unit = m[2].downcase
-        if unit.start_with?("degf") || unit.start_with?("°f")
+        if unit.start_with?('degf') || unit.start_with?('°f')
           ((num - 32) * 5.0 / 9.0).round(1)
         else
           num.round(1)
@@ -536,22 +538,22 @@ module GitFit
       def normalize_date(date_str)
         t = Time.parse(date_str) rescue nil
         return date_str unless t
-        t.utc.strftime("%Y-%m-%dT%H:%M:%SZ")
+        t.utc.strftime('%Y-%m-%dT%H:%M:%SZ')
       end
 
       def map_sport_type(workout_type)
-        return "other" unless workout_type
-        key = workout_type.sub(/\AHKWorkoutActivityType/, "")
-        return "workout" if key.downcase.include?("strength")
+        return 'other' unless workout_type
+        key = workout_type.sub(/\AHKWorkoutActivityType/, '')
+        return 'workout' if key.downcase.include?('strength')
         GitFit::SportMapper.canonicalize(key)
       end
 
       def display_name(workout_type, _start_date)
-        workout_type.sub(/\AHKWorkoutActivityType/, "")
+        workout_type.sub(/\AHKWorkoutActivityType/, '')
       end
 
       def build_title(workout_type, start_date)
-        label = workout_type.sub(/\AHKWorkoutActivityType/, "")
+        label = workout_type.sub(/\AHKWorkoutActivityType/, '')
         date = start_date ? start_date[0..9] : nil
         date ? "#{date} · #{label}" : label
       end

data/lib/git_fit/import/local_file.rb

@@ -1,23 +1,26 @@
-require "digest"
-require "fileutils"
-require "json"
-require "nokogiri"
-require "tzinfo"
+# frozen_string_literal: true
+
+require 'digest'
+require 'fileutils'
+require 'json'
+require 'nokogiri'
+require 'tzinfo'
 
 module GitFit
   module Import
     class LocalFile < Source::Base
       IMPORT_DIRS = {
-        "gpx" => "data/import/gpx",
-        "tcx" => "data/import/tcx",
-        "fit" => "data/import/fit"
+        'gpx' => 'data/import/gpx',
+        'tcx' => 'data/import/tcx',
+        'fit' => 'data/import/fit',
       }.freeze
 
       SUPPORTED_EXTENSIONS = %w[.gpx .tcx .fit].freeze
 
-      NS_TCX = { tcx: "http://www.garmin.com/xmlschemas/TrainingCenterDatabase/v2" }.freeze
+      NS_TCX = { tcx: 'http://www.garmin.com/xmlschemas/TrainingCenterDatabase/v2' }.freeze
 
-      def initialize(config:, db:, activity_filter: nil, privacy: nil, time_budget: nil, sources: nil, sport_override: nil)
+      def initialize(config:, db:, activity_filter: nil, privacy: nil, time_budget: nil, sources: nil,
+                     sport_override: nil)
         super(config:, db:, activity_filter:, privacy:, time_budget:)
         @sources = sources
         @sport_override = sport_override
@@ -32,7 +35,7 @@ module GitFit
           dir = IMPORT_DIRS[source]
           next unless dir && File.directory?(dir)
 
-          files = Dir.glob(File.join(dir, "*"), File::FNM_CASEFOLD)
+          files = Dir.glob(File.join(dir, '*'), File::FNM_CASEFOLD)
                      .select { |f| SUPPORTED_EXTENSIONS.include?(File.extname(f).downcase) && File.file?(f) }
 
           total_imported += process_files(files, source)
@@ -50,15 +53,15 @@ module GitFit
       end
 
       def source_label
-        "LocalFile"
+        'LocalFile'
       end
 
       def raw_dir_for(source)
-        File.join("data", "raw", source)
+        File.join('data', 'raw', source)
       end
 
       def std_dir_for(source)
-        File.join("data", "std", source)
+        File.join('data', 'std', source)
       end
 
       private
@@ -107,7 +110,7 @@ module GitFit
           activity[:source] = source
           activity[:run_id] = imported.zero? ? platform_id : "#{platform_id}_#{imported}"
 
-          if source == "gpx" && @sport_override
+          if source == 'gpx' && @sport_override
             activity[:sport_category] = SportMapper.canonicalize(@sport_override)
           end
 
@@ -141,18 +144,18 @@ module GitFit
         end
 
         { activity_count: imported }
-      rescue => e
+      rescue StandardError => e
         $stderr.puts "LocalFile: error importing #{file_path}: #{e.message}"
         nil
       end
 
       def parse_file(ext, content, file_path)
         case ext
-        when ".gpx"
+        when '.gpx'
           GitFit::Parser::GPX.new.call(content)
-        when ".tcx"
+        when '.tcx'
           GitFit::Parser::TCX.new.call(content)
-        when ".fit"
+        when '.fit'
           GitFit::Parser::FIT.new.call(file_path)
         else
           []
@@ -161,11 +164,11 @@ module GitFit
 
       def extract_l2_points(ext, content, file_path)
         case ext
-        when ".gpx"
+        when '.gpx'
           extract_gpx_points(content)
-        when ".tcx"
+        when '.tcx'
           extract_tcx_points(content)
-        when ".fit"
+        when '.fit'
           extract_fit_points(file_path)
         else
           nil
@@ -175,36 +178,36 @@ module GitFit
       def extract_gpx_points(xml)
         doc = Nokogiri::XML(xml)
         doc.remove_namespaces!
-        doc.xpath("//trkpt").filter_map do |pt|
-          lat = pt["lat"]&.to_f
-          lon = pt["lon"]&.to_f
+        doc.xpath('//trkpt').filter_map do |pt|
+          lat = pt['lat']&.to_f
+          lon = pt['lon']&.to_f
           next unless lat && lon
-          ele = pt.at_xpath("ele")&.text&.to_f
-          time_str = pt.at_xpath("time")&.text
+          ele = pt.at_xpath('ele')&.text&.to_f
+          time_str = pt.at_xpath('time')&.text
           time = parse_timestamp(time_str)
-          hr = pt.at_xpath("extensions//hr")&.text&.to_f
+          hr = pt.at_xpath('extensions//hr')&.text&.to_f
           [lat, lon, ele, time, hr]
         end
-      rescue => e
+      rescue StandardError => e
         $stderr.puts "LocalFile: GPX parse error: #{e.message}"
         nil
       end
 
       def extract_tcx_points(xml)
         doc = Nokogiri::XML(xml)
-        doc.xpath("//tcx:Trackpoint", NS_TCX).filter_map do |tp|
-          pos = tp.at_xpath("tcx:Position", NS_TCX)
+        doc.xpath('//tcx:Trackpoint', NS_TCX).filter_map do |tp|
+          pos = tp.at_xpath('tcx:Position', NS_TCX)
           next unless pos
-          lat = pos.at_xpath("tcx:LatitudeDegrees", NS_TCX)&.text&.to_f
-          lon = pos.at_xpath("tcx:LongitudeDegrees", NS_TCX)&.text&.to_f
+          lat = pos.at_xpath('tcx:LatitudeDegrees', NS_TCX)&.text&.to_f
+          lon = pos.at_xpath('tcx:LongitudeDegrees', NS_TCX)&.text&.to_f
           next unless lat && lon
-          ele = tp.at_xpath("tcx:AltitudeMeters", NS_TCX)&.text&.to_f
-          time_str = tp.at_xpath("tcx:Time", NS_TCX)&.text
+          ele = tp.at_xpath('tcx:AltitudeMeters', NS_TCX)&.text&.to_f
+          time_str = tp.at_xpath('tcx:Time', NS_TCX)&.text
           time = parse_timestamp(time_str)
-          hr = tp.at_xpath("tcx:HeartRateBpm/tcx:Value", NS_TCX)&.text&.to_f
+          hr = tp.at_xpath('tcx:HeartRateBpm/tcx:Value', NS_TCX)&.text&.to_f
           [lat, lon, ele, time, hr]
         end
-      rescue => e
+      rescue StandardError => e
         $stderr.puts "LocalFile: TCX parse error: #{e.message}"
         nil
       end
@@ -212,14 +215,14 @@ module GitFit
       def extract_fit_points(file_path)
         records = GitFit::FIT::Decoder.decode(file_path)
         records.filter_map do |r|
-          lat = r["positionLat"]
-          lon = r["positionLong"]
+          lat = r['positionLat']
+          lon = r['positionLong']
           next unless lat && lon
-          ts = r["timestamp"]
+          ts = r['timestamp']
           time = ts.is_a?(Numeric) ? (Time.utc(1989, 12, 31) + ts) : (Time.parse(ts.to_s) rescue nil)
-          [lat, lon, r["altitude"], time, r["heartRate"]]
+          [lat, lon, r['altitude'], time, r['heartRate']]
         end
-      rescue => e
+      rescue StandardError => e
         $stderr.puts "LocalFile: FIT decode error: #{e.message}"
         nil
       end

data/lib/git_fit/import.rb

@@ -1,2 +1,4 @@
-require_relative "import/local_file"
-require_relative "import/apple_health"
+# frozen_string_literal: true
+
+require_relative 'import/local_file'
+require_relative 'import/apple_health'

data/lib/git_fit/install/actions.rb

@@ -1,10 +1,12 @@
-require "erb"
-require "digest"
+# frozen_string_literal: true
+
+require 'erb'
+require 'digest'
 
 module GitFit
   module Install
     module Actions
-      TEMPLATE_DIR = File.expand_path("actions", __dir__)
+      TEMPLATE_DIR = File.expand_path('actions', __dir__)
       VERSION_STAMP = "# Generated by git-fit v#{GitFit::VERSION} — do not edit manually\n"
 
       def self.render(name, db_path:)
@@ -20,11 +22,11 @@ module GitFit
 
       def self.install_family(family, check: false, dry_run: false)
         dir = File.join(TEMPLATE_DIR, family)
-        db_path = ENV["GIT_FIT_DATABASE_PATH"] || "db/fit.sqlite3"
+        db_path = ENV['GIT_FIT_DATABASE_PATH'] || 'db/fit.sqlite3'
         results = []
 
-        Dir.glob("*.yml.erb", base: dir).sort.map do |tpl|
-          action_name = tpl.delete_suffix(".yml.erb")
+        Dir.glob('*.yml.erb', base: dir).sort.map do |tpl|
+          action_name = tpl.delete_suffix('.yml.erb')
           content = render("#{family}/#{action_name}", db_path: db_path)
           stamped = "#{VERSION_STAMP}\n#{content}"
           out_path = ".github/actions/#{family}/#{action_name}/action.yml"
@@ -61,9 +63,9 @@ module GitFit
       end
 
       def self.diff(a, b)
-        require "tempfile"
-        ta = Tempfile.new("a")
-        tb = Tempfile.new("b")
+        require 'tempfile'
+        ta = Tempfile.new('a')
+        tb = Tempfile.new('b')
         ta.write(a)
         tb.write(b)
         ta.close
@@ -76,14 +78,14 @@ module GitFit
         families.flat_map { |f| install_family(f, check: check, dry_run: dry_run) }
       end
 
-      def self.restore_action(db_path = "db/fit.sqlite3")
-        render("db-store/restore", db_path: db_path)
+      def self.restore_action(db_path = 'db/fit.sqlite3')
+        render('db-store/restore', db_path: db_path)
       end
 
       RESTORE_ACTION = restore_action
 
-      def self.save_action(db_path = "db/fit.sqlite3")
-        render("db-store/save", db_path: db_path)
+      def self.save_action(db_path = 'db/fit.sqlite3')
+        render('db-store/save', db_path: db_path)
       end
 
       SAVE_ACTION = save_action