Use TC metadata to provide ThreadClient info about TUI

Since TUI needs to display up-to-date information once enabled, I think
it makes sense to pass its information to ThreadClient in every request.

This information will help ThreadClient

1. know when to send back TUI data when not to.
2. send back data that fits the best on TUI. For example: the line of content.

Author: st0012

lib/debug/local.rb

@@ -113,10 +113,9 @@ def after_fork_parent
   class UI_LocalTuiConsole < UI_LocalConsole
     SUPPORTED_TYPES = [:locals, :src]
 
-    attr_reader :type
+    attr_reader :screen
 
     def initialize(type)
-      @type = type
       @screen = Screen.new(width, height, top_frame: type)
       super()
     end
@@ -211,7 +210,7 @@ def clear_screen!
     end
 
     class Window
-      attr_reader :width, :height
+      attr_reader :width, :height, :content_width, :content_height
 
       def initialize(width, height)
         @width = width
@@ -258,6 +257,8 @@ def clear!
     class TopFrame < Window
       HEIGHT = 15
 
+      attr_reader :type
+
       def initialize(width, height, type:)
         super(width, height)
         # border and padding

lib/debug/session.rb

@@ -202,15 +202,28 @@ def session_server_main
       deactivate
     end
 
+    def tui_metadata
+      if @ui.tui?
+        top_frame = @ui.screen.top_frame
+        { type: top_frame.type, height: top_frame.content_height }
+      end
+    end
+
     def request_tc(req)
-      @tc << req
+      metadata = {}
+
+      if tui_meta = tui_metadata
+        metadata[:tui] = tui_meta
+      end
+
+      @tc << [req, metadata]
     end
 
     def process_event evt
       # variable `@internal_info` is only used for test
       @tc, output, ev, @internal_info, *ev_args = evt
 
-      capture_tui_data if @ui.tui?
+      capture_tui_data(@internal_info[:tui]) if @ui.tui?
 
       output.each{|str| @ui.puts str} if ev != :suspend
 
@@ -1770,9 +1783,9 @@ def after_fork_parent
       @ui.after_fork_parent
     end
 
-    def capture_tui_data
-      if tui_data = @internal_info[:tui]
-        @ui.store_tui_data(tui_data[@ui.type])
+    def capture_tui_data(tui_data)
+      if tui_data
+        @ui.store_tui_data(tui_data[@ui.screen.top_frame.type])
       end
     end
 
@@ -1784,7 +1797,8 @@ def start_tui(type)
         end
 
         @ui = UI_LocalTuiConsole.new(type)
-        capture_tui_data
+        tui_data = @tc.generate_tui_data(tui_metadata)
+        capture_tui_data(tui_data)
         @ui.puts
       else
         @ui.puts("TUI can't be activated under this mode")

lib/debug/thread_client.rb

@@ -187,33 +187,35 @@ def << req
       @q_cmd << req
     end
 
-    def generate_tui_data(type)
-      case type
-      when :src
-        frame = current_frame
-        start_line, end_line, lines = get_src(frame: frame, max_lines: 15)
-
-        if start_line
-          lines[start_line, end_line]
-        else
-          "# No sourcefile available for #{frame.path}"
-        end
-      when :locals
-        locals = collect_locals(current_frame).map do |var, val|
-          variable_info(var,  val, nil)
+    def generate_tui_data(metadata)
+      type = metadata[:type]
+      data =
+        case type
+        when :src
+          frame = current_frame
+          start_line, end_line, lines = get_src(frame: frame, max_lines: metadata[:height])
+
+          if start_line
+            lines[start_line, end_line]
+          else
+            "# No sourcefile available for #{frame.path}"
+          end
+        when :locals
+          locals = collect_locals(current_frame).map do |var, val|
+            variable_info(var,  val, nil)
+          end
         end
-      end
+
+      { type => data }
     end
 
     def generate_info
       return {} unless current_frame
 
       result = { location: current_frame.location_str, line: current_frame.location.lineno }
 
-      result[:tui] = {}
-
-      [:locals, :src].each do |type|
-        result[:tui][type] = generate_tui_data(type)
+      if @tui_metadata
+        result[:tui] = generate_tui_data(@tui_metadata)
       end
 
       result
@@ -755,13 +757,14 @@ def wait_next_action_
       while true
         begin
           set_mode :waiting if !waiting?
-          cmds = @q_cmd.pop
+          cmds, metadata = @q_cmd.pop
           # pp [self, cmds: cmds]
           break unless cmds
         ensure
           set_mode :running
         end
 
+        @tui_metadata = metadata[:tui]
         cmd, *args = *cmds
 
         case cmd