abt-cli 0.0.15 → 0.0.20

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: f176b71200cc6fcc13ab56507e4d48234785e2307ab350afca7be13193ef2a5e
-  data.tar.gz: '093172885ed64949153cd73457748f446e012887f78d092f2483164f005fbfd0'
+  metadata.gz: 6348f086170cb21625ec22595423cebbca76b0455f6dc2ad552ef072019b3929
+  data.tar.gz: e53c442f505ba9141e62d2b4c7883ec010c966b4d1deef0d71651b94ea11db93
 SHA512:
-  metadata.gz: 6350132f05617d7e7222f0b7e07255091119ad0edac59277795c891668a6de4fb3ea9e376cdc93ab2672e130b5222d6ff310d80da8979dfaf060ff0e2479e447
-  data.tar.gz: 14eea7d7cb849f39d1aef4643df214541178d4f8702cecc962ffcff099b0c9143be68b8562aac0fdbc75b2bb2424872bd34661136104ac233df2097031d49394
+  metadata.gz: 977ff0fbc908433afdf996cdd354bd33ea9a315cfebcbc456f42df731309f0502801516995f0ded85a58f8efcfeb0fe9830dd31c51efcb05f4acb58ddb3e3833
+  data.tar.gz: 45d25bbae2c077af24c304132703ca8ebc7a0968c04aafc598ca48a66cd3f303fe5cc108e193e24bd8f5cbb09e37d831eeba3518c360bafb1112f4ae443138cd

data/bin/abt

@@ -5,7 +5,7 @@ require_relative '../lib/abt.rb'
 
 begin
   Abt::Cli.new.perform
-rescue Abt::Cli::AbortError => e
+rescue Abt::Cli::Abort => e
   abort e.message
 rescue Interrupt
   abort 'Aborted'

data/lib/abt.rb

@@ -5,6 +5,7 @@ require 'faraday'
 require 'oj'
 require 'open3'
 require 'stringio'
+require 'optparse'
 
 Dir.glob("#{File.dirname(File.absolute_path(__FILE__))}/abt/*.rb").sort.each do |file|
   require file
@@ -13,12 +14,12 @@ end
 module Abt
   module Providers; end
 
-  def self.provider_names
+  def self.schemes
     Providers.constants.sort.map { |constant_name| Helpers.const_to_command(constant_name) }
   end
 
-  def self.provider_module(name)
-    const_name = Helpers.command_to_const(name)
+  def self.scheme_provider(scheme)
+    const_name = Helpers.command_to_const(scheme)
     Providers.const_get(const_name) if Providers.const_defined?(const_name)
   end
 end

data/lib/abt/ari.rb

@@ -0,0 +1,20 @@
+# frozen_string_literal: true
+
+module Abt
+  class Ari
+    attr_reader :scheme, :path, :flags
+
+    def initialize(scheme:, path: nil, flags: [])
+      @scheme = scheme
+      @path = path
+      @flags = flags
+    end
+
+    def to_s
+      str = scheme
+      str += ":#{path}" if path
+
+      [str, *flags].join(' ')
+    end
+  end
+end

data/lib/abt/ari_list.rb

@@ -0,0 +1,13 @@
+# frozen_string_literal: true
+
+module Abt
+  class AriList < Array
+    def to_s
+      map(&:to_s).join(' -- ')
+    end
+
+    def -(other)
+      AriList.new(to_a - other)
+    end
+  end
+end

data/lib/abt/base_command.rb

@@ -0,0 +1,63 @@
+# frozen_string_literal: true
+
+module Abt
+  class BaseCommand
+    extend Forwardable
+
+    def self.usage
+      raise NotImplementedError, 'Command classes must implement .usage'
+    end
+
+    def self.description
+      raise NotImplementedError, 'Command classes must implement .description'
+    end
+
+    def self.flags
+      []
+    end
+
+    attr_reader :ari, :cli, :flags
+
+    def_delegators(:@cli, :warn, :puts, :print, :abort, :exit_with_message)
+
+    def initialize(ari:, cli:)
+      @cli = cli
+      @ari = ari
+      @flags = parse_flags(ari.flags)
+    end
+
+    def perform
+      raise NotImplementedError, 'Command classes must implement #perform'
+    end
+
+    private
+
+    def parse_flags(flags)
+      result = {}
+
+      flag_parser.parse!(flags.dup, into: result)
+
+      exit_with_message(flag_parser.help) if result[:help]
+
+      result
+    rescue OptionParser::InvalidOption => e
+      abort e.message
+    end
+
+    def flag_parser
+      @flag_parser ||= OptionParser.new do |opts|
+        opts.banner = <<~TXT
+          #{self.class.description}
+
+          Usage: #{self.class.usage}
+        TXT
+
+        opts.on('-h', '--help')
+
+        self.class.flags.each do |(*flag)|
+          opts.on(*flag)
+        end
+      end
+    end
+  end
+end

data/lib/abt/cli.rb

@@ -6,31 +6,30 @@ end
 
 module Abt
   class Cli
-    class AbortError < StandardError; end
+    class Abort < StandardError; end
+    class Exit < StandardError; end
 
-    attr_reader :command, :args, :input, :output, :err_output, :prompt
+    attr_reader :command, :aris, :input, :output, :err_output, :prompt
 
     def initialize(argv: ARGV, input: STDIN, output: STDOUT, err_output: STDERR)
-      (@command, *@args) = argv
-
+      (@command, *remaining_args) = argv
       @input = input
       @output = output
       @err_output = err_output
       @prompt = Abt::Cli::Prompt.new(output: err_output)
-
-      @args += args_from_input unless input.isatty # Add piped arguments
+      @aris = ArgumentsParser.new(sanitized_piped_args + remaining_args).parse
     end
 
     def perform
       return if handle_global_commands!
 
-      abort('No provider arguments') if args.empty?
+      abort('No ARIs') if aris.empty?
 
-      process_providers
+      process_aris
     end
 
-    def print_provider_command(provider, arg_str, description = nil)
-      command = "#{provider}:#{arg_str}"
+    def print_ari(scheme, path, description = nil)
+      command = "#{scheme}:#{path}"
       command += " # #{description}" unless description.nil?
       output.puts command
     end
@@ -48,77 +47,97 @@ module Abt
     end
 
     def abort(message)
-      raise AbortError, message
+      raise Abort, message
+    end
+
+    def exit_with_message(message)
+      raise Exit, message
     end
 
     private
 
-    def handle_global_commands! # rubocop:disable Metrics/MethodLength
+    def handle_global_commands!
       case command
       when nil
         warn("No command specified\n\n")
-        puts(Abt::Docs::Cli.content)
-        true
-      when '--help', '-h', 'help', 'commands'
-        puts(Abt::Docs::Cli.content)
-        true
-      when 'help-md'
-        puts(Abt::Docs::Markdown.content)
+        puts(Abt::Docs::Cli.help)
         true
       when '--version', '-v', 'version'
         puts(Abt::VERSION)
         true
+      when '--help', '-h', 'help'
+        puts(Abt::Docs::Cli.help)
+        true
+      when 'commands'
+        puts(Abt::Docs::Cli.commands)
+        true
+      when 'examples'
+        puts(Abt::Docs::Cli.examples)
+        true
+      when 'readme'
+        puts(Abt::Docs::Markdown.readme)
+        true
       else
         false
       end
     end
 
-    def args_from_input
-      input_string = input.read.strip
+    def sanitized_piped_args
+      return [] if input.isatty
 
-      abort 'No input from pipe' if input_string.nil? || input_string.empty?
+      @sanitized_piped_args ||= begin
+        input_string = input.read.strip
 
-      # Exclude comment part of piped input lines
-      lines_without_comments = input_string.lines.map do |line|
-        line.split(' # ').first
-      end
+        abort 'No input from pipe' if input_string.nil? || input_string.empty?
+
+        # Exclude comment part of piped input lines
+        lines_without_comments = input_string.lines.map do |line|
+          line.split(' # ').first
+        end
 
-      # Allow multiple provider arguments on a single piped input line
-      joined_lines = lines_without_comments.join(' ').strip
-      joined_lines.split(/\s+/)
+        # Allow multiple ARIs on a single piped input line
+        # TODO: Force the user to pick a single ARI
+        joined_lines = lines_without_comments.join(' ').strip
+        joined_lines.split(/\s+/)
+      end
     end
 
-    def process_providers
-      used_providers = []
-      args.each do |provider_args|
-        (provider, arg_str) = provider_args.split(':')
+    def process_aris
+      used_schemes = []
 
-        if used_providers.include?(provider)
-          warn "Dropping command for already used provider: #{provider_args}"
+      aris.each do |ari|
+        if used_schemes.include?(ari.scheme)
+          warn "Dropping command for already used scheme: #{ari}"
           next
         end
 
-        used_providers << provider if process_provider_command(provider, command, arg_str)
-      end
+        command_class = get_command_class(ari.scheme)
+        next if command_class.nil?
 
-      abort 'No matching providers found for command' if used_providers.empty? && output.isatty
-    end
+        print_command(command, ari) if output.isatty
+        begin
+          command_class.new(ari: ari, cli: self).perform
+        rescue Exit => e
+          puts e.message
+        end
 
-    def process_provider_command(provider_name, command_name, arg_str)
-      provider = Abt.provider_module(provider_name)
-      return false if provider.nil?
+        used_schemes << ari.scheme
+      end
 
-      command = provider.command_class(command_name)
-      return false if command.nil?
+      return unless used_schemes.empty? && output.isatty
+
+      abort 'No providers found for command and ARI(s)'
+    end
 
-      print_command(command_name, provider_name, arg_str) if output.isatty
+    def get_command_class(scheme)
+      provider = Abt.scheme_provider(scheme)
+      return nil if provider.nil?
 
-      command.new(arg_str: arg_str, cli: self).call
-      true
+      provider.command_class(command)
     end
 
-    def print_command(name, provider, arg_str)
-      warn "===== #{name} #{provider}#{arg_str.nil? ? '' : ":#{arg_str}"} =====".upcase
+    def print_command(name, ari)
+      warn "===== #{name.upcase} #{ari} ====="
     end
   end
 end

data/lib/abt/cli/arguments_parser.rb

@@ -0,0 +1,48 @@
+# frozen_string_literal: true
+
+module Abt
+  class Cli
+    class ArgumentsParser
+      attr_reader :arguments
+
+      def initialize(arguments)
+        @arguments = arguments
+      end
+
+      def parse
+        result = AriList.new
+        rest = arguments.dup
+
+        until rest.empty?
+          (scheme, path) = rest.shift.split(':')
+          flags = take_flags(rest)
+
+          result << Ari.new(scheme: scheme, path: path, flags: flags)
+        end
+
+        result
+      end
+
+      private
+
+      def take_flags(rest)
+        flags = []
+
+        if flag?(rest.first)
+          flags << rest.shift until rest.empty? || delimiter?(rest.first)
+          rest.shift if delimiter?(rest.first)
+        end
+
+        flags
+      end
+
+      def flag?(part)
+        part && part[0] == '-'
+      end
+
+      def delimiter?(part)
+        part == '--'
+      end
+    end
+  end
+end

data/lib/abt/cli/prompt.rb

@@ -34,7 +34,7 @@ module Abt
         output.puts "#{text}:"
 
         if options.length.zero?
-          raise AbortError, 'No available options' unless nil_option
+          raise Abort, 'No available options' unless nil_option
 
           output.puts 'No available options'
           return nil
@@ -69,10 +69,11 @@ module Abt
       end
 
       def read_option_number(options_length, nil_option)
-        output.print '('
-        output.print options_length > 1 ? "1-#{options_length}" : '1'
-        output.print nil_option_string(nil_option)
-        output.print '): '
+        str = '('
+        str += options_length > 1 ? "1-#{options_length}" : '1'
+        str += nil_option_string(nil_option)
+        str += '): '
+        output.print str
 
         input = read_user_input
 
@@ -114,7 +115,7 @@ module Abt
         @tty_path ||= begin
           candidates = ['/dev/tty', 'CON:'] # Unix: '/dev/tty', Windows: 'CON:'
           selected = candidates.find { |candidate| File.exist?(candidate) }
-          raise AbortError, 'Unable to prompt for user input' if selected.nil?
+          raise Abort, 'Unable to prompt for user input' if selected.nil?
 
           selected
         end

data/lib/abt/docs.rb

@@ -7,52 +7,59 @@ end
 module Abt
   module Docs
     class << self
-      def examples # rubocop:disable Metrics/MethodLength
+      def basic_examples
         {
           'Getting started:' => {
-            'abt init asana harvest' => 'Setup asana and harvest project git repo in working dir',
-            'abt pick harvest' => 'Pick harvest tasks, for most projects this will stay the same',
-            'abt pick asana | abt start harvest' => 'Pick asana task and start working',
+            'abt init asana harvest' => 'Setup asana and harvest project for local git repo',
+            'abt pick harvest' => 'Pick harvest task. This will likely stay the same throughout the project',
+            'abt pick asana | abt start harvest' => 'Pick asana task and start tracking time',
             'abt stop harvest' => 'Stop time tracker',
-            'abt start asana harvest' => 'Continue working, e.g. after a break',
+            'abt start asana harvest' => 'Continue working, e.g., after a break',
             'abt finalize asana' => 'Finalize the selected asana task'
+          }
+        }
+      end
+
+      def extended_examples
+        {
+          'Tracking meetings (without switching current task setting):' => {
+            'abt pick asana -d | abt track harvest' => 'Track on asana meeting task',
+            'abt pick harvest -d | abt track harvest -c "Name of meeting"' => 'Track on separate harvest-task'
           },
-          'Tracking meetings (without changing the config):' => {
-            'abt tasks asana | grep -i standup | abt track harvest' => 'Track on asana meeting task without changing any configuration',
-            'abt tasks harvest | grep -i comment | abt track harvest' => 'Track on harvest "Comment"-task (will prompt for a comment)'
-          },
-          'Command output can be piped, e.g.:' => {
+          'Many commands output ARIs that can be piped into other commands:' => {
             'abt tasks asana | grep -i <name of task>' => nil,
             'abt tasks asana | grep -i <name of task> | abt start' => nil
           },
-          'Sharing configuration:' => {
-            'abt share asana harvest | tr "\n" " "' => 'Print current configuration',
-            'abt share asana harvest | tr "\n" " " | pbcopy' => 'Copy configuration (mac only)',
-            'abt start <shared configuration>' => 'Start a shared configuration'
+          'Sharing ARIs:' => {
+            'abt share asana harvest | tr "\n" " "' => 'Print current asana and harvest ARIs on a single line',
+            'abt share asana harvest | tr "\n" " " | pbcopy' => 'Copy ARIs to clipboard (mac only)',
+            'abt start <ARIs from coworker>' => 'Work on a task your coworker shared with you',
+            'abt current <ARIs from coworker> | abt start' => 'Set task as current, then start it'
+          },
+          'Flags:' => {
+            'abt start harvest -c "comment"' => 'Add command flags after ARIs',
+            'abt start harvest -c "comment" -- asana' => 'Use -- to end a list of flags, so that it can be followed by another ARI',
+            'abt pick harvest | abt start -c "comment"' => 'Flags placed directly after a command applies to the piped in ARI'
           }
         }
       end
 
       def providers
-        provider_definitions
+        @providers ||= Abt.schemes.sort.each_with_object({}) do |scheme, definition|
+          definition[scheme] = command_definitions(scheme)
+        end
       end
 
       private
 
-      def provider_definitions
-        Abt.provider_names.sort.each_with_object({}) do |name, definition|
-          provider_module = Abt.provider_module(name)
-
-          definition[name] = command_definitions(provider_module)
-        end
-      end
-
-      def command_definitions(provider_module)
-        provider_module.command_names.each_with_object({}) do |name, definition|
-          command_class = provider_module.command_class(name)
+      def command_definitions(scheme)
+        provider = Abt.scheme_provider(scheme)
+        provider.command_names.each_with_object({}) do |name, definition|
+          command_class = provider.command_class(name)
+          full_name = "abt #{name} #{scheme}"
 
-          if command_class.respond_to?(:command) && command_class.respond_to?(:description)
-            definition[command_class.command] = command_class.description
+          if command_class.respond_to?(:usage) && command_class.respond_to?(:description)
+            definition[full_name] = [command_class.usage, command_class.description]
           end
         end
       end