ghi 0.2.0 → 1.2.0

data/lib/ghi/commands/list.rb

@@ -0,0 +1,204 @@
+require 'date'
+
+module GHI
+  module Commands
+    class List < Command
+      attr_accessor :web
+      attr_accessor :reverse
+      attr_accessor :quiet
+      attr_accessor :exclude_pull_requests
+      attr_accessor :pull_requests_only
+
+      def options
+        OptionParser.new do |opts|
+          opts.banner = 'usage: ghi list [options]'
+          opts.separator ''
+          opts.on '-g', '--global', 'all of your issues on GitHub' do
+            assigns[:filter] = 'all'
+            @repo = nil
+          end
+          opts.on(
+            '-s', '--state <in>', %w(open closed),
+            {'o'=>'open', 'c'=>'closed'}, "'open' or 'closed'"
+          ) do |state|
+            assigns[:state] = state
+          end
+          opts.on(
+            '-L', '--label <labelname>...', Array, 'by label(s)'
+          ) do |labels|
+            (assigns[:labels] ||= []).concat labels
+          end
+          opts.on(
+            '-N', '--not-label <labelname>...', Array, 'exclude with label(s)'
+          ) do |labels|
+            (assigns[:exclude_labels] ||= []).concat labels
+          end
+          opts.on(
+            '--no-labels', 'do not print labels'
+          ) do
+            assigns[:dont_print_labels] = true
+          end
+          opts.on(
+            '-S', '--sort <by>', %w(created updated comments),
+            {'c'=>'created','u'=>'updated','m'=>'comments'},
+            "'created', 'updated', or 'comments'"
+          ) do |sort|
+            assigns[:sort] = sort
+          end
+          opts.on '--reverse', 'reverse (ascending) sort order' do
+            self.reverse = !reverse
+          end
+          opts.on('-p', '--pulls','list only pull requests') { self.pull_requests_only = true }
+          opts.on('-P', '--no-pulls','exclude pull requests') { self.exclude_pull_requests = true }
+          opts.on(
+            '--since <date>', 'issues more recent than',
+            "e.g., '2011-04-30'"
+          ) do |date|
+            begin
+              assigns[:since] = DateTime.parse date # TODO: Better parsing.
+            rescue ArgumentError => e
+              raise OptionParser::InvalidArgument, e.message
+            end
+          end
+          opts.on('-v', '--verbose') { self.verbose = true }
+          opts.on('-w', '--web') { self.web = true }
+          opts.separator ''
+          opts.separator 'Global options'
+          opts.on(
+            '-f', '--filter <by>',
+            filters = %w[all assigned created mentioned subscribed],
+            Hash[filters.map { |f| [f[0, 1], f] }],
+            filters.map { |f| "'#{f}'" }.join(', ')
+          ) do |filter|
+            assigns[:filter] = filter
+          end
+          opts.on '--mine', 'assigned to you' do
+            assigns[:filter] = 'assigned'
+            assigns[:assignee] = Authorization.username
+          end
+          opts.separator ''
+          opts.separator 'Project options'
+          opts.on(
+            '-M', '--[no-]milestone [<n>]', Integer,
+            'with (specified) milestone'
+          ) do |milestone|
+            assigns[:milestone] = any_or_none_or milestone
+          end
+          opts.on(
+            '-u', '--[no-]assignee [<user>]', 'assigned to specified user'
+          ) do |assignee|
+            assignee = assignee.sub /^@/, '' if assignee
+            assigns[:assignee] = any_or_none_or assignee
+          end
+          opts.on '--mine', 'assigned to you' do
+            assigns[:filter] = 'assigned'
+            assigns[:assignee] = Authorization.username
+          end
+          opts.on(
+            '--creator [<user>]', 'created by you or specified user'
+          ) do |creator|
+            creator = creator.sub /^@/, '' if creator
+            assigns[:creator] = creator || Authorization.username
+          end
+          opts.on(
+            '-U', '--mentioned [<user>]', 'mentioning you or specified user'
+          ) do |mentioned|
+            assigns[:mentioned] = mentioned || Authorization.username
+          end
+          opts.on(
+            '-O', '--org <organization>', 'in repos within an organization you belong to'
+          ) do |org|
+	    assigns[:org] = org
+            @repo = nil
+          end
+          opts.separator ''
+        end
+      end
+
+      def execute
+        if index = args.index { |arg| /^@/ === arg }
+          assigns[:assignee] = args.delete_at(index)[1..-1]
+        end
+
+        begin
+          options.parse! args
+          @repo ||= ARGV[0] if ARGV.one?
+        rescue OptionParser::InvalidOption => e
+          fallback.parse! e.args
+          retry
+        end
+        assigns[:labels] = assigns[:labels].join ',' if assigns[:labels]
+        if assigns[:exclude_labels]
+          assigns[:exclude_labels] = assigns[:exclude_labels].join ','
+        end
+        if reverse
+          assigns[:sort] ||= 'created'
+          assigns[:direction] = 'asc'
+        end
+        if web
+          Web.new(repo || 'dashboard').open 'issues', assigns
+        else
+          assigns[:per_page] = 100
+          unless quiet
+            print header = format_issues_header
+            print "\n" unless paginate?
+          end
+          res = throb(
+            0, format_state(assigns[:state], quiet ? CURSOR[:up][1] : '#')
+          ) { api.get uri, assigns }
+          print "\r#{CURSOR[:up][1]}" if header && paginate?
+          page header do
+            issues = res.body
+
+            if exclude_pull_requests || pull_requests_only
+              prs, issues = issues.partition { |i| i['pull_request'].values.any? }
+              issues = prs if pull_requests_only
+            end
+            if assigns[:exclude_labels]
+              issues = issues.reject  do |i|
+                i["labels"].any? do |label|
+                  assigns[:exclude_labels].include? label["name"]
+                end
+              end
+            end
+            if verbose
+              puts issues.map { |i| format_issue i }
+            else
+              puts format_issues(issues, repo.nil?)
+            end
+            break unless res.next_page
+            res = throb { api.get res.next_page }
+          end
+        end
+      rescue Client::Error => e
+        if e.response.code == '422'
+          e.errors.any? { |err|
+            err['code'] == 'missing' && err['field'] == 'milestone'
+          } and abort 'No such milestone.'
+        end
+
+        raise
+      end
+
+      private
+
+      def uri
+        url = ''
+        if repo
+          url = "/repos/#{repo}"
+        end
+	if assigns[:org]
+          url = "/orgs/#{assigns[:org]}"
+        end
+        return url << '/issues'
+      end
+
+      def fallback
+        OptionParser.new do |opts|
+          opts.on('-c', '--closed') { assigns[:state] = 'closed' }
+          opts.on('-q', '--quiet')  { self.quiet = true }
+        end
+      end
+    end
+  end
+end

data/lib/ghi/commands/milestone.rb

@@ -0,0 +1,206 @@
+require 'date'
+
+module GHI
+  module Commands
+    class Milestone < Command
+      attr_accessor :edit
+      attr_accessor :reverse
+      attr_accessor :web
+
+      #--
+      # FIXME: Opt for better interface, e.g.,
+      #
+      #   ghi milestone [-v | --verbose] [--[no-]closed]
+      #   ghi milestone add <name> <description>
+      #   ghi milestone rm <milestoneno>
+      #++
+      def options
+        OptionParser.new do |opts|
+          opts.banner = <<EOF
+usage: ghi milestone [<modification options>] [<milestoneno>]
+   or: ghi milestone -D <milestoneno>
+   or: ghi milestone -l [-c] [-v]
+EOF
+          opts.separator ''
+          opts.on '-l', '--list', 'list milestones' do
+            self.action = 'index'
+          end
+          opts.on '-c', '--[no-]closed', 'show closed milestones' do |closed|
+            assigns[:state] = closed ? 'closed' : 'open'
+          end
+          opts.on(
+            '-S', '--sort <on>', %w(due_date completeness),
+            {'d'=>'due_date', 'due'=>'due_date', 'c'=>'completeness'},
+            "'due_date' or 'completeness'"
+          ) do |sort|
+            assigns[:sort] = sort
+          end
+          opts.on '--reverse', 'reverse (ascending) sort order' do
+            self.reverse = !reverse
+          end
+          opts.on '-v', '--verbose', 'list milestones verbosely' do
+            self.verbose = true
+          end
+          opts.on('-w', '--web') { self.web = true }
+          opts.separator ''
+          opts.separator 'Milestone modification options'
+          opts.on(
+            '-m', '--message [<text>]', 'change milestone description'
+          ) do |text|
+            self.action = 'create'
+            self.edit = true
+            next unless text
+            assigns[:title], assigns[:description] = text.split(/\n+/, 2)
+          end
+          # FIXME: We already describe --[no-]closed; describe this, too?
+          opts.on(
+            '-s', '--state <in>', %w(open closed),
+            {'o'=>'open', 'c'=>'closed'}, "'open' or 'closed'"
+          ) do |state|
+            self.action = 'create'
+            assigns[:state] = state
+          end
+          opts.on(
+            '--due <on>', 'when milestone should be complete',
+            "e.g., '2012-04-30'"
+          ) do |date|
+            self.action = 'create'
+            begin
+              # TODO: Better parsing.
+              assigns[:due_on] = DateTime.parse(date).strftime
+            rescue ArgumentError => e
+              raise OptionParser::InvalidArgument, e.message
+            end
+          end
+          opts.on '-D', '--delete', 'delete milestone' do
+            self.action = 'destroy'
+          end
+          opts.separator ''
+        end
+      end
+
+      def execute
+        self.action = 'index'
+        require_repo
+        extract_milestone
+
+        begin
+          options.parse! args
+        rescue OptionParser::AmbiguousOption => e
+          fallback.parse! e.args
+        end
+
+        milestone and case action
+          when 'create' then self.action = 'update'
+          when 'index'  then self.action = 'show'
+        end
+
+        if reverse
+          assigns[:sort] ||= 'created'
+          assigns[:direction] = 'asc'
+        end
+
+        case action
+        when 'index'
+          if web
+            Web.new(repo).open 'milestones', assigns
+          else
+            assigns[:per_page] = 100
+            state = assigns[:state] || 'open'
+            print format_state state, "# #{repo} #{state} milestones"
+            print "\n" unless paginate?
+            res = throb(0, format_state(state, '#')) { api.get uri, assigns }
+            page do
+              milestones = res.body
+              if verbose
+                puts milestones.map { |m| format_milestone m }
+              else
+                puts format_milestones(milestones)
+              end
+              break unless res.next_page
+              res = throb { api.get res.next_page }
+            end
+          end
+        when 'show'
+          if web
+            List.execute %W(-w -M #{milestone} -- #{repo})
+          else
+            m = throb { api.get uri }.body
+            page do
+              puts format_milestone(m)
+              puts 'Issues:'
+              args.unshift(*%W(-q -M #{milestone} -- #{repo}))
+              args.unshift '-v' if verbose
+              List.execute args
+              break
+            end
+          end
+        when 'create'
+          if web
+            Web.new(repo).open 'issues/milestones/new'
+          else
+            if assigns[:title].nil?
+              e = Editor.new 'GHI_MILESTONE'
+              message = e.gets format_milestone_editor
+              e.unlink 'Empty milestone.' if message.nil? || message.empty?
+              assigns[:title], assigns[:description] = message.split(/\n+/, 2)
+            end
+            m = throb { api.post uri, assigns }.body
+            puts 'Milestone #%d created.' % m['number']
+            e.unlink if e
+          end
+        when 'update'
+          if web
+            Web.new(repo).open "issues/milestones/#{milestone}/edit"
+          else
+            if edit || assigns.empty?
+              m = throb { api.get "/repos/#{repo}/milestones/#{milestone}" }.body
+              e = Editor.new "GHI_MILESTONE_#{milestone}"
+              message = e.gets format_milestone_editor(m)
+              e.unlink 'Empty milestone.' if message.nil? || message.empty?
+              assigns[:title], assigns[:description] = message.split(/\n+/, 2)
+            end
+            if assigns[:title] && m
+              t_match = assigns[:title].strip == m['title'].strip
+              if assigns[:description]
+                b_match = assigns[:description].strip == m['description'].strip
+              end
+              if t_match && b_match
+                e.unlink if e
+                abort 'No change.' if assigns.dup.delete_if { |k, v|
+                  [:title, :description].include? k
+                }
+              end
+            end
+            m = throb { api.patch uri, assigns }.body
+            puts format_milestone(m)
+            puts 'Updated.'
+            e.unlink if e
+          end
+        when 'destroy'
+          require_milestone
+          throb { api.delete uri }
+          puts 'Milestone deleted.'
+        end
+      end
+
+      private
+
+      def uri
+        if milestone
+          "/repos/#{repo}/milestones/#{milestone}"
+        else
+          "/repos/#{repo}/milestones"
+        end
+      end
+
+      def fallback
+        OptionParser.new do |opts|
+          opts.on '-d' do
+            self.action = 'destroy'
+          end
+        end
+      end
+    end
+  end
+end

data/lib/ghi/commands/open.rb

@@ -0,0 +1,102 @@
+module GHI
+  module Commands
+    class Open < Command
+      attr_accessor :editor
+      attr_accessor :web
+
+      def options
+        OptionParser.new do |opts|
+          opts.banner = <<EOF
+usage: ghi open [options]
+   or: ghi reopen [options] <issueno>
+EOF
+          opts.separator ''
+          opts.on '-l', '--list', 'list open tickets' do
+            self.action = 'index'
+          end
+          opts.on('-w', '--web') { self.web = true }
+          opts.separator ''
+          opts.separator 'Issue modification options'
+          opts.on(
+              '-m', '--message [<text>]', 'describe issue',
+              "use line breaks to separate title from description"
+          ) do |text|
+            if text
+              assigns[:title], assigns[:body] = text.split(/\n+/, 2)
+            else
+              self.editor = true
+            end
+          end
+          opts.on(
+            '-u', '--[no-]assign [<user>]', 'assign to specified user'
+          ) do |assignee|
+            assigns[:assignee] = assignee
+          end
+          opts.on '--claim', 'assign to yourself' do
+            assigns[:assignee] = Authorization.username
+          end
+          opts.on(
+            '-M', '--milestone <n>', 'associate with milestone'
+          ) do |milestone|
+            assigns[:milestone] = milestone
+          end
+          opts.on(
+            '-L', '--label <labelname>...', Array, 'associate with label(s)'
+          ) do |labels|
+            (assigns[:labels] ||= []).concat labels
+          end
+          opts.separator ''
+        end
+      end
+
+      def execute
+        require_repo
+        self.action = 'create'
+
+        options.parse! args
+
+        if GHI.config('ghi.infer-issue') != 'false' && extract_issue
+          Edit.execute args.push('-so', issue, '--', repo)
+          exit
+        end
+
+        case action
+        when 'index'
+          if assigns.key? :assignee
+            args.unshift assigns[:assignee] if assigns[:assignee]
+            args.unshift '-u'
+          end
+          args.unshift '-w' if web
+          List.execute args.push('--', repo)
+        when 'create'
+          if web
+            Web.new(repo).open 'issues/new'
+          else
+            unless args.empty?
+              assigns[:title], assigns[:body] = args.join(' '), assigns[:title]
+            end
+            assigns[:title] = args.join ' ' unless args.empty?
+            if assigns[:title].nil? || editor
+              e = Editor.new 'GHI_ISSUE'
+              message = e.gets format_editor(assigns)
+              e.unlink "There's no issue?" if message.nil? || message.empty?
+              assigns[:title], assigns[:body] = message.split(/\n+/, 2)
+            end
+            i = throb { api.post "/repos/#{repo}/issues", assigns }.body
+            e.unlink if e
+            puts format_issue(i)
+            puts "Opened on #{repo}."
+          end
+        end
+      rescue Client::Error => e
+        raise unless error = e.errors.first
+        abort "%s %s %s %s." % [
+          error['resource'],
+          error['field'],
+          [*error['value']].join(', '),
+          error['code']
+        ]
+      end
+    end
+  end
+end

data/lib/ghi/commands/show.rb

@@ -0,0 +1,65 @@
+module GHI
+  module Commands
+    class Show < Command
+      attr_accessor :patch, :web
+
+      def options
+        OptionParser.new do |opts|
+          opts.banner = 'usage: ghi show <issueno>'
+          opts.separator ''
+          opts.on('-p', '--patch') { self.patch = true }
+          opts.on('-w', '--web') { self.web = true }
+        end
+      end
+
+      def execute
+        require_issue
+        require_repo
+        options.parse! args
+        patch_path = "pull/#{issue}.patch" if patch # URI also in API...
+        if web
+          Web.new(repo).open patch_path || "issues/#{issue}"
+        else
+          if patch_path
+            i = throb { Web.new(repo).curl patch_path }
+            unless i.start_with? 'From'
+              warn 'Patch not found'
+              abort
+            end
+            page do
+              no_color { puts i }
+              break
+            end
+          else
+            i = throb { api.get "/repos/#{repo}/issues/#{issue}" }.body
+            determine_merge_status(i) if pull_request?(i)
+            page do
+              puts format_issue(i)
+              n = i['comments']
+              if n > 0
+                puts "#{n} comment#{'s' unless n == 1}:\n\n"
+                Comment.execute %W(-l #{issue} -- #{repo})
+              end
+              break
+            end
+          end
+        end
+      end
+
+      private
+
+      def pull_request?(issue)
+        issue['pull_request']['html_url']
+      end
+
+      def determine_merge_status(pr)
+        pr['merged'] = true if pr['state'] == 'closed' && merged?
+      end
+
+      def merged?
+        # API returns with a Not Found error when the PR is not merged
+        api.get "/repos/#{repo}/pulls/#{issue}/merge" rescue false
+      end
+    end
+  end
+end

data/lib/ghi/commands/status.rb

@@ -0,0 +1,32 @@
+module GHI
+  module Commands
+    class Status < Command
+
+      def options
+        OptionParser.new do |opts|
+          opts.banner = 'usage: ghi status'
+        end
+      end
+
+      def execute
+        begin
+          options.parse! args
+          @repo ||= ARGV[0] if ARGV.one?
+        rescue OptionParser::InvalidOption => e
+          fallback.parse! e.args
+          retry
+        end
+        
+        require_repo
+        res = throb { api.get "/repos/#{repo}" }.body
+
+        if res['has_issues']
+          puts "Issues are enabled for this repo"
+        else
+          puts "Issues are not enabled for this repo"
+        end
+
+      end
+    end
+  end
+end

data/lib/ghi/commands/version.rb

@@ -0,0 +1,16 @@
+module GHI
+  module Commands
+    module Version
+      MAJOR   = 1
+      MINOR   = 2
+      PATCH   = 0
+      PRE     = nil
+
+      VERSION = [MAJOR, MINOR, PATCH, PRE].compact.join '.'
+
+      def self.execute args
+        puts "ghi version #{VERSION}"
+      end
+    end
+  end
+end

data/lib/ghi/commands.rb

@@ -0,0 +1,23 @@
+module GHI
+  module Commands
+    autoload :Command,   'ghi/commands/command'
+
+    autoload :List,      'ghi/commands/list'
+    autoload :Open,      'ghi/commands/open'
+    autoload :Assign,    'ghi/commands/assign'
+    autoload :Close,     'ghi/commands/close'
+    autoload :Comment,   'ghi/commands/comment'
+    autoload :Config,    'ghi/commands/config'
+		autoload :Disable,   'ghi/commands/disable'
+    autoload :Edit,      'ghi/commands/edit'
+		autoload :Enable,    'ghi/commands/enable'
+    autoload :Help,      'ghi/commands/help'
+    autoload :Label,     'ghi/commands/label'
+    autoload :Milestone, 'ghi/commands/milestone'
+    autoload :Reopen,    'ghi/commands/reopen'
+    autoload :Show,      'ghi/commands/show'
+		autoload :Status,    'ghi/commands/status'
+    autoload :Unassign,  'ghi/commands/unassign'
+    autoload :Version,   'ghi/commands/version'
+  end
+end

data/lib/ghi/editor.rb

@@ -0,0 +1,48 @@
+require 'tmpdir'
+
+module GHI
+  class Editor
+    attr_reader :filename
+    def initialize filename
+      @filename = filename
+    end
+
+    def gets prefill
+      File.open path, 'a+' do |f|
+        f << prefill if File.zero? path
+        f.rewind
+        system "#{editor} #{f.path}"
+        return File.read(f.path).gsub(/(?:^#.*$\n?)+\s*\z/, '').strip
+      end
+    end
+
+    def unlink message = nil
+      File.delete path
+      abort message if message
+    end
+
+    private
+
+    def editor
+      editor   = GHI.config 'ghi.editor'
+      editor ||= GHI.config 'core.editor'
+      editor ||= ENV['VISUAL']
+      editor ||= ENV['EDITOR']
+      editor ||= 'vi'
+    end
+
+    def path
+      File.join dir, filename
+    end
+
+    def dir
+      @dir ||= git_dir || Dir.tmpdir
+    end
+
+    def git_dir
+      return unless Commands::Command.detected_repo
+      dir = `git rev-parse --git-dir 2>/dev/null`.chomp
+      dir unless dir.empty?
+    end
+  end
+end