abak-flow 1.0.10 → 1.1.0

data/lib/abak-flow/locales/ru.yml

@@ -1,44 +1,82 @@
 ru:
-  configuration:
-    name: Корфигурация
-    errors:
-      oauth_user_not_setup: Не установлена опция oauth_user
-      oauth_token_not_setup: Не установлена опция oauth_token
-    recommendations: "Проверьте секцию [abak-flow] в файле ~/.gitcofig"
+  abak:
+    flow:
+      commands:
+        checkup:
+          fail: Вы не готовы!
+          success: Поздравляем, вы готовы чтобы жечь :)
 
-  repository:
-    name: Репозиторий
-    errors:
-      origin_not_setup: Репозиторий с именем 'origin' не найден
-      upstream_not_setup: Репозиторий с именем 'upstream' не найден
-    recommendations: Проверьте файл .git/cofig
+        compare:
+          words:
+            updating: "Обновляю %{branch} → %{upstream}"
+            diverging: "Ветки могут расходиться\nСовет: переключитесь в ветку '%{branch}' и повторите команду"
 
-  pull_request:
-    name: Пулл реквест
-    errors:
-      head_is_incorrect: Что-то не так с 'head' веткой
-      base_is_incorrect: Что-то не так с 'base' веткой
-      title_is_incorrect: Не указан заголовок
-      body_is_incorrect: Не указано содержание
+        configure:
+          fail: Что-то пошло не так!
+          success: Конфигурация сгененирована и сохранена ↓
+          words:
+            email: Почта
+            password: Пароль
+            locale: Язык
+            http_proxy: HTTP прокси
+            sms_otp: SMS пароль
+            configuring: Настраиваю
+          errors:
+            execution_failed: Ошибка при запрос к апи Github
+            empty_response: Почему-то ответ от апи Github был пустотой
 
-  commands:
-    default:
-      fail: Что-то пошло не так!
-    checkup:
-      fail: "Вы не готовы!"
-      success: "Поздравляем, вы готовы чтобы жечь :)"
-    compare:
-      fail: Что-то пошло не так!
-      updating: "Обновление %{branch} → %{upstream}"
-      diverging: "Ветки могут расходиться.\nСовет: переключитесь в ветку '%{branch}' и повторите операцию"
-    publish:
-      fail: Что-то пошло не так!
-      success: "Пулл реквест успешно создан %{link}"
-      requesting: "Создание пулл реквеста %{branch} → %{upstream}"
-      updating: "Обновление %{branch} → %{upstream}"
-      nothing: Мне нечего добавить ...
-    done:
-      fail: Что-то пошло не так!
-      deleting: "Удаление %{branch} из %{upstream}"
-      errors:
-        branch_is_incorrect: "Вы не можете удалить %{branch}"
+        done:
+          words:
+            deleting: Удаляю %{branch} в %{upstream}
+            done: √
+          errors:
+            branch:
+              delete_now_allowed: Вы не можете удалить %{branch}
+              missed_on: Ветка %{branch} не существует на %{upstream}
+
+        publish:
+          fail: Черт побери, только не ошибка!
+          words:
+            updating: Обновляю %{branch} → %{upstream}
+            publicating: Создаю пул реквест %{branch} → %{upstream}
+          errors:
+            publication:
+              failed: Во время публикации произошли ошибки
+
+      configuration:
+        name: Конфигурация
+        fields:
+          login: Логин
+          password: Пароль
+        errors:
+          login:
+            blank: Не может быть пустым
+          password:
+            blank: Не может быть пустым
+
+      repository:
+        name: Репозиторий
+        fields:
+          origin: Origin
+          upstream: Upstream
+        errors:
+          origin:
+            not_set: Удаленный репозиторий 'origin' не объявлен
+          upstream:
+            not_set: Удаленный репозиторий 'upstream' не объявлен
+
+      pullrequest:
+        name: Пул реквест
+        fields:
+          title: Заголовок
+          body: Описание
+          head: Ветка Head
+          base: Ветка Base
+          exception: Исключение
+        errors:
+          head:
+            invalid: Название ветки неверно
+          base:
+            invalid: Название ветки неверно
+          exception:
+            message: "%{backtrace}"

data/lib/abak-flow/manager.rb

@@ -7,23 +7,40 @@ module Abak::Flow
   class Manager
     include Singleton
 
-    def initialize
-      # preload dependencies
-      configuration
-      repository
+    class << self
+      def git
+        instance.git
+      end
+
+      def github
+        instance.github
+      end
+
+      def locale
+        instance.locale
+      end
+
+      def configuration
+        instance.configuration
+      end
+
+      def repository
+        instance.repository
+      end
     end
 
     def configuration
-      @configuration ||= Configuration.new(self)
+      @configuration ||= Configuration.new
     end
 
     def repository
-      @repository ||= Repository.new(self)
+      @repository ||= Repository.new
     end
 
     def github
-      @github ||= Octokit::Client.new(login: configuration.oauth_user,
-        oauth_token: configuration.oauth_token,
+      @github ||= Octokit::Client.new(
+        login: configuration.login,
+        password: configuration.password,
         proxy: configuration.http_proxy)
     end
 
@@ -31,5 +48,8 @@ module Abak::Flow
       @git ||= Git.open(".")
     end
 
+    def locale
+      @locale ||= Locale.new(configuration.locale)
+    end
   end
 end

data/lib/abak-flow/pull_request.rb

@@ -1,15 +1,10 @@
 # coding: utf-8
-require "ruler"
-
 module Abak::Flow
   class PullRequest
-    include Ruler
-
-    attr_reader :errors, :link
+    attr_reader :link
 
-    def initialize(params, manager)
-      @manager = manager
-      @errors = []
+    def initialize(params)
+      @_errors = Hash.new
 
       @head = params.fetch(:head)
       @base = params.fetch(:base)
@@ -17,42 +12,38 @@ module Abak::Flow
       @body = params.fetch(:body)
     end
 
-    def ready?
-      @errors = []
-
-      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)  { @head.tracker_task? ? @body.empty? : false }
+    def valid?
+      @_errors = Hash.new
+      @_errors["head"] = ["invalid"] unless @head.valid?
+      @_errors["base"] = ["invalid"] unless @base.valid?
+      @_errors["title"] = ["blank"] if @title.empty?
 
-        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
-
-      @errors.empty? ? true : false
+      @_errors.empty?
     end
 
-    def display_name
-      I18n.t("pull_request.name")
+    def errors
+      ErrorsPresenter.new(self, @_errors)
     end
 
     def publish
+      @_errors = Hash.new
+
       begin
-        head_with_repo = [@manager.repository.origin.owner, @head] * ':'
+        head_with_repo = [Manager.repository.origin.owner, @head] * ':'
 
-        response = @manager.github.create_pull_request(
-          @manager.repository.upstream.to_s, @base.to_s, head_with_repo, @title, @body)
+        response = Manager.github.create_pull_request(
+          Manager.repository.upstream.to_s, @base.to_s, head_with_repo, @title, @body)
 
-        @link = response._links.html.href
+        @link = response[:html_url]
 
         true
       rescue Exception => exception
         backtrace = exception.backtrace[0...10] * "\n"
 
-        @errors = ["#{exception.message}\n\n#{backtrace}"]
+        @_errors["exception"] = [{
+          field: "message",
+          options: {backtrace: "#{exception.message}\n\n#{backtrace}"}
+        }]
 
         false
       end

data/lib/abak-flow/repository.rb

@@ -1,64 +1,49 @@
 # coding: utf-8
-require "i18n"
-require "ruler"
-require "forwardable"
 
 module Abak::Flow
   class Repository
-    include Ruler
     extend Forwardable
 
-    REMOTES = [:origin, :upstream].freeze
+    REMOTES = %w{origin upstream}.map(&:freeze)
 
-    def_delegators :@manager, :git
+    def initialize
+      @_errors = Hash.new
 
-    attr_reader :errors
-
-    def initialize(manager)
-      @manager = manager
-      @errors  = []
-
-      configure!
+      create_public_instance_methods
     end
 
-    def ready?
-      @errors = []
+    def valid?
+      @_errors = Hash.new
+      @_errors["origin"] = ['not_set'] if origin.nil?
+      @_errors["upstream"] = ['not_set'] if upstream.nil?
 
-      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
+      @_errors.empty?
     end
 
-    def display_name
-      I18n.t("repository.name")
+    def errors
+      ErrorsPresenter.new(self, @_errors)
     end
 
     private
-    def configure!
+
+    def create_public_instance_methods
       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
+      Manager.git.remotes.select { |remote| REMOTES.include?(remote.name) }
+        .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)]
+        [remote.name, Remote.new(matches[:owner], matches[:project], remote)]
       end
     end
 

data/lib/abak-flow/request.rb

@@ -1,7 +1,8 @@
 # coding: utf-8
 require "commander/import"
-require "ansi/code"
 
+# TODO : I18n
+# TODO : Переименовать в CLI
 module Abak::Flow
   program :name, "Утилита для оформления pull request на github.com"
   program :version, Abak::Flow::VERSION
@@ -25,89 +26,26 @@ module Abak::Flow
     cmd.action Commands::Compare, :run
   end # command :compare
 
-  command :publish do |c|
-    c.syntax      = "git request publish"
-    c.description = "Оформить pull request в upstream репозиторий"
+  command :configure do |cmd|
+    cmd.syntax      = "git request setup"
+    cmd.description = "Настроить приложение abak-flow для работы с github"
+    cmd.action Commands::Configure, :run
+  end # command :configure
 
-    c.option "--title STRING", String, "Заголовок для вашего pull request"
-    c.option "--body STRING", String, "Текст для вашего pull request"
-    c.option "--base STRING", String, "Имя ветки, в которую нужно принять изменения"
+  command :publish do |cmd|
+    cmd.syntax      = "git request publish"
+    cmd.description = "Оформить pull request в upstream репозиторий"
 
-    c.action do |args, options|
-      m = Manager.instance
+    cmd.option "--title STRING", String, "Заголовок для вашего pull request"
+    cmd.option "--body STRING", String, "Текст для вашего pull request"
+    cmd.option "--base STRING", String, "Имя ветки, в которую нужно принять изменения"
 
-      head = Branch.new(m.git.current_branch)
-      base = Branch.new(options.base || head.extract_base_name)
-
-      title = options.title || head.extract_title
-      body  = options.body || head.extract_body
-
-      p = PullRequest.new({base: base, head: head, title: title, body: body}, m)
-
-      v = Visitor.new(m.configuration, m.repository, p, call: :ready?, inspect: :errors)
-      v.exit_on_fail(:publish, 1)
-
-      say ANSI.white {
-        I18n.t("commands.publish.updating",
-          branch: ANSI.bold { head },
-          upstream: ANSI.bold { "#{m.repository.origin}" }) }
-
-      head.update
-
-      say ANSI.white {
-        I18n.t("commands.publish.requesting",
-          branch: ANSI.bold { "#{m.repository.origin.owner}:#{head}" },
-          upstream: ANSI.bold { "#{m.repository.upstream.owner}:#{base}" }) }
-
-      v = Visitor.new(p, call: :publish, inspect: :errors)
-      v.exit_on_fail(:publish, 2)
-
-      say ANSI.green { I18n.t("commands.publish.success", link: p.link) }
-    end
+    cmd.action Commands::Publish, :run
   end # command :publish
 
-  command :done do |c|
-    c.syntax      = "git request done"
-    c.description = "Удалить ветки (local и origin) в которых велась работа"
-
-    c.action do |args, options|
-      m = Manager.instance
-      v = Visitor.new(m.configuration, m.repository, call: :ready?, inspect: :errors)
-      v.exit_on_fail(:done, 1)
-
-      branch = Branch.new(m.git.current_branch)
-
-      if branch.develop? || branch.master?
-        say ANSI.red {
-          I18n.t("commands.done.errors.branch_is_incorrect",
-            branch: ANSI.bold { branch }) }
-        exit 2
-      end
-
-      say ANSI.white {
-        I18n.t("commands.done.deleting",
-          branch: ANSI.bold { branch },
-          upstream: ANSI.bold { "#{m.repository.origin}" }) }
-
-      # FIXME : Исправить молчаливую ситуацию
-      #         Возможно стоит предупредить о ее отсутствии
-      branch.delete_on_remote rescue nil
-
-      say ANSI.white {
-        I18n.t("commands.done.deleting",
-          branch: ANSI.bold { branch },
-          upstream: ANSI.bold { "working tree" }) }
-
-      # TODO : Добавить проверку, что ветка,
-      #        в которую надо попасть (master/develop)
-      #        существует
-
-      # TODO : Быть может стоит вынести это в настройки
-      #        и позволить выбирать, куда отправлять
-      #        при удалении ветки, а по умолчанию использовать master
-      m.git.checkout(
-        branch.extract_base_name(if_undef: Branch::DEVELOPMENT))
-      branch.delete_on_local
-    end
+  command :done do |cmd|
+    cmd.syntax      = "git request done"
+    cmd.description = "Удалить ветки (local и origin) в которых велась работа"
+    cmd.action Commands::Done, :run
   end # command :done
 end