heathrow 0.7.5 → 0.7.7

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: e5276cc34b418c9d3b080eb711a1870857679ca87681c58436e6207ff37e7106
-  data.tar.gz: 8a10f16633f2ac610bc191434566f883319b570a2ad773297373a8d5a4e477de
+  metadata.gz: d7c8e6773f02f56ba4ac097d508bafaf526e2aed3bb327220614640d5c1bc3bd
+  data.tar.gz: 78db591713e7d944436a7c797c7099f337e5050bd4bce8689b38637330665056
 SHA512:
-  metadata.gz: 4c98df39af32e449f46f7a4d7c58436d88cfd6e32499394624ad74c2b16c4e390512ecbe18fff5fbe0462c1bb76b18921b806a72403aa08e7e39ccfc7ac69ccc
-  data.tar.gz: 9fb82e3608cb1fd6d63078b8584aad96deb3e43072421468b4c712d8c3ead53c192b7378c4d1cef75ada0f5f8d430e81d5847d71c27a04accfdf333f59fe01c8
+  metadata.gz: c7b866bfaaf9c2f2edd272be1950fd1730d0e40df9500779dbc51fa8f166e9a149472edd7b1099eea0ece4b713b62201e010cd165161c1a8a0f15725400a00ec
+  data.tar.gz: b1bde82078786c429268cf54cce8bef68de0ef03368612e90414c9b484e76181fe3aba59e1157daa7808c968128c750e565d273ea4c30044fa19a847314d76fb

data/lib/heathrow/database.rb

@@ -375,9 +375,9 @@ module Heathrow
       # Handle sender pattern (supports regex via pipe separation)
       if filters[:sender_pattern]
         patterns = filters[:sender_pattern].split('|')
-        conditions = patterns.map { "sender LIKE ?" }.join(' OR ')
+        conditions = patterns.map { "(sender LIKE ? OR sender_name LIKE ?)" }.join(' OR ')
         query += " AND (#{conditions})"
-        params += patterns.map { |p| "%#{p}%" }
+        patterns.each { |p| params += ["%#{p}%", "%#{p}%"] }
       end
       
       # Handle subject pattern

data/lib/heathrow/ui/application.rb

@@ -206,14 +206,24 @@ module Heathrow
       selected ? base.merge(selected) : base
     end
 
+    def header_message?(msg)
+      msg['is_header'] || msg['is_channel_header'] || msg['is_thread_header'] || msg['is_dm_header']
+    end
+
+    def invalidate_counts
+      @cached_unread = nil
+      @cached_starred = nil
+      @cached_total = nil
+    end
+
     # Get the current message at @index (threaded or flat view)
     def current_message
-      respond_to?(:current_message_for_navigation) ? current_message_for_navigation : @filtered_messages[@index]
+      current_message_for_navigation
     end
 
     # Get the current message count (threaded or flat view)
     def message_count
-      respond_to?(:filtered_messages_size) ? filtered_messages_size : @filtered_messages.size
+      filtered_messages_size
     end
 
     # Mark the previous message as read when moving away from it
@@ -223,7 +233,7 @@ module Heathrow
       # Remember current message so it gets marked read when we leave it
       msg = current_message
       return unless msg
-      return if msg['is_header'] || msg['is_channel_header'] || msg['is_thread_header'] || msg['is_dm_header']
+      return if header_message?(msg)
       return unless msg['id'] && !msg['id'].to_s.start_with?('header_')
 
       if msg['is_read'].to_i == 0
@@ -246,6 +256,7 @@ module Heathrow
 
       @db.mark_as_read(msg['id'])
       msg['is_read'] = 1
+      invalidate_counts
       sync_maildir_flag(msg, 'S', true)
 
       # Also update any other references to this message in filtered/display lists
@@ -281,21 +292,22 @@ module Heathrow
       count = @browsed_message_ids.size
       return if count == 0
 
+      msg_by_id = @filtered_messages.each_with_object({}) { |m, h| h[m['id']] = m }
       @browsed_message_ids.each do |msg_id|
         @db.mark_as_read(msg_id)
-        # Update in current message list
-        msg = @filtered_messages.find { |m| m['id'] == msg_id }
+        msg = msg_by_id[msg_id]
         msg['is_read'] = 1 if msg
       end
 
       @browsed_message_ids.clear
+      invalidate_counts
       set_feedback("Marked #{count} browsed messages as read", 156, 3)
 
       # Remove from view if in unread view
       if is_unread_view?
         @filtered_messages.select! { |m| m['is_read'].to_i == 0 }
         @index = 0
-        reset_threading if respond_to?(:reset_threading)
+        reset_threading
       end
 
       render_all
@@ -391,16 +403,13 @@ module Heathrow
 
       # EXACTLY LIKE RTFM
       # Initialize rcurses
-      File.write('/tmp/heathrow_start.txt', "Starting run method\n") if ENV['DEBUG']
       Rcurses.init!
-      File.write('/tmp/heathrow_start.txt', "Rcurses initialized\n", mode: 'a') if ENV['DEBUG']
 
       # Clear screen to remove any artifacts
       Rcurses.clear_screen
 
       # Get terminal size
       setup_display
-      File.write('/tmp/heathrow_start.txt', "Display setup complete: #{@w}x#{@h}\n", mode: 'a') if ENV['DEBUG']
 
       # Create panes and show skeleton immediately
       create_panes
@@ -440,8 +449,8 @@ module Heathrow
           end
           sort_messages
           @index = 0
-          reset_threading if respond_to?(:reset_threading)
-          restore_view_thread_mode if respond_to?(:restore_view_thread_mode)
+          reset_threading
+          restore_view_thread_mode
           @initial_load_done = true
           @needs_redraw = true
           # Preload the heavy mail gem so 'v' (attachments) doesn't lag
@@ -481,14 +490,14 @@ module Heathrow
               sort_messages
             end
             # Force re-organization in threaded mode
-            if @show_threaded && respond_to?(:reset_threading)
+            if @show_threaded
               reset_threading(true)
-              organize_current_messages(true) if respond_to?(:organize_current_messages)
+              organize_current_messages(true)
             end
             # Defer index restoration for threaded views (display_messages is empty
             # after reset_threading; it gets populated during render). For flat views,
             # resolve immediately since @filtered_messages is the navigation list.
-            if @show_threaded && respond_to?(:reset_threading)
+            if @show_threaded
               @pending_restore_id = selected_id
               @index = 0
             elsif selected_id
@@ -826,6 +835,8 @@ module Heathrow
         label_message
       when 'v'
         view_attachments
+      when 'Z'
+        open_in_timely
       when 'I'
         ai_assistant
       when 'V'
@@ -1143,7 +1154,7 @@ module Heathrow
       set_feedback("Deleted: #{name} (#{deleted_msgs} messages removed)", 156, 3)
 
       # Re-load the view to reflect removed messages
-      reset_threading if respond_to?(:reset_threading)
+      reset_threading
       switch_to_view(@current_view) if @current_view
     end
 
@@ -1362,10 +1373,15 @@ module Heathrow
         count_text = "#{total} sources"
       else
         # For message views, show unread/total plus starred
-        real_msgs = @filtered_messages.reject { |m| m['is_header'] || m['is_channel_header'] || m['is_thread_header'] || m['is_dm_header'] }
-        unread = real_msgs.count { |m| m['is_read'].to_i == 0 }
-        starred = real_msgs.count { |m| m['is_starred'].to_i == 1 }
-        total = real_msgs.size
+        if @cached_unread.nil?
+          real_msgs = @filtered_messages.reject { |m| header_message?(m) }
+          @cached_unread = real_msgs.count { |m| m['is_read'].to_i == 0 }
+          @cached_starred = real_msgs.count { |m| m['is_starred'].to_i == 1 }
+          @cached_total = real_msgs.size
+        end
+        unread = @cached_unread
+        starred = @cached_starred
+        total = @cached_total
         count_text = "#{unread} unread / #{total} msgs"
         count_text += " / #{starred}*" if starred > 0
       end
@@ -1381,6 +1397,7 @@ module Heathrow
                    when 'latest' then 'Latest'
                    when 'alphabetical' then 'A-Z'
                    when 'sender' then 'Sender'
+                   when 'from' then 'From'
                    when 'unread' then 'Unread'
                    when 'source' then 'Source'
                    else @sort_order.capitalize
@@ -1392,7 +1409,7 @@ module Heathrow
 
       # Threading mode indicator
       mode_part = ""
-      if @current_view != 'S' && respond_to?(:thread_mode_label)
+      if @current_view != 'S'
         mode_part = " [#{thread_mode_label}]".fg(245)
       end
 
@@ -1520,7 +1537,7 @@ module Heathrow
       end
 
       # Build the line with timestamp, icon and sender
-      icon = respond_to?(:get_source_icon) ? get_source_icon(msg['source_type']) : '•'
+      icon = get_source_icon(msg['source_type'])
       line_prefix = "#{timestamp} #{icon} #{sender_display} "
 
       # Calculate remaining space for subject (use display_width for CJK)
@@ -1837,7 +1854,7 @@ module Heathrow
       show_loading("Loading sources...")
 
       # Reset threading state when changing views
-      reset_threading if respond_to?(:reset_threading)
+      reset_threading
       
       # Reload source colors to ensure they're fresh
       load_source_colors
@@ -1885,6 +1902,7 @@ module Heathrow
     
     def show_all_messages
       flush_pending_read
+      invalidate_counts
       @current_view = 'A'
       @current_folder = nil
       @in_source_view = false
@@ -1895,8 +1913,8 @@ module Heathrow
       @last_rendered_index = nil  # Force right pane refresh
 
       # Restore per-view threading mode
-      reset_threading if respond_to?(:reset_threading)
-      restore_view_thread_mode if respond_to?(:restore_view_thread_mode)
+      reset_threading
+      restore_view_thread_mode
 
       # Show view name instantly
       render_top_bar
@@ -1912,14 +1930,6 @@ module Heathrow
     def render_message_content
       current_msg = current_message
 
-      if ENV['DEBUG']
-        File.open('/tmp/heathrow_debug.log', 'a') do |f|
-          f.puts "render_message_content START"
-          f.puts "  @index: #{@index}"
-          f.puts "  @filtered_messages[@index] exists: #{!current_msg.nil?}"
-        end
-      end
-
       # Reset scroll position when switching messages
       if @last_rendered_index != @index
         @panes[:right].ix = 0
@@ -1947,23 +1957,11 @@ module Heathrow
       end
 
       # Special handling for headers (channel headers, thread headers, etc.)
-      if msg['is_header'] || msg['is_channel_header'] || msg['is_thread_header']
+      if header_message?(msg)
         render_header_summary(msg)
         return
       end
       
-      if ENV['DEBUG']
-        File.open('/tmp/heathrow_debug.log', 'a') do |f|
-          f.puts "  msg details:"
-          f.puts "    id: #{msg['id']}"
-          f.puts "    sender: #{msg['sender']}"
-          f.puts "    recipient: #{msg['recipient']}"
-          f.puts "    subject: #{msg['subject']}"
-          f.puts "    timestamp: #{msg['timestamp'].inspect}"
-          f.puts "    source_type: #{msg['source_type'].inspect}"
-        end
-      end
-      
       # Format message header
       header = []
       
@@ -2000,23 +1998,20 @@ module Heathrow
       else
         # Regular message display
         header << "From: #{msg['sender']}".fg(2) if msg['sender']
-        # Show recipients (To field)
+        # Show recipients (To field, already parsed by normalize_message_row)
         to = msg['recipients'] || msg['recipient']
         if to
-          to_list = to.is_a?(String) ? (JSON.parse(to) rescue [to]) : to
-          to_str = to_list.is_a?(Array) ? to_list.join(', ') : to_list.to_s
+          to_str = to.is_a?(Array) ? to.join(', ') : to.to_s
           header << "To: #{to_str}".fg(2) unless to_str.empty?
         end
-        # Show CC recipients
+        # Show CC recipients (already parsed by normalize_message_row)
         cc = msg['cc']
         if cc
-          cc_list = cc.is_a?(String) ? (JSON.parse(cc) rescue [cc]) : cc
-          cc_str = cc_list.is_a?(Array) ? cc_list.join(', ') : cc_list.to_s
+          cc_str = cc.is_a?(Array) ? cc.join(', ') : cc.to_s
           header << "Cc: #{cc_str}".fg(2) unless cc_str.empty?
         end
         # For weechat, show channel name from metadata instead of content preview
         meta = msg['metadata']
-        meta = JSON.parse(meta) if meta.is_a?(String) rescue nil
         if meta.is_a?(Hash) && meta['channel_name']
           header << "Subject: #{meta['channel_name']}".b.fg(1)
         elsif msg['subject']
@@ -2024,22 +2019,9 @@ module Heathrow
         end
       end
       
-      if ENV['DEBUG']
-        File.open('/tmp/heathrow_debug.log', 'a') do |f|
-          f.puts "  About to parse timestamp: #{msg['timestamp'].inspect}"
-        end
-      end
-      
       # Parse timestamp using helper method
       date_str = parse_timestamp(msg['timestamp'], '%Y-%m-%d %H:%M:%S') || "Unknown date"
-      
-      if ENV['DEBUG']
-        File.open('/tmp/heathrow_debug.log', 'a') do |f|
-          f.puts "  Final date_str: #{date_str}"
-          f.puts "  About to add header lines"
-        end
-      end
-      
+
       header << "Date: #{date_str}".fg(240)
       if msg['source_type']
         source_label = case msg['source_type']
@@ -2050,10 +2032,10 @@ module Heathrow
         header << "Type: #{source_label}".fg(get_source_color(msg))
       end
 
-      # Show labels (if any beyond the folder name)
+      # Show labels (if any beyond the folder name, already parsed by normalize_message_row)
       labels = msg['labels']
-      labels = JSON.parse(labels) if labels.is_a?(String) rescue []
-      if labels.is_a?(Array) && labels.size > 0
+      labels = [] unless labels.is_a?(Array)
+      if labels.size > 0
         # Skip folder name (first label) if it matches the folder
         display_labels = labels.reject { |l| l == msg['folder'] }
         unless display_labels.empty?
@@ -2061,11 +2043,6 @@ module Heathrow
         end
       end
       
-      if ENV['DEBUG']
-        File.open('/tmp/heathrow_debug.log', 'a') do |f|
-          f.puts "  Header built successfully, about to format content"
-        end
-      end
       header << ("─" * 60).fg(238)
       
       # Format message body — prefer HTML rendered via w3m
@@ -2134,13 +2111,6 @@ module Heathrow
         content = content_parts.join("\n")
       end
       
-      if ENV['DEBUG']
-        File.open('/tmp/heathrow_debug.log', 'a') do |f|
-          f.puts "  Content formatted, about to set pane text"
-          f.puts "  Content length: #{content.length}"
-        end
-      end
-
       # Ensure content is UTF-8 compatible (handle emails with various encodings)
       # Create a mutable copy if frozen, then fix encoding
       content = content.dup if content.frozen?
@@ -2154,9 +2124,8 @@ module Heathrow
       # Colorize email content (quote levels + signature)
       content = colorize_email_content(content)
 
-      # Store maildir file path for calendar parser
+      # Store maildir file path for calendar parser (metadata already parsed by normalize_message_row)
       meta = msg['metadata']
-      meta = JSON.parse(meta) if meta.is_a?(String) rescue nil
       @_current_render_msg_file = meta['maildir_file'] if meta.is_a?(Hash)
 
       # Attachment list under header, before body
@@ -2193,11 +2162,6 @@ module Heathrow
       @panes[:right].text = full_text
       @panes[:right].refresh
 
-      if ENV['DEBUG']
-        File.open('/tmp/heathrow_debug.log', 'a') do |f|
-          f.puts "render_message_content END - success"
-        end
-      end
     end
 
     def render_header_summary(header_msg)
@@ -2392,7 +2356,7 @@ module Heathrow
       end
 
       # Force threaded view to rebuild organizer with new messages
-      if @show_threaded && respond_to?(:organize_current_messages)
+      if @show_threaded
         organize_current_messages(true)
       end
 
@@ -2455,7 +2419,7 @@ module Heathrow
       }
     end
 
-    def apply_view_filters_with_limit(view, limit)
+    def build_db_filters(view)
       filters = view[:filters] || {}
       db_filters = {}
       if filters['rules'].is_a?(Array)
@@ -2476,6 +2440,11 @@ module Heathrow
           end
         end
       end
+      db_filters
+    end
+
+    def apply_view_filters_with_limit(view, limit)
+      db_filters = build_db_filters(view)
       @filtered_messages = @db.get_messages(db_filters, limit, 0, light: true)
     end
 
@@ -2552,6 +2521,7 @@ module Heathrow
     # View switching
     def show_new_messages
       flush_pending_read
+      invalidate_counts
       @current_view = 'N'
       @current_folder = nil
       @in_source_view = false
@@ -2561,8 +2531,8 @@ module Heathrow
       @last_rendered_index = nil  # Force right pane refresh
 
       # Restore per-view threading mode
-      reset_threading if respond_to?(:reset_threading)
-      restore_view_thread_mode if respond_to?(:restore_view_thread_mode)
+      reset_threading
+      restore_view_thread_mode
 
       render_top_bar
 
@@ -2585,6 +2555,7 @@ module Heathrow
 
     def switch_to_view(key)
       flush_pending_read
+      invalidate_counts
       @current_view = key
       @current_folder = nil
       @in_source_view = false
@@ -2592,8 +2563,8 @@ module Heathrow
       @last_rendered_index = nil  # Force right pane refresh
 
       # Restore per-view threading mode
-      reset_threading if respond_to?(:reset_threading)
-      restore_view_thread_mode if respond_to?(:restore_view_thread_mode)
+      reset_threading
+      restore_view_thread_mode
 
       render_top_bar
 
@@ -2631,12 +2602,12 @@ module Heathrow
 
     def apply_view_filters(view)
       filters = view[:filters] || {}
-      
+
       # Check for special filter types
       if filters['special'] == 'uncategorized'
         # Get all messages first
         all_messages = @db.get_messages({}, 1000, 0, light: true)
-        
+
         # Get messages from all other configured views
         categorized_ids = Set.new
 
@@ -2648,7 +2619,7 @@ module Heathrow
             other_view[:filters].each do |key, value|
               symbolized_filters[key.to_sym] = value
             end
-            
+
             view_messages = @db.get_messages(symbolized_filters, 1000, 0, light: true)
             view_messages.each { |msg| categorized_ids.add(msg['id']) }
           end
@@ -2657,32 +2628,12 @@ module Heathrow
         # Filter to only uncategorized messages
         @filtered_messages = all_messages.reject { |msg| categorized_ids.include?(msg['id']) }
       else
-        # Parse rules-based filters into database filter format
-        db_filters = {}
-
-        if filters['rules'].is_a?(Array)
-          # Convert rules array to simple database filters
-          filters['rules'].each do |rule|
-            field = rule['field']
-            value = rule['value']
-            case rule['op']
-            when '='
-              db_filters[field.to_sym] = value
-            when 'like'
-              case field
-              when 'search'  then db_filters[:search] = value
-              when 'sender'  then db_filters[:sender_pattern] = value
-              when 'subject' then db_filters[:subject_pattern] = value
-              when 'folder'  then db_filters[:maildir_folder] = value
-              when 'label'   then db_filters[:label] = value
-              when 'source'  then db_filters[:source_name] = value
-              end
-            end
-          end
-        else
-          # Legacy simple filters - convert string keys to symbols
+        db_filters = build_db_filters(view)
+
+        # Legacy simple filters (no rules array)
+        if !filters['rules'].is_a?(Array)
           filters.each do |key, value|
-            next if key == 'rules'  # Skip rules key
+            next if key == 'rules'
             db_filters[key.to_sym] = value
           end
         end
@@ -2694,7 +2645,6 @@ module Heathrow
           ensure_all_feeds_loaded(db_filters[:source_id].to_i)
         end
       end
-
     end
     
     # Message operations
@@ -2748,7 +2698,7 @@ module Heathrow
       end
       
       # Re-render
-      render_message_list_threaded if respond_to?(:render_message_list_threaded)
+      render_message_list_threaded
       render_message_content
     end
     
@@ -2786,7 +2736,7 @@ module Heathrow
       end
       
       # Re-render
-      render_message_list_threaded if respond_to?(:render_message_list_threaded)
+      render_message_list_threaded
       
       # Restore position if we had one saved
       if restore_position
@@ -2820,7 +2770,7 @@ module Heathrow
     def toggle_tag
       msg = current_message
       return unless msg
-      return if msg['is_header'] || msg['is_channel_header'] || msg['is_thread_header'] || msg['is_dm_header']
+      return if header_message?(msg)
       return unless msg['id']
 
       if @tagged_messages.include?(msg['id'])
@@ -2847,7 +2797,7 @@ module Heathrow
 
       count = 0
       @filtered_messages.each do |msg|
-        next if msg['is_header'] || msg['is_channel_header'] || msg['is_thread_header'] || msg['is_dm_header']
+        next if header_message?(msg)
         next unless msg['id']
 
         match = [msg['sender'], msg['subject'], msg['content']].compact.any? { |f| f.match?(regex) }
@@ -2867,7 +2817,7 @@ module Heathrow
 
     # Tag/untag all messages in current view
     def tag_all_toggle
-      msgs = @filtered_messages.reject { |m| m['is_header'] || m['is_channel_header'] || m['is_thread_header'] || m['is_dm_header'] }
+      msgs = @filtered_messages.reject { |m| header_message?(m) }
       ids = msgs.map { |m| m['id'] }.compact
       if ids.all? { |id| @tagged_messages.include?(id) }
         # All tagged, untag all
@@ -2895,7 +2845,7 @@ module Heathrow
         idx = (@index + 1 + i) % display.size
         msg = display[idx]
         next unless msg
-        next if msg['is_header'] || msg['is_channel_header'] || msg['is_thread_header'] || msg['is_dm_header']
+        next if header_message?(msg)
         if msg['is_read'].to_i == 0
           @index = idx
           track_browsed_message
@@ -2914,14 +2864,14 @@ module Heathrow
         if unread_msgs.any?
           # Re-render to rebuild display_messages with uncollapsed sections
           organize_current_messages(true)
-          render_message_list_threaded if respond_to?(:render_message_list_threaded)
+          render_message_list_threaded
           new_display = @display_messages || @filtered_messages
           # Find the first visible unread after current position
           new_display.size.times do |i|
             idx = (@index + 1 + i) % new_display.size
             m = new_display[idx]
             next unless m && m['id'] && m['is_read'].to_i == 0
-            next if m['is_header'] || m['is_channel_header'] || m['is_thread_header'] || m['is_dm_header']
+            next if header_message?(m)
             @index = idx
             track_browsed_message
             render_all
@@ -2943,7 +2893,7 @@ module Heathrow
         idx = (@index - 1 - i) % size
         msg = display[idx]
         next unless msg
-        next if msg['is_header'] || msg['is_channel_header'] || msg['is_thread_header'] || msg['is_dm_header']
+        next if header_message?(msg)
         if msg['is_read'].to_i == 0
           @index = idx
           track_browsed_message
@@ -2960,14 +2910,14 @@ module Heathrow
         end
         if unread_msgs.any?
           organize_current_messages(true)
-          render_message_list_threaded if respond_to?(:render_message_list_threaded)
+          render_message_list_threaded
           new_display = @display_messages || @filtered_messages
           new_size = new_display.size
           new_size.times do |i|
             idx = (@index - 1 - i) % new_size
             m = new_display[idx]
             next unless m && m['id'] && m['is_read'].to_i == 0
-            next if m['is_header'] || m['is_channel_header'] || m['is_thread_header'] || m['is_dm_header']
+            next if header_message?(m)
             @index = idx
             track_browsed_message
             render_all
@@ -3050,10 +3000,87 @@ module Heathrow
         (msg['content'] && msg['content'] =~ /\A\s*<(!DOCTYPE|html|head|body)\b/i)
     end
 
+    def open_in_timely
+      msg = current_message
+      return unless msg
+      msg = ensure_full_message(msg)
+
+      # Try to find a date from calendar data or message timestamp
+      timely_home = File.expand_path('~/.timely')
+      return set_feedback("Timely not configured (~/.timely missing)", 196, 3) unless File.directory?(timely_home)
+
+      # Check for ICS attachment or inline calendar data
+      date_str = nil
+      meta = msg['metadata']
+      meta = JSON.parse(meta) if meta.is_a?(String) rescue nil
+      file = meta['maildir_file'] if meta.is_a?(Hash)
+
+      if file && File.exist?(file)
+        begin
+          require 'mail'
+          mail = Mail.read(file)
+          if mail.multipart?
+            mail.parts.each do |part|
+              ct = (part.content_type || '').downcase
+              if ct.include?('calendar') || ct.include?('ics')
+                ics = part.decoded
+                vevent = ics[/BEGIN:VEVENT(.*?)END:VEVENT/m, 1]
+                if vevent
+                  vevent = vevent.gsub(/\r?\n[ \t]/, '')
+                  if vevent =~ /^DTSTART;TZID=[^:]*:(\d{8})/i ||
+                     vevent =~ /^DTSTART:(\d{8})/i ||
+                     vevent =~ /^DTSTART;VALUE=DATE:(\d{8})/i
+                    d = $1
+                    date_str = "#{d[0,4]}-#{d[4,2]}-#{d[6,2]}"
+                  end
+                end
+                break
+              end
+            end
+          end
+        rescue => e
+          # Fall through to timestamp
+        end
+      end
+
+      # Fallback: use message timestamp
+      unless date_str
+        ts = msg['timestamp'].to_i
+        date_str = Time.at(ts).strftime('%Y-%m-%d') if ts > 0
+      end
+
+      return set_feedback("Could not determine date for Timely", 245, 2) unless date_str
+
+      # Write goto file for Timely
+      File.write(File.join(timely_home, 'goto'), date_str)
+
+      # Also copy ICS to incoming if it has calendar data
+      if file && File.exist?(file)
+        begin
+          incoming = File.join(timely_home, 'incoming')
+          FileUtils.mkdir_p(incoming)
+          require 'mail'
+          mail = Mail.read(file)
+          mail.parts.each do |part|
+            ct = (part.content_type || '').downcase
+            if ct.include?('calendar') || ct.include?('ics')
+              ics_file = File.join(incoming, "heathrow_#{msg['id']}.ics")
+              File.write(ics_file, part.decoded) unless File.exist?(ics_file)
+              break
+            end
+          end
+        rescue => e
+          # Non-fatal
+        end
+      end
+
+      set_feedback("Sent to Timely: #{date_str}", 156, 0)
+    end
+
     def open_message_external
       msg = current_message
       return unless msg
-      return if msg['is_header'] || msg['is_channel_header'] || msg['is_thread_header']
+      return if header_message?(msg)
       msg = ensure_full_message(msg)
 
       # Mark as read
@@ -3147,7 +3174,7 @@ module Heathrow
     def view_attachments
       msg = current_message
       return unless msg
-      return set_feedback("Select a message first", 245, 2) if msg['is_header'] || msg['is_channel_header'] || msg['is_thread_header'] || msg['is_dm_header']
+      return set_feedback("Select a message first", 245, 2) if header_message?(msg)
 
       # Load full message if needed (use numeric id only)
       msg_id = msg['id']
@@ -3354,7 +3381,7 @@ module Heathrow
           if @sort_order == 'unread'
             sort_messages
             # Reset threading to reorganize with new unread counts (preserve collapsed state)
-            reset_threading(true) if respond_to?(:reset_threading)
+            reset_threading(true)
           end
           
           # Update displays if UI is initialized
@@ -3451,6 +3478,8 @@ module Heathrow
         rows = nil
       end
 
+      organized_sections = (@show_threaded && @organizer) ? @organizer.get_organized_view(@sort_order, @sort_inverted) : nil
+
       if rows
         # Sync maildir flags for all affected messages
         count = rows.size
@@ -3465,8 +3494,8 @@ module Heathrow
           next unless list
           list.each { |m| m['is_read'] = 1 if m['id'] && m['is_read'].to_i == 0 }
         end
-        if @show_threaded && @organizer
-          @organizer.get_organized_view(@sort_order, @sort_inverted).each do |section|
+        if organized_sections
+          organized_sections.each do |section|
             next unless section[:messages]
             section[:messages].each { |m| m['is_read'] = 1 if m['is_read'].to_i == 0 }
           end
@@ -3481,14 +3510,15 @@ module Heathrow
         return if msgs.nil? || msgs.empty?
         count = mark_messages_read(msgs)
         # Also mark messages inside collapsed sections
-        if @show_threaded && @organizer
-          @organizer.get_organized_view(@sort_order, @sort_inverted).each do |section|
+        if organized_sections
+          organized_sections.each do |section|
             next unless section[:messages]
             count += mark_messages_read(section[:messages])
           end
         end
       end
 
+      invalidate_counts
       set_feedback("Marked #{count} messages as read", 156, 3)
       render_top_bar
       render_message_list
@@ -3499,7 +3529,7 @@ module Heathrow
     def mark_messages_read(msgs)
       count = 0
       msgs.each do |msg|
-        next if msg['is_header'] || msg['is_channel_header'] || msg['is_thread_header'] || msg['is_dm_header']
+        next if header_message?(msg)
         next if msg['is_read'].to_i == 1
         next unless msg['id']
         @db.mark_as_read(msg['id'])
@@ -3516,7 +3546,7 @@ module Heathrow
       return nil unless msg && @show_threaded && @organizer
 
       # If standing on a header, use its section_messages directly
-      if msg['is_header'] || msg['is_channel_header'] || msg['is_thread_header'] || msg['is_dm_header']
+      if header_message?(msg)
         return msg['section_messages'] if msg['section_messages']
       end
 
@@ -3524,7 +3554,7 @@ module Heathrow
       idx = @index - 1
       while idx >= 0
         prev = @display_messages[idx]
-        if prev && (prev['is_header'] || prev['is_channel_header'] || prev['is_thread_header'] || prev['is_dm_header'])
+        if prev && (header_message?(prev))
           return prev['section_messages'] if prev['section_messages']
           break
         end
@@ -3566,7 +3596,7 @@ module Heathrow
       # Toggle all messages in this section
       messages.each do |msg|
         next unless msg['id'] && !msg['id'].to_s.start_with?('header_')  # Skip synthetic headers
-        
+
         if all_read
           @db.mark_as_unread(msg['id'])
           msg['is_read'] = 0
@@ -3575,12 +3605,13 @@ module Heathrow
           msg['is_read'] = 1
         end
       end
-      
+      invalidate_counts
+
       # Re-sort if sorting by unread
       if @sort_order == 'unread'
         sort_messages
         # Reset threading to reorganize with new unread counts (preserve collapsed state)
-        reset_threading(true) if respond_to?(:reset_threading)
+        reset_threading(true)
       end
       
       # Update displays if UI is initialized
@@ -3601,6 +3632,7 @@ module Heathrow
 
       # Toggle the specific message
       current_read_status = msg['is_read'].to_i
+      invalidate_counts
 
       if current_read_status == 1
         success = @db.mark_as_unread(msg['id'])
@@ -3620,10 +3652,10 @@ module Heathrow
           if is_unread_view?
             if @show_threaded
               # In threaded view, need to reload filtered messages to rebuild threads
-              reset_threading if respond_to?(:reset_threading)
+              reset_threading
               @filtered_messages = @db.get_messages({is_read: false}, 1000, 0, light: true)
               sort_messages
-              organize_current_messages(force_reinit: true) if respond_to?(:organize_current_messages)
+              organize_current_messages(force_reinit: true)
               @index = [@index, filtered_messages_size - 1].min
               @index = 0 if @index < 0
             else
@@ -3640,7 +3672,7 @@ module Heathrow
       if @sort_order == 'unread'
         sort_messages
         # Reset threading to reorganize with new unread counts (preserve collapsed state)
-        reset_threading(true) if respond_to?(:reset_threading)
+        reset_threading(true)
       end
       
       # Update displays immediately if UI is initialized
@@ -3696,6 +3728,7 @@ module Heathrow
       return unless msg
       @db.toggle_star(msg['id'])
       msg['is_starred'] = msg['is_starred'] == 1 ? 0 : 1
+      invalidate_counts
 
       # Sync flagged status to Maildir file
       sync_maildir_flag(msg, 'F', msg['is_starred'] == 1)
@@ -3986,7 +4019,7 @@ module Heathrow
       @current_source_filter = "Folder: #{folder_name}"
 
       # Reset threading state so old @display_messages don't persist
-      reset_threading if respond_to?(:reset_threading)
+      reset_threading
 
       # Show progress while loading
       @panes[:bottom].text = " Loading #{folder_name}...".fg(226)
@@ -4273,7 +4306,7 @@ module Heathrow
       if @show_threaded
         @display_messages&.reject! { |m| m['id'] && filed_ids.include?(m['id']) }
         # Force organizer to rebuild from updated @filtered_messages
-        organize_current_messages(true) if respond_to?(:organize_current_messages)
+        organize_current_messages(true)
       end
       @tagged_messages.clear if @tagged_messages.size > 0
       @index = [@index, (@filtered_messages.size - 1)].min
@@ -4963,7 +4996,7 @@ module Heathrow
       @filtered_messages = results
       sort_messages
       @index = 0
-      reset_threading if respond_to?(:reset_threading)
+      reset_threading
       set_feedback("#{results.size} results for: #{query}", 156, 0)
       render_all
     end
@@ -4987,7 +5020,7 @@ module Heathrow
       else
         msg = current_message
         return unless msg
-        return if msg['is_header'] || msg['is_channel_header'] || msg['is_thread_header']
+        return if header_message?(msg)
 
         if @delete_marked.include?(msg['id'])
           @delete_marked.delete(msg['id'])
@@ -5052,7 +5085,7 @@ module Heathrow
       @delete_marked.clear
 
       # Force threaded view to rebuild with purged messages gone
-      reset_threading(true) if respond_to?(:reset_threading)
+      reset_threading(true)
 
       # Position cursor on the message above the first deleted one
       new_display = @display_messages || @filtered_messages
@@ -5100,7 +5133,7 @@ module Heathrow
       msg = ensure_full_message(msg)
 
       # Don't allow replying to header messages
-      if msg['is_header'] || msg['is_channel_header'] || msg['is_thread_header'] || msg['is_dm_header']
+      if header_message?(msg)
         set_feedback("Cannot reply to section headers. Select a message.", 226, 3)
         render_bottom_bar
         return
@@ -5241,7 +5274,7 @@ module Heathrow
     def edit_message_content
       msg = current_message
       return unless msg
-      return if msg['is_header'] || msg['is_channel_header'] || msg['is_thread_header']
+      return if header_message?(msg)
       msg = ensure_full_message(msg)
 
       source_id = msg['source_id']
@@ -5866,7 +5899,7 @@ module Heathrow
         ['%d %b %H:%M', 'DD Mon HH:MM'],
         ['%b %d %H:%M', 'Mon DD HH:MM']
       ]
-      sort_orders = ['latest', 'alphabetical', 'sender', 'conversation', 'unread', 'source']
+      sort_orders = ['latest', 'alphabetical', 'sender', 'from', 'conversation', 'unread', 'source']
       border_labels = ['none', 'right', 'both', 'left']
       # Build view choices: A, N, plus any user-defined views
       view_choices = [['A', 'All'], ['N', 'New/Unread']]
@@ -6051,9 +6084,9 @@ module Heathrow
       @panes[:right].w = @w - @panes[:left].w - 4
 
       # Reset threading for sort changes
-      reset_threading(true) if respond_to?(:reset_threading)
+      reset_threading(true)
       sort_messages
-      organize_current_messages(true) if respond_to?(:organize_current_messages)
+      organize_current_messages(true)
 
       Rcurses.clear_screen
       @panes.each_value { |p| p.cleanup if p.respond_to?(:cleanup) }
@@ -6296,11 +6329,12 @@ module Heathrow
     end
     
     def cycle_sort_order
-      # Cycle through: latest -> alphabetical -> sender -> conversation -> unread -> source -> latest
+      # Cycle through: latest -> alphabetical -> sender -> from -> conversation -> unread -> source -> latest
       @sort_order = case @sort_order
       when 'latest' then 'alphabetical'
       when 'alphabetical' then 'sender'
-      when 'sender' then 'conversation'
+      when 'sender' then 'from'
+      when 'from' then 'conversation'
       when 'conversation' then 'unread'
       when 'unread' then 'source'
       else 'latest'  # This handles 'source' and any other value
@@ -6322,13 +6356,13 @@ module Heathrow
       end
       
       # Reset threading state to force reorganization with new sort (preserve collapsed state)
-      reset_threading(true) if respond_to?(:reset_threading)
+      reset_threading(true)
       
       # Re-sort and redisplay messages
       sort_messages
       
       # Force reinit the organizer with the newly sorted messages
-      organize_current_messages(true) if respond_to?(:organize_current_messages)
+      organize_current_messages(true)
       
       @index = 0  # Reset to top
       render_all  # Re-render everything to show the new sort
@@ -6354,13 +6388,13 @@ module Heathrow
       end
       
       # Reset threading state to force reorganization with new sort (preserve collapsed state)
-      reset_threading(true) if respond_to?(:reset_threading)
+      reset_threading(true)
       
       # Re-sort and redisplay messages
       sort_messages
       
       # Force reinit the organizer with the newly sorted messages
-      organize_current_messages(true) if respond_to?(:organize_current_messages)
+      organize_current_messages(true)
       
       @index = 0  # Reset to top
       render_all  # Re-render everything
@@ -6386,7 +6420,11 @@ module Heathrow
 
       # Make sure we have a mutable array
       @filtered_messages = @filtered_messages.dup if @filtered_messages.frozen?
-      
+
+      # Pre-compute timestamp cache to avoid repeated parsing in sort comparisons
+      ts_cache = {}
+      @filtered_messages.each { |m| ts_cache[m.object_id] = timestamp_to_time(m['timestamp']) }
+
       begin
         case @sort_order
         when 'alphabetical'
@@ -6410,7 +6448,7 @@ module Heathrow
             else
               # If subjects are the same, sort by timestamp
               begin
-                timestamp_to_time(b["timestamp"]) <=> timestamp_to_time(a["timestamp"])
+                ts_cache[b.object_id] <=> ts_cache[a.object_id]
               rescue
                 0
               end
@@ -6428,12 +6466,30 @@ module Heathrow
             else
               # Safe timestamp comparison
               begin
-                timestamp_to_time(b["timestamp"]) <=> timestamp_to_time(a["timestamp"])
+                ts_cache[b.object_id] <=> ts_cache[a.object_id]
               rescue
                 0
               end
             end
           end
+        when 'from'
+          # Group by sender, most recently active sender first
+          # Within each sender group, newest message first
+          latest_per_sender = {}
+          @filtered_messages.each do |m|
+            s = display_sender(m).downcase
+            t = ts_cache[m.object_id] || Time.at(0)
+            latest_per_sender[s] = t if !latest_per_sender[s] || t > latest_per_sender[s]
+          end
+          @filtered_messages.sort! do |a, b|
+            sa = display_sender(a).downcase
+            sb = display_sender(b).downcase
+            if sa == sb
+              (ts_cache[b.object_id] || Time.at(0)) <=> (ts_cache[a.object_id] || Time.at(0))
+            else
+              (latest_per_sender[sb] || Time.at(0)) <=> (latest_per_sender[sa] || Time.at(0))
+            end
+          end
         when 'unread'
           # Sort by unread first, then by timestamp
           @filtered_messages.sort! do |a, b|
@@ -6443,7 +6499,7 @@ module Heathrow
             else
               # Safe timestamp comparison
               begin
-                timestamp_to_time(b["timestamp"]) <=> timestamp_to_time(a["timestamp"])
+                ts_cache[b.object_id] <=> ts_cache[a.object_id]
               rescue
                 0  # If timestamp parsing fails, consider them equal
               end
@@ -6460,7 +6516,7 @@ module Heathrow
               conv_cmp
             else
               begin
-                timestamp_to_time(b["timestamp"]) <=> timestamp_to_time(a["timestamp"])
+                ts_cache[b.object_id] <=> ts_cache[a.object_id]
               rescue
                 0
               end
@@ -6478,7 +6534,7 @@ module Heathrow
             else
               # Safe timestamp comparison
               begin
-                timestamp_to_time(b["timestamp"]) <=> timestamp_to_time(a["timestamp"])
+                ts_cache[b.object_id] <=> ts_cache[a.object_id]
               rescue
                 0  # If timestamp parsing fails, consider them equal
               end
@@ -6488,7 +6544,7 @@ module Heathrow
           # Sort by timestamp descending (newest first)
           @filtered_messages.sort! do |a, b|
             begin
-              timestamp_to_time(b["timestamp"]) <=> timestamp_to_time(a["timestamp"])
+              ts_cache[b.object_id] <=> ts_cache[a.object_id]
             rescue
               0  # If timestamp parsing fails, consider them equal
             end
@@ -6512,8 +6568,7 @@ module Heathrow
       @panes[:right].ix = 0  # Reset scroll position
       help_text = get_help_text  # Use the colored version directly
 
-      # Debug: write help text to file to verify colors are included
-      File.write('/tmp/heathrow_help_debug.txt', help_text) if ENV['DEBUG']
+
 
       # Just set the text and let rcurses handle everything
       @panes[:right].text = help_text

data/lib/heathrow/ui/threaded_view.rb

@@ -21,6 +21,8 @@ module Heathrow
         @all_start_collapsed = true  # Start with everything collapsed
         @section_order = nil  # Custom section order (array of names)
         @view_thread_modes = {}  # Per-view threading mode: key => {threaded:, folder:}
+        @organized_cache = nil
+        @organized_cache_key = nil
       end
       
       # Toggle between flat and threaded view
@@ -69,20 +71,10 @@ module Heathrow
       def save_view_thread_mode
         return unless @current_view
         @view_thread_modes[@current_view] = { threaded: @show_threaded, folder: @group_by_folder }
-
-        # Persist to DB
-        view = @views[@current_view]
-        if view && view[:filters].is_a?(Hash)
-          view[:filters]['view_thread_mode'] = thread_mode_key
-          @db.execute("UPDATE views SET filters = ?, updated_at = ? WHERE id = ?",
-                      [JSON.generate(view[:filters]), Time.now.to_i, view[:id]])
-        else
-          # Built-in views (A, N): store in settings table
-          @db.execute(
-            "INSERT OR REPLACE INTO settings (key, value, updated_at) VALUES (?, ?, ?)",
-            ["thread_mode_#{@current_view}", thread_mode_key, Time.now.to_i]
-          )
-        end
+        @db.execute(
+          "INSERT OR REPLACE INTO settings (key, value, updated_at) VALUES (?, ?, ?)",
+          ["thread_mode_#{@current_view}", thread_mode_key, Time.now.to_i]
+        )
       end
 
       # Restore threading mode for the active view
@@ -97,19 +89,12 @@ module Heathrow
           return true
         end
 
-        # Load from DB
-        mode_key = nil
-        view = @views[@current_view]
-        if view && view[:filters].is_a?(Hash)
-          mode_key = view[:filters]['view_thread_mode']
-        end
-        unless mode_key
-          row = @db.db.get_first_row(
-            "SELECT value FROM settings WHERE key = ?",
-            ["thread_mode_#{@current_view}"]
-          )
-          mode_key = row && row['value']
-        end
+        # Load from settings table
+        row = @db.db.get_first_row(
+          "SELECT value FROM settings WHERE key = ?",
+          ["thread_mode_#{@current_view}"]
+        )
+        mode_key = row && row['value']
         return false unless mode_key
 
         apply_thread_mode_key(mode_key)
@@ -161,6 +146,8 @@ module Heathrow
         @threading_initialized = false
         @display_messages = []
         @scroll_offset = 0  # Reset scroll position
+        @organized_cache = nil
+        @organized_cache_key = nil
 
         # Preserve or reset collapsed states
         if !preserve_collapsed_state
@@ -195,17 +182,15 @@ module Heathrow
       # Organize messages for current view
       def organize_current_messages(force_reinit = false)
         return unless @show_threaded
-        
+
         # Only organize once per message set - capture the base messages on first run
         # OR if we're forcing reinitialization after a sort change
         if !@threading_initialized || force_reinit
           @base_messages = @filtered_messages.dup
           @threading_initialized = true
-          
-          File.open('/tmp/heathrow_debug.log', 'a') do |f|
-            f.puts "THREADING: Creating organizer with #{@base_messages.size} base messages (#{force_reinit ? 'forced reinit' : 'first time'})"
-          end if ENV['DEBUG']
-          
+          @organized_cache = nil
+          @organized_cache_key = nil
+
           require_relative '../message_organizer'
           @organizer = MessageOrganizer.new(@base_messages, @db, group_by_folder: @group_by_folder)
         end
@@ -305,8 +290,15 @@ module Heathrow
         visible_messages = []
         current_index = 0
         @scroll_offset ||= 0  # Track scroll position
-        
-        organized = @organizer.get_organized_view(@sort_order, @sort_inverted)
+
+        cache_key = "#{@sort_order}|#{@sort_inverted}|#{@filtered_messages.size}"
+        if @organized_cache && @organized_cache_key == cache_key
+          organized = @organized_cache
+        else
+          organized = @organizer.get_organized_view(@sort_order, @sort_inverted)
+          @organized_cache = organized
+          @organized_cache_key = cache_key
+        end
 
         # Apply custom section order if set
         if @section_order && !@section_order.empty?
@@ -404,10 +396,6 @@ module Heathrow
           end
         end
 
-        File.open('/tmp/heathrow_debug.log', 'a') do |f|
-          f.puts "THREADING: Created #{@display_messages.size} display messages for navigation"
-        end if ENV['DEBUG']
-
         # Give full text to rcurses and use its scrolling with markers
         @panes[:left].scroll = true
         new_text = lines.join("\n")

data/lib/heathrow/version.rb

@@ -1,3 +1,3 @@
 module Heathrow
-  VERSION = '0.7.5'
+  VERSION = '0.7.7'
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: heathrow
 version: !ruby/object:Gem::Version
-  version: 0.7.5
+  version: 0.7.7
 platform: ruby
 authors:
 - Geir Isene
@@ -9,7 +9,7 @@ authors:
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2026-03-20 00:00:00.000000000 Z
+date: 2026-03-24 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rcurses