pry-moves 1.0.3 → 1.0.5

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: cf124819313d4f06ebe4b9d81d1b392cf06e96028ebef3ef67604642f9529c70
-  data.tar.gz: d07e5afae2c9bf9ce303a1731706af1921f79892496d188b9c5bf371f0160ed1
+  metadata.gz: 301fded3cbdfbcafed7b5387b820b8c1366be09023f0462a8ab1a36a7bbcc563
+  data.tar.gz: bfb140a999ccc293ca3f81ab11b2ea456c48f21c5a62902c6a44e6c6453e2baa
 SHA512:
-  metadata.gz: ea4608c426807fbc02121a5bc32dddb36fe35fdf988872d964fe97424f9ce0d14d1b8bf28dd96012610ffcfdbb7d2567ceb97589a37b12e49a835c40eddc8fbc
-  data.tar.gz: d1fca032e8f5c76bd10012b9a170849238c335238ea1b899577c0d2ccb2109b2578b6f171ebcd2f6465f3fdba247c05feef0c9e0a77f481452dfd4c3c270e591
+  metadata.gz: 794091fb89ac6ab0b9b2de60dc960137dd72068368caafb6302e6b5731835c3d0d36db7526524c7d6b47104eaf1293aec779a9bb4bfcb45aa1a22a3dd71484c2
+  data.tar.gz: 816e6e90426305ea3b08ed9bfa1899106a3870b782a25cef59abfc3462d7ffa99fa081cae74acb0acfced7126267176c2b6af32fae6510b0e72a4217bd6bb95c

data/Gemfile.lock

@@ -1,7 +1,7 @@
 PATH
   remote: .
   specs:
-    pry-moves (1.0.3)
+    pry-moves (1.0.5)
       awesome_print (>= 1.8.0)
       binding_of_caller (~> 0.7)
       colorize (~> 0.8)

data/README.md

@@ -93,6 +93,7 @@ PryMoves::Backtrace::filter =
 
 Turn off features with environment variables:
 ```bash
+PRY_MOVES=off
 PRY_MOVES_DEBUG_MISSING=off
 PRY_MOVES_RELOADER=off
 ```

data/lib/commands/next_breakpoint.rb

@@ -4,6 +4,10 @@ class PryMoves::NextBreakpoint < PryMoves::TraceCommand
   end
 
   def trace(event, file, line, method, binding_)
+    # this command will stuck in tracing when no breakpoint met (usually)
+    PryMoves.messages << "⚠️  Command NextBreakpoint is not implemented"
+    return true
+
     if binding_.local_variable_defined?(:pry_breakpoint) and
         binding_.local_variable_get(:pry_breakpoint)
       binding_.local_variable_set :pry_breakpoint, nil # reset breakpoint at visited method instance

data/lib/pry-moves/backtrace.rb

@@ -3,6 +3,7 @@ require 'fileutils'
 class PryMoves::Backtrace
 
   FILTERS = %w[/gems/ /rubygems/ /bin/ /lib/ruby/]
+  FILTERS << File.expand_path('..', __dir__)
   @@backtrace = nil
 
   class << self

data/lib/pry-moves/code_reloader.rb

@@ -10,15 +10,25 @@ class CodeReloader
   def reload
     traverse_files do |path|
       if @timestamps[path] != File.mtime(path)
-        @timestamps[path] = File.mtime(path)
-        load path
-        # Log.info "⚡️ file reloaded #{path}"
+        if reload_file path
+          @timestamps[path] = File.mtime(path)
+          # Log.info "⚡️ file reloaded #{path}"
+        end
       end
     end
   end
 
   private
 
+  def reload_file path
+    hide_from_stack = true
+    load path
+    true
+  rescue SyntaxError => e
+    PryMoves.debug_error ["🛠  Syntax error:".red, e.message].join "\n"
+    false
+  end
+
   def traverse_files
     paths = PryMoves.reload_ruby_scripts[:monitor]
     except = PryMoves.reload_ruby_scripts[:except]

data/lib/pry-moves/commands.rb

@@ -112,12 +112,15 @@ module PryMoves
           return
         end
 
-        input = Pry.config.original_user_input || action
-        binding_value = target.eval(input) rescue nil
-        unless binding_value.nil?
+        input = Pry.config.original_user_input || action.to_s
+        return if %w[next debug].include? input # next - ruby keyword
+        begin
+          binding_value = target.eval(input)
           puts "ℹ️️  Variable \"#{input}\" found. To execute command type its alias or \\#{input}"
           puts PryMoves::Painter.colorize binding_value
           true
+        rescue => e
+        #   puts (e.backtrace.reverse + ["var_precedence exception:".red, "#{e}".red]).join "\n"
         end
       end
 

data/lib/pry-moves/formatter.rb

@@ -40,7 +40,11 @@ class PryMoves::Formatter
   end
 
   def format_arg binding, arg_name
-    arg = binding.eval(arg_name.to_s)
+    arg = begin
+      binding.eval(arg_name.to_s)
+    rescue Exception
+      "?"
+    end
     format_obj arg
   end
 

data/lib/pry-moves/painter.rb

@@ -12,10 +12,20 @@ module PryMoves::Painter
 
   end
 
+  class ShortInspector
+    def initialize obj
+      @obj = obj
+    end
+    def inspect
+      @obj.short_inspect
+    end
+  end
+
   def self.colorize(obj)
     colored_str = Canvas.new
     i = obj.inspect
     obj = obj.class if i.is_a?(String) && i.start_with?("#<")
+    obj = ShortInspector.new(obj) if obj.respond_to?(:short_inspect)
     catch (:cut) do
       Pry::ColorPrinter.pp obj, colored_str
     end

data/lib/pry-moves/pry_ext.rb

@@ -42,9 +42,9 @@ Pry::Command.class_eval do
   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
+    run_origin_for_pry_moves command_string, *args
+  ensure
     Pry.config.original_user_input = nil
-    result
   end
 end
 
@@ -86,6 +86,7 @@ Pry::Command::Whereami.class_eval do
 
     formatter = PryMoves::Formatter.new
     prefix = Thread.current[:pry_moves_debug] ? "👾 " : ""
+    # lines << "🍱 #{PryMoves.test_example}" if PryMoves.test_example
     lines << "🦆 step_in_everywhere" if PryMoves.step_in_everywhere
     lines << "#{prefix}#{formatter.shorten_path location}:#{@line} #{me}"
     lines << "   ." + formatter.method_signature(target)
@@ -93,7 +94,11 @@ Pry::Command::Whereami.class_eval do
     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
+    lines.concat(PryMoves.messages.map do |message|
+      message
+      # message.length > MAX_MESSAGE_CHARS ?
+      #   message[0 .. MAX_MESSAGE_CHARS] + "... (cut)" : message
+    end)
     PryMoves.messages.clear
 
     lines << ''

data/lib/pry-moves/pry_wrapper.rb

@@ -44,6 +44,7 @@ class PryWrapper
       nil    # Nothing thrown == no navigational command
     end
 
+  ensure
     PryMoves.is_open = false
     Pry.config.marker = "=>"
   end

data/lib/pry-moves/restartable.rb

@@ -7,13 +7,12 @@ module PryMoves::Restartable
   def restartable context
     trigger :each_new_run, context
     context[:retry] ||= 0
+    PryMoves.reloader&.reload if context[:retry] > 0
     yield context
-    re_execution
+    re_execution # todo: maybe mark restart_request for thread? not globally?
   rescue PryMoves::Restart
     puts "🔄️  Restarting execution"
-    self.restart_requested = false
     PryMoves.reset
-    PryMoves.reloader&.reload
     trigger :restart, context
     context[:retry] += 1
     retry
@@ -23,7 +22,10 @@ module PryMoves::Restartable
   end
 
   def re_execution
-    raise PryMoves::Restart if restart_requested
+    if restart_requested
+      self.restart_requested = false
+      raise PryMoves::Restart
+    end
     raise PryMoves::Reload if reload_requested
   end
 

data/lib/pry-moves/version.rb

@@ -1,3 +1,3 @@
 module PryMoves
-  VERSION = '1.0.3'
+  VERSION = '1.0.5'
 end

data/lib/pry-moves.rb

@@ -7,14 +7,23 @@ module PryMoves
   extend PryMoves::Restartable
 
   attr_accessor :is_open, :trace, :stack_tips,
-    :stop_on_breakpoints, :launched_specs_examples,
+    :stop_on_breakpoints,
+    :test_example, :launched_specs_examples,
     :debug_called_times, :step_in_everywhere
 
+  def loop
+    Kernel.loop do
+      result = yield
+      debug "⏸  execution loop complete\n#{result}"
+      PryMoves.reloader&.reload
+    end
+  end
+
   def init
     reset
     self.trace = true if ENV['TRACE_MOVES']
     self.reload_ruby_scripts = {
-      monitor: %w(app spec),
+      monitor: %w(app test spec),
       except: %w(app/assets app/views)
     }
     self.reloader = CodeReloader.new unless ENV['PRY_MOVES_RELOADER'] == 'off'
@@ -23,13 +32,17 @@ module PryMoves
 
   def reset
     self.launched_specs_examples = 0
-    self.stop_on_breakpoints = true
+    unless ENV['PRY_MOVES'] == 'off' ||
+        (defined?(Rails) and Rails.env.production?)
+      self.stop_on_breakpoints = STDIN.tty? && STDOUT.tty?
+    end
     self.debug_called_times = 0
     self.step_in_everywhere = false
   end
 
-  def debug(message = nil, at: nil, from: nil, options: nil)
+  def debug(message = nil, data: nil, at: nil, from: nil, options: nil)
     pry_moves_stack_end = true
+    message ||= data
     PryMoves.re_execution
     if PryMoves.stop_on_breakpoints
       self.debug_called_times += 1
@@ -45,24 +58,41 @@ module PryMoves
 
   ROOT_DIR = File.expand_path(".")
 
-  def runtime_debug(instance)
+  def runtime_debug(instance, external: false)
     do_debug = (
       stop_on_breakpoints and
         not open? and
-        caller[1].start_with?(ROOT_DIR) and
+        (external or is_project_file?) and
         not [RubyVM::InstructionSequence].include?(instance)
     )
     if do_debug
       hide_from_stack = true
-      err = yield
+      err, obj = yield
       # HINT: when pry failed to start use: caller.reverse
-      PryMoves.debug_error err
+      PryMoves.debug_error err, obj
       true
     end
   end
 
-  def debug_error(message)
+  def is_project_file?
+    files = caller[2..4] # -2 steps upside: runtime_debug, debug sugar function
+    files.any? do |file|
+      !file.start_with?("/") || file.start_with?(ROOT_DIR)
+    end
+  end
+
+  MAX_MESSAGE_CHARS = 520
+  def format_debug_object obj
+    output = obj.ai rescue "#{obj.class} #{obj}"
+    output.length > MAX_MESSAGE_CHARS ?
+      output[0 .. MAX_MESSAGE_CHARS] + "... (cut)" : output
+  end
+
+  def debug_error(message, debug_object=nil)
     pry_moves_stack_end = true
+    if debug_object
+      message = [format_debug_object(debug_object), message].join "\n"
+    end
     debug message, options: {is_error: true}
   end
 

data/lib/sugar/debug_of_missing.rb

@@ -5,30 +5,20 @@ Object.class_eval do
     pry_cancel_debug = true
 
     debug_missing_method = (
-      not ([:begin, :to_s, :to_str, :to_int, :to_ary, :to_io, :to_hash].include? method) and
-        not caller[0].match PryMoves::Backtrace::filter
+      not ([:begin, :to_s, :to_str, :to_int, :to_r, :to_ary, :to_io, :to_hash].include? method)
+      #   not caller[0].match PryMoves::Backtrace::filter
     )
 
     PryMoves.runtime_debug(self) do
       message = self.nil? ?
         "\e[31mCalling \e[1m#{method}\e[0m\e[31m on nil\e[0m" :
         "\e[31mMethod \e[1m#{method}\e[0m\e[31m missing\e[0m"
-      subject = self.ai rescue "#{self.class} #{self}"
-      "#{subject}\n" +
-        "😱  #{message}"
+      [message, self]
     end if debug_missing_method
 
     super
   end
 
-  def should_be *classes
-    hide_from_stack = true
-    if self && !classes.some?{self.is_a?(_1)}
-      error("Expected class #{classes.join ", "}, got #{self.class.ai}", self)
-    end
-    self
-  end
-
   def self.const_missing(name)
     super
   rescue => e
@@ -40,5 +30,5 @@ Object.class_eval do
     raise
   end unless defined?(Rails)
 
-end if ENV['PRY_MOVES_DEBUG_MISSING'] != 'off' and
+end if ENV['PRY_MOVES_DEBUG_MISSING'] != 'off' and ENV['PRY_MOVES'] != 'off' and
   not (defined?(Rails) and Rails.env.production?)

data/lib/sugar/debug_sugar.rb

@@ -1,5 +1,5 @@
 def debug *args
-  return binding.pry_forced if args.first == :forced
+  return binding.pry_forced if [:force, :forced].include? args.first
   pry_moves_stack_end = true
   PryMoves.debug *args
 end
@@ -9,11 +9,9 @@ def error(msg = "Error", debug_object = nil)
   err = "😱  #{msg}"
   unless PryMoves.open?
     if PryMoves.stop_on_breakpoints
-      lines = [err.red]
-      lines.prepend debug_object.ai if debug_object
-      PryMoves.debug_error lines.join("\n")
+      PryMoves.debug_error err.red, debug_object
     else
-      STDERR.puts debug_object.ai if debug_object
+      STDERR.puts PryMoves.format_debug_object(debug_object) if debug_object
       STDERR.puts err.ljust(80, ' ').red
     end
   end
@@ -35,7 +33,15 @@ Object.class_eval do
 
   def required!
     pry_moves_stack_end = true
-    error("required parameter is missing") if self.nil?
+    error("required parameter is missing", self) if self.nil?
+    self
+  end
+
+  def should_be *classes
+    hide_from_stack = true
+    if self && !classes.some?{self.is_a?(_1)}
+      error("Expected class #{classes.join ", "}, got #{self.class.ai}", self)
+    end
     self
   end
 
@@ -55,7 +61,7 @@ RSpec.configure do |config|
     end
   end
 
-end if defined? RSpec
+end if ENV['PRY_MOVES'] != 'off' and defined? RSpec
 
 Rake::Task.class_eval do
 
@@ -63,7 +69,7 @@ Rake::Task.class_eval do
 
   def execute(args=nil)
     args ||= EMPTY_TASK_ARGS
-    PryMoves.restartable(rake_args: args) do |context|
+    PryMoves.restartable(rake_args: args, name: self.name) do |context|
       reload_actions if PryMoves.reload_rake_tasks and context[:retry] > 0
       execute_origin_for_pry_moves args
     end
@@ -75,7 +81,7 @@ Rake::Task.class_eval do
     load rake_task_path
   end
 
-end if defined? Rake and defined? Rake::Task
+end if ENV['PRY_MOVES'] != 'off' and defined? Rake and defined? Rake::Task
 
 Diffy.module_eval do
 

data/publish.sh

@@ -1,3 +1,6 @@
-bundle exec rspec && \
-gem build pry-moves.gemspec && \
+set -e
+set -o pipefail
+
+#bundle exec rspec
+gem build pry-moves.gemspec
 gem push pry-moves-`ruby -e 'require "./lib/pry-moves/version.rb"; puts PryMoves::VERSION'`.gem

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: pry-moves
 version: !ruby/object:Gem::Version
-  version: 1.0.3
+  version: 1.0.5
 platform: ruby
 authors:
 - garmoshka-mo
-autorequire:
+autorequire: 
 bindir: bin
 cert_chain: []
-date: 2022-12-21 00:00:00.000000000 Z
+date: 2023-05-31 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: pry
@@ -179,7 +179,7 @@ homepage: https://github.com/garmoshka-mo/pry-moves
 licenses:
 - MIT
 metadata: {}
-post_install_message:
+post_install_message: 
 rdoc_options: []
 require_paths:
 - lib
@@ -198,7 +198,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
 requirements: []
 rubygems_version: 3.1.6
-signing_key:
+signing_key: 
 specification_version: 4
 summary: Debugger for ruby
 test_files: