abt-cli 0.0.21 → 0.0.22

data/lib/abt/providers/harvest/base_command.rb

@@ -22,28 +22,26 @@ module Abt
         def require_project!
           return if project_id
 
-          abort 'No current/specified project. Did you initialize Harvest?'
+          abort("No current/specified project. Did you initialize Harvest?")
         end
 
         def require_task!
-          unless project_id
-            abort 'No current/specified project. Did you initialize Harvest and pick a task?'
-          end
+          abort("No current/specified project. Did you initialize Harvest and pick a task?") unless project_id
 
-          abort 'No current/specified task. Did you pick a Harvest task?' if task_id.nil?
+          abort("No current/specified task. Did you pick a Harvest task?") if task_id.nil?
         end
 
         def print_project(project)
           cli.print_ari(
-            'harvest',
-            project['id'],
+            "harvest",
+            project["id"],
             "#{project['client']['name']} > #{project['name']}"
           )
         end
 
         def print_task(project, task)
           cli.print_ari(
-            'harvest',
+            "harvest",
             "#{project['id']}/#{task['id']}",
             "#{project['name']} > #{task['name']}"
           )

data/lib/abt/providers/harvest/commands/clear.rb

@@ -6,29 +6,28 @@ module Abt
       module Commands
         class Clear < BaseCommand
           def self.usage
-            'abt clear harvest'
+            "abt clear harvest"
           end
 
           def self.description
-            'Clear harvest configuration'
+            "Clear harvest configuration"
           end
 
           def self.flags
             [
-              ['-g', '--global', 'Clear global instead of local harvest configuration (credentials etc.)'],
-              ['-a', '--all', 'Clear all harvest configuration']
+              ["-g", "--global",
+               "Clear global instead of local harvest configuration (credentials etc.)"],
+              ["-a", "--all", "Clear all harvest configuration"]
             ]
           end
 
           def perform
-            if flags[:global] && flags[:all]
-              abort('Flags --global and --all cannot be used at the same time')
-            end
+            abort("Flags --global and --all cannot be used at the same time") if flags[:global] && flags[:all]
 
             config.clear_local unless flags[:global]
             config.clear_global if flags[:global] || flags[:all]
 
-            warn 'Configuration cleared'
+            warn("Configuration cleared")
           end
         end
       end

data/lib/abt/providers/harvest/commands/current.rb

@@ -6,22 +6,22 @@ module Abt
       module Commands
         class Current < BaseCommand
           def self.usage
-            'abt current harvest[:<project-id>[/<task-id>]]'
+            "abt current harvest[:<project-id>[/<task-id>]]"
           end
 
           def self.description
-            'Get or set project and or task for current git repository'
+            "Get or set project and or task for current git repository"
           end
 
           def perform
-            abort 'Must be run inside a git repository' unless config.local_available?
+            abort("Must be run inside a git repository") unless config.local_available?
 
             require_project!
             ensure_valid_configuration!
 
             if path != config.path
               config.path = path
-              warn 'Configuration updated'
+              warn("Configuration updated")
             end
 
             print_configuration
@@ -38,36 +38,36 @@ module Abt
           end
 
           def ensure_valid_configuration!
-            abort "Invalid project: #{project_id}" if project.nil?
-            abort "Invalid task: #{task_id}" if task_id && task.nil?
+            abort("Invalid project: #{project_id}") if project.nil?
+            abort("Invalid task: #{task_id}") if task_id && task.nil?
           end
 
           def project
-            return @project if instance_variable_defined? :@project
+            return @project if instance_variable_defined?(:@project)
 
             @project = if project_assignment
-                         project_assignment['project'].merge('client' => project_assignment['client'])
+                         project_assignment["project"].merge("client" => project_assignment["client"])
                        end
           end
 
           def task
-            return @task if instance_variable_defined? :@task
+            return @task if instance_variable_defined?(:@task)
 
             @task = if project_assignment
-                      project_assignment['task_assignments'].map { |ta| ta['task'] }.find do |task|
-                        task['id'].to_s == task_id
+                      project_assignment["task_assignments"].map { |ta| ta["task"] }.find do |task|
+                        task["id"].to_s == task_id
                       end
                     end
           end
 
           def project_assignment
             @project_assignment ||= begin
-              project_assignments.find { |pa| pa['project']['id'].to_s == project_id }
+              project_assignments.find { |pa| pa["project"]["id"].to_s == project_id }
             end
           end
 
           def project_assignments
-            @project_assignments ||= api.get_paged('users/me/project_assignments')
+            @project_assignments ||= api.get_paged("users/me/project_assignments")
           end
         end
       end

data/lib/abt/providers/harvest/commands/init.rb

@@ -6,74 +6,46 @@ module Abt
       module Commands
         class Init < BaseCommand
           def self.usage
-            'abt init harvest'
+            "abt init harvest"
           end
 
           def self.description
-            'Pick Harvest project for current git repository'
+            "Pick Harvest project for current git repository"
           end
 
           def perform
-            abort 'Must be run inside a git repository' unless config.local_available?
+            abort("Must be run inside a git repository") unless config.local_available?
 
             projects # Load projects up front to make it obvious that searches are instant
-            project = find_search_result
+            project = cli.prompt.search("Select a project", searchable_projects)["project"]
 
-            config.path = Path.from_ids(project['id'])
+            config.path = Path.from_ids(project["id"])
 
             print_project(project)
           end
 
           private
 
-          def find_search_result
-            warn 'Select a project'
-
-            loop do
-              matches = matches_for_string cli.prompt.text('Enter search')
-              if matches.empty?
-                warn 'No matches'
-                next
-              end
-
-              warn 'Showing the 10 first matches' if matches.size > 10
-              choice = cli.prompt.choice 'Select a project', matches[0...10], true
-              break choice['project'] unless choice.nil?
-            end
-          end
-
-          def matches_for_string(string)
-            search_string = sanitize_string(string)
-
-            searchable_projects.select do |project|
-              sanitize_string(project['name']).include?(search_string)
-            end
-          end
-
-          def sanitize_string(string)
-            string.downcase.gsub(/[^\w]/, '')
-          end
-
           def searchable_projects
             @searchable_projects ||= projects.map do |project|
               {
-                'name' => "#{project['client']['name']} > #{project['name']}",
-                'project' => project
+                "name" => "#{project['client']['name']} > #{project['name']}",
+                "project" => project
               }
             end
           end
 
           def projects
             @projects ||= begin
-              warn 'Fetching projects...'
+              warn("Fetching projects...")
               project_assignments.map do |project_assignment|
-                project_assignment['project'].merge('client' => project_assignment['client'])
+                project_assignment["project"].merge("client" => project_assignment["client"])
               end
             end
           end
 
           def project_assignments
-            @project_assignments ||= api.get_paged('users/me/project_assignments')
+            @project_assignments ||= api.get_paged("users/me/project_assignments")
           end
         end
       end

data/lib/abt/providers/harvest/commands/pick.rb

@@ -6,51 +6,51 @@ module Abt
       module Commands
         class Pick < BaseCommand
           def self.usage
-            'abt pick harvest[:<project-id>]'
+            "abt pick harvest[:<project-id>]"
           end
 
           def self.description
-            'Pick task for current git repository'
+            "Pick task for current git repository"
           end
 
           def self.flags
             [
-              ['-d', '--dry-run', 'Keep existing configuration']
+              ["-d", "--dry-run", "Keep existing configuration"]
             ]
           end
 
           def perform
-            abort 'Must be run inside a git repository' unless config.local_available?
+            abort("Must be run inside a git repository") unless config.local_available?
             require_project!
 
-            warn project['name']
-            task = cli.prompt.choice 'Select a task', tasks
+            warn(project["name"])
+            task = cli.prompt.choice("Select a task", tasks)
 
             print_task(project, task)
 
             return if flags[:"dry-run"]
 
-            config.path = Path.from_ids(project_id, task['id'])
+            config.path = Path.from_ids(project_id, task["id"])
           end
 
           private
 
           def project
-            project_assignment['project']
+            project_assignment["project"]
           end
 
           def tasks
-            @tasks ||= project_assignment['task_assignments'].map { |ta| ta['task'] }
+            @tasks ||= project_assignment["task_assignments"].map { |ta| ta["task"] }
           end
 
           def project_assignment
             @project_assignment ||= begin
-              project_assignments.find { |pa| pa['project']['id'].to_s == project_id }
+              project_assignments.find { |pa| pa["project"]["id"].to_s == project_id }
             end
           end
 
           def project_assignments
-            @project_assignments ||= api.get_paged('users/me/project_assignments')
+            @project_assignments ||= api.get_paged("users/me/project_assignments")
           end
         end
       end

data/lib/abt/providers/harvest/commands/projects.rb

@@ -6,11 +6,11 @@ module Abt
       module Commands
         class Projects < BaseCommand
           def self.usage
-            'abt projects harvest'
+            "abt projects harvest"
           end
 
           def self.description
-            'List all available projects - useful for piping into grep etc.'
+            "List all available projects - useful for piping into grep etc."
           end
 
           def perform
@@ -23,15 +23,15 @@ module Abt
 
           def projects
             @projects ||= begin
-              warn 'Fetching projects...'
+              warn("Fetching projects...")
               project_assignments.map do |project_assignment|
-                project_assignment['project'].merge('client' => project_assignment['client'])
+                project_assignment["project"].merge("client" => project_assignment["client"])
               end
             end
           end
 
           def project_assignments
-            @project_assignments ||= api.get_paged('users/me/project_assignments')
+            @project_assignments ||= api.get_paged("users/me/project_assignments")
           end
         end
       end

data/lib/abt/providers/harvest/commands/share.rb

@@ -6,18 +6,18 @@ module Abt
       module Commands
         class Share < BaseCommand
           def self.usage
-            'abt share harvest[:<project-id>[/<task-id>]]'
+            "abt share harvest[:<project-id>[/<task-id>]]"
           end
 
           def self.description
-            'Print project/task ARI'
+            "Print project/task ARI"
           end
 
           def perform
-            if path != ''
-              cli.print_ari('harvest', path)
+            if path != ""
+              cli.print_ari("harvest", path)
             elsif cli.output.isatty
-              warn 'No configuration for project. Did you initialize Harvest?'
+              warn("No configuration for project. Did you initialize Harvest?")
             end
           end
         end

data/lib/abt/providers/harvest/commands/start.rb

@@ -1,6 +1,6 @@
 # frozen_string_literal: true
 
-require_relative 'track'
+require_relative "track"
 
 module Abt
   module Providers
@@ -8,11 +8,13 @@ module Abt
       module Commands
         class Start < Track
           def self.usage
-            'abt start harvest[:<project-id>/<task-id>] [options]'
+            "abt start harvest[:<project-id>/<task-id>] [options]"
           end
 
           def self.description
-            'Alias for: `abt track harvest`. Meant to used in combination with other ARIs, e.g. `abt start harvest asana`'
+            <<~TXT
+              Alias for: `abt track harvest`. Meant to used in combination with other ARIs, e.g. `abt start harvest asana`
+            TXT
           end
         end
       end

data/lib/abt/providers/harvest/commands/stop.rb

@@ -6,19 +6,19 @@ module Abt
       module Commands
         class Stop < BaseCommand
           def self.usage
-            'abt stop harvest'
+            "abt stop harvest"
           end
 
           def self.description
-            'Stop running harvest tracker'
+            "Stop running harvest tracker"
           end
 
           def perform
-            abort 'No running time entry' if time_entry.nil?
+            abort("No running time entry") if time_entry.nil?
 
             stop_time_entry
 
-            warn 'Harvest time entry stopped'
+            warn("Harvest time entry stopped")
             print_task(project, task)
           end
 
@@ -27,28 +27,28 @@ module Abt
           def stop_time_entry
             api.patch("time_entries/#{time_entry['id']}/stop")
           rescue Abt::HttpError::HttpError => e
-            warn e
-            abort 'Unable to stop time entry'
+            warn(e)
+            abort("Unable to stop time entry")
           end
 
           def project
-            time_entry['project']
+            time_entry["project"]
           end
 
           def task
-            time_entry['task']
+            time_entry["task"]
           end
 
           def time_entry
             @time_entry ||= begin
               api.get_paged(
-                'time_entries',
+                "time_entries",
                 is_running: true,
                 user_id: config.user_id
               ).first
-            rescue Abt::HttpError::HttpError => e # rubocop:disable Layout/RescueEnsureAlignment
-              warn e
-              abort 'Unable to fetch running time entry'
+            rescue Abt::HttpError::HttpError => e
+              warn(e)
+              abort("Unable to fetch running time entry")
             end
           end
         end

data/lib/abt/providers/harvest/commands/tasks.rb

@@ -6,11 +6,11 @@ module Abt
       module Commands
         class Tasks < BaseCommand
           def self.usage
-            'abt tasks harvest'
+            "abt tasks harvest"
           end
 
           def self.description
-            'List available tasks on project - useful for piping into grep etc.'
+            "List available tasks on project - useful for piping into grep etc."
           end
 
           def perform
@@ -24,24 +24,24 @@ module Abt
           private
 
           def project
-            project_assignment['project']
+            project_assignment["project"]
           end
 
           def tasks
             @tasks ||= begin
-              warn 'Fetching tasks...'
-              project_assignment['task_assignments'].map { |ta| ta['task'] }
+              warn("Fetching tasks...")
+              project_assignment["task_assignments"].map { |ta| ta["task"] }
             end
           end
 
           def project_assignment
             @project_assignment ||= begin
-              project_assignments.find { |pa| pa['project']['id'].to_s == project_id }
+              project_assignments.find { |pa| pa["project"]["id"].to_s == project_id }
             end
           end
 
           def project_assignments
-            @project_assignments ||= api.get_paged('users/me/project_assignments')
+            @project_assignments ||= api.get_paged("users/me/project_assignments")
           end
         end
       end