truenorth 0.6.1 → 0.7.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: bf116b555b67f39c34c1579a5d1212ee86d26c33a2d1bc88bf681d6145728b2f
-  data.tar.gz: d6ed28ab2f74be19204f331e1c37a4c925c021c00689c33d0b0a56880abdb9b1
+  metadata.gz: 8122d3972ae1ccc7041a3218e320b50665ee2df333fe998ab7127e2e3f04dec9
+  data.tar.gz: 2cb16851538859493d237c29cdb73e0d70b49cf43723fc99be4542878a2d3ad0
 SHA512:
-  metadata.gz: '079000943acd8e2f5a8cd278c5c20f52fe400d2df7189e6baa9aa4b64c7c814fa07f614af06fb1c6251fa302b125f8d1beb3cfa84b58287fdb04b8265706df7b'
-  data.tar.gz: 77a071e50db72b2c239f6b1d1beba278ff8e728aea8cd955f9a73bdd13d4535493b1412e3527fc9ebeff9fc964a8dbcff31eae2f2bf9addbf77a523965004756
+  metadata.gz: 324a9afac6e47cb66f141567ddcb65195c0f2a001cc1b48e1b24824f0aada5779d6ae620c5c2a2df60f09097ce424f7db1b30f7bc14b3f0d7479dbc988b03f4b
+  data.tar.gz: 8cfdc09c006491569b05f256096d55003d4d1a40bd3b9925ff1f1f7378745a3c657ce4cf1ed2350d7e0bb237fb485842288f631e2e82661f00f49d1eabf13685

data/lib/truenorth/cli.rb

@@ -2,6 +2,7 @@
 
 require 'thor'
 require 'date'
+require 'chronic'
 require 'tty-table'
 require 'io/console'
 require_relative '../truenorth'
@@ -114,15 +115,16 @@ module Truenorth
           say "Confirmation: #{result[:confirmation]}" if result[:confirmation]
         end
       end
-    rescue Error => e
+    rescue BookingError => e
       say "Error: #{e.message}", :red
 
-      # If no slot available, show nearby available times using HTTP client
-      if e.message.include?('No slot available')
-        http_client = Client.new(debug: options[:debug])
-        show_nearby_availability(http_client, date, time, activity)
-      end
+      # Show nearby available times using HTTP client
+      http_client = Client.new(debug: options[:debug])
+      show_nearby_availability(http_client, date, time, activity)
 
+      exit 1
+    rescue Error => e
+      say "Error: #{e.message}", :red
       exit 1
     end
 
@@ -410,67 +412,35 @@ module Truenorth
         input = input.gsub(/\b(squash|golf|music|room)\b/i, '').strip
       end
 
-      # Extract date if present
-      date = nil
-      input_lower = input.downcase
-
-      # Try patterns like "feb 15", "february 15th", "2/15"
-      if input_lower =~ /\b(jan|feb|mar|apr|may|jun|jul|aug|sep|oct|nov|dec)[a-z]*\s+(\d{1,2})(st|nd|rd|th)?\b/i
-        month_str = ::Regexp.last_match(1)
-        day = ::Regexp.last_match(2).to_i
-
-        month_map = {
-          'jan' => 1, 'feb' => 2, 'mar' => 3, 'apr' => 4,
-          'may' => 5, 'jun' => 6, 'jul' => 7, 'aug' => 8,
-          'sep' => 9, 'oct' => 10, 'nov' => 11, 'dec' => 12
-        }
-
-        month = month_map[month_str[0..2].downcase]
-        year = Date.today.year
-        year += 1 if month && month < Date.today.month # Next year if month has passed
-
-        date = Date.new(year, month, day) if month
-        input = input.gsub(/\b(jan|feb|mar|apr|may|jun|jul|aug|sep|oct|nov|dec)[a-z]*\s+(\d{1,2})(st|nd|rd|th)?\b/i, '').strip
-      elsif input =~ %r{\b(\d{1,2})/(\d{1,2})\b}
-        month = ::Regexp.last_match(1).to_i
-        day = ::Regexp.last_match(2).to_i
-        year = Date.today.year
-        date = Date.new(year, month, day)
-        input = input.gsub(%r{\b\d{1,2}/\d{1,2}\b}, '').strip
-      end
+      # Normalize shorthand am/pm ("4:45p" -> "4:45pm", "10a" -> "10am")
+      input = input.gsub(/(\d)\s*([ap])\b(?!m)/i) { "#{::Regexp.last_match(1)}#{::Regexp.last_match(2)}m" }
 
-      # Remove day names (monday, tuesday, etc.)
-      input = input.gsub(/\b(monday|tuesday|wednesday|thursday|friday|saturday|sunday)\b/i, '').strip
+      # Use Chronic for natural language date/time parsing
+      parsed_time = Chronic.parse(input, context: :future)
 
-      # Extract time - look for patterns like "10am", "10:00am", "10:00 AM", "10"
-      time = nil
-      if input =~ /\b(\d{1,2})(:(\d{2}))?\s*(am|pm)?\b/i
-        hour = ::Regexp.last_match(1).to_i
-        minute = ::Regexp.last_match(3) || '00'
-        period = ::Regexp.last_match(4) || (hour < 8 ? 'PM' : 'AM')
-
-        time = "#{hour}:#{minute} #{period.upcase}"
+      if parsed_time
+        date = parsed_time.to_date
+        time = parsed_time.strftime('%-l:%M %p')
+      else
+        # Fallback: use date option and treat input as time
+        date = parse_date(date_option)
+        time = input.strip
       end
 
-      # Use provided date option if date wasn't extracted
-      date ||= parse_date(date_option)
-
-      # If no time found, use the remaining input as-is
-      time ||= input.strip
-
       { date: date, time: time, activity: activity }
     end
 
     def parse_date(date_str)
       return Date.today if date_str.nil? || date_str.empty?
 
-      case date_str
-      when /^\+(\d+)$/
-        Date.today + ::Regexp.last_match(1).to_i
-      when 'today'
-        Date.today
-      when 'tomorrow'
-        Date.today + 1
+      # Handle +N shorthand for days from today
+      if date_str =~ /^\+(\d+)$/
+        return Date.today + ::Regexp.last_match(1).to_i
+      end
+
+      parsed = Chronic.parse(date_str, context: :future)
+      if parsed
+        parsed.to_date
       else
         Date.parse(date_str)
       end

data/lib/truenorth/client.rb

@@ -56,6 +56,7 @@ module Truenorth
       @debug = debug
       @debug_log = StringIO.new
       @logged_in = !@cookies.empty?  # If we have cookies, will verify on first use
+      @login_time = nil
       @last_verified_response = nil
 
       log "Loaded #{@cookies.length} cookies from cache" if @logged_in && @debug
@@ -103,6 +104,7 @@ module Truenorth
 
       if response.body.include?('Sign Out') || response.body.include?('My Reservations')
         @logged_in = true
+        @login_time = Time.now
         Config.save_cookies(@cookies)
         log 'Login successful (cookies saved)'
         true
@@ -123,70 +125,37 @@ module Truenorth
       response = get(BOOKING_PATH)
       html = Nokogiri::HTML(response.body)
 
-      # Navigate to the requested date first if needed
-      current_date = html.at_css('input[name*="sheetDate"]')&.[]('value')
       requested_date = date.strftime('%m/%d/%Y')
       activity_id = ACTIVITIES[activity.to_s.downcase] || '5'
 
-      if current_date != requested_date
-        log "Navigating from #{current_date} to #{requested_date}"
-        html = change_date(html, requested_date, activity_id)
-      end
-
-      # Then change activity type - this should now return slots for the correct date
-      current_activity = html.at_css('input[name*="activityId"]')&.[]('value')
-
-      if current_activity && current_activity != activity_id
-        log "Changing activity from #{current_activity} to #{activity_id}"
-        html = change_activity(html, activity_id)
-      end
+      # Preserve full-page form state before AJAX navigation
+      form_id = extract_form_id(html)
+      view_state = extract_view_state(html)
+      components = extract_primefaces_components(html)
+      form_fields = extract_all_form_fields(html, form_id)
 
-      # Debug: Save HTML after activity change
-      if @debug
-        debug_file = "/tmp/truenorth_after_activity_#{Time.now.to_i}.html"
-        File.write(debug_file, html.to_s)
-        log "Saved HTML after activity change to: #{debug_file}"
+      # Step 1: Navigate to the correct date
+      log "Navigating to #{requested_date}"
+      html = change_date(html, requested_date, activity_id)
+      view_state = extract_view_state(html) || view_state
+
+      # Step 2: Change activity type
+      # The dateSelect event only changes the date; a separate change event
+      # is needed to switch the activity (e.g., from golf to squash).
+      log "Changing activity to #{activity_id} (#{activity})"
+      updated_fields = extract_all_form_fields(html, form_id)
+      updated_fields = form_fields.dup if updated_fields.empty?
+      updated_fields["#{form_id}:activityId"] = activity_id
+      updated_fields["#{form_id}:sheetDate"] = requested_date
+
+      result = change_activity_ajax(form_id, view_state, activity_id, updated_fields, components)
+      if result[:success]
+        view_state = result[:view_state] || view_state
+        html = parse_ajax_cdata(result[:body]) || html
       end
 
-      # Court dropdown not available in HTML response - use hardcoded IDs
-      court_ids = COURT_IDS_BY_ACTIVITY[activity.to_s.downcase] || []
-
-      if court_ids.length >= 3
-        log "Querying #{court_ids.length} courts individually (IDs: #{court_ids.join(', ')})"
-        all_slots = {}
-
-        court_ids.each do |court_id|
-          court_name = COURTS[court_id] || "Court #{court_id}"
-          log "Querying #{court_name} (ID: #{court_id})"
-
-          # Set activityAreaId and refresh
-          form_id = extract_form_id(html)
-          view_state = extract_view_state(html)
-          form_fields = extract_all_form_fields(html, form_id)
-          form_fields["#{form_id}:activityAreaId"] = court_id
-
-          # Make AJAX request to update table for this court
-          result = change_date_ajax(form_id, view_state, requested_date, form_fields)
-          if result[:success]
-            court_html = Nokogiri::HTML(result[:body].scan(/<!\[CDATA\[(.*?)\]\]>/m).flatten.join("\n"))
-            court_slots = parse_slots(court_html)
-
-            # Merge slots
-            court_slots.each do |time, courts|
-              all_slots[time] ||= []
-              all_slots[time].concat(courts)
-              all_slots[time].uniq!
-            end
-          end
-        end
-
-        slots = all_slots
-        log "Combined #{slots.count} time slots from all courts"
-      else
-        # Fallback to original parsing
-        slots = parse_slots(html)
-        log "Found #{slots.count} available time slots"
-      end
+      slots = parse_slots(html)
+      log "Found #{slots.count} available time slots"
 
       {
         success: true,
@@ -216,18 +185,28 @@ module Truenorth
 
       raise BookingError, 'Could not extract form state' unless view_state && form_id
 
-      # Change activity if needed
       activity_id = ACTIVITIES[activity.to_s.downcase] || '5'
-      current_activity = form_fields["#{form_id}:activityId"]
+      requested_date = date.strftime('%m/%d/%Y')
+
+      # Step 1: Navigate to the correct date
+      log "Navigating to #{requested_date}"
+      html = change_date(html, requested_date, activity_id)
+      view_state = extract_view_state(html) || view_state
 
-      if current_activity != activity_id
-        log "Changing activity from #{current_activity} to #{activity_id}"
-        result = change_activity_ajax(form_id, view_state, activity_id, form_fields, components)
-        raise BookingError, 'Failed to change activity type' unless result[:success]
+      # Step 2: Change activity type (dateSelect only changes date, not activity)
+      log "Changing activity to #{activity_id} (#{activity})"
+      updated_fields = extract_all_form_fields(html, form_id)
+      updated_fields = form_fields.dup if updated_fields.empty?
+      updated_fields["#{form_id}:activityId"] = activity_id
+      updated_fields["#{form_id}:sheetDate"] = requested_date
 
+      result = change_activity_ajax(form_id, view_state, activity_id, updated_fields, components)
+      if result[:success]
         view_state = result[:view_state] || view_state
-        html = Nokogiri::HTML(result[:body]) if result[:body]
+        html = parse_ajax_cdata(result[:body]) || html
+        components = extract_components_from_ajax(result[:body]) if result[:body]
         form_fields = extract_all_form_fields(html, form_id)
+        form_fields = updated_fields if form_fields.empty?
       end
 
       # Find the slot (or use provided slot_info)
@@ -247,14 +226,23 @@ module Truenorth
         log "Found slot: #{slot[:court]} at #{slot[:start_time]}"
       end
 
-      # Select slot via AJAX
+      # Select slot via AJAX (opens reservation dialog, starts hold timer)
       select_result = select_slot_ajax(form_id, view_state, slot, components, form_fields)
       raise BookingError, 'Failed to select slot' unless select_result[:success]
 
       new_view_state = select_result[:view_state] || view_state
       new_components = select_result[:components] || components
+
+      # Extract ALL form fields from the select response (includes clientId, dialog fields)
       dialog_fields = extract_fields_from_ajax_response(select_result[:body], form_id) if select_result[:body]
 
+      # Verify hold timer started (confirms server accepted the slot selection)
+      if select_result[:body]&.include?('startHoldTimeTimer')
+        log 'Slot hold timer started - slot reserved temporarily'
+      else
+        log 'WARNING: No hold timer detected in select response'
+      end
+
       if dry_run
         return {
           success: true,
@@ -480,11 +468,17 @@ module Truenorth
     private
 
     def ensure_logged_in!
+      # Skip verification if we logged in recently (within 5 minutes)
+      if @logged_in && @login_time && (Time.now - @login_time < 300)
+        return
+      end
+
       if @logged_in
         # Verify cached session is still valid with a lightweight check
         response = get(RESERVATIONS_PATH)
         if authenticated_response?(response)
           @last_verified_response = response
+          @login_time = Time.now  # Reset timer on successful verification
           return
         end
 
@@ -580,7 +574,23 @@ module Truenorth
     def find_slot(html, target_time, preferred_court = nil)
       target_normalized = normalize_time(target_time)
 
-      html.css('td.slot.open div[data-start-time]').each do |div|
+      all_slots = html.css('td.slot.open div[data-start-time]')
+      sample_times = all_slots.first(5).map { |d| d['data-start-time'] }
+      log "find_slot: looking for '#{target_normalized}', found #{all_slots.length} open slot divs, sample times: #{sample_times.join(', ')}"
+      if all_slots.length.zero?
+        # Try without 'open' class to see what's there
+        any_slots = html.css('td.slot div[data-start-time]')
+        log "find_slot: #{any_slots.length} total slot divs (including non-open)"
+        if any_slots.length.positive?
+          sample_td = any_slots.first.parent
+          sample_td = sample_td.parent while sample_td && sample_td.name != 'td'
+          log "find_slot: sample td classes: '#{sample_td&.[]('class')}'"
+          times = any_slots.first(3).map { |d| d['data-start-time'] }
+          log "find_slot: sample times: #{times.join(', ')}"
+        end
+      end
+
+      all_slots.each do |div|
         slot_time = normalize_time(div['data-start-time'])
         next unless slot_time == target_normalized
 
@@ -754,12 +764,18 @@ module Truenorth
     end
 
     def change_date(html, date_str, activity_id = nil)
-      # Extract what we can from the HTML
       form_id = extract_form_id(html)
       view_state = extract_view_state(html)
 
-      # Build minimal form data manually to avoid extraction issues
-      form_data = build_minimal_form_data(html, form_id, activity_id)
+      # Use full form fields (not just hidden) so dynamic ID detection works
+      form_data = extract_all_form_fields(html, form_id)
+      if form_data.empty?
+        form_data = build_minimal_form_data(html, form_id, activity_id)
+      elsif activity_id && form_id
+        form_data["#{form_id}:activityId"] = activity_id.to_s
+        activity_source = find_activity_source(form_data, form_id)
+        form_data["#{activity_source}_input"] = activity_id.to_s
+      end
 
       result = change_date_ajax(form_id, view_state, date_str, form_data)
       return html unless result[:success]
@@ -809,7 +825,9 @@ module Truenorth
       # Override activity ID if provided
       if activity_id
         data["#{form_id}:activityId"] = activity_id.to_s
-        data["#{form_id}:j_idt51_input"] = activity_id.to_s
+        # Also set the activity dropdown input (dynamically detected)
+        activity_source = find_activity_source(data, form_id)
+        data["#{activity_source}_input"] = activity_id.to_s
       end
 
       data
@@ -838,6 +856,33 @@ module Truenorth
       components
     end
 
+    # Dynamically find the activity dropdown component ID (PrimeFaces selectOneMenu).
+    # SelectOneMenu has both _focus and _input suffixed fields; calendar only has _input.
+    def find_activity_source(form_fields, form_id)
+      prefix = "#{form_id}:"
+      form_fields.each_key do |key|
+        next unless key.start_with?(prefix) && key.end_with?('_focus')
+
+        base = key.sub(/_focus$/, '')
+        return base if form_fields.key?("#{base}_input") && base =~ /j_idt\d+$/
+      end
+      "#{form_id}:j_idt67" # fallback
+    end
+
+    # Dynamically find the date picker component ID (PrimeFaces calendar).
+    # Calendar has _input but no _focus (selectOneMenu has both).
+    def find_date_picker_source(form_fields, form_id)
+      prefix = "#{form_id}:"
+      form_fields.each_key do |key|
+        next unless key.start_with?(prefix) && key.end_with?('_input') && key =~ /j_idt\d+_input$/
+
+        base = key.sub(/_input$/, '')
+        # Calendar has _input but no _focus
+        return base unless form_fields.key?("#{base}_focus")
+      end
+      "#{form_id}:j_idt79" # fallback
+    end
+
     def extract_all_form_fields(html, form_id)
       form = html.at_css("form[id='#{form_id}']")
       return {} unless form
@@ -888,18 +933,16 @@ module Truenorth
     end
 
     def change_activity_ajax(form_id, view_state, activity_id, form_fields, _components)
-      activity_dropdown = "#{form_id}:j_idt57"
+      activity_dropdown = find_activity_source(form_fields, form_id)
       ajax_url = build_ajax_url
       encoded_url = URI.encode_www_form_component(ajax_url)
 
       form_data = form_fields.dup
-      form_data["#{form_id}:j_idt51_input"] = activity_id
+      form_data["#{activity_dropdown}_input"] = activity_id
       form_data["#{form_id}:activityId"] = activity_id
       form_data["#{form_id}:showAllAreasOrTrainers"] = 'true'  # Show all courts!
 
-      # Debug: Log what we're sending
-      log "Sending to #{form_id}:showAllAreasOrTrainers = #{form_data["#{form_id}:showAllAreasOrTrainers"]}"
-      log "Sending to #{form_id}:mobileViewDisplay = #{form_data["#{form_id}:mobileViewDisplay"]}"
+      log "Activity dropdown source: #{activity_dropdown}"
 
       form_data.merge!(
         'javax.faces.partial.ajax' => 'true',
@@ -974,16 +1017,16 @@ module Truenorth
     def change_date_ajax(form_id, view_state, date_str, form_data)
       return { success: false, error: 'No form_id' } unless form_id
 
-      date_picker = "#{form_id}:j_idt57"
+      date_picker = find_date_picker_source(form_data, form_id)
       ajax_url = build_ajax_url
       encoded_url = URI.encode_www_form_component(ajax_url)
 
-      # Use form_data as-is (already built correctly by build_minimal_form_data)
-      # Just add our date values and AJAX control parameters
+      log "Date picker source: #{date_picker}"
+
       form_data = form_data.dup
       form_data["#{form_id}:sheetDate"] = date_str
-      form_data["#{form_id}:j_idt57_input"] = date_str
-      form_data["#{form_id}:showAllAreasOrTrainers"] = 'true'  # Show all courts!
+      form_data["#{date_picker}_input"] = date_str
+      form_data["#{form_id}:showAllAreasOrTrainers"] = 'true'
 
       form_data.merge!(
         'javax.faces.partial.ajax' => 'true',
@@ -1006,20 +1049,31 @@ module Truenorth
     end
 
     def select_slot_ajax(form_id, view_state, slot, components, form_fields)
-      source_id = components['showReservationScreen'] || "#{form_id}:j_idt146"
+      source_id = components['showReservationScreen']
+      unless source_id
+        # The showReservationScreen function is triggered when clicking a slot.
+        # Look for it in the page JavaScript or use a dynamic approach.
+        source_id = "#{form_id}:j_idt146"
+        log "select_slot: using fallback source_id #{source_id} (showReservationScreen not in components)"
+      end
+      log "select_slot source_id: #{source_id}"
       ajax_url = build_ajax_url
       encoded_url = URI.encode_www_form_component(ajax_url)
 
       form_data = form_fields.dup
+      # Set the activityAreaId form field to the slot's area
+      form_data["#{form_id}:activityAreaId"] = slot[:area_id]
       form_data.merge!(
         'javax.faces.partial.ajax' => 'true',
         'javax.faces.source' => source_id,
-        'javax.faces.partial.execute' => '@all',
+        # PrimeFaces uses p: sourceId, meaning only the source component is processed
+        'javax.faces.partial.execute' => source_id,
         'javax.faces.partial.render' => form_id,
         source_id => source_id,
         form_id => form_id,
         'javax.faces.encodedURL' => encoded_url,
         'javax.faces.ViewState' => view_state,
+        # These are PrimeFaces pa (params) - sent as plain form data
         'activityAreaId' => slot[:area_id],
         'startTime' => slot[:start_time],
         'endTime' => slot[:end_time]
@@ -1028,6 +1082,10 @@ module Truenorth
       response = post_ajax(ajax_url, form_data)
       if response.is_a?(Net::HTTPSuccess)
         new_components = extract_components_from_ajax(response.body)
+        log "select_slot response components: #{new_components.inspect}"
+        # Also try to find save button dynamically from the dialog HTML
+        new_components['saveButton'] ||= find_save_button_id(response.body)
+        log "select_slot save button: #{new_components['saveButton'] || 'NOT FOUND'}"
         { success: true, view_state: extract_view_state_from_ajax(response.body), components: new_components, body: response.body }
       else
         { success: false, error: "HTTP #{response.code}" }
@@ -1035,15 +1093,16 @@ module Truenorth
     end
 
     def save_booking_ajax(form_id, view_state, slot, components, dialog_fields)
-      save_button_id = components['saveButton'] || "#{form_id}:j_idt378"
+      save_button_id = components['saveButton']
+      unless save_button_id
+        log 'WARNING: Save button not found in components, cannot save booking'
+        return { success: false, error: 'Save button not found - slot selection may have failed' }
+      end
+      log "save_booking using button: #{save_button_id}"
       ajax_url = build_ajax_url
       encoded_url = URI.encode_www_form_component(ajax_url)
 
       form_data = (dialog_fields || {}).dup
-      form_data["#{form_id}:selectedSlotId"] = slot[:id]
-      form_data["#{form_id}:selectedAreaId"] = slot[:area_id]
-      form_data["#{form_id}:selectedStartTime"] = slot[:start_time]
-      form_data["#{form_id}:selectedEndTime"] = slot[:end_time]
       form_data.merge!(
         'javax.faces.partial.ajax' => 'true',
         'javax.faces.source' => save_button_id,
@@ -1058,39 +1117,100 @@ module Truenorth
       response = post_ajax(ajax_url, form_data)
       if response.is_a?(Net::HTTPSuccess)
         body = response.body
-        body_lower = body.downcase
-
-        # Check for explicit error indicators
-        if body_lower.include?('error') || body_lower.include?('failed') ||
-           body_lower.include?('unable') || body_lower.include?('invalid')
-          log "Booking save returned error indicators in response"
-          { success: false, error: 'Booking save failed' }
-        # Check for success indicators OR assume success if reasonable response
-        elsif body_lower.include?('success') || body_lower.include?('confirmed') ||
-              body_lower.include?('booked') || body_lower.include?('reserved') ||
-              (body.length < 5000 && !body_lower.include?('exception'))
-          log "Booking save appears successful (HTTP 200)"
-          { success: true, confirmation: 'Booking confirmed' }
-        else
-          log "Uncertain booking result, response length: #{body.length}"
-          { success: false, error: 'No confirmation in response' }
-        end
+        parse_save_response(body)
       else
         { success: false, error: "HTTP #{response.code}" }
       end
     end
 
+    def parse_save_response(body)
+      # Check for advance booking restriction overlay
+      if body =~ /advance/i && body =~ /not allowed|only be made/i
+        restriction = body.match(/Advance reservation[^.]*\./i)&.[](0) ||
+                      body.match(/Reservations can only be made[^.]*\./i)&.[](0)
+        log "Booking rejected: #{restriction || 'advance booking restriction'}"
+        return { success: false, error: restriction || 'Advance booking restriction' }
+      end
+
+      # Check for PrimeFaces error messages (ui-messages-error or growl)
+      if body.include?('ui-messages-error') || body =~ /severity["']?\s*:\s*["']?error/i
+        error_html = Nokogiri::HTML(body)
+        error_text = error_html.at_css('.ui-messages-error-detail, .ui-growl-message')&.text&.strip
+        log "Booking save returned error: #{error_text || 'unknown'}"
+        return { success: false, error: error_text || 'Booking save failed' }
+      end
+
+      # Check for server exception
+      if body.include?('exception') && body.include?('stacktrace')
+        log 'Booking save returned server exception'
+        return { success: false, error: 'Server error during booking' }
+      end
+
+      # Check for restriction/warning overlays
+      if body =~ /not allowed|cannot be booked|restriction|already reserved/i
+        msg = body.match(/((?:not allowed|cannot be booked|restriction|already reserved)[^.<]*)/i)&.[](0)
+        log "Booking rejected: #{msg}"
+        return { success: false, error: msg || 'Booking not allowed' }
+      end
+
+      # HTTP 200 with no error indicators = success
+      log "Booking save appears successful (HTTP 200, #{body.length} bytes)"
+      { success: true, confirmation: 'Booking confirmed' }
+    end
+
+    # Find the save button ID dynamically from the AJAX response body.
+    # Parses CDATA sections with Nokogiri for reliable detection.
+    def find_save_button_id(response_body)
+      return nil unless response_body
+
+      # Parse each CDATA update section with Nokogiri
+      response_body.scan(/<update[^>]*>\s*<!\[CDATA\[(.*?)\]\]>/m) do |match|
+        frag = Nokogiri::HTML::DocumentFragment.parse(match[0])
+
+        # Look for btn-save class on buttons or links
+        save_el = frag.at_css('.btn-save[id]') ||
+                  frag.at_css('button[id]') { |el| el.text.strip =~ /\bSave\b/i } ||
+                  frag.at_css('a[id]') { |el| el.text.strip =~ /\bSave\b/i }
+        return save_el['id'] if save_el&.[]('id')
+      end
+
+      # Fallback: regex for btn-save (handles either class-before-id or id-before-class)
+      if (match = response_body.match(/class="[^"]*btn-save[^"]*"[^>]*id="([^"]+)"/))
+        return match[1]
+      end
+      if (match = response_body.match(/id="([^"]+)"[^>]*class="[^"]*btn-save/))
+        return match[1]
+      end
+
+      nil
+    end
+
     def extract_components_from_ajax(response_body)
       components = {}
-      if (save_match = response_body.match(/id="([^"]+)"[^>]*class="[^"]*btn-save/))
-        components['saveButton'] = save_match[1]
-      end
+
+      # Find save button via Nokogiri CDATA parsing
+      save_id = find_save_button_id(response_body)
+      components['saveButton'] = save_id if save_id
+
+      # Extract rc_ PrimeFaces remote command functions
       response_body.scan(/rc_(\w+)\s*=\s*function\(\)\s*\{PrimeFaces\.ab\(\{s:"([^"]+)"/) do |match|
         components[match[0]] = match[1]
       end
       components
     end
 
+    # Parse CDATA content from a PrimeFaces AJAX response
+    def parse_ajax_cdata(body)
+      return nil unless body
+
+      cdata_content = body.scan(/<!\[CDATA\[(.*?)\]\]>/m).flatten.join("\n")
+      has_slot = cdata_content.include?('slot')
+      log "parse_ajax_cdata: #{cdata_content.length} bytes, has slot: #{has_slot}"
+      return nil if cdata_content.empty? || !has_slot
+
+      Nokogiri::HTML(cdata_content)
+    end
+
     def extract_view_state_from_ajax(response_body)
       match = response_body.match(/ViewState[^>]*>(?:<!\[CDATA\[)?([^<\]]+)/)
       match&.[](1)

data/lib/truenorth/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module Truenorth
-  VERSION = '0.6.1'
+  VERSION = '0.7.0'
 end

data/truenorth.gemspec

@@ -30,6 +30,7 @@ Gem::Specification.new do |spec|
   spec.require_paths = ['lib']
 
   spec.add_dependency 'base64', '~> 0.2'
+  spec.add_dependency 'chronic', '~> 0.10'
   spec.add_dependency 'ferrum', '~> 0.15'
   spec.add_dependency 'nokogiri', '~> 1.15'
   spec.add_dependency 'thor', '~> 1.3'

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: truenorth
 version: !ruby/object:Gem::Version
-  version: 0.6.1
+  version: 0.7.0
 platform: ruby
 authors:
 - usiegj00
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2026-02-09 00:00:00.000000000 Z
+date: 2026-02-10 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: base64
@@ -24,6 +24,20 @@ dependencies:
     - - "~>"
       - !ruby/object:Gem::Version
         version: '0.2'
+- !ruby/object:Gem::Dependency
+  name: chronic
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '0.10'
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '0.10'
 - !ruby/object:Gem::Dependency
   name: ferrum
   requirement: !ruby/object:Gem::Requirement