pry-moves 0.1.13 → 1.0.0

data/lib/pry-moves/commands.rb

@@ -21,6 +21,11 @@ module PryMoves
       breakout_navigation :next, 'blockless'
     end
 
+    block_command 'next_breakpoint', 'Go to next breakpoint' do |param|
+      breakout_navigation :next_breakpoint, param
+    end
+    alias_command 'b', 'next_breakpoint'
+
     block_command 'iterate', 'Go to next iteration of current block' do |param|
       breakout_navigation :iterate, param
     end
@@ -41,7 +46,7 @@ module PryMoves
     end
 
     block_command 'bt', 'Backtrace' do |param, param2|
-      PryMoves::Backtrace.new(target, _pry_).run_command param, param2
+      PryMoves::Backtrace.new(_pry_).run_command param, param2
     end
 
     block_command 'debug', '' do
@@ -49,6 +54,23 @@ module PryMoves
       breakout_navigation :debug, cmd
     end
 
+    block_command :restart, '' do
+      PryMoves.restart_requested = true
+      run 'continue'
+    end
+    alias_command '@', 'restart'
+
+    block_command :reload, '' do
+      PryMoves.reload_requested = true
+      run 'continue'
+    end
+    alias_command '#', 'reload'
+
+    block_command /^\\(\w+)$/, 'Execute command explicitly' do |param|
+      Pry.config.ignore_once_var_precedence = true
+      run param
+    end
+
     block_command '!', 'exit' do
       PryMoves.unlock
       Pry.config.exit_requested = true
@@ -62,16 +84,32 @@ module PryMoves
 
     helpers do
       def breakout_navigation(action, param)
+        return if var_precedence action
+
         check_file_context
         _pry_.binding_stack.clear     # Clear the binding stack.
         throw :breakout_nav, {        # Break out of the REPL loop and
           action: action,          #   signal the tracer.
           param:  param,
-          binding: target,
-          pry: _pry_
+          binding: target
         }
       end
 
+      def var_precedence action
+        if Pry.config.ignore_once_var_precedence
+          Pry.config.ignore_once_var_precedence = false
+          return
+        end
+
+        input = Pry.config.original_user_input || action
+        binding_value = target.eval(input) rescue nil
+        unless binding_value.nil?
+          puts "ℹ️️  Variable \"#{input}\" found. To execute command type its alias or \\#{input}"
+          puts PryMoves::Painter.colorize binding_value
+          true
+        end
+      end
+
       # Ensures that a command is executed in a local file context.
       def check_file_context
         unless PryMoves.check_file_context(target)
@@ -79,6 +117,7 @@ module PryMoves
         end
       end
     end
+
   end
 end
 

data/lib/pry-moves/formatter.rb

@@ -0,0 +1,74 @@
+class PryMoves::Formatter
+
+  attr_accessor :colorize
+
+  def initialize colorize = true
+    @colorize = colorize
+  end
+
+  def method_signature(binding)
+    meth = binding.eval('__method__')
+    meth_obj = meth ? Pry::Method.from_binding(binding) : nil
+    if !meth_obj
+      ""
+    elsif meth_obj.undefined?
+      "#{meth_obj.name}(UNKNOWN) (undefined method)"
+    else
+      args = meth_obj.parameters.map.with_index do |(type, name), i|
+        if name
+          value = format_arg binding, name.to_s
+          show_value = true
+        else
+          name = (type == :block ? 'block' : "arg#{i + 1}")
+        end
+        name = case type
+          when :req   then "#{name} ="
+          when :key   then "#{name}:"
+          when :opt   then "#{name}=?"
+          when :rest  then "*#{name}"
+          when :block then "&#{name}"
+          else '?'
+        end
+        show_value ? "#{name} #{value}" : name
+      end
+      "#{meth_obj.name}(#{args.join(', ')})"
+    end
+  end
+
+  def format_arg binding, arg_name
+    arg = binding.eval(arg_name.to_s)
+    format_obj arg
+  end
+
+  def first_line str
+    str.split("\n").first
+  end
+
+  def cut_string str
+    str.length > 50 ? "#{str.first 50}..." : str
+  end
+
+  PATH_TRASH = defined?(Rails) ? Rails.root.to_s : Dir.pwd
+
+  def shorten_path(path)
+    path.gsub( /^#{PATH_TRASH}\//, '')
+  end
+
+  def format_obj(obj)
+    if obj.is_a? String
+      format_obj2 cut_string first_line obj
+    else
+      first_line format_obj2 obj
+    end
+  end
+
+  def format_obj2(obj)
+    if @colorize
+      PryMoves::Painter.colorize obj
+    else
+      i = obj.inspect
+      i.start_with?('#<') ? obj.class.to_s : i
+    end
+  end
+
+end

data/lib/pry-moves/painter.rb

@@ -14,13 +14,14 @@ module PryMoves::Painter
 
   def self.colorize(obj)
     colored_str = Canvas.new
-    obj = obj.class if obj.inspect.start_with? "#<"
+    i = obj.inspect
+    obj = obj.class if i.is_a?(String) && i.start_with?("#<")
     catch (:cut) do
       Pry::ColorPrinter.pp obj, colored_str
     end
     colored_str.chomp
   rescue => e
-    "Inspect error: #{e}\n" +
+    "⛔️ Inspect error: #{e}\n" +
       "#{e.backtrace.first(3).join("\n")}"
   end
 

data/lib/pry-moves/pry_ext.rb

@@ -1,31 +1,31 @@
 class << Pry
-  alias_method :start_without_pry_nav, :start
-
-  def start_with_pry_nav(target = TOPLEVEL_BINDING, options = {})
-    old_options = options.reject { |k, _| k == :pry_remote }
+  alias pry_moves_origin_start start
 
+  def start(target = TOPLEVEL_BINDING, options = {})
     if target.is_a?(Binding) && PryMoves.check_file_context(target)
       # Wrap the tracer around the usual Pry.start
-      PryMoves::PryWrapper.new(target, options).run do
-        start_without_pry_nav(target, old_options)
-      end
+      original_verbosity = $VERBOSE
+      $VERBOSE = nil # Disable warnings for pry-moves
+      PryMoves::PryWrapper.new(target, options, self).run
+      $VERBOSE = original_verbosity
     else
       # No need for the tracer unless we have a file context to step through
-      start_without_pry_nav(target, old_options)
+      pry_moves_origin_start(target, options)
     end
   end
 
-  alias_method :start, :start_with_pry_nav
 end
 
 Binding.class_eval do
 
+  attr_accessor :index
+
   alias pry_forced pry
 
   def pry
-    unless Pry.config.disable_breakpoints
-      PryMoves.synchronize_threads ||
-        return # Don't start binding.pry when semaphore locked by current thread
+    if !Pry.config.disable_breakpoints and
+        # Don't start binding.pry when semaphore locked by current thread
+        PryMoves.synchronize_threads
       pry_forced
     end
   end
@@ -34,6 +34,32 @@ end
 
 Pry.config.pager = false
 
+Pry::Command.class_eval do
+  class << self
+    attr_accessor :original_user_input
+  end
+
+  alias run_origin_for_pry_moves run
+  def run(command_string, *args)
+    Pry.config.original_user_input = self.class.original_user_input
+    result = run_origin_for_pry_moves command_string, *args
+    Pry.config.original_user_input = nil
+    result
+  end
+end
+
+Pry::CommandSet.class_eval do
+
+  alias alias_command_origin_for_pry_moves alias_command
+
+  def alias_command(match, action, options = {})
+    cmd = alias_command_origin_for_pry_moves match, action, options
+    cmd.original_user_input = match
+    cmd
+  end
+
+end
+
 Pry::Command::Whereami.class_eval do
   # Negligent function from Pry - evidently poor output format
   # would be wanted to be changed often by developers,
@@ -56,11 +82,19 @@ Pry::Command::Whereami.class_eval do
   end
 
   def build_output
-    lines = []
-    lines << "#{text.bold('From:')} #{PryMoves::Helpers.shorten_path location}"
-    lines << PryMoves::Watch.instance.output(target) unless PryMoves::Watch.instance.empty?
+    lines = ['']
+
+    formatter = PryMoves::Formatter.new
+    prefix = Thread.current[:pry_moves_debug] ? "👾 " : ""
+    lines << "#{prefix}#{formatter.shorten_path location}"
+    lines << "   ." + formatter.method_signature(target)
     lines << ''
     lines << "#{code.with_line_numbers(use_line_numbers?).with_marker(marker).highlighted}"
+
+    lines << PryMoves::Watch.instance.output(target) unless PryMoves::Watch.instance.empty?
+    lines.concat PryMoves.messages
+    PryMoves.messages.clear
+
     lines << ''
     lines.join "\n"
   end
@@ -82,4 +116,18 @@ Pry::Code::LOC.class_eval do
     tuple[0] = " #{marker} #{ line }"
   end
 
-end
+end
+
+Pry::Output.class_eval do
+
+  alias pry_moves_origin_for_puts puts
+
+  def puts *args
+    first = args[0]
+    if first.is_a? String and first.start_with? "(pry) output error"
+      first.slice! 400..-1
+    end
+    pry_moves_origin_for_puts *args
+  end
+
+end if defined? Pry::Output

data/lib/pry-moves/pry_wrapper.rb

@@ -2,24 +2,24 @@ require 'pry' unless defined? Pry
 
 module PryMoves
 class PryWrapper
-  def initialize(binding_, pry_start_options = {})
+  def initialize(binding_, pry_start_options, pry)
     @init_binding = binding_
     @pry_start_options = pry_start_options   # Options to use for Pry.start
+    @pry = pry
   end
 
-  def run(&block)
+  def run
     PryMoves.lock
 
-    Pry.config.marker = "⛔️" if @pry_start_options[:exit_from_method]
-
-    return_value = nil
-    PryMoves.is_open = true
-    @command = catch(:breakout_nav) do      # Coordinates with PryMoves::Commands
-      return_value = yield
-      nil    # Nothing thrown == no navigational command
+    initial_frame = PryMoves::BindingsStack.new.initial_frame
+    if not @pry_start_options[:pry_moves_loop] and
+        initial_frame.local_variable_defined? :debug_redirect
+      debug_redirect = initial_frame.local_variable_get(:debug_redirect)
+      PryMoves.messages << "⏩ redirected to #{debug_redirect}"
+      @command = {action: :step, binding: initial_frame}
+    else
+      start_pry
     end
-    PryMoves.is_open = false
-    Pry.config.marker = "=>"
 
     if @command
       trace_command
@@ -30,11 +30,24 @@ class PryWrapper
       end
     end
 
-    return_value
+    @return_value
   end
 
   private
 
+  def start_pry
+    Pry.config.marker = "⛔️" if @pry_start_options[:exit_from_method]
+    PryMoves.is_open = true
+
+    @command = catch(:breakout_nav) do      # Coordinates with PryMoves::Commands
+      @return_value = @pry.pry_moves_origin_start(@init_binding, @pry_start_options)
+      nil    # Nothing thrown == no navigational command
+    end
+
+    PryMoves.is_open = false
+    Pry.config.marker = "=>"
+  end
+
   def trace_command
     if @command[:action] == :debug
       wrap_debug
@@ -61,9 +74,9 @@ class PryWrapper
       tracer = start_tracing
       begin
         @command[:binding].eval @command[:param]
-      rescue => e
+      rescue StandardError, SyntaxError => e
         Thread.current.set_trace_func nil
-        puts e
+        puts "❌️ #{e}"
       end
       tracer.stop_tracing
     end.join
@@ -73,9 +86,9 @@ class PryWrapper
 
   def start_tracing
     @last_runtime_binding = @command[:binding]
-    tracer = PryMoves::Tracer.new @command, @pry_start_options
-    tracer.trace
-    tracer
+    PryMoves::TraceCommand.trace @command, @pry_start_options do |binding|
+      Pry.start(binding, @pry_start_options)
+    end
   end
 
 end

data/lib/pry-moves/restartable.rb

@@ -0,0 +1,38 @@
+module PryMoves::Restartable
+
+  attr_accessor :restart_requested, :reload_requested,
+    :reload_rake_tasks
+
+  def restartable
+    trigger :new_run
+    yield
+    re_execution
+  rescue PryMoves::Restart
+    self.restart_requested = false
+    PryMoves.reset
+    trigger :restart
+    retry
+  rescue PryMoves::Reload
+    puts "🔮  try to use @ with reload"
+    exit 3
+  end
+
+  def re_execution
+    raise PryMoves::Restart if restart_requested
+    raise PryMoves::Reload if reload_requested
+  end
+
+
+end
+
+class PryMoves::Restart < RuntimeError
+end
+class PryMoves::Reload < RuntimeError
+end
+RSpec::Support::AllExceptionsExceptOnesWeMustNotRescue::AVOID_RESCUING.concat [PryMoves::Restart, PryMoves::Reload] if defined? RSpec
+
+Pry.config.hooks.add_hook(:after_eval, :exit_on_re_execution) do |_, _, _pry_|
+  if PryMoves.restart_requested or PryMoves.reload_requested
+    Pry.run_command 'exit-all'
+  end
+end

data/lib/pry-moves/version.rb

@@ -1,3 +1,3 @@
 module PryMoves
-  VERSION = '0.1.13'
+  VERSION = '1.0.0'
 end

data/lib/pry-moves/watch.rb

@@ -1,4 +1,5 @@
 require 'singleton'
+require 'set'
 
 class PryMoves::Watch
 
@@ -44,6 +45,8 @@ class PryMoves::Watch
     "\033[1m#{cmd}\033[0m: #{format binding_.eval(cmd)}"
   rescue NameError
     "\033[1m#{cmd}\033[0m: <undefined>"
+  rescue => e
+    "\033[1m#{cmd}\033[0m: <#{e}>"
   end
 
   def format(text)

data/lib/pry-moves.rb

@@ -1,19 +1,30 @@
 require 'pry' unless defined? Pry
 
 require 'pry-moves/version'
-require 'pry-moves/trace_commands'
-require 'pry-moves/traced_method'
-require 'pry-moves/tracer'
 require 'pry-moves/pry_ext'
 require 'pry-moves/commands'
-require 'pry-moves/traversing'
+require 'pry-moves/add_suffix'
 require 'pry-moves/pry_wrapper'
+require 'pry-moves/bindings_stack'
+require 'pry-moves/formatter'
 require 'pry-moves/backtrace'
 require 'pry-moves/watch'
-require 'pry-moves/helpers'
 require 'pry-moves/painter'
+require 'pry-moves/restartable'
+
+require 'commands/traced_method'
+require 'commands/trace_helpers'
+require 'commands/trace_command'
+require 'commands/debug'
+require 'commands/finish'
+require 'commands/goto'
+require 'commands/iterate'
+require 'commands/next'
+require 'commands/next_breakpoint'
+require 'commands/step'
 
 require 'pry-stack_explorer/pry-stack_explorer'
+require 'debug_sugar'
 
 # Optionally load pry-remote monkey patches
 require 'pry-moves/pry_remote_ext' if defined? PryRemote
@@ -22,8 +33,30 @@ module PryMoves
   TRACE_IGNORE_FILES = Dir[File.join(File.dirname(__FILE__), '**', '*.rb')].map { |f| File.expand_path(f) }
 
   extend self
+  extend PryMoves::Restartable
+
+  attr_accessor :is_open, :trace,
+    :stop_on_breakpoints, :launched_specs_examples, :debug_called_times
+
+  def reset
+    self.launched_specs_examples = 0
+    self.stop_on_breakpoints = true
+    self.debug_called_times = 0
+  end
 
-  attr_accessor :is_open
+  def debug(message = nil, at: nil)
+    pry_moves_stack_root = true
+    PryMoves.re_execution
+    if PryMoves.stop_on_breakpoints
+      if at
+        self.debug_called_times += 1
+        return unless self.debug_called_times == at
+      end
+      PryMoves.messages << message if message
+      binding.pry
+      PryMoves.re_execution
+    end
+  end
 
   # Checks that a binding is in a local file context. Extracted from
   # https://github.com/pry/pry/blob/master/lib/pry/default_commands/context.rb
@@ -36,9 +69,18 @@ module PryMoves
     @semaphore ||= Mutex.new
   end
 
+  def messages
+    @messages ||= []
+  end
+
+  def add_command(command, &block)
+    Pry.commands.block_command command, "", &block
+  end
+
   def locked?
     semaphore.locked?
   end
+  alias tracing? locked?
 
   def lock
     semaphore.lock unless semaphore.locked?
@@ -59,6 +101,23 @@ module PryMoves
     true
   end
 
+  def trigger(event)
+    triggers[event].each &:call
+  end
+
+  def triggers
+    @triggers ||= Hash.new do |hash, key|
+      hash[key] = []
+    end
+  end
+
+  def on(trigger, &block)
+    triggers[trigger] << block
+  end
+
   # Reference to currently running pry-remote server. Used by the tracer.
   attr_accessor :current_remote_server
 end
+
+PryMoves.reset
+PryMoves.trace = true if ENV['TRACE_MOVES']