dommy 0.6.0 → 0.7.0

data/lib/dommy/media_query_list.rb

@@ -4,7 +4,7 @@ module Dommy
   # `MediaQueryList` — what `window.matchMedia(query)` returns.
   #
   # Dommy has no layout / viewport, so `matches` is `false` by default.
-  # Tests drive media query changes via `__set_matches__(bool)`, which
+  # Tests drive media query changes via `__test_set_matches__(bool)`, which
   # flips the boolean and fires a `change` event — exactly the surface
   # libraries like Material-UI / Bootstrap / @testing-library consult.
   #
@@ -42,7 +42,7 @@ module Dommy
 
     # Test seam: flip the match state and dispatch a `change` event so
     # subscribers re-render.
-    def __set_matches__(value)
+    def __test_set_matches__(value)
       return if @matches == !!value
 
       @matches = !!value
@@ -89,7 +89,7 @@ module Dommy
       end
     end
 
-    def __event_parent__
+    def __internal_event_parent__
       nil
     end
   end

data/lib/dommy/message_channel.rb

@@ -12,8 +12,8 @@ module Dommy
     def initialize(window)
       @port1 = MessagePort.new(window)
       @port2 = MessagePort.new(window)
-      @port1.__entangle__(@port2)
-      @port2.__entangle__(@port1)
+      @port1.__internal_entangle__(@port2)
+      @port2.__internal_entangle__(@port1)
     end
 
     def __js_get__(key)
@@ -39,7 +39,7 @@ module Dommy
       @pending = []
     end
 
-    def __entangle__(other)
+    def __internal_entangle__(other)
       @entangled = other
     end
 
@@ -50,10 +50,10 @@ module Dommy
       @window.scheduler.queue_microtask(
         proc do
           evt = MessageEvent.new("message", "data" => Dommy.structured_clone(data))
-          if port.__started__?
+          if port.__internal_started?
             port.dispatch_event(evt)
           else
-            port.__enqueue__(evt)
+            port.__internal_enqueue__(evt)
           end
         end
       )
@@ -76,11 +76,11 @@ module Dommy
       nil
     end
 
-    def __started__?
+    def __internal_started?
       @started || !@inline_message_handler.nil?
     end
 
-    def __enqueue__(event)
+    def __internal_enqueue__(event)
       @pending << event
     end
 
@@ -122,7 +122,7 @@ module Dommy
       end
     end
 
-    def __event_parent__
+    def __internal_event_parent__
       nil
     end
   end
@@ -242,7 +242,7 @@ module Dommy
       end
     end
 
-    def __event_parent__
+    def __internal_event_parent__
       nil
     end
   end

data/lib/dommy/mutation_observer.rb

@@ -148,17 +148,27 @@ module Dommy
         raise TypeError, "MutationObserver.observe: at least one of childList, attributes, characterData must be true"
       end
 
-      @observed <<
-        {
-          target: target,
-          child_list: child_list_on,
-          subtree: truthy_option(opts, "subtree"),
-          attributes: attributes_on,
-          attribute_filter: attribute_filter,
-          attribute_old_value: truthy_option(opts, "attributeOldValue"),
-          character_data: character_data_on,
-          character_data_old_value: truthy_option(opts, "characterDataOldValue")
-        }
+      entry = {
+        target: target,
+        child_list: child_list_on,
+        subtree: truthy_option(opts, "subtree"),
+        attributes: attributes_on,
+        attribute_filter: attribute_filter,
+        attribute_old_value: truthy_option(opts, "attributeOldValue"),
+        character_data: character_data_on,
+        character_data_old_value: truthy_option(opts, "characterDataOldValue")
+      }
+
+      # WHATWG MutationObserver §observe: if `target` is already
+      # observed, replace the existing registration's options
+      # (don't merge or stack).
+      existing_index = @observed.index { |e| e[:target].equal?(target) }
+      if existing_index
+        @observed[existing_index] = entry
+      else
+        @observed << entry
+      end
+
       @document.register_observer(self)
       nil
     end

data/lib/dommy/navigator.rb

@@ -31,7 +31,7 @@ module Dommy
     attr_reader :clipboard, :permissions, :geolocation, :wake_lock, :locks, :storage
 
     # Web Share API. Returns a Promise; tests can inspect
-    # `__last_shared__` to verify what was offered.
+    # `__test_last_shared__` to verify what was offered.
     def share(data = nil)
       @last_shared = data
       PromiseValue.resolve(@window, nil)
@@ -51,11 +51,11 @@ module Dommy
       true
     end
 
-    def __vibration_log__
+    def __test_vibration_log__
       @vibration_log.dup
     end
 
-    def __last_shared__
+    def __test_last_shared__
       @last_shared
     end
 
@@ -193,7 +193,7 @@ module Dommy
       end
     end
 
-    def __event_parent__
+    def __internal_event_parent__
       nil
     end
   end
@@ -237,7 +237,7 @@ module Dommy
       @overrides[key] = state.to_s
       @statuses ||= {}
       status = @statuses[key]
-      status&.__set_state__(state.to_s)
+      status&.__internal_set_state__(state.to_s)
       nil
     end
 
@@ -286,7 +286,7 @@ module Dommy
       @onchange = nil
     end
 
-    def __set_state__(new_state)
+    def __internal_set_state__(new_state)
       return if @state == new_state
 
       @state = new_state
@@ -327,14 +327,14 @@ module Dommy
       end
     end
 
-    def __event_parent__
+    def __internal_event_parent__
       nil
     end
   end
 
   # `navigator.geolocation` — stub Geolocation API. Real implementations
   # query the OS; dommy holds a mock position tests configure via
-  # `__set_position__(coords)` or `__set_error__(error_code)`.
+  # `__test_set_position__(coords)` or `__test_set_error__(error_code)`.
   #
   # Spec: https://www.w3.org/TR/geolocation/
   class Geolocation
@@ -357,7 +357,7 @@ module Dommy
     end
 
     # Test seam: install a mock position.
-    def __set_position__(coords = {})
+    def __test_set_position__(coords = {})
       merged = DEFAULT_COORDS.merge(coords.transform_keys(&:to_s))
       @position = {"coords" => merged, "timestamp" => @window.scheduler.now_ms}
       @error = nil
@@ -365,7 +365,7 @@ module Dommy
 
     # Test seam: install a permission/positioning error (code 1=PERMISSION_DENIED,
     # 2=POSITION_UNAVAILABLE, 3=TIMEOUT).
-    def __set_error__(code, message = "")
+    def __test_set_error__(code, message = "")
       @position = nil
       @error = {"code" => code.to_i, "message" => message.to_s}
     end
@@ -486,7 +486,7 @@ module Dommy
       end
     end
 
-    def __event_parent__
+    def __internal_event_parent__
       nil
     end
   end
@@ -518,7 +518,7 @@ module Dommy
       end
     end
 
-    def __event_parent__
+    def __internal_event_parent__
       nil
     end
   end

data/lib/dommy/node.rb

@@ -10,6 +10,12 @@ module Dommy
   # snapshots tree state at the time of the query, matching what
   # most happy-dom test patterns expect.
   class NodeList < Array
+    # Methods routed through __js_call__ (keep in sync with its when-arms).
+    JS_METHOD_NAMES = %w[item forEach entries keys values].freeze
+    def __js_method_names__
+      JS_METHOD_NAMES
+    end
+
     # Spec-compliant: out-of-range returns nil, not raise (Array#[] is
     # close but we make negative indices fail too — DOM `item(-1)` is
     # nil, not Array#[-1]'s last element).
@@ -88,6 +94,12 @@ module Dommy
   class LiveNodeList
     include Enumerable
 
+    # Methods routed through __js_call__ (keep in sync with its when-arms).
+    JS_METHOD_NAMES = %w[item forEach entries keys values].freeze
+    def __js_method_names__
+      JS_METHOD_NAMES
+    end
+
     def initialize(&block)
       @compute = block
     end

data/lib/dommy/notification.rb

@@ -3,7 +3,7 @@
 module Dommy
   # `Notification` polyfill. Real browsers prompt the user; dommy
   # exposes the permission state as a class-level slot tests can
-  # toggle via `Notification.__set_permission__("granted")`.
+  # toggle via `Notification.__test_set_permission__("granted")`.
   #
   # Spec: https://notifications.spec.whatwg.org/
   class Notification
@@ -14,7 +14,7 @@ module Dommy
     class << self
       attr_reader :permission
 
-      def __set_permission__(value)
+      def __test_set_permission__(value)
         @permission = value.to_s
       end
 
@@ -82,7 +82,7 @@ module Dommy
       end
     end
 
-    def __event_parent__
+    def __internal_event_parent__
       nil
     end
   end

data/lib/dommy/parser.rb

@@ -1,28 +1,28 @@
 # frozen_string_literal: true
 
-require "nokogiri"
-
 module Dommy
-  # Thin wrapper around Nokogiri's HTML5 fragment parser. Pinned to
-  # `max_errors: 0` for silent recovery on malformed HTML (matching
-  # browser behavior).
+  # Thin wrapper around the backend's HTML5 fragment parser. Delegates
+  # to `Dommy::Backend.fragment` so backends can supply their own
+  # implementation.
   #
-  # Known quirks: `<table>`-only fragments wrap children in an
-  # implicit `<tbody>`; `<select>` reparents non-option children
-  # outside itself.
+  # Known quirks (vary by backend):
+  # - Nokogiri (libxml2): `<table>`-only fragments wrap children in
+  #   an implicit `<tbody>`; `<select>` reparents non-option children.
+  # - Nokolexbor (Lexbor): similar behavior, slightly different edge
+  #   cases for malformed input.
   #
   # `owner_doc` is critical: when a node parsed via a detached
   # fragment gets `add_child`'d into a Document with a different
-  # Nokogiri owner, libxml2 silently **copies** the node (new
-  # object_id) instead of moving it. That breaks identity-dependent
-  # caches (e.g. `Document#wrap_node` and any reconciler that keys
-  # off node identity). Always pass the destination document.
+  # owner, libxml2 silently **copies** the node (new object_id)
+  # instead of moving it. That breaks identity-dependent caches
+  # (e.g. `Document#wrap_node` and any reconciler that keys off
+  # node identity). Always pass the destination document.
   module Parser
     def self.fragment(html, owner_doc: nil)
       if owner_doc
         owner_doc.fragment(html.to_s)
       else
-        Nokogiri::HTML5.fragment(html.to_s, max_errors: 0)
+        Backend.fragment(html.to_s, owner_doc: nil)
       end
     end
   end

data/lib/dommy/performance_observer.rb

@@ -2,7 +2,7 @@
 
 module Dommy
   # `PerformanceObserver` — stub. Same observe/disconnect shape, with
-  # `__trigger__` for tests.
+  # `__test_trigger__` for tests.
   #
   # Spec: https://w3c.github.io/performance-timeline/
   class PerformanceObserver
@@ -37,7 +37,7 @@ module Dommy
       @entry_types.dup
     end
 
-    def __trigger__(entries)
+    def __test_trigger__(entries)
       invoke_callback(entries)
     end
 

data/lib/dommy/range.rb

@@ -142,8 +142,8 @@ module Dommy
       collect_nodes_in_range.each do |node|
         if node.respond_to?(:remove)
           node.remove
-        elsif node.respond_to?(:__node__)
-          node.__node__.unlink
+        elsif node.respond_to?(:__dommy_backend_node__)
+          node.__dommy_backend_node__.unlink
         end
       end
 

data/lib/dommy/resize_observer.rb

@@ -3,7 +3,7 @@
 module Dommy
   # `ResizeObserver` — stub for the element-resize API. Same shape as
   # IntersectionObserver: observe / unobserve / disconnect, plus
-  # `__trigger__` for tests to drive callbacks.
+  # `__test_trigger__` for tests to drive callbacks.
   #
   # Spec: https://drafts.csswg.org/resize-observer/
   class ResizeObserver
@@ -35,7 +35,7 @@ module Dommy
       @targets.dup
     end
 
-    def __trigger__(entries)
+    def __test_trigger__(entries)
       invoke_callback(entries)
     end
 

data/lib/dommy/shadow_root.rb

@@ -14,7 +14,9 @@ module Dommy
     include EventTarget
     include Node
 
-    attr_reader :__node__, :host, :mode, :delegates_focus, :slot_assignment, :document
+    attr_reader :host, :mode, :delegates_focus, :slot_assignment, :document
+
+    def __dommy_backend_node__ = @__node__
 
     def initialize(host, mode:, delegates_focus: false, slot_assignment: "named")
       @host = host
@@ -23,7 +25,7 @@ module Dommy
       @slot_assignment = slot_assignment.to_s
       @document = host.document
       @__node__ = @document.nokogiri_doc.fragment("")
-      @document.__register_shadow_fragment__(@__node__, self)
+      @document.__internal_register_shadow_fragment__(@__node__, self)
     end
 
     # ---- Public Ruby API (ParentNode + DocumentFragment mixin) ----
@@ -48,7 +50,7 @@ module Dommy
 
     def text_content=(value)
       @__node__.children.each(&:unlink)
-      @__node__.add_child(Nokogiri::XML::Text.new(value.to_s, @document.nokogiri_doc))
+      @__node__.add_child(Backend.create_text(value.to_s, @document.nokogiri_doc))
     end
 
     def children
@@ -140,9 +142,9 @@ module Dommy
     end
 
     def contains?(other)
-      return false unless other.respond_to?(:__node__)
+      return false unless other.respond_to?(:__dommy_backend_node__)
 
-      other_node = other.__node__
+      other_node = other.__dommy_backend_node__
       return true if other_node == @__node__
 
       Internal::NodeTraversal.ancestor_of?(@__node__, other_node)
@@ -233,7 +235,7 @@ module Dommy
     # Event bubbling stops at the ShadowRoot unless event has
     # `composed: true`. The host is the bubble-path successor when
     # composition crosses the boundary (handled in Event dispatch).
-    def __event_parent__
+    def __internal_event_parent__
       nil
     end
 
@@ -242,9 +244,9 @@ module Dommy
     def detach_dom_nodes(value)
       case value
       when String
-        [Nokogiri::XML::Text.new(value, @document.nokogiri_doc)]
+        [Backend.create_text(value, @document.nokogiri_doc)]
       else
-        node = value.respond_to?(:__node__) ? value.__node__ : nil
+        node = value.respond_to?(:__dommy_backend_node__) ? value.__dommy_backend_node__ : nil
         return [] unless node
 
         node.unlink if node.parent

data/lib/dommy/streams.rb

@@ -51,7 +51,7 @@ module Dommy
     end
 
     # Convenience: iterate all chunks synchronously into an Array.
-    def __drain__
+    def __internal_drain__
       out = []
       until @queue.empty?
         out << @queue.shift
@@ -60,25 +60,25 @@ module Dommy
       out
     end
 
-    def __enqueue__(chunk)
+    def __internal_enqueue__(chunk)
       raise Error, "stream is not readable" unless @state == :readable
 
       @queue << chunk
       flush_pending_reads
     end
 
-    def __close__
+    def __internal_close__
       @state = :closed
       flush_pending_reads
     end
 
-    def __error__(reason)
+    def __internal_error__(reason)
       @state = :errored
       @error_reason = reason
       flush_pending_reads
     end
 
-    def __read__
+    def __internal_read__
       promise = PromiseValue.new(@window)
 
       if !@queue.empty?
@@ -147,15 +147,15 @@ module Dommy
     end
 
     def enqueue(chunk)
-      @stream.__enqueue__(chunk)
+      @stream.__internal_enqueue__(chunk)
     end
 
     def close
-      @stream.__close__
+      @stream.__internal_close__
     end
 
     def error(reason)
-      @stream.__error__(reason)
+      @stream.__internal_error__(reason)
     end
 
     def __js_call__(method, args)
@@ -177,7 +177,7 @@ module Dommy
     end
 
     def read
-      @stream.__read__
+      @stream.__internal_read__
     end
 
     def release_lock
@@ -227,18 +227,18 @@ module Dommy
       !@writer.nil?
     end
 
-    def __write__(chunk)
+    def __internal_write__(chunk)
       invoke(@sink["write"], [chunk])
       PromiseValue.resolve(@window, nil)
     end
 
-    def __close__
+    def __internal_close__
       @state = :closed
       invoke(@sink["close"], [])
       PromiseValue.resolve(@window, nil)
     end
 
-    def __abort__(reason)
+    def __internal_abort__(reason)
       @state = :errored
       invoke(@sink["abort"], [reason])
       PromiseValue.resolve(@window, nil)
@@ -256,9 +256,9 @@ module Dommy
       when "getWriter"
         get_writer
       when "close"
-        __close__
+        __internal_close__
       when "abort"
-        __abort__(args[0])
+        __internal_abort__(args[0])
       end
     end
 
@@ -284,15 +284,15 @@ module Dommy
     end
 
     def write(chunk)
-      @stream.__write__(chunk)
+      @stream.__internal_write__(chunk)
     end
 
     def close
-      @stream.__close__
+      @stream.__internal_close__
     end
 
     def abort(reason = nil)
-      @stream.__abort__(reason)
+      @stream.__internal_abort__(reason)
     end
 
     def __js_call__(method, args)
@@ -331,8 +331,8 @@ module Dommy
               controller.enqueue(chunk)
             end
           end,
-          "close" => proc { @readable.__close__ },
-          "abort" => proc { |reason| @readable.__error__(reason) }
+          "close" => proc { @readable.__internal_close__ },
+          "abort" => proc { |reason| @readable.__internal_error__(reason) }
         }
       )
 
@@ -361,15 +361,15 @@ module Dommy
     end
 
     def enqueue(chunk)
-      @readable.__enqueue__(chunk)
+      @readable.__internal_enqueue__(chunk)
     end
 
     def terminate
-      @readable.__close__
+      @readable.__internal_close__
     end
 
     def error(reason)
-      @readable.__error__(reason)
+      @readable.__internal_error__(reason)
     end
 
     def __js_call__(method, args)

data/lib/dommy/text_codec.rb

@@ -118,8 +118,8 @@ module Dommy
         window,
         {
           "write" => proc { |chunk| controller.enqueue(encoder.encode(chunk)) },
-          "close" => proc { @readable.__close__ },
-          "abort" => proc { |r| @readable.__error__(r) }
+          "close" => proc { @readable.__internal_close__ },
+          "abort" => proc { |r| @readable.__internal_error__(r) }
         }
       )
     end
@@ -155,8 +155,8 @@ module Dommy
         window,
         {
           "write" => proc { |chunk| controller.enqueue(decoder.decode(chunk)) },
-          "close" => proc { @readable.__close__ },
-          "abort" => proc { |r| @readable.__error__(r) }
+          "close" => proc { @readable.__internal_close__ },
+          "abort" => proc { |r| @readable.__internal_error__(r) }
         }
       )
     end