completely 0.7.2 → 0.7.3

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 3e2d4ef0dc7d503f77387c199d75989c8aca7408f73421086219bfab91823dd0
-  data.tar.gz: 4c5650a43490f9d35417a808c532391c9a0e7cf52d47b36ef5ffe56ab0d28ed2
+  metadata.gz: 1d6353c59bdf0d9ea55a2618a06f40b14d844ad7d40827d50f3bb38949b5263f
+  data.tar.gz: bbdcddef491b55bc6554fdcdf804b48bfddddbe744360bff99a19e9e105f2b2b
 SHA512:
-  metadata.gz: 655e9090c78451ffc8ba5f8b4f309e92756c02b769b4f60d9df2520f5ddc572b6eb7e761aeb8d0a2e1f3113bd1bcf05be186c75a95d833fc9938ad6bf706057c
-  data.tar.gz: 81b519ae5db4f3eed6475b1a8b601c8bb873ed6cf0a67c2862b893e878342e031c60b3c58b600c2bc80b080f9c63a0a6ead68d37ef5b42924d5e687ac514feb1
+  metadata.gz: 5fab1170310c441fded2bdac3770edbaab6ce20100be7b005971371361513926b5492016e4abacf0bceb528824c1f80b30d1874d4bd6d8bb566e4f3ebaebd23e
+  data.tar.gz: 01cdc71f95b09bc73492c2d65fd70e272505e1d5145d5aabe69a58cdab1a924316f9919951dbbf64243a24c0c435d60ca4e2f559d843716c82e27352f5f4bf1c

data/lib/completely/commands/base.rb

@@ -35,7 +35,13 @@ module Completely
       end
 
       def completions
-        @completions ||= Completions.load(config_path, function_name: args['--function'])
+        @completions ||= if config_path == '-'
+          raise Error, 'Nothing is piped on stdin' if $stdin.tty?
+
+          Completions.read $stdin, function_name: args['--function']
+        else
+          Completions.load config_path, function_name: args['--function']
+        end
       end
 
       def config_path
@@ -43,7 +49,11 @@ module Completely
       end
 
       def output_path
-        @output_path ||= args['OUTPUT_PATH'] || ENV['COMPLETELY_OUTPUT_PATH'] || "#{config_basename}.bash"
+        @output_path ||= args['OUTPUT_PATH'] || ENV['COMPLETELY_OUTPUT_PATH'] || stdout || "#{config_basename}.bash"
+      end
+
+      def stdout
+        @stdout ||= config_path == '-' ? '-' : nil
       end
 
       def config_basename

data/lib/completely/commands/generate.rb

@@ -3,19 +3,31 @@ require 'completely/commands/base'
 module Completely
   module Commands
     class Generate < Base
-      help 'Generate the bash completion script to a file'
+      help 'Generate the bash completion script to file or stdout'
 
       usage 'completely generate [CONFIG_PATH OUTPUT_PATH --function NAME --wrap NAME]'
+      usage 'completely generate [CONFIG_PATH --install PROGRAM --function NAME]'
       usage 'completely generate (-h|--help)'
 
       option_function
       option '-w --wrap NAME', 'Wrap the completion script inside a function that echos the ' \
         'script. This is useful if you wish to embed it directly in your script.'
 
-      param_config_path
+      option '-i --install PROGRAM', 'Install the generated script as completions for PROGRAM.'
+
+      param 'CONFIG_PATH', <<~USAGE
+        Path to the YAML configuration file [default: completely.yaml].
+        Use '-' to read from stdin.
+
+        Can also be set by an environment variable.
+      USAGE
+
       param 'OUTPUT_PATH', <<~USAGE
         Path to the output bash script.
-        When not provided, the name of the input file will be used with a .bash extension.
+        Use '-' for stdout.
+
+        When not provided, the name of the input file will be used with a .bash extension, unless the input is stdin - in this case the default will be to output to stdout.
+
         Can also be set by an environment variable.
       USAGE
 
@@ -26,13 +38,35 @@ module Completely
       def run
         wrap = args['--wrap']
         output = wrap ? wrapper_function(wrap) : script
-        File.write output_path, output
-        say "Saved m`#{output_path}`"
-        syntax_warning unless completions.valid?
+
+        if args['--install']
+          install output
+        elsif output_path == '-'
+          show output
+        else
+          save output
+        end
       end
 
     private
 
+      def install(content)
+        installer = Installer.from_string program: args['--install'], string: content
+        success = installer.install force: true
+        raise InstallError, "Failed running command:\nnb`#{installer.install_command_string}`" unless success
+
+        say "Saved m`#{installer.target_path}`"
+        say 'You may need to restart your session to test it'
+      end
+
+      def show(content) = puts content
+      
+      def save(content)
+        File.write output_path, content
+        say "Saved m`#{output_path}`"
+        syntax_warning unless completions.valid?
+      end
+
       def wrapper_function(wrapper_name)
         completions.wrapper_function wrapper_name
       end

data/lib/completely/commands/install.rb

@@ -17,7 +17,10 @@ module Completely
       option '-d --dry', 'Show the installation command but do not run it'
 
       param 'PROGRAM', 'Name of the program the completions are for.'
-      param 'SCRIPT_PATH', 'Path to the source bash script [default: completely.bash].'
+      param 'SCRIPT_PATH', <<~USAGE
+        Path to the source bash script [default: completely.bash].
+        Use '-' to provide the script via stdin.
+      USAGE
 
       def run
         if args['--dry']
@@ -32,15 +35,19 @@ module Completely
         say 'You may need to restart your session to test it'
       end
 
-    private
-
       def installer
-        Installer.new program: args['PROGRAM'], script_path: script_path
+        @installer ||= if stdin?
+          Installer.from_io program:
+        else
+          Installer.new program:, script_path: input_script_path
+        end
       end
 
-      def script_path
-        args['SCRIPT_PATH'] || 'completely.bash'
-      end
+    private
+
+      def program = args['PROGRAM']
+      def stdin? = input_script_path == '-'
+      def input_script_path = args['SCRIPT_PATH'] || 'completely.bash'
     end
   end
 end

data/lib/completely/commands/preview.rb

@@ -3,7 +3,7 @@ require 'completely/commands/base'
 module Completely
   module Commands
     class Preview < Base
-      help 'Generate the bash completion script to STDOUT'
+      help 'Generate the bash completion script to stdout'
 
       usage 'completely preview [CONFIG_PATH --function NAME]'
       usage 'completely preview (-h|--help)'

data/lib/completely/completions.rb

@@ -6,9 +6,12 @@ module Completely
     attr_reader :config
 
     class << self
-      def load(config_path, function_name: nil)
-        config = Config.load config_path
-        new config, function_name: function_name
+      def load(path, function_name: nil)
+        new Config.load(path), function_name: function_name
+      end
+
+      def read(io, function_name: nil)
+        new Config.read(io), function_name: function_name
       end
     end
 
@@ -26,7 +29,7 @@ module Completely
     end
 
     def valid?
-      pattern_prefixes.uniq.count == 1
+      pattern_prefixes.uniq.one?
     end
 
     def script

data/lib/completely/config.rb

@@ -3,17 +3,14 @@ module Completely
     attr_reader :config, :options
 
     class << self
-      def load(config_path)
-        begin
-          config = YAML.load_file config_path, aliases: true
-        rescue ArgumentError
-          # :nocov:
-          config = YAML.load_file config_path
-          # :nocov:
-        end
-
-        new config
+      def parse(str)
+        new YAML.load(str, aliases: true)
+      rescue Psych::Exception => e
+        raise ParseError, "Invalid YAML: #{e.message}"
       end
+
+      def load(path) = parse(File.read(path))
+      def read(io) = parse(io.read)
     end
 
     def initialize(config)

data/lib/completely/exceptions.rb

@@ -1,4 +1,5 @@
 module Completely
   class Error < StandardError; end
   class InstallError < Error; end
+  class ParseError < Error; end
 end

data/lib/completely/installer.rb

@@ -1,5 +1,36 @@
 module Completely
   class Installer
+    class << self
+      def from_io(program:, io: nil)
+        io ||= $stdin
+
+        raise InstallError, 'io must respond to #read' unless io.respond_to?(:read)
+        raise InstallError, 'io is closed' if io.respond_to?(:closed?) && io.closed?
+
+        from_string program:, string: io.read
+      end
+
+      def from_string(program:, string:)
+        tempfile = create_tempfile
+        script_path = tempfile.path
+        begin
+          File.write script_path, string
+        ensure
+          tempfile.close
+        end
+
+        new program:, script_path:
+      end
+
+      def create_tempfile
+        tempfile = Tempfile.new ['completely-', '.bash']
+        tempfiles.push tempfile
+        tempfile
+      end
+
+      def tempfiles = @tempfiles ||= []
+    end
+
     attr_reader :program, :script_path
 
     def initialize(program:, script_path: nil)
@@ -73,15 +104,17 @@ module Completely
     end
 
     def completions_path
-      @completions_path ||= completions_path!
-    end
-
-    def completions_path!
-      target_directories.each do |target|
-        return target if Dir.exist? target
+      @completions_path ||= begin
+        result = nil
+        target_directories.each do |target|
+          if Dir.exist? target
+            result = target
+            break
+          end
+        end
+
+        result
       end
-
-      nil
     end
   end
 end

data/lib/completely/templates/template.erb

@@ -9,24 +9,46 @@
   local cur=${COMP_WORDS[COMP_CWORD]}
   local result=()
 
+  # words the user already typed (excluding the command itself)
+  local used=()
+  if ((COMP_CWORD > 1)); then
+    used=("${COMP_WORDS[@]:1:$((COMP_CWORD - 1))}")
+  fi
+
   if [[ "${cur:0:1}" == "-" ]]; then
+    # Completing an option: offer everything (including options)
     echo "$words"
 
   else
+    # Completing a non-option: offer only non-options,
+    # and don't re-offer ones already used earlier in the line.
     for word in $words; do
-      [[ "${word:0:1}" != "-" ]] && result+=("$word")
+      [[ "${word:0:1}" == "-" ]] && continue
+
+      local seen=0
+      for u in "${used[@]}"; do
+        if [[ "$u" == "$word" ]]; then
+          seen=1
+          break
+        fi
+      done
+      ((!seen)) && result+=("$word")
     done
 
     echo "${result[*]}"
-
   fi
 }
 
 <%= function_name %>() {
   local cur=${COMP_WORDS[COMP_CWORD]}
-  local compwords=("${COMP_WORDS[@]:1:$COMP_CWORD-1}")
+  local compwords=()
+  if ((COMP_CWORD > 0)); then
+    compwords=("${COMP_WORDS[@]:1:$((COMP_CWORD - 1))}")
+  fi
   local compline="${compwords[*]}"
 
+  COMPREPLY=()
+
 % if ENV['COMPLETELY_DEBUG']
   if [[ -n "$COMPLETELY_DEBUG" ]]; then
     echo "compline: '$compline'" > 'completely-debug.txt'

data/lib/completely/version.rb

@@ -1,3 +1,3 @@
 module Completely
-  VERSION = '0.7.2'
+  VERSION = '0.7.3'
 end

data/lib/completely.rb

@@ -4,3 +4,4 @@ require 'completely/pattern'
 require 'completely/completions'
 require 'completely/tester'
 require 'completely/installer'
+require 'debug' if ENV['COMPLETELY_DEV']

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: completely
 version: !ruby/object:Gem::Version
-  version: 0.7.2
+  version: 0.7.3
 platform: ruby
 authors:
 - Danny Ben Shitrit