kagu 3.2.0 → 4.0.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: e1c1858a37f00f8823ce4ebacf2b9bb96837da05c1e7be296fcbcd1461afe03f
-  data.tar.gz: f8d1d56f7e6e90a3afa12aeabf4f2804dda4d7e3bd4c0be9cfcb1b282a4078d3
+  metadata.gz: d1cedd40180e7fdaaafb0a589b9497904dc1fea117988356a1b9825720d906eb
+  data.tar.gz: 6bd45283ddd23b1151928c3d5fe01fa57591df612c90604a1994271500dcf380
 SHA512:
-  metadata.gz: 88e72ddf7e6960712558490d614f9e82aa78c6908f5300ac8d29ff587a2e40ee59a471ed9538b03155e0b130bfb15211b3b2baf6ce45e34e338c0f8a3cc1e909
-  data.tar.gz: 9d0fe80ce5dc1a8a857409f60425ff349a555a374893ba0c26cd5fea177c0774f4acb21cd0c920d5d0cf28701edbf024a3351e865a3ef80bb9a5129234682305
+  metadata.gz: 5c58699f791c9f6451aa0e2c40b98bbd8cf7749655e27b96856ed73a5273e323948891025df00d6f84f510b079b4fab00ab629e54c7d4cb6ed3a1f3268438b53
+  data.tar.gz: cb1ceb390210ae2ce2408cd9c801dd9853aa8712767e178eebac86864bb2d670a4042c190be832b55714c9570b33993ac27a314ae404f107bbd8b53f61158320

data/VERSION

@@ -1 +1 @@
-3.2.0
+4.0.0

data/kagu.gemspec

@@ -9,17 +9,20 @@ Gem::Specification.new do |s|
   s.description = 'API to manage macOS Music tracks and playlists'
   s.license = 'MIT'
 
-  s.files = `git ls-files | grep -vE '^(spec/|test/|\\.|Gemfile|Rakefile)'`.split("\n")
-  s.executables = `git ls-files -- bin/*`.split("\n").map{ |f| File.basename(f) }
+  s.files = %x(git ls-files | grep -vE '^(spec/|test/|\\.|Gemfile|Rakefile)').split("\n")
+  s.executables = %x(git ls-files -- bin/*).split("\n").map { |f| File.basename(f) }
   s.require_paths = ['lib']
 
-  s.required_ruby_version = '>= 2.0.0'
+  s.required_ruby_version = '>= 3.1.0'
 
-  s.add_dependency 'activesupport', '>= 4.1.0', '< 7.0.0'
+  s.add_dependency 'activesupport', '>= 7.0.0', '< 8.0.0'
   s.add_dependency 'addressable', '>= 2.7.0', '< 3.0.0'
   s.add_dependency 'applescript', '>= 1.0', '< 2.0'
 
   s.add_development_dependency 'byebug', '>= 3.2.0', '< 12.0.0'
   s.add_development_dependency 'rake', '>= 10.3.0', '< 14.0.0'
   s.add_development_dependency 'rspec', '>= 3.1.0', '< 3.11.0'
+  s.add_development_dependency 'rubocop', '>= 1.25.0', '< 2.0.0'
+  s.add_development_dependency 'rubocop-rake', '>= 0.6.0', '< 1.0.0'
+  s.add_development_dependency 'rubocop-rspec', '>= 2.8.0', '< 3.0.0'
 end

data/lib/kagu/finder.rb

@@ -2,7 +2,7 @@ module Kagu
 
   class Finder
 
-    MANDATORY_ATTRIBUTES = []
+    MANDATORY_ATTRIBUTES = [].freeze
 
     delegate :replace, :transliterate, to: 'self.class'
 
@@ -29,7 +29,7 @@ module Kagu
         tracks_digests.each_with_index do |hash, digests_index|
           digests(attributes).each_with_index do |digest, digest_index|
             tracks = hash[digest].presence || next
-            tracks = tracks.select { |track| !matches.any? { |match| match.try(:include?, track) } }
+            tracks = tracks.reject { |track| matches.any? { |match| match.try(:include?, track) } }
             next if tracks.empty?
             index = [digests_index, digest_index].max
             matches[index] ||= []
@@ -83,10 +83,11 @@ module Kagu
 
     def ignored=(values)
       @ignored = [values].flatten.map do |value|
-        if value.is_a?(Hash)
+        case value
+        when Hash
           value = value.stringify_keys
           value = "#{value['artist']} #{value['title']}"
-        elsif value.is_a?(Track)
+        when Track
           value = "#{value.artist} #{value.title}"
         else
           value = value.to_s

data/lib/kagu/playlist.rb

@@ -2,12 +2,12 @@ module Kagu
 
   class Playlist
 
-    MANDATORY_ATTRIBUTES = %w(name)
+    MANDATORY_ATTRIBUTES = %w(name).freeze
 
     include AttributesInitializer
     include Enumerable
 
-    attr_reader :name, :tracks
+    attr_reader :name
 
     delegate :each, to: :tracks
 
@@ -31,7 +31,7 @@ module Kagu
       return if tracks.empty?
       Kagu.logger.info('Kagu') { "Adding #{tracks.size} track(s) to playlist #{name.inspect}" }
       tracks.map(&:id).each_slice(500) do |ids|
-        AppleScript.execute(%Q{
+        AppleScript.execute("
           tell application #{Kagu::OSX_APP_NAME.inspect}
             set playlistToPush to user playlist #{name.inspect}
             set idsToAdd to {#{ids.map(&:inspect).join(',')}}
@@ -39,7 +39,7 @@ module Kagu
               duplicate (tracks of library playlist 1 whose persistent ID is idToAdd) to playlistToPush
             end repeat
           end tell
-        })
+        ")
       end
       true
     rescue => e
@@ -48,11 +48,11 @@ module Kagu
 
     def clear
       Kagu.logger.info('Kagu') { "Removing all tracks from playlist #{name.inspect}" }
-      AppleScript.execute(%Q{
+      AppleScript.execute("
         tell application #{Kagu::OSX_APP_NAME.inspect}
           delete tracks of playlist #{name.inspect}
         end tell
-      })
+      ")
       true
     rescue => e
       raise Error.new(e)
@@ -60,13 +60,13 @@ module Kagu
 
     def create
       Kagu.logger.info('Kagu') { "Creating playlist #{name.inspect}" }
-      AppleScript.execute(%Q{
+      AppleScript.execute("
         tell application #{Kagu::OSX_APP_NAME.inspect}
           if not (exists user playlist #{name.inspect}) then
             make new user playlist with properties { name: #{name.inspect} }
           end if
         end tell
-      })
+      ")
       true
     rescue => e
       raise Error.new(e)

data/lib/kagu/playlists.rb

@@ -12,27 +12,27 @@ module Kagu
       build(attributes).tap(&:save)
     end
 
-    def each(&block)
+    def each
       return unless block_given?
       Kagu.logger.debug('Kagu') { 'Loading library playlists' }
-      tracks = {}.tap do |tracks|
-        Tracks.new.each { |track| tracks[track.id] = track }
+      tracks = {}.tap do |items|
+        Tracks.new.each { |track| items[track.id] = track }
       end
       playlist_name = nil
       playlist_tracks = []
-      SwiftHelper.execute(%Q{
+      SwiftHelper.execute("
         import iTunesLibrary
 
-        let library = try! ITLibrary(apiVersion: "1")
+        let library = try! ITLibrary(apiVersion: \"1\")
         for playlist in library.allPlaylists.filter({ !$0.isMaster }) {
-          print("BEGIN_PLAYLIST")
+          print(\"BEGIN_PLAYLIST\")
           print(playlist.name)
           for track in playlist.items.filter({ $0.mediaKind == ITLibMediaItemMediaKind.kindSong }) {
             print(String(track.persistentID.uint64Value, radix: 16).uppercased())
           }
-          print("END_PLAYLIST")
+          print(\"END_PLAYLIST\")
         }
-      }).each do |line|
+      ").each do |line|
         if line == 'BEGIN_PLAYLIST'
           playlist_name = nil
           playlist_tracks = []

data/lib/kagu/track.rb

@@ -5,7 +5,7 @@ module Kagu
     include AttributesInitializer
     include Comparable
 
-    MANDATORY_ATTRIBUTES = %w(added_at id length)
+    MANDATORY_ATTRIBUTES = %w(added_at id length).freeze
 
     attr_reader :added_at, :album, :artist, :bpm, :genre, :id, :length, :path, :title, :year
 
@@ -37,9 +37,10 @@ module Kagu
     private
 
     def added_at=(value)
-      if value.is_a?(String)
+      case value
+      when String
         value = Time.parse(value)
-      elsif value.is_a?(Integer)
+      when Integer
         value = Time.at(value)
       end
       @added_at = value.is_a?(Time) ? value.utc : nil
@@ -55,7 +56,7 @@ module Kagu
 
     def bpm=(value)
       value = value.to_s =~ /\A[0-9]+\z/ ? value.to_i : nil
-      @bpm = (value && value > 0) ? value : nil
+      @bpm = value && value > 0 ? value : nil
     end
 
     def genre=(value)

data/lib/kagu/tracks.rb

@@ -6,41 +6,41 @@ module Kagu
 
     EXTENSIONS = %w(.aac .flac .mp3 .wav).freeze
 
-    def each(&block)
+    def each
       return unless block_given?
       Kagu.logger.debug('Kagu') { 'Loading library tracks' }
       attributes = {}
-      SwiftHelper.execute(%Q{
+      SwiftHelper.execute("
         import iTunesLibrary
 
         func printObjectProperty<T: Encodable>(name: String, value: T?) {
           let jsonEncoder = JSONEncoder()
           let jsonData = try! jsonEncoder.encode(value)
           let json = String(data: jsonData, encoding: String.Encoding.utf8)
-          print("\\(name)=\\(json!)")
+          print(\"\\(name)=\\(json!)\")
         }
 
-        let library = try! ITLibrary(apiVersion: "1")
+        let library = try! ITLibrary(apiVersion: \"1\")
         for track in library.allMediaItems.filter({ $0.mediaKind == ITLibMediaItemMediaKind.kindSong }) {
-          print("BEGIN_TRACK")
-          printObjectProperty(name: "added_at", value: track.addedDate!.timeIntervalSince1970)
-          printObjectProperty(name: "album", value: track.album.title)
-          printObjectProperty(name: "artist", value: track.artist!.name)
-          printObjectProperty(name: "bpm", value: track.beatsPerMinute)
-          printObjectProperty(name: "genre", value: track.genre)
-          printObjectProperty(name: "id", value: String(track.persistentID.uint64Value, radix: 16).uppercased())
-          printObjectProperty(name: "length", value: track.totalTime)
-          printObjectProperty(name: "path", value: track.location)
-          printObjectProperty(name: "title", value: track.title)
-          printObjectProperty(name: "year", value: track.year)
-          print("END_TRACK")
+          print(\"BEGIN_TRACK\")
+          printObjectProperty(name: \"added_at\", value: track.addedDate!.timeIntervalSince1970)
+          printObjectProperty(name: \"album\", value: track.album.title)
+          printObjectProperty(name: \"artist\", value: track.artist!.name)
+          printObjectProperty(name: \"bpm\", value: track.beatsPerMinute)
+          printObjectProperty(name: \"genre\", value: track.genre)
+          printObjectProperty(name: \"id\", value: String(track.persistentID.uint64Value, radix: 16).uppercased())
+          printObjectProperty(name: \"length\", value: track.totalTime)
+          printObjectProperty(name: \"path\", value: track.location)
+          printObjectProperty(name: \"title\", value: track.title)
+          printObjectProperty(name: \"year\", value: track.year)
+          print(\"END_TRACK\")
         }
-      }).each do |line|
+      ").each do |line|
         if line == 'BEGIN_TRACK'
           attributes = {}
         elsif line == 'END_TRACK'
           yield(Track.new(attributes))
-        elsif match = /(^\w+)=(.*)/.match(line)
+        elsif (match = /(^\w+)=(.*)/.match(line))
           attributes[match[1]] = JSON.parse(match[2])
         end
       end

data/lib/kagu.rb

@@ -15,9 +15,7 @@ module Kagu
   IS_MAC_OS = RUBY_PLATFORM =~ /darwin/
   OSX_APP_NAME = begin
     if IS_MAC_OS
-      `sw_vers -productVersion`.chomp.to_f >= 10.15 ? 'Music' : 'iTunes'
-    else
-      nil
+      %x(sw_vers -productVersion).chomp.to_f >= 10.15 ? 'Music' : 'iTunes'
     end
   end
 

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: kagu
 version: !ruby/object:Gem::Version
-  version: 3.2.0
+  version: 4.0.0
 platform: ruby
 authors:
 - Alexis Toulotte
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2021-06-14 00:00:00.000000000 Z
+date: 2022-02-06 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activesupport
@@ -16,20 +16,20 @@ dependencies:
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: 4.1.0
+        version: 7.0.0
     - - "<"
       - !ruby/object:Gem::Version
-        version: 7.0.0
+        version: 8.0.0
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: 4.1.0
+        version: 7.0.0
     - - "<"
       - !ruby/object:Gem::Version
-        version: 7.0.0
+        version: 8.0.0
 - !ruby/object:Gem::Dependency
   name: addressable
   requirement: !ruby/object:Gem::Requirement
@@ -130,6 +130,66 @@ dependencies:
     - - "<"
       - !ruby/object:Gem::Version
         version: 3.11.0
+- !ruby/object:Gem::Dependency
+  name: rubocop
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: 1.25.0
+    - - "<"
+      - !ruby/object:Gem::Version
+        version: 2.0.0
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: 1.25.0
+    - - "<"
+      - !ruby/object:Gem::Version
+        version: 2.0.0
+- !ruby/object:Gem::Dependency
+  name: rubocop-rake
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: 0.6.0
+    - - "<"
+      - !ruby/object:Gem::Version
+        version: 1.0.0
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: 0.6.0
+    - - "<"
+      - !ruby/object:Gem::Version
+        version: 1.0.0
+- !ruby/object:Gem::Dependency
+  name: rubocop-rspec
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: 2.8.0
+    - - "<"
+      - !ruby/object:Gem::Version
+        version: 3.0.0
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: 2.8.0
+    - - "<"
+      - !ruby/object:Gem::Version
+        version: 3.0.0
 description: API to manage macOS Music tracks and playlists
 email: al@alweb.org
 executables: []
@@ -162,14 +222,14 @@ required_ruby_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
-      version: 2.0.0
+      version: 3.1.0
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.2.15
+rubygems_version: 3.3.3
 signing_key:
 specification_version: 4
 summary: API for macOS Music