source_route 0.1.8 → 0.2.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 3b4bfd9535b257febd357416089fee4ff68b38b2
-  data.tar.gz: f2c7a60553434e0a0430f9919764f0d97dcd5780
+  metadata.gz: a900afbf3890bf31f01041e72ac6a65f61f3a477
+  data.tar.gz: 85e3915fb47c64ac5cbbe9312c2435a4720758f2
 SHA512:
-  metadata.gz: 5d551e9a2364336fb6603633b0da2748eb3e27b2f0eba4b51bdaaffcf579af2956e269536a1e4d7cf7f47b3b05c1821fca533fb716d2d7888e1216472e96c436
-  data.tar.gz: d4b009feb8524193ad9d90776b60ff8a37213b45189838e4ce910975e697c720e0f9c9e3e01b6729b5db2130286480e63b437ee325d5972cbd5e2ebfa1adf6da
+  metadata.gz: 91e0b7a0a43e00dddf0d1b8f278324c197f9d62179c066d490c8049913c7eb19e32bb500966938ae2914a5b76541caa77e84508da96b9454dd5a667bcfdeb8ec
+  data.tar.gz: bdc789e2660545ec2b6c0aaac975d2b276bbaf618d4414d46a944fafe383eccd8e9ba1cf38b92a6564541d4ff69e2ce246f3a00da3047cf499f63d695db470b7

data/Guardfile

@@ -0,0 +1,23 @@
+# A sample Guardfile
+# More info at https://github.com/guard/guard#readme
+
+## Uncomment and set this to only include directories you want to watch
+# directories %w(lib) \
+#  .select{|d| Dir.exists?(d) ? d : UI.warning("Directory #{d} does not exist")}
+
+## Note: if you are using the `directories` clause above and you are not
+## watching the project directory ('.'), then you will want to move
+## the Guardfile to a watched dir and symlink it back, e.g.
+#
+#  $ mkdir config
+#  $ mv Guardfile config/
+#  $ ln -s config/Guardfile .
+#
+# and, you'll have to watch "config/Guardfile" instead of "Guardfile"
+guard :shell do
+  watch(%r{^lib/(.+)/*$}) { `rake` }
+end
+
+guard :livereload, grace_period: 0.5, override_url: true do
+  watch(%r{.+\.(css|js|html)})
+end

data/README.md

@@ -62,7 +62,7 @@ It will output the trace when you run the application.
 see more usage in examples.
 see full usage in examples/study_callback.rb
 
-![Study Callback Example](https://cloud.githubusercontent.com/assets/490502/5468821/decbb8fe-860e-11e4-9b74-7aef7f8679fd.png)
+![Study Callback Example](https://cloud.githubusercontent.com/assets/490502/9052549/746b36d2-3a9b-11e5-9e3e-fc9f149bc56c.png)
 
 ## Why
 
@@ -93,12 +93,9 @@ Finally, I expect my working style can changes from searching workaround from in
 
 ### TODO
 
-Dynamic indent when new child level comes.
-
-Is it possible to easily open only direct child
-Animation when child was click.
+Cleanup Code and remove useless comments.
 
-Show loading when click clear. (when $apply() take more than 1 seconds to run)
+Dynamic indent when new child level comes.
 
 Add debug option to provide more verbose messages of what has happened
 

data/Rakefile

@@ -2,7 +2,7 @@ require "bundler/gem_tasks"
 
 task default: :test
 task :test do
-  ENV['ignore_html_generation'] = 'true'
+  ENV['ignore_html_generation'] = 'false'
   $LOAD_PATH.unshift('lib', 'test')
   Dir.glob('./test/**/*_test.rb') { |f| require f }
 end

data/examples/debug_how_logger_object_init_when_rails_start.rb

@@ -0,0 +1,23 @@
+#!/usr/bin/env ruby
+
+# usage: cover bin/rails with this file and run bundle exec rails c to start your console then exit your console
+
+require 'rails'
+
+require 'source_route'
+
+SourceRoute.enable do
+  defined_class 'ActiveSupport::Configurable'
+  method_id :logger
+  full_feature 10
+end
+
+APP_PATH = File.expand_path('../../config/application',  __FILE__)
+
+require_relative '../config/boot'
+
+require 'rails/commands'
+
+at_exit do
+  SourceRoute.output_html
+end

data/examples/show_init_files_when_rails_server_start.rb

@@ -31,3 +31,5 @@ end
 APP_PATH = File.expand_path('../../config/application',  __FILE__)
 require_relative '../config/boot'
 require 'rails/commands'
+
+SourceRoute.output_html

data/examples/show_task_trace_in_rails.rb

@@ -12,5 +12,5 @@ end
 Rails.application.load_tasks
 
 at_exit do
-  SourceRoute.build_html_output
+  SourceRoute.output_html
 end

data/lib/source_route/core_ext.rb

@@ -1,12 +1,6 @@
 class Object
   def source_route_display
-    to_s
-  end
-
-end
-
-class Numeric
-  def source_route_display
+    # to_s # cause crash?
     self
   end
 end

data/lib/source_route/formats/html.rb

@@ -7,17 +7,17 @@ module SourceRoute
 
       def self.slim_render(wrapper)
         result_config = wrapper.condition.result_config
-        template_path = File.expand_path "../html_template.slim", __FILE__
+        template_path = File.expand_path "../html_semantic.slim", __FILE__
         slim_template = Slim::Template.new(template_path, pretty: true)
 
         filename = result_config[:filename] || "#{Time.now.strftime('%H')}-source-route.html"
 
         if result_config.import_return_to_call and wrapper.condition.has_call_and_return_event
-          wrapper.import_return_value_to_call_chain
-          wrapper.treeize_call_chain
+          wrapper.result_builder.import_return_value_to_call_chain
+          wrapper.result_builder.treeize_call_chain
         end
         # TODO: any exception triggered in render method will be absorb totally, how to fix it?
-        html_output_str = slim_template.render(wrapper)
+        html_output_str = slim_template.render(wrapper.result_builder)
         File.open(filename, 'w') do |f|
           f << html_output_str
         end

data/lib/source_route/formats/html_semantic.slim

@@ -0,0 +1,242 @@
+doctype html
+html
+  head
+    title Source Route Result
+    link rel="stylesheet" href="https://cdn.rawgit.com/Urigo/angular-spinkit/master/build/angular-spinkit.min.css"
+    link rel="stylesheet" href="https://cdn.rawgit.com/mohsen1/json-formatter/master/dist/json-formatter.css"
+    link rel="stylesheet" href="http://oss.maxcdn.com/semantic-ui/2.0.7/semantic.min.css"
+
+    script async=true src="http://cdnjs.cloudflare.com/ajax/libs/lodash.js/3.10.0/lodash.js"
+    script src="http://cdnjs.cloudflare.com/ajax/libs/jquery/2.1.4/jquery.min.js"
+    script src="https://ajax.googleapis.com/ajax/libs/angularjs/1.4.3/angular.min.js"
+    script async=true src="http://oss.maxcdn.com/semantic-ui/2.0.7/semantic.min.js"
+
+    css:
+      .call-level-0 {}
+      .item.trace.call-level-1 { padding-left: 50px }
+      .item.trace.call-level-2 { padding-left: 100px }
+      .item.trace.call-level-3 { padding-left: 150px }
+      .item.trace.call-level-4 { padding-left: 200px }
+      .item.trace.call-level-5 { padding-left: 250px }
+      .item.trace.call-level-6 { padding-left: 300px }
+      .item.trace.call-level-7 { padding-left: 300px }
+      // level more than 7 seems not reasonable
+
+  body(ng-app="SourceRoute" ng-controller="MainCtrl" ng-cloak)
+
+    .ui.menu.pointing.stackable.attached
+      .ui.container
+        .item
+          a.navbar-brand(href="#" ng-click="::resetTraceFilter()") ALL
+        .item(ng-repeat="event in tpEvents" ng-class="{active: event == traceFilter.event}")
+          a(href="#" ng-click="traceFilter.event = event" ng-bind="::event")
+        .item
+          .ui.buttons
+            button.ui.labeled.icon.button.olive(ng-click="outlineTrace()" ng-class="{loading: outlineTraceLoading}")
+              i.angle.double.right.icon
+              span OutLine
+            .or
+            button.ui.right.labeled.icon.button.green(ng-click="expandAllTrace()" ng-class="{loading: expandAllTraceLoading}")
+              i.angle.double.down.icon
+              span Expand
+        .right.menu
+          .item
+            span Trace Count
+            .ui.teal.pointing.left.label(ng-bind="currentCounter()")
+
+    .ui.container
+      .row
+      .ui.relaxed.items(ng-class="{{traceFilter.event}}")
+        .item.trace(ng-repeat="trace in traces | filter:traceFilter:true | filter:childParentFilterFn" ng-class="callLevelClass(trace)" ng-controller="TpTraceCtrl")
+          .content(ng-init="showMoreDetail = false")
+            .ui.segment.left.floated.padded
+              .header
+                span(ng-bind="::tpSelfList[trace.tp_self_refer]")
+                i.circle.icon.grey(style="font-size: 0.3em")
+                span.method-value(ng-bind="::trace.method_id")
+              / workaround for return_value is 'false' and return_value always to be string when existed
+              .meta(ng-if="trace.hasOwnProperty('return_value')")
+                i.icon.pointing.right.small
+                json-formatter(open="1" json="::trace.return_value")
+            .description
+              .ui.vertical.buttons.basic
+                button.ui.labeled.icon.button(ng-show="::containsDetail(trace)" ng-click="showMoreDetail = !showMoreDetail")
+                  i.info.icon
+                  span Details
+                button.ui.labeled.icon.button(ng-if="::hasChild()" ng-click="toggleChild()" ng-class="{loading: togglingChild}")
+                  i.icon.angle.down(ng-show="trace.childOpened")
+                  i.icon.angle.right(ng-hide="trace.childOpened")
+                  span Child
+                pulse-spinner(ng-show="togglingChild")
+
+              .details.right.floated(ng-if="showMoreDetail")
+                .ui.segments(style="border-color: blue")
+                  .ui.segment(ng-if="::trace.hasOwnProperty('return_value')")
+                    .ui.grey.right.ribbon.label Return Value
+                    json-formatter(json="::trace.return_value")
+                  .ui.segment(ng-if="::containsOtherAttrs(trace)")
+                    .ui.orange.right.ribbon.label Trace Attrs
+                    json-formatter(open="1" json="::plusTraceAttrs[trace.order_id]")
+                  .ui.segment(ng-if="::trace.params_var")
+                    .ui.teal.right.ribbon.label Params Var
+                    json-formatter(open="1" json="::trace.params_var")
+                  .ui.segment(ng-if="::trace.local_var")
+                    .ui.teal.right.ribbon.label Local Var
+                    json-formatter(open="1" json="::trace.local_var")
+                  .ui.segment(ng-if="::trace.instance_var")
+                    .ui.blue.right.ribbon.label Instance Var
+                    json-formatter(open="1" json="::trace.instance_var")
+
+    script src="https://cdn.rawgit.com/Urigo/angular-spinkit/master/build/angular-spinkit.min.js"
+    script src="https://cdn.rawgit.com/mohsen1/json-formatter/master/dist/json-formatter.js"
+    // script async src="https://ajax.googleapis.com/ajax/libs/angularjs/1.4.3/angular-sanitize.js"
+
+    javascript:
+      sourceRoute = angular.module('SourceRoute', ['jsonFormatter', 'angular-spinkit'])
+      sourceRoute.controller('MainCtrl', function($scope, $filter, $timeout) {
+
+        // setup different color on menu item may not be a good solution
+        // $scope.menuColorList = ['yellow', 'olive', 'green', 'teal', 'violet', 'purple', 'brown']
+        $scope.trimString = function(str, length) {
+          return str.length > length ? str.substring(0, length - 3) + '...' : str;
+        }
+
+        $scope.traces = angular.element("#trace-data").data('trace')
+        $scope.tpSelfList = angular.element("#trace-data").data('tp-self-caches')
+        $scope.tpEvents = angular.element("#trace-data").data('tp-events')
+        $scope.childParentFilter = { hide_trace_ids: [] }
+
+        $scope.childParentFilterFn = function(trace) {
+          if (!trace.hasOwnProperty('parent_ids')) {
+            return true;
+          }
+          if (trace.parent_ids.length == 0) {
+            return true;
+          } else {
+            var shared_hide_parents = _.intersection(trace.parent_ids, $scope.childParentFilter.hide_trace_ids);
+            if (shared_hide_parents.length > 0 ) {
+              return false;
+            }
+          }
+          return true;
+        }
+
+        $scope.expandAllTrace = function() {
+          $scope.expandAllTraceLoading = true
+          $timeout(function() {
+            _.each($scope.traces, function(trace) { trace.childOpened = true; });
+            $scope.childParentFilter.hide_trace_ids = [];
+            $scope.expandAllTraceLoading = false
+            $scope.expandAllTraceLoading = false
+          }, 100)
+        }
+
+        $scope.outlineTrace = function() {
+          $scope.outlineTraceLoading = true
+          $scope.childParentFilter.hide_trace_ids = [];
+          _.chain($scope.traces).filter({parent_length: 0, event: $scope.traceFilter.event})
+            .each(function(trace) {
+              trace.childOpened = false;
+              $scope.childParentFilter.hide_trace_ids.push(trace.order_id) }
+            ).value()
+          $scope.outlineTraceLoading = false
+        }
+
+        $scope.traceFilter = {event: $scope.tpEvents[0]}
+        if ($scope.tpEvents.length == 1 && angular.isUndefined($scope.traces[0].event)) {
+          _.each($scope.traces, function(trace) {
+            trace.event = $scope.tpEvents[0]
+          })
+        }
+
+        $scope.definedClasses = _.uniq(_.map($scope.traces, 'defined_class'))
+
+        $scope.callLevelClass = function(trace) {
+          if (trace.parent_length > 7) {
+            return 'call-level-7';
+          } else {
+            return 'call-level-' + trace.parent_length;
+          }
+        }
+
+        $scope.resetTraceFilter = function() {
+          $scope.traceFilter = {};
+        }
+
+        $scope.currentCounter = function() {
+          return $filter('filter')($scope.traces, $scope.traceFilter, true).length;
+        }
+
+        $scope.containsOtherAttrs = function(trace) {
+          return trace.hasOwnProperty('path') || trace.hasOwnProperty('lineno')
+        }
+
+        $scope.containsDetail = function(trace) {
+          return $scope.containsOtherAttrs(trace) || trace.hasOwnProperty('local_var') ||
+            trace.hasOwnProperty('instance_var')
+        }
+
+        combinedAttrs = function(trace) {
+          var attrs = {}
+          if (trace.hasOwnProperty('lineno') && trace.hasOwnProperty('path')) {
+            attrs.method_defined_on = trace.path + ":" + trace.lineno;
+          } else if (trace.hasOwnProperty('path')) {
+            attrs.method_defined_on = trace.path;
+          }
+          if (trace.hasOwnProperty('defined_class')) {
+            attrs.method_defined_in = trace.defined_class;
+          }
+          return attrs;
+        }
+
+        $scope.plusTraceAttrs = _.map($scope.traces, combinedAttrs)
+
+        $scope.outlineTrace();
+      })
+
+      sourceRoute.controller('TpTraceCtrl', function($scope, $timeout) {
+
+        $scope.toggleChild = function() {
+          $scope.togglingChild = true
+          $timeout(function() {
+            if ($scope.trace.childOpened) {
+              $scope.hideChild();
+              $scope.togglingChild = false
+            } else {
+              $scope.showChild();
+              $scope.togglingChild = false
+            }
+          }, 0)
+        }
+
+        $scope.showChild = function() {
+          $scope.trace.childOpened = true;
+          _.pull($scope.childParentFilter.hide_trace_ids, $scope.trace.order_id);
+          if ($scope.trace.direct_child_order_ids.length > 0) {
+            $scope.childParentFilter.hide_trace_ids.push($scope.trace.direct_child_order_ids);
+            $scope.childParentFilter.hide_trace_ids = _.chain($scope.childParentFilter.hide_trace_ids).flatten().uniq().value();
+          }
+        }
+
+        $scope.hideChild = function() {
+          $scope.trace.childOpened = false;
+          $scope.childParentFilter.hide_trace_ids.push($scope.trace.order_id);
+          if ($scope.trace.direct_child_order_ids.length > 0) {
+            _.each($scope.trace.direct_child_order_ids, function(ele) {
+              _.pull($scope.childParentFilter.hide_trace_ids, ele);
+            });
+          }
+        }
+
+        $scope.hasChild = function() {
+          return _.find($scope.traces, function(trace) {
+            return _.contains(trace.parent_ids, $scope.trace.order_id)
+          });
+        }
+      })
+
+    .data-collect
+      / dont use local_trace_data.to_json, because ActiveSupport override it and can introduce unexpected crash for some data
+      #trace-data(data-trace="#{jsonify_tp_result_chain}"
+        data-tp-events="#{jsonify_events}"
+        data-tp-self-caches="#{jsonify_tp_self_caches}")

data/lib/source_route/generate_result.rb

@@ -1,11 +1,23 @@
 module SourceRoute
-
+  # How it work
+  # 0. Config collect route options
+  # 1. Proxy generate TracePoint Filter
+  # 2. Proxy generate TracePoint Monitor Block
+  # 3. Generator collect Wanted TracePoint
+  # 4. Parse and Generate Useful data from wanted TracePoint
+  # 5. Output data with correct format
   class GenerateResult
 
+    attr_reader :tp_result_chain, :tp_self_caches, :collected_data
+
+    extend Forwardable
+
+    def_delegators :@tp_result_chain, :import_return_value_to_call_chain, :treeize_call_chain, :call_chain, :return_chain, :parent_length_list
+
     Config = Struct.new(:format, :show_additional_attrs,
                         :include_local_var, :include_instance_var,
                         :filename, :import_return_to_call) do
-      def initialize(f="silence", s=[], ilr=false, iiv=false)
+      def initialize(f=:test, s=[], ilr=false, iiv=false)
         super(f, s, ilr, iiv)
       end
     end
@@ -32,39 +44,48 @@ module SourceRoute
 
       @config = @wrapper.condition.result_config
 
-      @tp_events = @wrapper.condition.events
+      @tp_result_chain = TpResultChain.new
+
+      @tp_self_caches = []
+      @wanted_attributes = {}
     end
 
-    def output_attributes(event)
-      attrs = DEFAULT_ATTRS[event] + Array(@config.show_additional_attrs)
-      attrs.push(:event) if @tp_events.size > 1
-      attrs.uniq
+    # it cached and only calculate once for one trace point block round
+    def self.wanted_attributes(eve)
+      event = eve.to_sym
+      @wanted_attributes.fetch event do
+        attrs = DEFAULT_ATTRS[event] + Array(SourceRoute.wrapper.condition.result_config.show_additional_attrs)
+        attrs.push(:event)
+        @wanted_attributes[event] = attrs.uniq
+        @wanted_attributes[event]
+      end
     end
 
-    def build(trace_point_instance)
-      @tp = trace_point_instance
-      collect_tp_data
-      collect_tp_self # NEED more check. Does TracePoint support self for all events?
-      collect_local_var_data if @config[:include_local_var]
-      collect_instance_var_data if @config[:include_instance_var]
-      @collect_data
+    def self.clear_wanted_attributes
+      @wanted_attributes = {}
     end
 
     def output(tp_ins)
-
       format = @config.format
       format = format.to_sym if format.respond_to? :to_sym
 
+      assign_tp_self_caches(tp_ins)
+      # we cant call method on tp_ins outside of track block,
+      # so we have to run it immediately
+
+      @collected_data = TpResult.new(tp_ins)
+
       case format
-      when :none
-        # do nothing
-      when :console # need @collect_data
-        console_put
+      when :console
+        console_put(tp_ins)
       when :html
-        # I cant solve the problem: to generate html at the end,
-        # I have to know when the application is end
-      when :test, :silence
-        # do nothing at now
+        # we cant generate html right now becase the tp collection is still in process
+        # so we collect tp here
+        @tp_result_chain.push(TpResult.new(tp_ins))
+      when :silence, :none
+      # do nothing at now
+      when :test
+        @tp_result_chain.push(TpResult.new(tp_ins))
       when :stack_overflow
         console_stack_overflow
       when Proc
@@ -75,52 +96,94 @@ module SourceRoute
       end
     end
 
-    private
+    # def build(trace_point_instance)
+    #   TpResult.new(trace_point_instance)
+    #   # tp_result.collect_self
 
-    def collect_tp_data
-      tp_data = output_attributes(@tp.event).inject({}) do |memo, key|
-        memo[key.to_sym] = @tp.send(key) if @tp.respond_to?(key)
-        memo
-      end
-      @collect_data = TpResult.new(tp_data)
-      puts @collect_data.inspect if @wrapper.condition.is_debug?
-    end
+    #   # @tp = trace_point_instance
+    #   # collect_tp_data
+    #   # collect_tp_self # NEED more check. Does TracePoint support self for all events?
+    #   # collect_local_var_data if @config[:include_local_var]
+    #   # collect_instance_var_data if @config[:include_instance_var]
+    #   # @collected_data
+    # end
+
+    # def collect_tp_result
+    #   tp_result = TpResult.new(tp)
+    # end
 
     # include? will evaluate @tp.self, if @tp.self is AR::Relation, it could cause problems
     # So that's why I use object_id as replace
-    def collect_tp_self
-      unless @wrapper.tp_self_caches.find { |tp_cache| tp_cache.equal? @tp.self }
-        @wrapper.tp_self_caches.push @tp.self
+    def assign_tp_self_caches(tp_ins)
+      unless tp_self_caches.find { |tp_cache| tp_cache.object_id.equal? tp_ins.self.object_id }
+        tp_self_caches.push tp_ins.self
       end
-      @collect_data[:tp_self] = @wrapper.tp_self_caches.map(&:__id__).index(@tp.self.__id__)
     end
 
-    def collect_local_var_data
-      local_var_hash = {}
+    def jsonify_events
+      Oj.dump(@wrapper.condition.events.map(&:to_s))
+    end
 
-      # Warn: @tp.binding.eval('local_variables') =! @tp.binding.send('local_variables')
-      @tp.binding.eval('local_variables').each do |v|
-        local_var_hash[v] = @tp.binding.local_variable_get(v).source_route_display
-      end
-      if local_var_hash != {}
-        @collect_data.merge!(local_var: local_var_hash)
-      end
+    def jsonify_tp_result_chain
+      Oj.dump(tp_result_chain.chain.map(&:to_hash))
+      # tp_result_chain.to_json
+      # json_array = tp_result_chain.map { |result| Jsonify.dump(result) }
+      # '[ ' + json_array.join(',') + ' ]'
     end
 
-    def collect_instance_var_data
-      instance_var_hash = {}
-      @tp.self.instance_variables.each do |key|
-        instance_var_hash[key] = @tp.self.instance_variable_get(key).source_route_display
-      end
-      if instance_var_hash != {}
-        @collect_data.merge!(instance_var: instance_var_hash)
-      end
+    def jsonify_tp_self_caches
+      Oj.dump(tp_self_caches.clone
+               .map(&:to_s))
     end
 
-    def console_put
+    private
+
+    # def collect_tp_data
+    #   tp_data = wanted_attributes(@tp.event).inject({}) do |memo, key|
+    #     memo[key.to_sym] = @tp.send(key) if @tp.respond_to?(key)
+    #     memo
+    #   end
+    #   @collected_data = TpResult.new(tp_data)
+    #   puts @collected_data.inspect if @wrapper.condition.is_debug?
+    # end
+
+    # def collect_tp_self(tp)
+    #   unless tp_self_caches.find { |tp_cache| tp_cache.object_id.equal? tp.self.object_id }
+    #     tp_self_caches.push tp.self
+    #   end
+    #   @collected_data[:tp_self_refer] = tp_self_caches.map(&:__id__).index(tp.self.__id__)
+    # end
+
+    # def collect_local_var_data
+    #   local_var_hash = {}
+    #   # Warn: @tp.binding.eval('local_variables') =! @tp.binding.send('local_variables')
+    #   @tp.binding.eval('local_variables').each do |v|
+    #     # I need rememeber why i need source_route_display
+    #     local_var_hash[v] = @tp.binding.local_variable_get(v).source_route_display
+    #   end
+    #   if local_var_hash != {}
+    #     @collected_data.merge!(local_var: local_var_hash)
+    #   end
+    # end
+
+    # def collect_instance_var_data
+    #   instance_var_hash = {}
+    #   @tp.self.instance_variables.each do |key|
+    #     instance_var_hash[key] = @tp.self.instance_variable_get(key).source_route_display
+    #   end
+    #   if instance_var_hash != {}
+    #     @collected_data.merge!(instance_var: instance_var_hash)
+    #   end
+    # end
+
+    def console_put(tp)
       ret = []
-      ret << "#{@collect_data[:defined_class].inspect}##{@collect_data[:method_id]}"
-      left_values = @collect_data.reject { |k, v| %w[defined_class method_id].include? k.to_s }
+      ret << "#{collected_data.defined_class.inspect}##{collected_data.method_id}"
+      left_attrs = self.class.wanted_attributes(tp.event) - [:defined_class, :method_id]
+      left_values = left_attrs.inject({}) do |memo, key|
+        memo[key] = collected_data.send(key)
+        memo
+      end
       unless left_values == {}
         ret << left_values
       end
@@ -128,7 +191,7 @@ module SourceRoute
     end
 
     def console_stack_overflow
-      ap "#{@collect_data[:defined_class].inspect}##{@collect_data[:method_id]}"
+      ap "#{collected_data.defined_class.inspect}##{collected_data.method_id}"
     end
 
   end # END GenerateResult

data/lib/source_route/jsonify.rb

@@ -4,7 +4,8 @@ module Jsonify
     if obj.respond_to? :stringify
       JSON.dump(obj.stringify)
     else
-      JSON.dump(obj.to_s)
+      # JSON.dump(obj.to_s)
+      JSON.dump(obj.inspect)
     end
   end