hyper-trace 1.0.alpha1.3 → 1.0.alpha1.8

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: cb2488badd7b9e7b836c075f53b027925092f3efa0946805a0f66ca46fbf0a1d
-  data.tar.gz: d8c1cd1adcd5c8874688242c19949e23d56083fd88fa1bbd40153c63c02d655f
+  metadata.gz: 0a22c24fb1118d3a13c8918174b6d16a961dd959938dd26023b0ca721d726046
+  data.tar.gz: 290bc4e8cf9230f37882546edf9ff85da60bd1a098a45f9140294f1df8ae2837
 SHA512:
-  metadata.gz: 9df0bfb46e922b71dd2a1c055d20da85e73fdc6b1a7b5b324f7947440cf898f0b5eed6ae5c4119d2d7a5204634cfa925229ec41d8f5b1f4be64a8fa86822b0fe
-  data.tar.gz: d692b47f002b678012a5710a819bb90b5f583c14bf2afcd64352c3c793029c2613426de64465b459533e0462ea086a3b2cce87596bb6071c4a0559ecad7dfa7c
+  metadata.gz: 5e833fe34affd757cf911425635569fa92fa90c7c0e279d6fcc9942fc2865c731f649742da718bbadb041c4e07cd288eb86103857a21f7f84d761483e6763bbe
+  data.tar.gz: 5c1abe9dad2cc411546e50202f7977de825fcc66d38c4f8ff2a66e55116750d6c6d54ff0ad29e41f27a06f85f689c7fc71e098f5c2b9432ba39faee770018f46

data/.gitignore

@@ -6,3 +6,7 @@
 /pkg/
 /spec/reports/
 /tmp/
+
+# ignore Gemfile.locks https://yehudakatz.com/2010/12/16/clarifying-the-roles-of-the-gemspec-and-gemfile/
+/spec/test_app/Gemfile.lock
+/Gemfile.lock

data/Gemfile

@@ -1,4 +1,5 @@
 source 'https://rubygems.org'
 gem 'hyperstack-config', path: '../hyperstack-config'
+gem 'hyper-spec',   path: '../hyper-spec'
 # Specify your gem's dependencies in hyper-trace.gemspec
 gemspec

data/hyper-trace.gemspec

@@ -19,7 +19,8 @@ Gem::Specification.new do |spec|
   spec.require_paths = ["lib"]
 
   spec.add_dependency 'hyperstack-config', HyperTrace::VERSION
-  spec.add_development_dependency "bundler", ['>= 1.17.3', '< 2.1']
+  spec.add_development_dependency 'hyper-spec', HyperTrace::VERSION
+  spec.add_development_dependency "bundler"
   spec.add_development_dependency 'chromedriver-helper'
-  spec.add_development_dependency "rake", "~> 10.0"
+  spec.add_development_dependency "rake"
 end

data/lib/hyper_trace/hyper_trace.rb

@@ -6,17 +6,7 @@ class Class
   alias hypertrace hyper_trace
 end
 
-class Method
-  def parameters
-    /.*function[^(]*\(([^)]*)\)/
-      .match(`#{@method}.toString()`)[1]
-      .split(',')
-      .collect { |param| [:req, param.strip.to_sym] }
-  end
-end
-
 module HyperTrace
-
   class Config
     def initialize(klass, instrument_class, opts, &block)
       @klass = klass
@@ -116,15 +106,11 @@ module HyperTrace
     def instrumentation_off(config)
       if config.instrument_class?
         config.klass.methods.grep(/^__hyper_trace_pre_.+$/).each do |method|
-          config.klass.class_eval do
-            class << self
-              alias_method method.gsub(/^__hyper_trace_pre_/, ''), method
-            end
-          end
+          config.klass.singleton_class.alias_method method.gsub(/^__hyper_trace_pre_/, ''), method
         end
       else
         config.klass.instance_methods.grep(/^__hyper_trace_pre_.+$/).each do |method|
-          config.klass.class_eval { alias_method method.gsub(/^__hyper_trace_pre_/, ''), method }
+          config.klass.alias_method method.gsub(/^__hyper_trace_pre_/, ''), method
         end
       end
     end
@@ -145,17 +131,12 @@ module HyperTrace
 
     def instrument_method(method, config)
       if config.instrument_class?
-        config.klass.class_eval do
-          class << self
-            alias_method "__hyper_trace_pre_#{method}", method unless method_defined? "__pre_hyper_trace_#{method}"
-          end
+        unless config.klass.singleton_class.method_defined? "__pre_hyper_trace_#{method}"  # is this right?
+          config.klass.singleton_class.alias_method "__hyper_trace_pre_#{method}", method
         end
-        add_hyper_trace_method(method, config)
       else
         unless config.klass.method_defined? "__pre_hyper_trace_#{method}"
-          config.klass.class_eval do
-            alias_method "__hyper_trace_pre_#{method}", method
-          end
+          config.klass.alias_method "__hyper_trace_pre_#{method}", method
         end
       end
       add_hyper_trace_method(method, config)
@@ -194,28 +175,77 @@ module HyperTrace
       end
     end
 
+    def required(arity, actual_length)
+      if arity.negative?
+        req_count = - arity - 1
+        raise ArgumentError, "missing required arguments: #{actual_length} for #{req_count}" if req_count > actual_length
+      else
+        req_count = arity
+        raise ArgumentError, "incorrect number of arguments, expected #{req_count} got #{actual_length}" if req_count != actual_length
+      end
+      req_count
+    end
+
+    # [["req", "x"], ["opt", "y"], ["rest"], ["keyreq", "z"], ["key", "opt"], ["block", "flop"]]
+
+    def map_parameters(instance, name, actual, format = true)
+      method = instance.method("__hyper_trace_pre_#{name}")
+      map_with_specs(method, actual.dup, format) || map_without_specs(actual, format)
+    end
+
+    def key?(actual, key)
+      actual&.last&.respond_to?(:key?) && actual.last.key?(key)
+    end
+
+    def map_with_specs(method, actual, show_blank_rest)
+      specs = method.parameters
+      req_count = required(method.arity, actual.length)
+      args = {}
+
+      specs.each do |type, key|
+        case type
+        when :req
+          args[key] = actual.shift
+          req_count -= 1
+        when :opt
+          args[key] = actual.shift if actual.length > req_count
+        when (key || show_blank_rest) && :rest
+          args[key || '*'] = [*actual[0..-req_count - 1]]
+        when :keyreq
+          raise ArgumentError, "missing keyword argument: #{key}" unless key?(actual, key)
+
+          args[key] = actual.last[key]
+        when key?(actual, key) && :key
+          args[key] = actual.last[key]
+        end
+      end
+      args
+    rescue ArgumentError, Interrupt, SignalException => e
+      raise e
+    rescue Exception => e
+      nil
+    end
+
+    def map_without_specs(actual, format)
+      args = {}
+      prefix = 'p' unless format
+      actual.each_with_index { |value, i| args["#{prefix}#{i + 1}"] = value }
+      args
+    end
+
     def format_head(instance, name, args, &block)
       @formatting = true
-      method = instance.method("__hyper_trace_pre_#{name}")
       if args.any?
         group(" #{name}(...)#{instance_tag(instance)}") do
-          params = method.parameters
-          group("args:", collapsed: true) do
-            params.each_with_index do |param_spec, i|
-              arg_name = param_spec[1]
-              if arg_name == '$a_rest'
-                arg_name = '*'
-                arg = args[i..-1]
-              else
-                arg = args[i]
-              end
-              if safe_i(arg).length > 30 || show_js_object(arg)
-                group "#{arg_name}: #{safe_s(arg)}"[0..29], collapsed: true do
-                  puts safe_i(arg)
-                  log arg if show_js_object(arg)
+          group("args: (#{args.length})", collapsed: true) do
+            map_parameters(instance, name, args).each do |arg, value|
+              if safe_i(value).length > 30 || show_js_object(value)
+                group "#{arg}: #{safe_s(value)}"[0..29], collapsed: true do
+                  puts safe_i(value)
+                  log value if show_js_object(value)
                 end
               else
-                group "#{arg_name}: #{safe_i(arg)}"
+                group "#{arg}: #{safe_i(value)}"
               end
             end
           end
@@ -286,7 +316,10 @@ module HyperTrace
       @formatting = true
       if safe_i(result).length > 40
         group "raised: #{safe_s(result)}"[0..40], collapsed: true do
-          puts safe_i(result)
+          puts result.backtrace[0]
+          result.backtrace[1..-1].each do |line|
+            log line
+          end
         end
       else
         group "raised: #{safe_i(result)}"
@@ -305,24 +338,23 @@ module HyperTrace
 
     def breakpoint(location, config, name, args, instance, result = nil)
       if should_break? location, config, name, args, instance, result
-        method = instance.method("__hyper_trace_pre_#{name}")
+        params = map_parameters(instance, name, args, false)
         fn_def = ['RESULT']
-        fn_def += method.parameters.collect { |p| p[1] }
+        fn_def += params.keys
         fn_def += ["//break on #{location} of #{name}\nvar self = this;\ndebugger;\n;"]
         puts "break on #{location} of #{name}"
         fn = `Function.apply(#{self}, #{fn_def}).bind(#{instance})`
-        fn.call(result, *args)
+        fn.call(result, *params.values)
       end
     end
 
-    def call_original(instance, method, *args, &block)
+    def call_original(instance, method, args, &block)
       @formatting = false
       instance.send "__hyper_trace_pre_#{method}", *args, &block
     ensure
       @formatting = true
     end
 
-
     def add_hyper_trace_method(method, config)
       def_method = config.instrument_class? ? :define_singleton_method : :define_method
       config.klass.send(def_method, method) do |*args, &block|
@@ -330,7 +362,7 @@ module HyperTrace
         if HyperTrace.formatting?
           begin
             send "__hyper_trace_pre_#{method}", *args, &block
-          rescue Exception
+          rescue StandardError
             "???"
           end
         else
@@ -338,11 +370,13 @@ module HyperTrace
             HyperTrace.format_head(self, method, args) do
               HyperTrace.format_instance_internal(self)
               HyperTrace.breakpoint(:enter, config, method, args, self)
-              result = HyperTrace.call_original self, method, *args, &block
+              result = HyperTrace.call_original self, method, args, &block
               HyperTrace.format_result(result)
               HyperTrace.breakpoint(:exit, config, method, args, self, result)
               result
             end
+          rescue Interrupt, SignalException => e
+            raise e
           rescue Exception => e
             HyperTrace.format_exception(e)
             debugger unless HyperTrace.exclusions[self.class][:rescue].include? :method

data/lib/hyper_trace/version.rb

@@ -1,3 +1,3 @@
 module HyperTrace
-  VERSION = '1.0.alpha1.3'
+  VERSION = '1.0.alpha1.8'
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: hyper-trace
 version: !ruby/object:Gem::Version
-  version: 1.0.alpha1.3
+  version: 1.0.alpha1.8
 platform: ruby
 authors:
 - catmando
 autorequire: 
 bindir: exe
 cert_chain: []
-date: 2019-01-17 00:00:00.000000000 Z
+date: 2021-04-13 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: hyperstack-config
@@ -16,34 +16,42 @@ dependencies:
     requirements:
     - - '='
       - !ruby/object:Gem::Version
-        version: 1.0.alpha1.3
+        version: 1.0.alpha1.8
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - '='
       - !ruby/object:Gem::Version
-        version: 1.0.alpha1.3
+        version: 1.0.alpha1.8
+- !ruby/object:Gem::Dependency
+  name: hyper-spec
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - '='
+      - !ruby/object:Gem::Version
+        version: 1.0.alpha1.8
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - '='
+      - !ruby/object:Gem::Version
+        version: 1.0.alpha1.8
 - !ruby/object:Gem::Dependency
   name: bundler
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: 1.17.3
-    - - "<"
-      - !ruby/object:Gem::Version
-        version: '2.1'
+        version: '0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: 1.17.3
-    - - "<"
-      - !ruby/object:Gem::Version
-        version: '2.1'
+        version: '0'
 - !ruby/object:Gem::Dependency
   name: chromedriver-helper
   requirement: !ruby/object:Gem::Requirement
@@ -62,16 +70,16 @@ dependencies:
   name: rake
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
       - !ruby/object:Gem::Version
-        version: '10.0'
+        version: '0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
       - !ruby/object:Gem::Version
-        version: '10.0'
+        version: '0'
 description: 
 email:
 - mitch@catprint.com
@@ -83,7 +91,6 @@ files:
 - CODE_OF_CONDUCT.md
 - DOCS.md
 - Gemfile
-- Gemfile.lock
 - Rakefile
 - hyper-trace.gemspec
 - lib/hyper-trace.rb
@@ -109,7 +116,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: 1.3.1
 requirements: []
-rubygems_version: 3.0.2
+rubygems_version: 3.0.8
 signing_key: 
 specification_version: 4
 summary: Method tracing and conditional breakpoints for Opal Ruby

data/Gemfile.lock

@@ -1,73 +0,0 @@
-PATH
-  remote: ../hyperstack-config
-  specs:
-    hyperstack-config (1.0.alpha1.3)
-      listen (~> 3.0)
-      mini_racer (~> 0.2.4)
-      opal (>= 0.11.0, < 0.12.0)
-      opal-browser (~> 0.2.0)
-      uglifier
-      websocket
-
-PATH
-  remote: .
-  specs:
-    hyper-trace (1.0.alpha1.3)
-      hyperstack-config (= 1.0.alpha1.3)
-
-GEM
-  remote: https://rubygems.org/
-  specs:
-    archive-zip (0.11.0)
-      io-like (~> 0.3.0)
-    ast (2.4.0)
-    chromedriver-helper (2.1.0)
-      archive-zip (~> 0.10)
-      nokogiri (~> 1.8)
-    execjs (2.7.0)
-    ffi (1.10.0)
-    hike (1.2.3)
-    io-like (0.3.0)
-    libv8 (6.7.288.46.1)
-    listen (3.1.5)
-      rb-fsevent (~> 0.9, >= 0.9.4)
-      rb-inotify (~> 0.9, >= 0.9.7)
-      ruby_dep (~> 1.2)
-    mini_portile2 (2.3.0)
-    mini_racer (0.2.4)
-      libv8 (>= 6.3)
-    nokogiri (1.8.5)
-      mini_portile2 (~> 2.3.0)
-    opal (0.11.4)
-      ast (>= 2.3.0)
-      hike (~> 1.2)
-      parser (= 2.3.3.1)
-      sourcemap (~> 0.1.0)
-    opal-browser (0.2.0)
-      opal
-      paggio
-    paggio (0.2.6)
-    parser (2.3.3.1)
-      ast (~> 2.2)
-    rake (10.5.0)
-    rb-fsevent (0.10.3)
-    rb-inotify (0.10.0)
-      ffi (~> 1.0)
-    ruby_dep (1.5.0)
-    sourcemap (0.1.1)
-    uglifier (4.1.20)
-      execjs (>= 0.3.0, < 3)
-    websocket (1.2.8)
-
-PLATFORMS
-  ruby
-
-DEPENDENCIES
-  bundler (>= 1.17.3, < 2.1)
-  chromedriver-helper
-  hyper-trace!
-  hyperstack-config!
-  rake (~> 10.0)
-
-BUNDLED WITH
-   2.0.1