byebug-skipper 0.3.4 → 0.3.5

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: acec768ce4a96b1dd844bbc86c6a0f4a41c1ee8cbfbc6448a86a887e888ccba2
-  data.tar.gz: 1a5a1c63a2f6f93accad8b1345bc8446e716bf5a90188f8f35a8e505f6b8c803
+  metadata.gz: 360120962b1a0dc04e8f98ab79c32e4d1c1f590a2382ff4da51dd475f2d8420b
+  data.tar.gz: 58b99f37beab966329e3ac9d2e2d4440d2bbf013b349640b55abf06b41fa24e2
 SHA512:
-  metadata.gz: c1573d50cf409116ac2a8d75b14c9770b792b8eb534d316b89877e8ca8fba0cb8093067f3aedfe9e9a397c8ed7b1813b0dbe94ebfd3a94251ebf04ff3086888c
-  data.tar.gz: 0adf8f2e8c975db29053a63a55c8c901a595037201ba96ac04bd622b6df6bfbad631f23206c584096e49904b1ce2a6585cdf7ca1f48bf4dead3c5f9cd8c9bce7
+  metadata.gz: de210ab93f2b9af685874b06997946b023a4713451443a32562876e7c629f38aa4df17c2b82f708a11526ca17a495a4223c9cea13fab11d548ea62be270e1aab
+  data.tar.gz: 20fba508092216fdbd868353bc6b406aa0fbc3e2a15241cc9391e7ef14265df4be221c0be3b3ad4ef1c0f400a6bfe424e618f0b8a962c8517d1ce9e621a16cf6

data/lib/byebug/skipper.rb

@@ -20,13 +20,19 @@ module Byebug::Skipper
 end
 
 require 'delegate'
+require 'tempfile'
+
 require 'byebug'
-require 'byebug/command'
 require 'byebug/helpers/frame'
+require 'byebug/command'
+require 'byebug/commands/skip'
+
 require_relative 'skipper/ups_command'
 require_relative 'skipper/downs_command'
 require_relative 'skipper/finishs_command'
 require_relative 'skipper/steps_command'
+require_relative 'skipper/comment_line_above'
+require_relative 'skipper/skip_bang_command'
 
 # Command classes need to be in the Byebug module or else they don't get picked
 # up. Cool, bruh.
@@ -35,6 +41,7 @@ require_relative 'skipper/steps_command'
   Byebug::Skipper::DownsCommand,
   Byebug::Skipper::FinishsCommand,
   Byebug::Skipper::StepsCommand,
+  Byebug::Skipper::SkipBangCommand,
 ].each do |command_class|
   Byebug.const_set(
     command_class.name.split('::').last,

data/lib/byebug/skipper/comment_line_above.rb

@@ -0,0 +1,31 @@
+# frozen_string_literal: true
+
+module Byebug::Skipper
+  module CommentLineAbove
+    extend self
+
+    def call(location)
+      path, _, line_no = location.rpartition(':')
+      lines = File.readlines(path)
+
+      idx = Integer(line_no) - 2
+      while ignore_line?(lines[idx])
+        idx -= 1
+        return if idx < 0 # tried to go above the first line of the file, so abort
+      end
+
+      lines[idx] = comment_out(lines.fetch(idx))
+      File.write(path, lines.join)
+    end
+
+    private
+
+      def comment_out(line)
+        line.sub(/\A\s*/, '\0# ')
+      end
+
+      def ignore_line?(line)
+        line.strip.empty? || line.strip.start_with?('#')
+      end
+  end
+end

data/lib/byebug/skipper/pry.rb

@@ -2,18 +2,44 @@ require_relative '../skipper'
 require 'pry-byebug'
 
 module Byebug::Skipper::Pry
-  COMMAND_NAMES = %w[ups downs steps finishs].freeze
-  COMMANDS_THAT_CONTINUE = %w[steps finishs].freeze
+  COMMANDS = [
+    {
+      cmd: 'ups',
+      const_name: 'Ups',
+      continues?: false,
+    },
+    {
+      cmd: 'downs',
+      const_name: 'Downs',
+      continues?: false,
+    },
+    {
+      cmd: 'steps',
+      const_name: 'Steps',
+      continues?: true,
+    },
+    {
+      cmd: 'finishs',
+      const_name: 'Finishs',
+      continues?: true,
+    },
+    {
+      cmd: 'skip!',
+      const_name: 'SkipBang',
+      continues?: true,
+    },
+  ].freeze
 end
 
 # All the Pry command classes are implemented the same
-Byebug::Skipper::Pry::COMMAND_NAMES.each do |command_name|
-  command_class = Class.new(Pry::ClassCommand) do
+Byebug::Skipper::Pry::COMMANDS.each do |command|
+  byebug_command_class = Byebug::Skipper.const_get("#{command.fetch(:const_name)}Command")
+  pry_command_class = Class.new(Pry::ClassCommand) do
     include ::PryByebug::Helpers::Navigation
 
-    match command_name
+    match command.fetch(:cmd)
     group "Byebug Skipper"
-    description "Same as Byebug's `#{command_name.chomp('s')}` command but skips garbage frames (e.g. from gems)"
+    description byebug_command_class.short_description
 
     def process
       PryByebug.check_file_context(target)
@@ -23,8 +49,8 @@ Byebug::Skipper::Pry::COMMAND_NAMES.each do |command_name|
     end
   end
 
-  Byebug::Skipper::Pry.const_set("#{command_name.capitalize}Command", command_class)
-  Pry::Commands.add_command(command_class)
+  Byebug::Skipper::Pry.const_set("#{command.fetch(:const_name)}Command", pry_command_class)
+  Pry::Commands.add_command(pry_command_class)
 end
 
 # This is a monkey patch for Byebug::PryProcessor in order to add extra Byebug
@@ -34,18 +60,17 @@ module Byebug::Skipper::Pry::ProcessorHacks
   def perform(action, options = {})
     # If it's not one of our commands, short circuit and use the typical
     # behaviour.
-    return super unless Byebug::Skipper::Pry::COMMAND_NAMES.include?(action.to_s)
+    command = Byebug::Skipper::Pry::COMMANDS.find { _1.fetch(:cmd) == action.to_s }
+    return super unless command
 
     # Call the Byebug command objects directly. This seems kind of fragile to
     # me, but I don't see any better options.
-    Byebug::Skipper.const_get("#{command_name.capitalize}Command")
-      .new(self, command_name)
+    Byebug::Skipper.const_get("#{command.fetch(:const_name)}Command")
+      .new(self, action.to_s)
       .execute
 
-    unless Byebug::Skipper::Pry::COMMANDS_THAT_CONTINUE.include?(command_name)
-      # This shows the REPL again, preventing execution from continuing.
-      resume_pry
-    end
+    # This shows the REPL again, preventing execution from continuing.
+    resume_pry unless command.fetch(:continues?)
   end
 end
 

data/lib/byebug/skipper/skip_bang_command.rb

@@ -0,0 +1,20 @@
+module Byebug::Skipper
+  class SkipBangCommand < Byebug::SkipCommand
+    def self.regexp
+      /^ \s* skip! \s* $/x
+    end
+
+    def self.short_description
+      "Same as `skip` but also comments out the line before the current one (where `byebug` or `binding.pry` usually is)"
+    end
+
+    def self.description
+      short_description
+    end
+
+    def execute
+      CommentLineAbove.(context.location)
+      super
+    end
+  end
+end

data/lib/byebug/skipper/version.rb

@@ -1,5 +1,5 @@
 module Byebug
   module Skipper
-    VERSION = '0.3.4'
+    VERSION = '0.3.5'
   end
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: byebug-skipper
 version: !ruby/object:Gem::Version
-  version: 0.3.4
+  version: 0.3.5
 platform: ruby
 authors:
 - Tom Dalling
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2021-06-07 00:00:00.000000000 Z
+date: 2021-06-19 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: byebug
@@ -73,9 +73,11 @@ extensions: []
 extra_rdoc_files: []
 files:
 - lib/byebug/skipper.rb
+- lib/byebug/skipper/comment_line_above.rb
 - lib/byebug/skipper/downs_command.rb
 - lib/byebug/skipper/finishs_command.rb
 - lib/byebug/skipper/pry.rb
+- lib/byebug/skipper/skip_bang_command.rb
 - lib/byebug/skipper/steps_command.rb
 - lib/byebug/skipper/ups_command.rb
 - lib/byebug/skipper/version.rb