abt-cli 0.0.17 → 0.0.22

data/lib/abt/providers/devops/path.rb

@@ -0,0 +1,51 @@
+# frozen_string_literal: true
+
+module Abt
+  module Providers
+    module Devops
+      class Path < String
+        ORGANIZATION_NAME_REGEX = %r{(?<organization_name>[^/ ]+)}.freeze
+        PROJECT_NAME_REGEX = %r{(?<project_name>[^/ ]+)}.freeze
+        BOARD_ID_REGEX = /(?<board_id>[a-z0-9\-]+)/.freeze
+        WORK_ITEM_ID_REGEX = /(?<work_item_id>\d+)/.freeze
+
+        PATH_REGEX =
+          %r{^(#{ORGANIZATION_NAME_REGEX}/#{PROJECT_NAME_REGEX}/#{BOARD_ID_REGEX})?(/#{WORK_ITEM_ID_REGEX})?}.freeze
+
+        def self.from_ids(organization_id = nil, project_name = nil, board_id = nil, work_item_id = nil)
+          return new unless organization_id && project_name && board_id
+
+          new([organization_id, project_name, board_id, *work_item_id].join("/"))
+        end
+
+        def initialize(path = "")
+          raise Abt::Cli::Abort, "Invalid path: #{path}" unless PATH_REGEX.match?(path)
+
+          super
+        end
+
+        def organization_name
+          match[:organization_name]
+        end
+
+        def project_name
+          match[:project_name]
+        end
+
+        def board_id
+          match[:board_id]
+        end
+
+        def work_item_id
+          match[:work_item_id]
+        end
+
+        private
+
+        def match
+          @match ||= PATH_REGEX.match(self)
+        end
+      end
+    end
+  end
+end

data/lib/abt/providers/git/commands/branch.rb

@@ -4,33 +4,33 @@ module Abt
   module Providers
     module Git
       module Commands
-        class Branch < Abt::Cli::BaseCommand
+        class Branch < Abt::BaseCommand
           def self.usage
-            'abt branch git <scheme>[:<path>]'
+            "abt branch git <scheme>[:<path>]"
           end
 
           def self.description
-            'Switch branch. Uses a compatible scheme to generate the branch-name: E.g. `abt branch git asana`'
+            "Switch branch. Uses a compatible scheme to generate the branch-name: E.g. `abt branch git asana`"
           end
 
           def perform
-            create_and_switch unless switch
-            cli.warn "Switched to #{branch_name}"
+            switch || create_and_switch
+            warn("Switched to #{branch_name}")
           end
 
           private
 
           def switch
             success = false
-            Open3.popen3("git switch #{branch_name}") do |_i, _o, _error_output, thread|
+            Open3.popen3("git switch #{branch_name}") do |_i, _o, _e, thread|
               success = thread.value.success?
             end
             success
           end
 
           def create_and_switch
-            cli.warn "No such branch: #{branch_name}"
-            cli.abort('Aborting') unless cli.prompt.boolean 'Create branch?'
+            warn("No such branch: #{branch_name}")
+            abort("Aborting") unless cli.prompt.boolean("Create branch?")
 
             Open3.popen3("git switch -c #{branch_name}") do |_i, _o, _e, thread|
               thread.value
@@ -39,31 +39,35 @@ module Abt
 
           def branch_name # rubocop:disable Metrics/MethodLength
             @branch_name ||= begin
-              if branch_names_from_scheme_arguments.empty?
-                cli.abort [
-                  'None of the specified scheme arguments responded to `branch-name`.',
-                  'Did you add compatible scheme? e.g.:',
-                  '   abt branch git asana',
-                  '   abt branch git devops'
-                ].join("\n")
+              if branch_names_from_aris.empty?
+                abort([
+                  "None of the specified ARIs responded to `branch-name`.",
+                  "Did you add compatible scheme? e.g.:",
+                  "   abt branch git asana",
+                  "   abt branch git devops"
+                ].join("\n"))
               end
 
-              if branch_names_from_scheme_arguments.length > 1
-                cli.abort [
-                  'Got branch names from multiple scheme arguments, only one is supported',
-                  'Branch names were:',
-                  *branch_names_from_scheme_arguments.map { |name| "   #{name}" }
-                ].join("\n")
+              if branch_names_from_aris.length > 1
+                abort([
+                  "Got branch names from multiple ARIs, only one is supported",
+                  "Branch names were:",
+                  *branch_names_from_aris.map { |name| "   #{name}" }
+                ].join("\n"))
               end
 
-              branch_names_from_scheme_arguments.first
+              branch_names_from_aris.first
             end
           end
 
-          def branch_names_from_scheme_arguments
-            input = StringIO.new(cli.scheme_arguments.to_s)
+          def branch_names_from_aris
+            other_aris = cli.aris - [ari]
+
+            abort("You must provide an additional ARI that responds to: branch-name. E.g., asana") if other_aris.empty?
+
+            input = StringIO.new(cli.aris.to_s)
             output = StringIO.new
-            Abt::Cli.new(argv: ['branch-name'], output: output, input: input).perform
+            Abt::Cli.new(argv: ["branch-name"], output: output, input: input).perform
 
             output.string.lines.map(&:strip).compact
           end

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

@@ -4,8 +4,8 @@ module Abt
   module Providers
     module Harvest
       class Api
-        API_ENDPOINT = 'https://api.harvestapp.com/v2'
-        VERBS = %i[get post patch].freeze
+        API_ENDPOINT = "https://api.harvestapp.com/v2"
+        VERBS = [:get, :post, :patch].freeze
 
         attr_reader :access_token, :account_id
 
@@ -21,7 +21,7 @@ module Abt
         end
 
         def get_paged(path, query = {})
-          result_key = path.split('?').first.split('/').last
+          result_key = path.split("?").first.split("/").last
 
           page = 1
           records = []
@@ -29,7 +29,7 @@ module Abt
           loop do
             result = get(path, query.merge(page: page))
             records += result[result_key]
-            break if result['total_pages'] == page
+            break if result["total_pages"] == page
 
             page += 1
           end
@@ -44,16 +44,16 @@ module Abt
             Oj.load(response.body)
           else
             error_class = Abt::HttpError.error_class_for_status(response.status)
-            encoded_response_body = response.body.force_encoding('utf-8')
+            encoded_response_body = response.body.force_encoding("utf-8")
             raise error_class, "Code: #{response.status}, body: #{encoded_response_body}"
           end
         end
 
         def connection
           @connection ||= Faraday.new(API_ENDPOINT) do |connection|
-            connection.headers['Authorization'] = "Bearer #{access_token}"
-            connection.headers['Harvest-Account-Id'] = account_id
-            connection.headers['Content-Type'] = 'application/json'
+            connection.headers["Authorization"] = "Bearer #{access_token}"
+            connection.headers["Harvest-Account-Id"] = account_id
+            connection.headers["Content-Type"] = "application/json"
           end
         end
       end

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

@@ -3,70 +3,50 @@
 module Abt
   module Providers
     module Harvest
-      class BaseCommand < Abt::Cli::BaseCommand
-        attr_reader :path, :flags, :project_id, :task_id, :cli, :config
+      class BaseCommand < Abt::BaseCommand
+        extend Forwardable
 
-        def initialize(path:, cli:, **)
+        attr_reader :config, :path
+
+        def_delegators(:@path, :project_id, :task_id)
+
+        def initialize(ari:, cli:)
           super
 
           @config = Configuration.new(cli: cli)
-
-          if path.nil?
-            use_current_path
-          else
-            use_path(path)
-          end
+          @path = ari.path ? Path.new(ari.path) : config.path
         end
 
         private
 
         def require_project!
-          cli.abort 'No current/specified project. Did you initialize Harvest?' if project_id.nil?
+          return if project_id
+
+          abort("No current/specified project. Did you initialize Harvest?")
         end
 
         def require_task!
-          if project_id.nil?
-            cli.abort 'No current/specified project. Did you initialize Harvest and pick a task?'
-          end
-          cli.abort 'No current/specified task. Did you pick a Harvest task?' if task_id.nil?
-        end
+          abort("No current/specified project. Did you initialize Harvest and pick a task?") unless project_id
 
-        def same_args_as_config?
-          project_id == config.project_id && task_id == config.task_id
+          abort("No current/specified task. Did you pick a Harvest task?") if task_id.nil?
         end
 
         def print_project(project)
-          cli.print_scheme_argument(
-            'harvest',
-            project['id'],
+          cli.print_ari(
+            "harvest",
+            project["id"],
             "#{project['client']['name']} > #{project['name']}"
           )
         end
 
         def print_task(project, task)
-          cli.print_scheme_argument(
-            'harvest',
+          cli.print_ari(
+            "harvest",
             "#{project['id']}/#{task['id']}",
             "#{project['name']} > #{task['name']}"
           )
         end
 
-        def use_current_path
-          @project_id = config.project_id
-          @task_id = config.task_id
-        end
-
-        def use_path(path)
-          args = path.to_s.split('/')
-          @project_id = args[0].to_s
-          @project_id = nil if project_id.empty?
-
-          return if project_id.nil?
-
-          @task_id = args[1].to_s
-          @task_id = nil if @task_id.empty?
-        end
-
         def api
           @api ||= Abt::Providers::Harvest::Api.new(access_token: config.access_token,
                                                     account_id: config.account_id)

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

@@ -6,27 +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")
           end
         end
       end

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

@@ -6,75 +6,68 @@ 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?
+
             require_project!
+            ensure_valid_configuration!
 
-            if same_args_as_config? || !config.local_available?
-              show_current_configuration
-            else
-              cli.warn 'Updating configuration'
-              update_configuration
+            if path != config.path
+              config.path = path
+              warn("Configuration updated")
             end
-          end
-
-          private
 
-          def show_current_configuration
-            if task_id.nil?
-              print_project(project)
-            else
-              print_task(project, task)
-            end
+            print_configuration
           end
 
-          def update_configuration
-            ensure_project_is_valid!
-            config.project_id = project_id
+          private
 
+          def print_configuration
             if task_id.nil?
               print_project(project)
-              config.task_id = nil
             else
-              ensure_task_is_valid!
-              config.task_id = task_id
-
               print_task(project, task)
             end
           end
 
-          def ensure_project_is_valid!
-            cli.abort "Invalid project: #{project_id}" if project.nil?
-          end
-
-          def ensure_task_is_valid!
-            cli.abort "Invalid task: #{task_id}" if task.nil?
+          def ensure_valid_configuration!
+            abort("Invalid project: #{project_id}") if project.nil?
+            abort("Invalid task: #{task_id}") if task_id && task.nil?
           end
 
           def project
-            @project ||= project_assignment['project'].merge('client' => project_assignment['client'])
+            return @project if instance_variable_defined?(:@project)
+
+            @project = if project_assignment
+                         project_assignment["project"].merge("client" => project_assignment["client"])
+                       end
           end
 
           def task
-            @task ||= project_assignment['task_assignments'].map { |ta| ta['task'] }.find do |task|
-              task['id'].to_s == task_id
-            end
+            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
+                      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,75 +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
-            cli.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.project_id = project['id']
-            config.task_id = nil
+            config.path = Path.from_ids(project["id"])
 
             print_project(project)
           end
 
           private
 
-          def find_search_result
-            cli.warn 'Select a project'
-
-            loop do
-              matches = matches_for_string cli.prompt.text('Enter search')
-              if matches.empty?
-                warn 'No matches'
-                next
-              end
-
-              cli.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
-              cli.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