octodown 1.8.1 → 1.8.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 873f8e9bccb9cea287afa902146a4932ca71e83a
-  data.tar.gz: 5d88a4c9b7a26f981d10671a76a7a5bfc5271959
+  metadata.gz: 89c51bf66ccf645f30d89b02527b0d7d4f976d69
+  data.tar.gz: 6bf5907da675618e307c7fbb6613ea93f8f66803
 SHA512:
-  metadata.gz: 358cc97c065769a35c69c608ab41268d7eb2d4fa23fd84b92702cd3f727e51a8e486249609bc7d6da1f1fb91bd59ff4c63ce4a00c7555269b19b3ebc914781ae
-  data.tar.gz: 720403b3d42ce1b2c1b83f908aebab01dca2e3d8a8c6783b165951a8acfd3495c8cf76823dfd1c55f2de5baa521b3136d85a27df7f2773ca41ac1854fd5f1520
+  metadata.gz: 661e7500c5ab971eed08b57c2e7d4550944a850769722eaf0b3aedd8fedde103ba1c003c7c633c813c501182a2aa041744d61231350d88d4bfe974a6ad766b04
+  data.tar.gz: 01a6a0e86ea8626a9b2676d4f06947db7e5a31b0991d36e2fe562c7b9d9773bd028e51c0efa9b7a995322cc1d777cae98e95defac2fba0097975b4e0b98459c4

data/bin/octodown

@@ -71,16 +71,12 @@ OptionParser.new do |opts|
   end
 end.parse!
 
-require 'tty-prompt'
-
-TUI = TTY::Prompt.new(enable_color: true)
-
 options[:file] = if ARGF.file == STDIN && options[:stdin]
                    STDIN
                  elsif ARGF.file != STDIN
                    ARGF.file
                  else
-                   FileChooser.new(prompt: TUI).choose_file
+                   Octodown::FileChooser.new(logger: options[:logger]).call
                  end
 
 Octodown.call options

data/lib/octodown/support/file_chooser.rb

@@ -1,26 +1,11 @@
-class FileChooser
-  attr_reader :prompt
+require 'English'
+require 'tty-prompt'
 
-  def initialize(prompt:)
-    @prompt = prompt
-  end
-
-  def choose_file
-    choices = all_markdown_files
-    choice = prompt.select('Which markdown file would you like to edit?', choices)
-    File.open(choice, 'r')
-  end
-
-  def abort_no_files_found!
-    prompt.error 'We could not find any markdown files in this folder.'
-    puts
-    prompt.error 'Try passing the file explicitly such as, i.e:'
-    prompt.error '    $ octodown README.md'
-    exit 1
-  end
+module Octodown
+  class FileChooser
+    TUI = ::TTY::Prompt.new(enable_color: true)
 
-  def all_markdown_files
-    extensions = %w[markdown
+    EXTENSIONS = %w[markdown
                     mdown
                     mkdn
                     md
@@ -29,12 +14,77 @@ class FileChooser
                     mdtxt
                     mdtext
                     text
-                    Rmd]
+                    Rmd].freeze
+
+    class MarkdownFileList
+      class Git
+        def call
+          ext_args = EXTENSIONS.map { |ext| "**/*.#{ext} *.#{ext}" }.join(' ')
+          `git ls-files --cached --others -z #{ext_args}`.split("\x0").uniq
+        end
+
+        def runnable?
+          `git rev-parse --is-inside-work-tree`
+          $CHILD_STATUS.success?
+        rescue StandardError
+          false
+        end
+      end
+
+      class Glob
+        def call
+          Dir.glob "*.{#{EXTENSIONS.join(',')}}"
+        end
+
+        def runnable?
+          true
+        end
+      end
+
+      attr_reader :logger
+
+      def initialize(logger)
+        @logger = logger
+      end
+
+      def call
+        logger.debug("File choose strategy: #{winning_strategy.class.name}")
+        winning_strategy.call
+      end
+
+      def winning_strategy
+        strats = [Git.new, Glob.new]
+        @winning_strategy ||= strats.find(&:runnable?)
+      end
+    end
+
+    attr_reader :prompt, :logger
+
+    def initialize(logger:)
+      @logger = logger
+      @prompt = TUI
+    end
+
+    def call
+      choices = all_markdown_files
+      choice = prompt.select('Which file would you like to edit?', choices)
+      File.open(choice, 'r')
+    end
+
+    def abort_no_files_found!
+      prompt.error 'We could not find any markdown files in this folder.'
+      puts
+      prompt.error 'Try passing the file explicitly such as, i.e:'
+      prompt.error '    $ octodown README.md'
+      exit 1
+    end
 
-    choices = Dir.glob "**/*.{#{extensions.join(',')}}"
+    def all_markdown_files
+      choices = MarkdownFileList.new(logger).call
 
-    abort_no_files_found! if choices.empty?
+      abort_no_files_found! if choices.empty?
 
-    choices.sort_by! { |c| c.split(File::SEPARATOR).length }
+      choices.sort_by! { |c| c.split(File::SEPARATOR).length }
+    end
   end
 end

data/lib/octodown/version.rb

@@ -1,3 +1,3 @@
 module Octodown
-  VERSION = '1.8.1'.freeze
+  VERSION = '1.8.2'.freeze
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: octodown
 version: !ruby/object:Gem::Version
-  version: 1.8.1
+  version: 1.8.2
 platform: ruby
 authors:
 - Ian Ker-Seymer