pry-moves 1.0.3 → 1.0.4

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: cf124819313d4f06ebe4b9d81d1b392cf06e96028ebef3ef67604642f9529c70
-  data.tar.gz: d07e5afae2c9bf9ce303a1731706af1921f79892496d188b9c5bf371f0160ed1
+  metadata.gz: 311be09c5ffb0f48a6aae45f2cf138b4ac8f1c5b285335dd1b358ac5153ca6ab
+  data.tar.gz: 66b1eb2eea22582332db51580159ff3ad470dcdd26c2952b5b8846db4928cfb8
 SHA512:
-  metadata.gz: ea4608c426807fbc02121a5bc32dddb36fe35fdf988872d964fe97424f9ce0d14d1b8bf28dd96012610ffcfdbb7d2567ceb97589a37b12e49a835c40eddc8fbc
-  data.tar.gz: d1fca032e8f5c76bd10012b9a170849238c335238ea1b899577c0d2ccb2109b2578b6f171ebcd2f6465f3fdba247c05feef0c9e0a77f481452dfd4c3c270e591
+  metadata.gz: 55f69a03e9109a9f55c277242318475ef22cbf4776f885f4a2a3d8bab2ec66e83f6896ab305443dc7d5a6797db6bf7bc97644185e98ba18185792161d87dda8d
+  data.tar.gz: ec9e017b99cd33292fd49c5458a9fbe5abb3391a8966032cc9d7aa40f69051fddd83bffbe12b24e1f2e4724458b01893bf8a193934babb3b02e26c1f816b8e3f

data/Gemfile.lock

@@ -1,7 +1,7 @@
 PATH
   remote: .
   specs:
-    pry-moves (1.0.3)
+    pry-moves (1.0.4)
       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/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,8 +112,14 @@ module PryMoves
           return
         end
 
-        input = Pry.config.original_user_input || action
+        input = Pry.config.original_user_input || action.to_s
+        return if input == 'next' # ruby keyword
         binding_value = target.eval(input) rescue nil
+        # begin # for debug
+        #   binding_value = target.eval(input)
+        # rescue => e
+        #   puts (e.backtrace.reverse + ["var_precedence exception:".red, "#{e}".red]).join "\n"
+        # end
         unless binding_value.nil?
           puts "ℹ️️  Variable \"#{input}\" found. To execute command type its alias or \\#{input}"
           puts PryMoves::Painter.colorize binding_value

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/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
 
@@ -93,7 +93,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/restartable.rb

@@ -7,13 +7,13 @@ 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
   rescue PryMoves::Restart
     puts "🔄️  Restarting execution"
     self.restart_requested = false
     PryMoves.reset
-    PryMoves.reloader&.reload
     trigger :restart, context
     context[:retry] += 1
     retry

data/lib/pry-moves/version.rb

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

data/lib/pry-moves.rb

@@ -23,7 +23,8 @@ module PryMoves
 
   def reset
     self.launched_specs_examples = 0
-    self.stop_on_breakpoints = true
+    self.stop_on_breakpoints = true unless ENV['PRY_MOVES'] == 'off' ||
+      (defined?(Rails) and Rails.env.production?)
     self.debug_called_times = 0
     self.step_in_everywhere = false
   end
@@ -45,24 +46,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..3] # -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,22 @@ 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_ary, :to_io, :to_hash].include? 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
     )
 
     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 +32,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

@@ -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
@@ -39,6 +37,14 @@ Object.class_eval do
     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
+
 end
 
 RSpec.configure do |config|
@@ -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
 
@@ -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.4
 platform: ruby
 authors:
 - garmoshka-mo
-autorequire:
+autorequire: 
 bindir: bin
 cert_chain: []
-date: 2022-12-21 00:00:00.000000000 Z
+date: 2022-12-29 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: