puppeteer-ruby 0.50.0.alpha8 → 0.50.0.alpha10

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: cc07137db57af2ee149486e37d5ee9268baf3f928cd2e51e8e2bfb733bf3aaf3
-  data.tar.gz: 36f6b3d362e99c2c5b4a52a2039264b078f605e0ba249acbf8116b1706ec724a
+  metadata.gz: 6ad92e4a471db943a70ea1f6e928ba29e611cba0fc4b62b001bafb0cf444c797
+  data.tar.gz: 226103f1934651f038099e58a7b94414e248db9290f928c4fc2d4b262df59807
 SHA512:
-  metadata.gz: 1e89b2eed4753830edb5dcd8c54e6d923a24fd802c2e57bf4c7c92ebf9e5e620d4fda0a1096511177e5ba689785aa0ff8d56d2c024d8469bd75a729df738e8fe
-  data.tar.gz: 32cd0c2eb2e677c661b74801d6922a3876a3454557f0d12f7468708951f44e789404d389d041dfed17cada032101e5b4efc513fb9cf41d93ee1a6b130975b4b9
+  metadata.gz: 1ba290b15ba557824715b8aa82ad8695cdc9d08200381c7e004cb3c3fe63e041bc732e5e27f904e6bf75ff4eef51e6fdb905972d4225cfa495bbde9637a5f2ac
+  data.tar.gz: c7a208b38765265b77cef6f81dc1945daa6b4f4c09957c0b9b8e20484630a20a0359bbd3bbde26d5d88c1b55ea5c17cb02d522e358e294aa3db814685559a736

data/docs/api_coverage.md

@@ -1,6 +1,6 @@
 # API coverages
 - Puppeteer version: v24.35.0
-- puppeteer-ruby version: 0.50.0.alpha4
+- puppeteer-ruby version: 0.50.0.alpha10
 
 ## Puppeteer
 

data/lib/puppeteer/connection.rb

@@ -1,4 +1,5 @@
 require 'json'
+require 'async/queue'
 
 class Puppeteer::Connection
   include Puppeteer::DebugPrint
@@ -44,11 +45,16 @@ class Puppeteer::Connection
     @delay = delay
     @protocol_timeout = protocol_timeout
 
+    @network_message_queue = Async::Queue.new
+    @network_message_task = nil
+
     @transport = transport
     @transport.on_message do |data|
       message = JSON.parse(data)
       sleep_before_handling_message(message)
-      if should_handle_synchronously?(message)
+      if network_event_message?(message)
+        enqueue_network_message(message)
+      elsif should_handle_synchronously?(message)
         handle_message(message)
       else
         async_handle_message(message)
@@ -87,7 +93,8 @@ class Puppeteer::Connection
     when nil
       false
     when /^Network\./
-      true
+      # Network events are queued for ordered async processing.
+      false
     when /^Page\.frame/
       # Page.frameAttached
       # Page.frameNavigated
@@ -112,6 +119,34 @@ class Puppeteer::Connection
     end
   end
 
+  private def network_event_message?(message)
+    message['method']&.start_with?('Network.')
+  end
+
+  private def ensure_network_message_task
+    return if @network_message_task&.alive?
+    return unless Async::Task.current?
+
+    @network_message_task = Async do
+      while (queued = @network_message_queue.dequeue)
+        handle_message(queued)
+      end
+    end
+  end
+
+  private def enqueue_network_message(message)
+    if Async::Task.current?
+      ensure_network_message_task
+      begin
+        @network_message_queue.enqueue(message)
+      rescue Async::Queue::ClosedError
+        # Connection is closing; ignore late network events.
+      end
+    else
+      handle_message(message)
+    end
+  end
+
   def self.from_session(session)
     session.connection
   end
@@ -310,6 +345,12 @@ class Puppeteer::Connection
     @closed = true
     @transport.on_message
     @transport.on_close
+    @network_message_queue.close
+    begin
+      @network_message_task&.stop
+    rescue Async::Stop
+      # Task already stopping; ignore.
+    end
     callbacks = @callbacks_mutex.synchronize do
       @callbacks.values.tap { @callbacks.clear }
     end

data/lib/puppeteer/frame_manager.rb

@@ -34,32 +34,49 @@ class Puppeteer::FrameManager
 
     # Keeps track of frames that are in the process of being attached in #onFrameAttached.
     @frames_pending_attachment = {}
+    @frame_tree_handled_promise = nil
+    @queued_frame_events = []
+    @frame_tree_mutex = Mutex.new
 
     setup_listeners(@client)
   end
 
   private def setup_listeners(client)
     client.on_event('Page.frameAttached') do |event|
-      handle_frame_attached(client, event['frameId'], event['parentFrameId'])
+      with_frame_tree_handled do
+        handle_frame_attached(client, event['frameId'], event['parentFrameId'])
+      end
     end
     client.on_event('Page.frameNavigated') do |event|
-      @frame_naviigated_received << event['frame']['id']
-      handle_frame_navigated(event['frame'])
+      with_frame_tree_handled do
+        @frame_naviigated_received << event['frame']['id']
+        handle_frame_navigated(event['frame'])
+      end
     end
     client.on_event('Page.navigatedWithinDocument') do |event|
-      handle_frame_navigated_within_document(event['frameId'], event['url'])
+      with_frame_tree_handled do
+        handle_frame_navigated_within_document(event['frameId'], event['url'])
+      end
     end
     client.on_event('Page.frameDetached') do |event|
-      handle_frame_detached(event['frameId'], event['reason'])
+      with_frame_tree_handled do
+        handle_frame_detached(event['frameId'], event['reason'])
+      end
     end
     client.on_event('Page.frameStartedLoading') do |event|
-      handle_frame_started_loading(event['frameId'])
+      with_frame_tree_handled do
+        handle_frame_started_loading(event['frameId'])
+      end
     end
     client.on_event('Page.frameStoppedLoading') do |event|
-      handle_frame_stopped_loading(event['frameId'])
+      with_frame_tree_handled do
+        handle_frame_stopped_loading(event['frameId'])
+      end
     end
     client.on_event('Runtime.executionContextCreated') do |event|
-      handle_execution_context_created(event['context'], client)
+      with_frame_tree_handled do
+        handle_execution_context_created(event['context'], client)
+      end
     end
     client.on_event('Runtime.executionContextDestroyed') do |event|
       handle_execution_context_destroyed(event['executionContextId'], client)
@@ -68,7 +85,9 @@ class Puppeteer::FrameManager
       handle_execution_contexts_cleared(client)
     end
     client.on_event('Page.lifecycleEvent') do |event|
-      handle_lifecycle_event(event)
+      with_frame_tree_handled do
+        handle_lifecycle_event(event)
+      end
     end
   end
 
@@ -78,6 +97,7 @@ class Puppeteer::FrameManager
     @frames_pending_target_init[target_id] ||= Async::Promise.new
     client = cdp_session || @client
 
+    prepare_frame_tree_handling
     promises = [
       client.async_send_message('Page.enable'),
       client.async_send_message('Page.getFrameTree'),
@@ -85,6 +105,7 @@ class Puppeteer::FrameManager
     results = Puppeteer::AsyncUtils.await_promise_all(*promises)
     frame_tree = results[1]['frameTree']
     handle_frame_tree(client, frame_tree)
+    finish_frame_tree_handling
     Puppeteer::AsyncUtils.await_promise_all(
       client.async_send_message('Page.setLifecycleEventsEnabled', enabled: true),
       client.async_send_message('Runtime.enable'),
@@ -93,10 +114,14 @@ class Puppeteer::FrameManager
     @network_manager.init unless cdp_session
   rescue => err
     # The target might have been closed before the initialization finished.
-    return if err.message.include?('Target closed') || err.message.include?('Session closed')
+    if err.message.include?('Target closed') || err.message.include?('Session closed')
+      finish_frame_tree_handling
+      return
+    end
 
     raise
   ensure
+    finish_frame_tree_handling if @frame_tree_handled_promise && !@frame_tree_handled_promise.resolved?
     @frames_pending_target_init.delete(target_id)&.resolve(nil)
   end
 
@@ -220,6 +245,46 @@ class Puppeteer::FrameManager
     emit_event(FrameManagerEmittedEvents::LifecycleEvent, frame)
   end
 
+  private def with_frame_tree_handled(&block)
+    promise = @frame_tree_handled_promise
+    if promise && !promise.resolved?
+      queued = false
+      @frame_tree_mutex.synchronize do
+        promise = @frame_tree_handled_promise
+        if promise && !promise.resolved?
+          @queued_frame_events << block
+          queued = true
+        end
+      end
+      return if queued
+    end
+    block.call
+  end
+
+  private def prepare_frame_tree_handling
+    queued = nil
+    @frame_tree_mutex.synchronize do
+      if @frame_tree_handled_promise && !@frame_tree_handled_promise.resolved?
+        @frame_tree_handled_promise.resolve(nil)
+        queued = @queued_frame_events
+      end
+      @queued_frame_events = []
+      @frame_tree_handled_promise = Async::Promise.new
+    end
+    queued&.each(&:call)
+  end
+
+  private def finish_frame_tree_handling
+    queued = nil
+    @frame_tree_mutex.synchronize do
+      return unless @frame_tree_handled_promise
+      @frame_tree_handled_promise.resolve(nil) unless @frame_tree_handled_promise.resolved?
+      queued = @queued_frame_events
+      @queued_frame_events = []
+    end
+    queued&.each(&:call)
+  end
+
   # @param frame_id [String]
   def handle_frame_started_loading(frame_id)
     frame = @frames[frame_id]
@@ -352,8 +417,11 @@ class Puppeteer::FrameManager
     if is_main_frame
       if frame
         # Update frame id to retain frame identity on cross-process navigation.
-        @frames.delete(frame.id)
-        frame.id = frame_id
+        if frame.id != frame_id
+          old_frame_id = frame.id
+          @frames.delete(old_frame_id)
+          frame.id = frame_id
+        end
       else
         # Initial main frame navigation.
         frame = Puppeteer::Frame.new(self, nil, frame_id, @client)

data/lib/puppeteer/version.rb

@@ -1,3 +1,3 @@
 module Puppeteer
-  VERSION = '0.50.0.alpha8'
+  VERSION = '0.50.0.alpha10'
 end

data/sig/_supplementary.rbs

@@ -178,6 +178,9 @@ class Puppeteer::FrameManager
 
   def setup_listeners: (Puppeteer::CDPSession client) -> void
   def init: (String target_id, ?Puppeteer::CDPSession cdp_session) -> void
+  def with_frame_tree_handled: () { () -> void } -> void
+  def prepare_frame_tree_handling: () -> void
+  def finish_frame_tree_handling: () -> void
   def attach_child_frame: (Puppeteer::Frame parent_frame, String parent_frame_id, String frame_id, Puppeteer::CDPSession? session) -> void
   def reattach_frame: (Puppeteer::Frame frame, String frame_id, bool is_main_frame, Hash[String, untyped] frame_payload) -> void
   def ensure_isolated_world: (Puppeteer::CDPSession session, String name) -> void
@@ -286,6 +289,9 @@ class Puppeteer::Connection
 
   def sleep_before_handling_message: (Hash[String, untyped] message) -> void
   def should_handle_synchronously?: (Hash[String, untyped] message) -> bool
+  def network_event_message?: (Hash[String, untyped] message) -> bool
+  def ensure_network_message_task: () -> void
+  def enqueue_network_message: (Hash[String, untyped] message) -> void
   def request_debug_printer: () -> untyped
   def response_debug_printer: () -> untyped
   def handle_message: (Hash[String, untyped] message) -> void

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: puppeteer-ruby
 version: !ruby/object:Gem::Version
-  version: 0.50.0.alpha8
+  version: 0.50.0.alpha10
 platform: ruby
 authors:
 - YusukeIwaki
 autorequire:
 bindir: exe
 cert_chain: []
-date: 2026-01-24 00:00:00.000000000 Z
+date: 2026-01-25 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: async