debug 1.3.4 → 1.4.0

data/lib/debug/server_cdp.rb

@@ -4,20 +4,178 @@ require 'json'
 require 'digest/sha1'
 require 'base64'
 require 'securerandom'
+require 'stringio'
+require 'open3'
+require 'tmpdir'
 
 module DEBUGGER__
   module UI_CDP
     SHOW_PROTOCOL = ENV['RUBY_DEBUG_CDP_SHOW_PROTOCOL'] == '1'
 
-    class WebSocket
+    class << self
+      def setup_chrome addr
+        return if CONFIG[:chrome_path] == ''
+
+        port, path, pid = run_new_chrome
+        begin
+          s = Socket.tcp '127.0.0.1', port
+        rescue Errno::ECONNREFUSED, Errno::EADDRNOTAVAIL
+          return
+        end
+
+        ws_client = WebSocketClient.new(s)
+        ws_client.handshake port, path
+        ws_client.send id: 1, method: 'Target.getTargets'
+
+        4.times do
+          res = ws_client.extract_data
+          case
+          when res['id'] == 1 && target_info = res.dig('result', 'targetInfos')
+            page = target_info.find{|t| t['type'] == 'page'}
+            ws_client.send id: 2, method: 'Target.attachToTarget',
+                          params: {
+                            targetId: page['targetId'],
+                            flatten: true
+                          }
+          when res['id'] == 2
+            s_id = res.dig('result', 'sessionId')
+            sleep 0.1
+            ws_client.send sessionId: s_id, id: 1,
+                          method: 'Page.navigate',
+                          params: {
+                            url: "devtools://devtools/bundled/inspector.html?ws=#{addr}"
+                          }
+          end
+        end
+        pid
+      rescue Errno::ENOENT
+        nil
+      end
+
+      def get_chrome_path
+        return CONFIG[:chrome_path] if CONFIG[:chrome_path]
+
+        # The process to check OS is based on `selenium` project.
+        case RbConfig::CONFIG['host_os']
+        when /mswin|msys|mingw|cygwin|emc/
+          'C:\Program Files (x86)\Google\Chrome\Application\chrome.exe'
+        when /darwin|mac os/
+          '/Applications/Google\ Chrome.app/Contents/MacOS/Google\ Chrome'
+        when /linux/
+          'google-chrome'
+        else
+          raise "Unsupported OS"
+        end
+      end
+
+      def run_new_chrome
+        dir = Dir.mktmpdir
+        # The command line flags are based on: https://developer.mozilla.org/en-US/docs/Tools/Remote_Debugging/Chrome_Desktop#connecting
+        stdin, stdout, stderr, wait_thr = *Open3.popen3("#{get_chrome_path} --remote-debugging-port=0 --no-first-run --no-default-browser-check --user-data-dir=#{dir}")
+        stdin.close
+        stdout.close
+
+        data = stderr.readpartial 4096
+        if data.match /DevTools listening on ws:\/\/127.0.0.1:(\d+)(.*)/
+          port = $1
+          path = $2
+        end
+        stderr.close
+
+        at_exit{
+          CONFIG[:skip_path] = [//] # skip all
+          FileUtils.rm_rf dir
+        }
+
+        [port, path, wait_thr.pid]
+      end
+    end
+
+    class WebSocketClient
       def initialize s
         @sock = s
       end
 
-      def handshake  
+      def handshake port, path
+        key = SecureRandom.hex(11)
+        @sock.print "GET #{path} HTTP/1.1\r\nHost: 127.0.0.1:#{port}\r\nConnection: Upgrade\r\nUpgrade: websocket\r\nSec-WebSocket-Version: 13\r\nSec-WebSocket-Key: #{key}==\r\n\r\n"
+        res = @sock.readpartial 4092
+        $stderr.puts '[>]' + res if SHOW_PROTOCOL
+
+        if res.match /^Sec-WebSocket-Accept: (.*)\r\n/
+          correct_key = Base64.strict_encode64 Digest::SHA1.digest "#{key}==258EAFA5-E914-47DA-95CA-C5AB0DC85B11"
+          raise "The Sec-WebSocket-Accept value: #{$1} is not valid" unless $1 == correct_key
+        else
+          raise "Unknown response: #{res}"
+        end
+      end
+
+      def send **msg
+        msg = JSON.generate(msg)
+        frame = []
+        fin = 0b10000000
+        opcode = 0b00000001
+        frame << fin + opcode
+
+        mask = 0b10000000 # A client must mask all frames in a WebSocket Protocol.
+        bytesize = msg.bytesize
+        if bytesize < 126
+          payload_len = bytesize
+        elsif bytesize < 2 ** 16
+          payload_len = 0b01111110
+          ex_payload_len = [bytesize].pack('n*').bytes
+        else
+          payload_len = 0b01111111
+          ex_payload_len = [bytesize].pack('Q>').bytes
+        end
+
+        frame << mask + payload_len
+        frame.push *ex_payload_len if ex_payload_len
+
+        frame.push *masking_key = 4.times.map{rand(1..255)}
+        masked = []
+        msg.bytes.each_with_index do |b, i|
+          masked << (b ^ masking_key[i % 4])
+        end
+
+        frame.push *masked
+        @sock.print frame.pack 'c*'
+      end
+
+      def extract_data
+        first_group = @sock.getbyte
+        fin = first_group & 0b10000000 != 128
+        raise 'Unsupported' if fin
+        opcode = first_group & 0b00001111
+        raise "Unsupported: #{opcode}" unless opcode == 1
+
+        second_group = @sock.getbyte
+        mask = second_group & 0b10000000 == 128
+        raise 'The server must not mask any frames' if mask
+        payload_len = second_group & 0b01111111
+        # TODO: Support other payload_lengths
+        if payload_len == 126
+          payload_len = @sock.read(2).unpack('n*')[0]
+        end
+
+        data = JSON.parse @sock.read payload_len
+        $stderr.puts '[>]' + data.inspect if SHOW_PROTOCOL
+        data
+      end
+    end
+
+    class Detach < StandardError
+    end
+
+    class WebSocketServer
+      def initialize s
+        @sock = s
+      end
+
+      def handshake
         req = @sock.readpartial 4096
         $stderr.puts '[>]' + req if SHOW_PROTOCOL
-  
+
         if req.match /^Sec-WebSocket-Key: (.*)\r\n/
           accept = Base64.strict_encode64 Digest::SHA1.digest "#{$1}258EAFA5-E914-47DA-95CA-C5AB0DC85B11"
           @sock.print "HTTP/1.1 101 Switching Protocols\r\nUpgrade: websocket\r\nConnection: Upgrade\r\nSec-WebSocket-Accept: #{accept}\r\n\r\n"
@@ -32,7 +190,7 @@ module DEBUGGER__
         fin = 0b10000000
         opcode = 0b00000001
         frame << fin + opcode
-  
+
         mask = 0b00000000 # A server must not mask any frames in a WebSocket Protocol.
         bytesize = msg.bytesize
         if bytesize < 126
@@ -44,7 +202,7 @@ module DEBUGGER__
           payload_len = 0b01111111
           ex_payload_len = [bytesize].pack('Q>').bytes
         end
-  
+
         frame << mask + payload_len
         frame.push *ex_payload_len if ex_payload_len
         frame.push *msg.bytes
@@ -55,7 +213,9 @@ module DEBUGGER__
         first_group = @sock.getbyte
         fin = first_group & 0b10000000 != 128
         raise 'Unsupported' if fin
+
         opcode = first_group & 0b00001111
+        raise Detach if opcode == 8
         raise "Unsupported: #{opcode}" unless opcode == 1
 
         second_group = @sock.getbyte
@@ -80,34 +240,40 @@ module DEBUGGER__
 
     def send_response req, **res
       if res.empty?
-        @web_sock.send id: req['id'], result: {}
+        @ws_server.send id: req['id'], result: {}
       else
-        @web_sock.send id: req['id'], result: res
+        @ws_server.send id: req['id'], result: res
       end
     end
 
+    def send_fail_response req, **res
+      @ws_server.send id: req['id'], error: res
+    end
+
     def send_event method, **params
       if params.empty?
-        @web_sock.send method: method, params: {}
+        @ws_server.send method: method, params: {}
       else
-        @web_sock.send method: method, params: params
+        @ws_server.send method: method, params: params
       end
     end
 
+    INVALID_REQUEST = -32600
+
     def process
-      bps = []
+      bps = {}
       @src_map = {}
       loop do
-        req = @web_sock.extract_data
+        req = @ws_server.extract_data
         $stderr.puts '[>]' + req.inspect if SHOW_PROTOCOL
 
         case req['method']
 
         ## boot/configuration
         when 'Page.getResourceTree'
-          abs = File.absolute_path($0)
-          src = File.read(abs)
-          @src_map[abs] = src
+          path = File.absolute_path($0)
+          src = File.read(path)
+          @src_map[path] = src
           send_response req,
                         frameTree: {
                           frame: {
@@ -120,8 +286,8 @@ module DEBUGGER__
                           ]
                         }
           send_event 'Debugger.scriptParsed',
-                      scriptId: abs,
-                      url: "http://debuggee#{abs}",
+                      scriptId: path,
+                      url: "http://debuggee#{path}",
                       startLine: 0,
                       startColumn: 0,
                       endLine: src.count("\n"),
@@ -141,7 +307,7 @@ module DEBUGGER__
           @q_msg << req
         when 'Page.startScreencast', 'Emulation.setTouchEmulationEnabled', 'Emulation.setEmitTouchEventsForMouse',
           'Runtime.compileScript', 'Page.getResourceContent', 'Overlay.setPausedInDebuggerMessage',
-          'Debugger.setBreakpointsActive', 'Runtime.releaseObjectGroup'
+          'Runtime.releaseObjectGroup', 'Runtime.discardConsoleEntries', 'Log.clear'
           send_response req
 
         ## control
@@ -151,20 +317,52 @@ module DEBUGGER__
           send_response req
           send_event 'Debugger.resumed'
         when 'Debugger.stepOver'
-          @q_msg << 'n'
-          @q_msg << req
-          send_response req
-          send_event 'Debugger.resumed'
+          begin
+            @session.check_postmortem
+            @q_msg << 'n'
+            send_response req
+            send_event 'Debugger.resumed'
+          rescue PostmortemError
+            send_fail_response req,
+                              code: INVALID_REQUEST,
+                              message: "'stepOver' is not supported while postmortem mode"
+          ensure
+            @q_msg << req
+          end
         when 'Debugger.stepInto'
-          @q_msg << 's'
-          @q_msg << req
-          send_response req
-          send_event 'Debugger.resumed'
+          begin
+            @session.check_postmortem
+            @q_msg << 's'
+            send_response req
+            send_event 'Debugger.resumed'
+          rescue PostmortemError
+            send_fail_response req,
+                              code: INVALID_REQUEST,
+                              message: "'stepInto' is not supported while postmortem mode"
+          ensure
+            @q_msg << req
+          end
         when 'Debugger.stepOut'
-          @q_msg << 'fin'
-          @q_msg << req
+          begin
+            @session.check_postmortem
+            @q_msg << 'fin'
+            send_response req
+            send_event 'Debugger.resumed'
+          rescue PostmortemError
+            send_fail_response req,
+                              code: INVALID_REQUEST,
+                              message: "'stepOut' is not supported while postmortem mode"
+          ensure
+            @q_msg << req
+          end
+        when 'Debugger.setSkipAllPauses'
+          skip = req.dig('params', 'skip')
+          if skip
+            deactivate_bp
+          else
+            activate_bp bps
+          end
           send_response req
-          send_event 'Debugger.resumed'
 
         # breakpoint
         when 'Debugger.getPossibleBreakpoints'
@@ -181,31 +379,72 @@ module DEBUGGER__
                         ]
         when 'Debugger.setBreakpointByUrl'
           line = req.dig('params', 'lineNumber')
-          path = req.dig('params', 'url').match('http://debuggee(.*)')[1]
-          cond = req.dig('params', 'condition')
-          src = get_source_code path
-          end_line = src.count("\n")
-          line = end_line  if line > end_line
-          if cond != ''
-            bps << SESSION.add_line_breakpoint(path, line + 1, cond: cond)
+          url = req.dig('params', 'url')
+          locations = []
+          if url.match /http:\/\/debuggee(.*)/
+            path = $1
+            cond = req.dig('params', 'condition')
+            src = get_source_code path
+            end_line = src.count("\n")
+            line = end_line  if line > end_line
+            b_id = "1:#{line}:#{path}"
+            if cond != ''
+              SESSION.add_line_breakpoint(path, line + 1, cond: cond)
+            else
+              SESSION.add_line_breakpoint(path, line + 1)
+            end
+            bps[b_id] = bps.size
+            locations << {scriptId: path, lineNumber: line}
           else
-            bps << SESSION.add_line_breakpoint(path, line + 1)
+            b_id = "1:#{line}:#{url}"
           end
           send_response req,
-                        breakpointId: (bps.size - 1).to_s,
-                        locations: [
-                          scriptId: path,
-                          lineNumber: line
-                        ]
+                        breakpointId: b_id,
+                        locations: locations
         when 'Debugger.removeBreakpoint'
           b_id = req.dig('params', 'breakpointId')
-          @q_msg << "del #{b_id}"
+          bps = del_bp bps, b_id
+          send_response req
+        when 'Debugger.setBreakpointsActive'
+          active = req.dig('params', 'active')
+          if active
+            activate_bp bps
+          else
+            deactivate_bp # TODO: Change this part because catch breakpoints should not be deactivated.
+          end
+          send_response req
+        when 'Debugger.setPauseOnExceptions'
+          state = req.dig('params', 'state')
+          ex = 'Exception'
+          case state
+          when 'none'
+            @q_msg << 'config postmortem = false'
+            bps = del_bp bps, ex
+          when 'uncaught'
+            @q_msg << 'config postmortem = true'
+            bps = del_bp bps, ex
+          when 'all'
+            @q_msg << 'config postmortem = false'
+            SESSION.add_catch_breakpoint ex
+            bps[ex] = bps.size
+          end
           send_response req
 
         when 'Debugger.evaluateOnCallFrame', 'Runtime.getProperties'
           @q_msg << req
         end
       end
+    rescue Detach
+      @q_msg << 'continue'
+    end
+
+    def del_bp bps, k
+      return bps unless idx = bps[k]
+
+      bps.delete k
+      bps.each_key{|i| bps[i] -= 1 if bps[i] > idx}
+      @q_msg << "del #{idx}"
+      bps
     end
 
     def get_source_code path
@@ -216,9 +455,32 @@ module DEBUGGER__
       src
     end
 
+    def activate_bp bps
+      bps.each_key{|k|
+        if k.match /^\d+:(\d+):(.*)/
+          line = $1
+          path = $2
+          SESSION.add_line_breakpoint(path, line.to_i + 1)
+        else
+          SESSION.add_catch_breakpoint 'Exception'
+        end
+      }
+    end
+
+    def deactivate_bp
+      @q_msg << 'del'
+      @q_ans << 'y'
+    end
+
+    def cleanup_reader
+      Process.kill :KILL, @chrome_pid if @chrome_pid
+    end
+
     ## Called by the SESSION thread
 
     def readline prompt
+      return 'c' unless @q_msg
+
       @q_msg.pop || 'kill!'
     end
 
@@ -226,6 +488,10 @@ module DEBUGGER__
       send_response req, **result
     end
 
+    def respond_fail req, **result
+      send_fail_response req, **result
+    end
+
     def fire_event event, **result
       if result.empty?
         send_event event
@@ -245,26 +511,48 @@ module DEBUGGER__
   end
 
   class Session
+    def fail_response req, **result
+      @ui.respond_fail req, result
+      return :retry
+    end
+
+    INVALID_PARAMS = -32602
+
     def process_protocol_request req
       case req['method']
       when 'Debugger.stepOver', 'Debugger.stepInto', 'Debugger.stepOut', 'Debugger.resume', 'Debugger.getScriptSource'
         @tc << [:cdp, :backtrace, req]
       when 'Debugger.evaluateOnCallFrame'
-        expr = req.dig('params', 'expression')
-        @tc << [:cdp, :evaluate, req, expr]
+        frame_id = req.dig('params', 'callFrameId')
+        if fid = @frame_map[frame_id]
+          expr = req.dig('params', 'expression')
+          @tc << [:cdp, :evaluate, req, fid, expr]
+        else
+          fail_response req,
+                        code: INVALID_PARAMS,
+                        message: "'callFrameId' is an invalid"
+        end
       when 'Runtime.getProperties'
         oid = req.dig('params', 'objectId')
-        case oid
-        when /(\d?):local/
-          @tc << [:cdp, :properties, req, $1.to_i]
-        when /\d?:script/
-          # TODO: Support a script type
-          @ui.respond req
-          return :retry
-        when /\d?:global/
-          # TODO: Support a global type
-          @ui.respond req
-          return :retry
+        if ref = @obj_map[oid]
+          case ref[0]
+          when 'local'
+            frame_id = ref[1]
+            fid = @frame_map[frame_id]
+            @tc << [:cdp, :scope, req, fid]
+          when 'properties'
+            @tc << [:cdp, :properties, req, oid]
+          when 'script', 'global'
+            # TODO: Support script and global types
+            @ui.respond req
+            return :retry
+          else
+            raise "Unknown type: #{ref.inspect}"
+          end
+        else
+          fail_response req,
+                        code: INVALID_PARAMS,
+                        message: "'objectId' is an invalid"
         end
       end
     end
@@ -274,7 +562,9 @@ module DEBUGGER__
 
       case type
       when :backtrace
-        result[:callFrames].each do |frame|
+        result[:callFrames].each.with_index do |frame, i|
+          frame_id = frame[:callFrameId]
+          @frame_map[frame_id] = i
           s_id = frame.dig(:location, :scriptId)
           if File.exist?(s_id) && !@script_paths.include?(s_id)
             src = File.read(s_id)
@@ -289,13 +579,60 @@ module DEBUGGER__
                             hash: src.hash
             @script_paths << s_id
           end
+
+          frame[:scopeChain].each {|s|
+            oid = s.dig(:object, :objectId)
+            @obj_map[oid] = [s[:type], frame_id]
+          }
+        end
+
+        if oid = result.dig(:data, :objectId)
+          @obj_map[oid] = ['properties']
         end
-        result[:reason] = 'other'
         @ui.fire_event 'Debugger.paused', **result
       when :evaluate
+        message = result.delete :message
+        if message
+          fail_response req,
+                        code: INVALID_PARAMS,
+                        message: message
+        else
+          rs = result.dig(:response, :result)
+          [rs].each{|obj|
+            if oid = obj[:objectId]
+              @obj_map[oid] = ['properties']
+            end
+          }
+          @ui.respond req, **result[:response]
+
+          out = result[:output]
+          if out && !out.empty?
+            @ui.fire_event 'Runtime.consoleAPICalled',
+                            type: 'log',
+                            args: [
+                              type: out.class,
+                              value: out
+                            ],
+                            executionContextId: 1, # Change this number if something goes wrong.
+                            timestamp: Time.now.to_f
+          end
+        end
+      when :scope
+        result.each{|obj|
+          if oid = obj.dig(:value, :objectId)
+            @obj_map[oid] = ['properties']
+          end
+        }
         @ui.respond req, result: result
       when :properties
-        @ui.respond req, result: result
+        result.each_value{|v|
+          v.each{|obj|
+            if oid = obj.dig(:value, :objectId)
+              @obj_map[oid] = ['properties']
+            end
+          }
+        }
+        @ui.respond req, **result
       end
     end
   end
@@ -307,43 +644,49 @@ module DEBUGGER__
 
       case type
       when :backtrace
-        event! :cdp_result, :backtrace, req, {
+        exception = nil
+        result = {
+          reason: 'other',
           callFrames: @target_frames.map.with_index{|frame, i|
+            exception = frame.raised_exception if frame == current_frame && frame.has_raised_exception
+
             path = frame.realpath || frame.path
             if path.match /<internal:(.*)>/
-              abs = $1
-            else
-              abs = path
+              path = $1
             end
 
-            local_scope = {
+            {
               callFrameId: SecureRandom.hex(16),
               functionName: frame.name,
+              functionLocation: {
+                scriptId: path,
+                lineNumber: 0
+              },
               location: {
-                scriptId: abs,
+                scriptId: path,
                 lineNumber: frame.location.lineno - 1 # The line number is 0-based.
               },
-              url: "http://debuggee#{abs}",
+              url: "http://debuggee#{path}",
               scopeChain: [
                 {
                   type: 'local',
                   object: {
                     type: 'object',
-                    objectId: "#{i}:local"
+                    objectId: rand.to_s
                   }
                 },
                 {
                   type: 'script',
                   object: {
                     type: 'object',
-                    objectId: "#{i}:script"
+                    objectId: rand.to_s
                   }
                 },
                 {
                   type: 'global',
                   object: {
                     type: 'object',
-                    objectId: "#{i}:global"
+                    objectId: rand.to_s
                   }
                 }
               ],
@@ -353,15 +696,83 @@ module DEBUGGER__
             }
           }
         }
+
+        if exception
+          result[:data] = evaluate_result exception
+          result[:reason] = 'exception'
+        end
+        event! :cdp_result, :backtrace, req, result
       when :evaluate
-        expr = args.shift
-        begin
-          result = current_frame.binding.eval(expr.to_s, '(DEBUG CONSOLE)')
-        rescue Exception => e
-          result = e
+        res = {}
+        fid, expr = args
+        frame = @target_frames[fid]
+        message = nil
+
+        if frame && (b = frame.binding)
+          b = b.dup
+          special_local_variables current_frame do |name, var|
+            b.local_variable_set(name, var) if /\%/ !~name
+          end
+
+          result = nil
+
+          case req.dig('params', 'objectGroup')
+          when 'popover'
+            case expr
+            # Chrome doesn't read instance variables
+            when /\A\$\S/
+              global_variables.each{|gvar|
+                if gvar.to_s == expr
+                  result = eval(gvar.to_s)
+                  break false
+                end
+              } and (message = "Error: Not defined global variable: #{expr.inspect}")
+            when /(\A[A-Z][a-zA-Z]*)/
+              unless result = search_const(b, $1)
+                message = "Error: Not defined constant: #{expr.inspect}"
+              end
+            else
+              begin
+                # try to check local variables
+                b.local_variable_defined?(expr) or raise NameError
+                result = b.local_variable_get(expr)
+              rescue NameError
+                # try to check method
+                if b.receiver.respond_to? expr, include_all: true
+                  result = b.receiver.method(expr)
+                else
+                  message = "Error: Can not evaluate: #{expr.inspect}"
+                end
+              end
+            end
+          else
+            begin
+              orig_stdout = $stdout
+              $stdout = StringIO.new
+              result = current_frame.binding.eval(expr.to_s, '(DEBUG CONSOLE)')
+            rescue Exception => e
+              result = e
+              b = result.backtrace.map{|e| "    #{e}\n"}
+              line = b.first.match('.*:(\d+):in .*')[1].to_i
+              res[:exceptionDetails] = {
+                exceptionId: 1,
+                text: 'Uncaught',
+                lineNumber: line - 1,
+                columnNumber: 0,
+                exception: evaluate_result(result),
+              }
+            ensure
+              output = $stdout.string
+              $stdout = orig_stdout
+            end
+          end
+        else
+          result = Exception.new("Error: Can not evaluate on this frame")
         end
-        event! :cdp_result, :evaluate, req, evaluate_result(result)
-      when :properties
+
+        res[:result] = evaluate_result(result)
+        event! :cdp_result, :evaluate, req, message: message, response: res, output: output
+      when :scope
         fid = args.shift
         frame = @target_frames[fid]
         if b = frame.binding
@@ -369,8 +780,9 @@ module DEBUGGER__
             v = b.local_variable_get(name)
             variable(name, v)
           }
-          vars.unshift variable('%raised', frame.raised_exception) if frame.has_raised_exception
-          vars.unshift variable('%return', frame.return_value) if frame.has_return_value
+          special_local_variables frame do |name, val|
+            vars.unshift variable(name, val)
+          end
           vars.unshift variable('%self', b.receiver)
         elsif lvars = frame.local_variables
           vars = lvars.map{|var, val|
@@ -378,46 +790,131 @@ module DEBUGGER__
           }
         else
           vars = [variable('%self', frame.self)]
-          vars.push variable('%raised', frame.raised_exception) if frame.has_raised_exception
-          vars.push variable('%return', frame.return_value) if frame.has_return_value
+          special_local_variables frame do |name, val|
+            vars.unshift variable(name, val)
+          end
+        end
+        event! :cdp_result, :scope, req, vars
+      when :properties
+        oid = args.shift
+        result = []
+        prop = []
+
+        if obj = @obj_map[oid]
+          case obj
+          when Array
+            result = obj.map.with_index{|o, i|
+              variable i.to_s, o
+            }
+          when Hash
+            result = obj.map{|k, v|
+              variable(k, v)
+            }
+          when Struct
+            result = obj.members.map{|m|
+              variable(m, obj[m])
+            }
+          when String
+            prop = [
+              property('#length', obj.length),
+              property('#encoding', obj.encoding)
+            ]
+          when Class, Module
+            result = obj.instance_variables.map{|iv|
+              variable(iv, obj.instance_variable_get(iv))
+            }
+            prop = [property('%ancestors', obj.ancestors[1..])]
+          when Range
+            prop = [
+              property('#begin', obj.begin),
+              property('#end', obj.end),
+            ]
+          end
+
+          result += obj.instance_variables.map{|iv|
+            variable(iv, obj.instance_variable_get(iv))
+          }
+          prop += [property('#class', obj.class)]
         end
-        event! :cdp_result, :properties, req, vars
+        event! :cdp_result, :properties, req, result: result, internalProperties: prop
       end
     end
 
+    def search_const b, expr
+      cs = expr.split('::')
+      [Object, *b.eval('Module.nesting')].reverse_each{|mod|
+        if cs.all?{|c|
+             if mod.const_defined?(c)
+               mod = mod.const_get(c)
+             else
+               false
+             end
+           }
+          # if-body
+          return mod
+        end
+      }
+      false
+    end
+
     def evaluate_result r
       v = variable nil, r
       v[:value]
     end
 
-    def variable_ name, obj, type, use_short: true
-      {
+    def property name, obj
+      v = variable name, obj
+      v.delete :configurable
+      v.delete :enumerable
+      v
+    end
+
+    def variable_ name, obj, type, description: obj.inspect, subtype: nil
+      oid = rand.to_s
+      @obj_map[oid] = obj
+      prop = {
         name: name,
         value: {
           type: type,
-          value: DEBUGGER__.short_inspect(obj, use_short)
+          description: description,
+          value: obj,
+          objectId: oid
         },
-        configurable: true,
-        enumerable: true
+        configurable: true, # TODO: Change these parts because
+        enumerable: true    #       they are not necessarily `true`.
       }
+
+      if type == 'object'
+        v = prop[:value]
+        v.delete :value
+        v[:subtype] = subtype if subtype
+        v[:className] = obj.class
+      end
+      prop
     end
 
     def variable name, obj
       case obj
-      when Array, Hash, Range, NilClass, Time
-        variable_ name, obj, 'object'
+      when Array
+        variable_ name, obj, 'object', description: "Array(#{obj.size})", subtype: 'array'
+      when Hash
+        variable_ name, obj, 'object', description: "Hash(#{obj.size})", subtype: 'map'
       when String
-        variable_ name, obj, 'string', use_short: false
-      when Class, Module, Struct
-        variable_ name, obj, 'function'
+        variable_ name, obj, 'string', description: obj
+      when Class, Module, Struct, Range, Time, Method
+        variable_ name, obj, 'object'
       when TrueClass, FalseClass
         variable_ name, obj, 'boolean'
       when Symbol
         variable_ name, obj, 'symbol'
-      when Float
-        variable_ name, obj, 'number'
-      when Integer
+      when Integer, Float
         variable_ name, obj, 'number'
+      when Exception
+        bt = nil
+        if log = obj.backtrace
+          bt = log.map{|e| "    #{e}\n"}.join
+        end
+        variable_ name, obj, 'object', description: "#{obj.inspect}\n#{bt}", subtype: 'error'
       else
         variable_ name, obj, 'undefined'
       end