abak-flow 0.3.2 → 1.0.0

data/lib/abak-flow/pull_request.rb

@@ -1,221 +1,61 @@
-# -*- encoding: utf-8 -*-
+# coding: utf-8
+require "ruler"
+
 module Abak::Flow
   class PullRequest
-    extend Forwardable
-
-    attr_reader :repository, :request_params
-    attr_reader :config, :validator
+    include Ruler
 
-    def initialize(config, options)
-      strategy = options.delete(:strategy) || :publish
+    attr_reader :errors, :link
 
-      @repository = Hub::Commands.send(:local_repo)
-      @request_params = OpenStruct.new(options)
-      @config = config
+    def initialize(params, manager)
+      @manager = manager
+      @errors = []
 
-      @validator = Validator.new(strategy, self)
+      @head = params.fetch(:head)
+      @base = params.fetch(:base)
+      @title = params.fetch(:title)
+      @body = params.fetch(:body)
     end
 
-    delegate [:validate!, :valid?] => :validator
-    delegate [:base=, :head=, :title=, :body=] => :request_params
+    def ready?
+      @errors = []
 
-    def default_task
-      @default_task ||= current_branch.split('/').push(nil)[1].to_s
-    end
-
-    def branch_prefix
-      @branch_prefix  ||= current_branch.include?('/') ? current_branch.split('/').first : ''
-    end
+      multi_ruleset do
+        fact(:head_is_incorrect)  { not @head.valid? }
+        fact(:base_is_incorrect)  { not @base.valid? }
+        fact(:title_is_incorrect) { @title.empty? }
+        fact(:body_is_incorrect)  { @body.empty? }
 
-    def current_branch
-      @current_branch ||= repository.current_branch.short_name
-    end
-
-    def from_repo
-      @from_repo ||= begin
-        upstream_project = repository.remote_by_name('upstream').project
-        "#{upstream_project.owner}/#{upstream_project.name}"
-      end
-    end
-    alias_method :upstream_project, :from_repo
-    
-    def origin_project
-      @origin_project ||= begin
-        origin_project = repository.remote_by_name('origin').project
-        "#{origin_project.owner}/#{origin_project.name}"
+        rule([:head_is_incorrect])  { @errors << I18n.t("pull_request.errors.head_is_incorrect") }
+        rule([:base_is_incorrect])  { @errors << I18n.t("pull_request.errors.base_is_incorrect") }
+        rule([:title_is_incorrect]) { @errors << I18n.t("pull_request.errors.title_is_incorrect") }
+        rule([:body_is_incorrect])  { @errors << I18n.t("pull_request.errors.body_is_incorrect") }
       end
-    end
-    
-    def origin_repo
-      @origin_repo ||= repository.main_project.remote.name
-    end
-
-    def base
-      exit unless validator.valid?
-
-      branch = Abak::Flow::PullRequest.branch_by_prefix(branch_prefix)
 
-      request_params.base || "#{repository.remote_by_name('upstream').project.owner}:#{branch}"
+      @errors.empty? ? true : false
     end
 
-    def head
-      exit unless validator.valid?
-
-      request_params.head || "#{repository.repo_owner}:#{current_branch}"
-    end
-
-    def title
-      exit unless validator.valid?
-
-      request_params.title
-    end
-
-    def body
-      exit unless validator.valid?
-
-      request_params.body
-    end
-
-    def self.branch_by_prefix(prefix)
-      {:feature => :develop, :hotfix  => :master}.fetch(prefix.to_sym, '')
-    end
-
-    # TODO Вынести
-    class Validator
-      attr_reader :strategy, :target_object
-      attr_reader :errors, :executed
-
-      def initialize(strategy_name, target_object)
-        @strategy = Abak::Flow::PullRequest.const_get("Strategy#{strategy_name.capitalize}".to_sym)
-        @target_object = target_object
-        @errors = []
-      end
-
-      def valid?
-        return errors.empty? if executed
-
-        validate!
-        errors.empty?
-      end
-
-      protected
-      def validate!
-        @executed = true
-
-        strategy.attributes.each do |attribute|
-          send("validate_#{attribute}")
-        end
-
-        print_errors
-      end
-
-      def print_errors
-        errors.each do |error|
-          say color(error[:message], :error).to_s
-          say color(error[:tip], :info).to_s
-        end
-      end
-
-      def validate_api_user
-        return if target_object.config.api_user?
-
-        @errors << {
-          :message => 'Необходимо указать своего пользователя API github',
-          :tip     => '=> https://github.com/Strech/abak-flow/blob/master/README.md'
-        }
-      end
-
-      def validate_api_token
-        return if target_object.config.api_token?
-
-        @errors << {
-          :message => 'Необходимо указать токен своего пользователя API github',
-          :tip     => '=> https://github.com/Strech/abak-flow/blob/master/README.md'
-        }
-      end
-
-      def validate_origin
-        return unless target_object.repository.remote_by_name('origin').nil?
-
-        @errors << {
-          :message => 'Необходимо настроить репозиторий origin (форк) для текущего пользователя',
-          :tip     => '=> git remote add origin https://Developer@github.com/abak-press/sample.git'
-        }
-      end
-
-      def validate_upstream
-        return unless target_object.repository.remote_by_name('upstream').nil?
-
-        @errors << {
-          :message => 'Необходимо настроить репозиторий upstream (главный) для текущего пользователя',
-          :tip     => '=> git remote add upstream https://Developer@github.com/abak-press/sample.git'
-        }
-      end
-
-      def validate_title
-        return unless target_object.request_params.title.empty?
-
-        @errors << {
-          :message => 'Пожалуйста, укажите что-нибудь для заголовка pull request, например номер вашей задачи вот так:',
-          :tip     => '=> git request publish "PC-001"'
-        }
-      end
-
-      def validate_branch
-        return if [:master, :develop].include?(target_object.current_branch.to_sym)
-
-        @errors << {
-          :message => 'Нельзя делать pull request из меток master или develop, попробуйде переключиться, например так:',
-          :tip     => '=> git checkout master'
-        }
-      end
-
-      def validate_deleted_branch
-        return unless [:master, :develop].include?(target_object.current_branch.to_sym)
-
-        @errors << {
-          :message => 'Извините, но нельзя удалить ветку develop или master',
-          :tip     => '=> git checkout feature/TASK-0001'
-        }
-      end
+    def display_name
+      I18n.t("pull_request.name")
     end
 
-    class Strategy
-      def self.attributes
-        raise NotImplementedError
-      end
-    end
+    def publish
+      begin
+        head_with_repo = [@manager.repository.origin.owner, @head] * ':'
 
-    class StrategyFeature < Strategy
-      def self.attributes
-        StrategyReadycheck.attributes | [:title, :branch]
-      end
-    end
+        response = @manager.github.create_pull_request(
+          @manager.repository.upstream.to_s, @base.to_s, head_with_repo, @title, @body)
 
-    class StrategyPublish < Strategy
-      def self.attributes
-        StrategyReadycheck.attributes | [:title]
-      end
-    end
+        @link = response._links.html.href
 
-    class StrategyUpdate < Strategy
-      def self.attributes
-        StrategyReadycheck.attributes
-      end
-    end
+        true
+      rescue Exception => exception
+        backtrace = exception.backtrace[0...10] * "\n"
 
-    class StrategyDone < Strategy
-      def self.attributes
-        StrategyReadycheck.attributes | [:deleted_branch]
-      end
-    end
+        @errors = ["#{exception.message}\n\n#{backtrace}"]
 
-    class StrategyReadycheck < Strategy
-      def self.attributes
-        [:origin, :upstream, :api_user, :api_token]
+        false
       end
     end
-    
-    class StrategyStatus < StrategyReadycheck; end
-  end
-end
+  end # class PullRequest
+end

data/lib/abak-flow/repository.rb

@@ -0,0 +1,71 @@
+# coding: utf-8
+require "i18n"
+require "ruler"
+require "forwardable"
+
+module Abak::Flow
+  class Repository
+    include Ruler
+    extend Forwardable
+
+    REMOTES = [:origin, :upstream].freeze
+
+    def_delegators :@manager, :git
+
+    attr_reader :errors
+
+    def initialize(manager)
+      @manager = manager
+      @errors  = []
+
+      configure!
+    end
+
+    def ready?
+      @errors = []
+
+      multi_ruleset do
+        fact(:origin_not_setup)   { origin.nil? }
+        fact(:upstream_not_setup) { upstream.nil? }
+
+        rule([:origin_not_setup])   { @errors << I18n.t("repository.errors.origin_not_setup") }
+        rule([:upstream_not_setup]) { @errors << I18n.t("repository.errors.upstream_not_setup") }
+      end
+
+      @errors.empty? ? true : false
+    end
+
+    def display_name
+      I18n.t("repository.name")
+    end
+
+    private
+    def configure!
+      remotes = Hash[fetch_remotes_from_git]
+      REMOTES.each do |name|
+        define_singleton_method(name) { remotes[name] }
+      end
+    end
+
+    def fetch_remotes_from_git
+      git.remotes.
+          select { |remote| REMOTES.include?(remote.name.to_sym) }.
+          map { |remote| create_remote(remote) }.
+          compact
+    end
+
+    def create_remote(remote)
+      matches = /.+.github\.com[\:|\/](?<owner>.+)\/(?<project>.+).git/.match(remote.url)
+
+      if !matches.nil? && matches.captures.length == 2
+        [remote.name.to_sym, Remote.new(matches[:owner], matches[:project], remote)]
+      end
+    end
+
+    class Remote < Struct.new(:owner, :project, :repo)
+      def to_s
+        "#{owner}/#{project}"
+      end
+    end
+  end
+end

data/lib/abak-flow/request.rb

@@ -1,234 +1,124 @@
-# -*- encoding: utf-8 -*-
+# coding: utf-8
+require "commander/import"
+require "ansi/code"
+
 module Abak::Flow
-  program :name, 'Утилита для оформления pull request на github.com'
+  program :name, "Утилита для оформления pull request на github.com"
   program :version, Abak::Flow::VERSION
-  program :description, 'Утилита, заточенная под git-flow но с использованием github.com'
+  program :description, "Утилита, заточенная под git-flow но с использованием github.com"
 
   default_command :help
-  command :publish do |c|
-    c.syntax      = 'git request publish <Заголовок>'
-    c.description = 'Оформить pull request из текущей ветки (feature -> develop, hotfix -> master)'
 
-    # Опции нужны, если человек хочет запушить  `` ветку, с именем отличным от стандарта
-    c.option '--head STRING', String, 'Имя ветки, которую нужно принять в качестве изменений'
-    c.option '--base STRING', String, 'Имя ветки, в которую нужно принять изменения'
+  command :checkup do |c|
+    c.syntax      = "git request checkup"
+    c.description = "Проверить все ли настроено для работы с github и удаленными репозиториями"
 
     c.action do |args, options|
-      jira_browse_url = 'http://jira.dev.apress.ru/browse/'
-
-      config = Abak::Flow::Config.current
-      github_client = Abak::Flow::GithubClient.connect(config)
-      request = Abak::Flow::PullRequest.new(config, :head => options.head, :base => options.base)
-
-      title = args.first.to_s.strip
-      body = 'Я забыл какая это задача :('
-
-      if request.default_task =~ /^\w+\-\d{1,}$/
-        title = request.default_task if title.empty?
-        body = jira_browse_url + request.default_task
-      end
-
-      request.title = title
-      request.body  = body
-
-      exit unless request.valid?
-
-      # Запушим текущую ветку на origin
-      say "=> Обновляю ветку #{request.current_branch} на origin"
-      Hub::Runner.execute('push', 'origin', request.current_branch)
-
-      # Запостим pull request на upstream
-      say '=> Делаю pull request на upstream'
-      begin
-        result = github_client.create_pull_request(request.from_repo, request.base, request.head, request.title, request.body)
-        say color(result._links.html.href, :green).to_s
-      rescue => e
-        say color(e.message, :error).to_s
-        say "\nПроблемы? Попробуйте заглянуть сюда:"
-        say color('=> cписок кодов статуса ответа http://www.w3.org/Protocols/rfc2616/rfc2616-sec10.html', :info).to_s
+      m = Manager.new
+      v = Visitor.new(m.configuration, m.repository, call: :ready?, look_for: :errors)
+
+      if v.ready?
+        say ANSI.green { I18n.t("commands.checkup.success") }
+      else
+        say ANSI.red { I18n.t("commands.checkup.fail") }
+        say ANSI.yellow { v.output }
       end
     end
-  end
-
-  command :update do |c|
-    c.syntax      = 'git request update'
-    c.description = 'Обновить ветку на удаленном (origin) репозитории'
-
-    c.option '--branch STRING', String, 'Имя ветки, которую нужно обновить'
-
-    c.action do |args, options|
-      config = Abak::Flow::Config.current
-      request = Abak::Flow::PullRequest.new(config, :strategy => :update)
-
-      exit unless request.valid?
-
-      # Запушим текущую ветку на origin
-      branch = options.branch || request.current_branch
-      say "=> Обновляю ветку #{branch} на origin"
-      Hub::Runner.execute('push', 'origin', branch)
-    end
-  end
-
-  command :feature do |c|
-    c.syntax      = 'git request feature <Название задачи>'
-    c.description = 'Создать ветку для выполнения задачи. Лучше всего, если название задачи, будет ее номером из jira'
+  end # command :checkup
 
-    c.action do |args, options|
-      config = Abak::Flow::Config.current
-
-      task = args.shift.to_s
+  command :compare do |c|
+    c.syntax      = "git request compare"
+    c.description = "Сравнить свою ветку с веткой upstream репозитория"
 
-      if task.empty?
-        say color('Необходимо указать имя задачи, а лучше всего ее номер из jira', :error).to_s
-        exit
-      end
-
-      unless task =~ /^\w+\-\d{1,}$/
-        say '=> Вы приняли верное решение :)' && exit if agree("Лучше всего завести задачу с именем примерно такого формата PC-001, может попробуем заново? [y/n]:")
-      end
-
-      Hub::Runner.execute('flow', 'feature', 'start', task)
-    end
-  end
-
-  command :hotfix do |c|
-    c.syntax      = 'git request hotfix <Название задачи>'
-    c.description = 'Создать ветку для выполнения bugfix задачи. Лучше всего, если название задачи, будет ее номером из jira'
+    c.option "--base STRING", String, "Имя ветки с которой нужно сравнить"
+    c.option "--head STRING", String, "Имя ветки которую нужно сравнить"
 
     c.action do |args, options|
-      config = Abak::Flow::Config.current
+      # TODO : Вот это дубль, хочется его как-то более красиво
+      m = Manager.new
+      v = Visitor.new(m.configuration, m.repository, call: :ready?, look_for: :errors)
 
-      task = args.shift.to_s
+      unless v.ready?
+        say ANSI.red { I18n.t("commands.compare.fail") }
+        say ANSI.yellow { v.output }
 
-      if task.empty?
-        say color('Необходимо указать имя задачи, а лучше всего ее номер из jira', :error).to_s
-        exit
+        exit 1
       end
 
-      unless task =~ /^\w+\-\d{1,}$/
-        say '=> Вы приняли верное решение :)' && exit if agree("Лучше всего завести задачу с именем примерно такого формата PC-001, может попробуем заново? [y/n]:")
+      current = m.git.current_branch
+      head = Branch.new(options.head || current, m)
+      base = Branch.new(options.base || head.pick_up_base_name, m)
+
+      if head.current?
+        say ANSI.white {
+          I18n.t("commands.compare.updating",
+            branch: ANSI.bold { head },
+            upstream: ANSI.bold { "#{m.repository.origin}" }) }
+
+        head.update
+      else
+        say ANSI.yellow {
+          I18n.t("commands.compare.diverging",
+            branch: ANSI.bold { head }) }
       end
 
-      Hub::Runner.execute('flow', 'hotfix', 'start', task)
+      say ANSI.green { head.compare_link(base) }
     end
-  end
+  end # command :compare
 
-  command :done do |c|
-    c.syntax      = 'git request done'
-    c.description = 'Завершить pull request. По умолчанию удаляются ветки как локальная (local), так и удаленная (origin)'
+  command :publish do |c|
+    c.syntax      = "git request publish"
+    c.description = "Оформить pull request в upstream репозиторий"
 
-    c.option '--branch STRING', String, 'Имя ветки pull request которой нужно закрыть'
-    c.option '--all', 'Удаляет ветку в локальном репозитории и в удалнном (local + origin) (по умолчанию)'
-    c.option '--local', 'Удаляет ветку только в локальном репозитории (local)'
-    c.option '--origin', 'Удаляет ветку в удаленном репозитории (origin)'
+    c.option "--title STRING", String, "Заголовок для вашего pull request"
+    c.option "--body STRING", String, "Текст для вашего pull request"
+    c.option "--base STRING", String, "Имя ветки, в которую нужно принять изменения"
 
     c.action do |args, options|
-      config  = Abak::Flow::Config.current
-      request = Abak::Flow::PullRequest.new(config, :strategy => :done)
-      branch  = options.branch || request.current_branch
+      m = Manager.new
 
-      exit unless request.valid?
+      head = Branch.new(m.git.current_branch, m)
+      base = Branch.new(options.base || head.pick_up_base_name, m)
 
-      type = :all
-      if [options.local, options.origin].compact.count == 1
-        type = options.local ? :local : :origin
-      end
+      title = options.title || head.pick_up_title
+      body = [
+        options.body || (head.mappable? ? nil : I18n.t("commands.publish.nothing")),
+        head.pick_up_body
+      ].compact * "\n\n"
 
-      warning = color('Внимание! Alarm! Danger! Achtung!', :error).to_s +
-                "\nЕсли вы удалите ветку на удаленном (remote) репозитории, а ваш pull request еще не приняли, вы рискуете потерять проделанную работу.\nВы уверены, что хотите продолжить?"
-      if [:all, :origin].include?(type)
-        say '=> Вы приняли верное решение :)' && exit unless agree("#{warning} [y/n]:")
-      end
+      p = PullRequest.new({base: base, head: head, title: title, body: body}, m)
+      v = Visitor.new(m.configuration, m.repository, p, call: :ready?, look_for: :errors)
 
-      # TODO Проверку на наличие ветки на origin
-      if [:all, :origin].include?(type)
-        say "=> Удаляю ветку #{branch} на origin"
-        Hub::Runner.execute('push', request.origin_repo, ':' + branch)
-      end
+      unless v.ready?
+        say ANSI.red { I18n.t("commands.publish.fail") }
+        say ANSI.yellow { v.output }
 
-      if [:all, :local].include?(type)
-        say "=> Удаляю локальную ветку #{branch}"
-        Hub::Runner.execute('checkout', 'develop')
-        Hub::Runner.execute('branch', '-D', branch)
+        exit 1
       end
-    end
-  end
 
-  # TODO Отрефакторить эту какашку
-  command :readycheck do |c|
-    c.syntax      = 'git request readycheck'
-    c.description = 'Проверить все ли настроено для работы с github и удаленным (origin) репозиторием'
+      say ANSI.white {
+        I18n.t("commands.publish.updating",
+          branch: ANSI.bold { head },
+          upstream: ANSI.bold { "#{m.repository.origin}" }) }
 
-    c.action do |args, options|
-      config  = Abak::Flow::Config.current
-      request = Abak::Flow::PullRequest.new(config, :strategy => :readycheck)
-
-      if config.proxy?
-        message = "== В качестве прокси будет установлено значение #{config.proxy} =="
-        say color('=' * message.length, :info).to_s
-        say color(message, :info).to_s
-        say color('=' * message.length + "\n", :info).to_s
-      end
+      head.update
 
-      say color('Хм ... кажется у вас все готово к работе', :debug).to_s if request.valid?
-    end
-  end
-  
-  command :garbage do |c|
-    c.syntax      = 'git request status'
-    c.description = 'Проверить пригодность удаленных (origin) веток и возможность их уничтожения (ветки master, develop игнорируются)'
+      say ANSI.white {
+        I18n.t("commands.publish.requesting",
+          branch: ANSI.bold { "#{m.repository.origin.owner}:#{head}" },
+          upstream: ANSI.bold { "#{m.repository.upstream.owner}:#{base}" }) }
 
-    c.action do |args, options|
-      config  = Abak::Flow::Config.current
-      github_client = Abak::Flow::GithubClient.connect(config)
-      request = Abak::Flow::PullRequest.new(config, :strategy => :status)
-      
-      exit unless request.valid?
-      
-      messages = {unused: ["отсутствует в upstream репозитории", :notice],
-                  differ: ["отличается от origin репозитория", :warning],
-                  missing: ["отсутствует в локальном репозитории", :warning]}
-      
-      say "=> Обновляю данные о репозитории upstream"
-      %w(origin upstream).each { |remote| Hub::Runner.execute('fetch', remote, '-p') }
-      
-      say "=> Загружаю список веток для origin"
-      branches = github_client.branches(request.origin_project).
-                               reject { |branch| %w(master develop).include? branch.name }
-
-      say "=> На origin найдено веток: #{branches.count}\n\n"
-      branches.each_with_index do |branch, index|
-        index += 1
-
-        base = Abak::Flow::PullRequest.branch_by_prefix branch.name.split('/').first
-        
-        upstream_branch = %x(git branch -r --contain #{branch.commit.sha} | grep upstream/#{base} 2> /dev/null).strip
-        local_sha = %x(git show #{branch.name} --format=%H --no-notes 2> /dev/null | head -n 1).strip
-
-        statuses = {
-          unused: upstream_branch.empty?,
-          differ: !local_sha.empty? && local_sha != branch.commit.sha,
-          missing: local_sha.empty?
-        }
-        
-        unless statuses.values.inject &:|
-          say color("#{index}) #{branch.name} → можно удалить", :debug).to_s
-          say "\n"
-          next
-        end
-        
-        diagnoses = statuses.select { |_,bool| bool }.
-                             map { |name,_| messages[name].first }.
-                             map { |msg| "#{' ' * (index.to_s.length + 2)}   ↪ #{msg}" }.
-                             join("\n")
-
-        if statuses.select { |_,bool| bool }.keys == [:missing]
-          say color("#{index}) #{branch.name} → потенциально можно удалить", :warning).to_s
-          say "#{diagnoses}\n\n"
-        else
-          say "#{index}) #{branch.name}\n#{diagnoses}\n\n"
-        end
+      v = Visitor.new(p, call: :publish, look_for: :errors)
+      if v.ready?
+        say ANSI.green {
+          I18n.t("commands.publish.success",
+            link: ANSI.bold { p.link }) }
+      else
+        say ANSI.red { I18n.t("commands.publish.fail") }
+        say ANSI.yellow { v.output }
+
+        exit 3
       end
     end
-  end
-end
+  end # command :publish
+end

data/lib/abak-flow/version.rb

@@ -1,5 +1,5 @@
 module Abak
   module Flow
-    VERSION = '0.3.2'
+    VERSION = '1.0.0'
   end
 end