pry 0.10.4 → 0.11.0.pre

data/lib/pry/commands/edit/file_and_line_locator.rb

@@ -7,7 +7,7 @@ class Pry
         end
 
         def from_code_object(code_object, filename_argument)
-          if File.exists?(code_object.source_file.to_s)
+          if File.exist?(code_object.source_file.to_s)
             [code_object.source_file, code_object.source_line]
           else
             raise CommandError, "Cannot find a file for #{filename_argument}!"

data/lib/pry/commands/find_method.rb

@@ -20,7 +20,7 @@ class Pry
       find-method re Pry
 
       # Find all methods that contain the code:
-      # output.puts inside the Pry namepsace.
+      # output.puts inside the Pry namespace.
       find-method -c 'output.puts' Pry
     BANNER
 

data/lib/pry/commands/gem_open.rb

@@ -16,7 +16,7 @@ class Pry
 
     def process(gem)
       Dir.chdir(Rubygem.spec(gem).full_gem_path) do
-        Pry::Editor.invoke_editor(".", 0, false)
+        Pry::Editor.new(_pry_).invoke_editor(".", 0, false)
       end
     end
 

data/lib/pry/commands/gem_readme.rb

@@ -0,0 +1,25 @@
+class Pry::Command::GemReadme < Pry::ClassCommand
+  match 'gem-readme'
+  description 'Show the readme bundled with a rubygem'
+  group 'Gems'
+  command_options argument_required: true
+  banner <<-BANNER
+    gem-readme gem
+    Show the readme bundled with a rubygem
+  BANNER
+
+  def process(name)
+    spec = Gem::Specification.find_by_name(name)
+    glob = File.join(spec.full_gem_path, 'README*')
+    readme = Dir[glob][0]
+    if File.exist?(readme.to_s)
+      _pry_.pager.page File.read(readme)
+    else
+      raise Pry::CommandError, "Gem '#{name}' doesn't appear to have a README"
+    end
+  rescue Gem::LoadError
+    raise Pry::CommandError, "Gem '#{name}' wasn't found. Are you sure it is installed?"
+  end
+
+  Pry::Commands.add_command(self)
+end

data/lib/pry/commands/gem_search.rb

@@ -0,0 +1,40 @@
+class Pry::Command::GemSearch < Pry::ClassCommand
+  match 'gem-search'
+  description 'Search for a gem with the rubygems.org JSON API'
+  group 'Gems'
+  command_options argument_required: true
+  banner <<-BANNER
+    gem-search [options] gem
+    Search for a gem with the rubygems.org HTTP API
+  BANNER
+
+  API_ENDPOINT = 'https://rubygems.org/api/v1/search.json'
+
+  def setup
+    require 'json' unless defined?(JSON)
+    require 'net/http' unless defined?(Net::HTTP)
+  end
+
+  def options(opt)
+    opt.on :l, :limit, 'Limit the number of results (max: 30)',
+      default: 10,
+      as: Integer,
+      argument: true
+  end
+
+  def process(str)
+    uri = URI.parse(API_ENDPOINT)
+    uri.query = URI.encode_www_form(query: str)
+    gems = JSON.load Net::HTTP.get(uri)
+    _pry_.pager.page list_as_string(gems, opts[:limit])
+  end
+
+private
+  def list_as_string(gems, limit = 10)
+    gems[0..limit-1].map do |gem|
+      name, version, info = gem.values_at 'name', 'version', 'info'
+      "#{text.bold(name)} #{text.bold('v'+version)} \n#{info}\n\n"
+    end.join
+  end
+  Pry::Commands.add_command(self)
+end

data/lib/pry/commands/hist.rb

@@ -170,8 +170,8 @@ class Pry
           else
             Pry.history.to_a.last(Pry.history.session_line_count)
           end
-      # The last value in history will be the 'hist' command itself.
-      Pry::Code(h[0..-2])
+
+      Pry::Code(Pry.history.filter(h[0..-2]))
     end
   end
 

data/lib/pry/commands/jump_to.rb

@@ -9,18 +9,18 @@ class Pry
     BANNER
 
     def process(break_level)
-      break_level = break_level.to_i
-      nesting_level = _pry_.binding_stack.size - 1
+      break_level    = break_level.to_i
+      nesting_level  = _pry_.binding_stack.size - 1
+      max_nest_level = nesting_level - 1
 
       case break_level
       when nesting_level
         output.puts "Already at nesting level #{nesting_level}"
-      when (0...nesting_level)
-        _pry_.binding_stack.slice!(break_level + 1, _pry_.binding_stack.size)
-
+      when 0..max_nest_level
+        _pry_.binding_stack = _pry_.binding_stack[0..break_level]
       else
-        max_nest_level = nesting_level - 1
-        output.puts "Invalid nest level. Must be between 0 and #{max_nest_level}. Got #{break_level}."
+        output.puts "Invalid nest level. Must be between 0 and " \
+          "#{max_nest_level}. Got #{break_level}."
       end
     end
   end

data/lib/pry/commands/ls/formatter.rb

@@ -7,6 +7,7 @@ class Pry
       def initialize(_pry_)
         @_pry_ = _pry_
         @target = _pry_.current_context
+        @default_switch = nil
       end
 
       def write_out

data/lib/pry/commands/ls/jruby_hacks.rb

@@ -19,7 +19,7 @@ module Pry::Command::Ls::JRubyHacks
       m.name.sub(/\A(is|get|set)(?=[A-Z_])/, '').gsub(/[_?=]/, '').downcase
     end
 
-    grouped.map do |key, values|
+    grouped.flat_map do |key, values|
       values = values.sort_by do |m|
         rubbishness(m.name)
       end
@@ -28,7 +28,7 @@ module Pry::Command::Ls::JRubyHacks
       values.select do |x|
         (!found.any? { |y| x == y }) && found << x
       end
-    end.flatten(1)
+    end
   end
 
   # When removing jruby aliases, we want to keep the alias that is

data/lib/pry/commands/ls/self_methods.rb

@@ -11,6 +11,8 @@ class Pry
         super(_pry_)
         @interrogatee = interrogatee
         @no_user_opts = no_user_opts
+        @ppp_switch = opts[:ppp]
+        @jruby_switch = opts['all-java']
       end
 
       def output_self

data/lib/pry/commands/play.rb

@@ -12,7 +12,7 @@ class Pry
 
       play --lines 149..153   # assumes current context
       play -i 20 --lines 1..3 # assumes lines of the input expression at 20
-      play -o 4               # the output of of an expression at 4
+      play -o 4               # the output of an expression at 4
       play Pry#repl -l 1..-1  # play the contents of Pry#repl method
       play -e 2               # play from specified line until end of valid expression
       play hello.rb           # play a file
@@ -91,7 +91,7 @@ class Pry
     end
 
     def file_content
-      if default_file && File.exists?(default_file)
+      if default_file && File.exist?(default_file)
         @cc.restrict_to_lines(File.read(default_file), @cc.line_range)
       else
         raise CommandError, "File does not exist! File was: #{default_file.inspect}"

data/lib/pry/commands/reload_code.rb

@@ -31,7 +31,7 @@ class Pry
     end
 
     def reload_current_file
-      if !File.exists?(current_file)
+      if !File.exist?(current_file)
         raise CommandError, "Current file: #{current_file} cannot be found on disk!"
       end
 
@@ -49,7 +49,7 @@ class Pry
     def check_for_reloadability(code_object, identifier)
       if !code_object || !code_object.source_file
         raise CommandError, "Cannot locate #{identifier}!"
-      elsif !File.exists?(code_object.source_file)
+      elsif !File.exist?(code_object.source_file)
         raise CommandError,
           "Cannot reload #{identifier} as it has no associated file on disk. " \
           "File found was: #{code_object.source_file}"

data/lib/pry/commands/ri.rb

@@ -14,6 +14,10 @@ class Pry
     BANNER
 
     def process(spec)
+      unless spec
+        return output.puts "Please provide a class, module, or method name (e.g: ri Array#push)"
+      end
+
       # Lazily load RI
       require 'rdoc/ri/driver'
 

data/lib/pry/commands/shell_command.rb

@@ -4,7 +4,7 @@ class Pry
     group 'Input and Output'
     description "All text following a '.' is forwarded to the shell."
     command_options :listing => '.<shell command>', :use_prefix => false,
-      :takes_block => true
+                    :takes_block => true
 
     banner <<-'BANNER'
       Usage: .COMMAND_NAME
@@ -30,18 +30,44 @@ class Pry
 
     private
 
-      def parse_destination(dest)
-        return "~" if dest.empty?
-        return dest unless dest == "-"
-        state.old_pwd || raise(CommandError, "No prior directory available")
-      end
+    def parse_destination(dest)
+      return "~" if dest.empty?
+      return dest unless dest == "-"
+      state.old_pwd || raise(CommandError, "No prior directory available")
+    end
 
-      def process_cd(dest)
+    def process_cd(dest)
+      begin
         state.old_pwd = Dir.pwd
-        Dir.chdir File.expand_path(dest)
+        Dir.chdir(File.expand_path(path_from_cd_path(dest) || dest))
       rescue Errno::ENOENT
         raise CommandError, "No such directory: #{dest}"
       end
+    end
+
+    def cd_path_env
+      ENV['CDPATH']
+    end
+
+    def cd_path_exists?
+      cd_path_env && cd_path_env.length.nonzero?
+    end
+
+    def path_from_cd_path(dest)
+      return if !(dest && cd_path_exists?) || special_case_path?(dest)
+
+      cd_path_env.split(File::PATH_SEPARATOR).each do |path|
+        if File.directory?(path) && path.split(File::SEPARATOR).last == dest
+          return path
+        end
+      end
+
+      return nil
+    end
+
+    def special_case_path?(dest)
+      ['.', '..', '-'].include?(dest) || dest =~ /\A[#{File::PATH_SEPARATOR}~]/
+    end
   end
 
   Pry::Commands.add_command(Pry::Command::ShellCommand)

data/lib/pry/commands/show_info.rb

@@ -22,7 +22,15 @@ class Pry
       raise CommandError, no_definition_message if !code_object
       @original_code_object = code_object
 
-      if show_all_modules?(code_object)
+      if !obj_name && code_object.c_module? && !opts[:all]
+        result = "Warning: You're inside an object, whose class is defined by means\n" +
+                 "         of the C Ruby API. Pry cannot display the information for\n" +
+                 "         this class."
+        if code_object.candidates.any?
+          result += "\n         However, you can view monkey-patches applied to this class.\n" +
+                    "         Just execute the same command with the '--all' switch."
+        end
+      elsif show_all_modules?(code_object)
         # show all monkey patches for a module
 
         result = content_and_headers_for_all_module_candidates(code_object)
@@ -85,7 +93,7 @@ class Pry
     end
 
     def no_definition_message
-      "Couldn't locate a definition for #{obj_name}!"
+      "Couldn't locate a definition for #{obj_name}"
     end
 
     # Generate a header (meta-data information) for all the code

data/lib/pry/commands/watch_expression/expression.rb

@@ -10,7 +10,7 @@ class Pry
       end
 
       def eval!
-        @previous_value = @value
+        @previous_value = value
         @value = Pry::ColorPrinter.pp(target_eval(target, source), "")
       end
 

data/lib/pry/commands/whereami.rb

@@ -159,12 +159,12 @@ class Pry
     end
 
     def class_code
-      return @class_code if @class_code
-
-      mod = @method ? Pry::WrappedModule(@method.owner) : target_class
-
-      idx = mod.candidates.find_index { |v| expand_path(v.source_file) == @file }
-      @class_code = idx && Pry::Code.from_module(mod, idx)
+      @class_code ||=
+        begin
+          mod = @method ? Pry::WrappedModule(@method.owner) : target_class
+          idx = mod.candidates.find_index { |v| expand_path(v.source_file) == @file }
+          idx && Pry::Code.from_module(mod, idx)
+        end
     end
 
     def valid_method?

data/lib/pry/config.rb

@@ -1,24 +1,11 @@
-class Pry::Config
+require_relative 'basic_object'
+class Pry::Config < Pry::BasicObject
   require_relative 'config/behavior'
+  require_relative 'config/lazy'
   require_relative 'config/default'
   require_relative 'config/convenience'
   include Pry::Config::Behavior
-
   def self.shortcuts
     Convenience::SHORTCUTS
   end
-
-  #
-  # FIXME
-  # @param [Pry::Hooks] hooks
-  #
-  def hooks=(hooks)
-    if hooks.is_a?(Hash)
-      warn "Hash-based hooks are now deprecated! Use a `Pry::Hooks` object " \
-           "instead! http://rubydoc.info/github/pry/pry/master/Pry/Hooks"
-      self["hooks"] = Pry::Hooks.from_hash(hooks)
-    else
-      self["hooks"] = hooks
-    end
-  end
 end

data/lib/pry/config/behavior.rb

@@ -2,130 +2,200 @@ module Pry::Config::Behavior
   ASSIGNMENT     = "=".freeze
   NODUP          = [TrueClass, FalseClass, NilClass, Symbol, Numeric, Module, Proc].freeze
   INSPECT_REGEXP = /#{Regexp.escape "default=#<"}/
+  ReservedKeyError = Class.new(RuntimeError)
 
   module Builder
-    def from_hash(hash, default = nil)
+    #
+    # @param [Hash] attributes
+    #   a hash to initialize an instance of self with.
+    #
+    # @param [Pry::Config, nil] default
+    #   a default, or nil for none.
+    #
+    # @return [Pry::Config]
+    #   returns an instance of self.
+    #
+    def from_hash(attributes, default = nil)
       new(default).tap do |config|
-        config.merge!(hash)
+        attributes.each do |key,value|
+          config[key] = Hash === value ? from_hash(value, nil) : value
+        end
       end
     end
   end
 
   def self.included(klass)
-    unless defined?(RESERVED_KEYS)
-      const_set :RESERVED_KEYS, instance_methods(false).map(&:to_s).freeze
-    end
     klass.extend(Builder)
   end
 
   def initialize(default = Pry.config)
     @default = default
     @lookup = {}
+    @reserved_keys = methods.map(&:to_s).freeze
   end
 
   #
   # @return [Pry::Config::Behavior]
-  #   returns the default used if a matching value for a key isn't found in self
+  #   returns the default used incase a key isn't found in self.
   #
   def default
     @default
   end
 
+  #
+  # @param [String] key
+  #   a key (as a String)
+  #
+  # @return [Object, BasicObject]
+  #   returns an object from self or one of its defaults.
+  #
   def [](key)
-    @lookup[key.to_s]
+    key = key.to_s
+    @lookup[key] or (@default and @default[key])
   end
 
+  #
+  # Add a key and value pair to self.
+  #
+  # @param [String] key
+  #   a key (as a String).
+  #
+  # @param [Object,BasicObject] value
+  #   a value.
+  #
+  # @raise [Pry::Config::ReservedKeyError]
+  #   when 'key' is a reserved key name.
+  #
   def []=(key, value)
     key = key.to_s
-    if RESERVED_KEYS.include?(key)
-      raise ArgumentError, "few things are reserved by pry, but using '#{key}' as a configuration key is."
+    if @reserved_keys.include?(key)
+      raise ReservedKeyError, "It is not possible to use '#{key}' as a key name, please choose a different key name."
     end
-    @lookup[key] = value
+    __push(key,value)
   end
 
-  def method_missing(name, *args, &block)
-    key = name.to_s
-    if key[-1] == ASSIGNMENT
-      short_key = key[0..-2]
-      self[short_key] = args[0]
-    elsif key?(key)
-      self[key]
-    elsif @default.respond_to?(name)
-      value = @default.public_send(name, *args, &block)
-      # FIXME: refactor Pry::Hook so that it stores config on the config object,
-      # so that we can use the normal strategy.
-      self[key] = value = value.dup if key == 'hooks'
-      value
-    else
-      nil
-    end
+  #
+  # Removes a key from self.
+  #
+  # @param [String] key
+  #  a key (as a String)
+  #
+  # @return [void]
+  #
+  def forget(key)
+    key = key.to_s
+    __remove(key)
   end
 
+  #
+  # @param [Hash, #to_h, #to_hash] other
+  #   a hash to merge into self.
+  #
+  # @return [void]
+  #
   def merge!(other)
-    other = try_convert_to_hash(other)
+    other = __try_convert_to_hash(other)
     raise TypeError, "unable to convert argument into a Hash" unless other
     other.each do |key, value|
       self[key] = value
     end
   end
 
+  #
+  # @param [Hash, #to_h, #to_hash] other
+  #   a hash to compare against the lookup table of self.
+  #
   def ==(other)
-    @lookup == try_convert_to_hash(other)
+    @lookup == __try_convert_to_hash(other)
   end
   alias_method :eql?, :==
 
-  def respond_to_missing?(key, include_private=false)
-    key?(key) or @default.respond_to?(key) or super(key, include_private)
-  end
-
+  #
+  # @param [String] key
+  #   a key (as a String)
+  #
+  # @return [Boolean]
+  #   returns true when "key" is a member of self.
+  #
   def key?(key)
     key = key.to_s
     @lookup.key?(key)
   end
 
+  #
+  # Clear the lookup table of self.
+  #
+  # @return [void]
+  #
   def clear
     @lookup.clear
     true
   end
-  alias_method :refresh, :clear
-
-  def forget(key)
-    @lookup.delete(key.to_s)
-  end
 
+  #
+  # @return [Array<String>]
+  #   returns an array of keys in self.
+  #
   def keys
     @lookup.keys
   end
 
+  def eager_load!
+    local_last_default = last_default
+    local_last_default.lazy_keys.each do |key|
+      self[key] = local_last_default.public_send(key)
+    end
+  end
+
+  def last_default
+    last = @default
+    last = last.default while last and last.default
+    last
+  end
+  
+  #
+  # @return [Hash]
+  #   returns a duplicate copy of the lookup table used by self.
+  #
   def to_hash
     @lookup.dup
   end
   alias_method :to_h, :to_hash
 
-
   def inspect
     key_str = keys.map { |key| "'#{key}'" }.join(",")
-    "#<#{_clip_inspect(self)} local_keys=[#{key_str}] default=#{@default.inspect}>"
+    "#<#{__clip_inspect(self)} keys=[#{key_str}] default=#{@default.inspect}>"
   end
 
   def pretty_print(q)
     q.text inspect[1..-1].gsub(INSPECT_REGEXP, "default=<")
   end
 
-private
-  def _clip_inspect(obj)
-    "#{obj.class}:0x%x" % obj.object_id << 1
-  end
-
-  def _dup(value)
-    if NODUP.any? { |klass| klass === value }
-      value
+  def method_missing(name, *args, &block)
+    key = name.to_s
+    if key[-1] == ASSIGNMENT
+      short_key = key[0..-2]
+      self[short_key] = args[0]
+    elsif key?(key)
+      self[key]
+    elsif @default.respond_to?(name)
+      value = @default.public_send(name, *args, &block)
+      self[key] = __dup(value)
     else
-      value.dup
+      nil
     end
   end
 
-  def try_convert_to_hash(obj)
+  def respond_to_missing?(key, include_private=false)
+    key?(key) or @default.respond_to?(key) or super(key, include_private)
+  end
+
+private
+  def __clip_inspect(obj)
+    "#{obj.class}:0x%x" % obj.object_id
+  end
+
+  def __try_convert_to_hash(obj)
     if Hash === obj
       obj
     elsif obj.respond_to?(:to_h)
@@ -136,4 +206,24 @@ private
       nil
     end
   end
+
+  def __dup(value)
+    if NODUP.any? { |klass| klass === value }
+      value
+    else
+      value.dup
+    end
+  end
+
+  def __push(key,value)
+    unless singleton_class.method_defined? key
+      define_singleton_method(key) { self[key] }
+      define_singleton_method("#{key}=") { |val| @lookup[key] = val }
+    end
+    @lookup[key] = value
+  end
+
+  def __remove(key)
+    @lookup.delete(key)
+  end
 end