leptris 1.9.174.4 → 1.9.174.6

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 3052fa875df89194fa904610e20786deaf2076de881242382afe9bbc9fd7fa0d
-  data.tar.gz: 86806a382948d948d2ec5c2550302a8ae52b976edd224aaff4c130f3a4606f3e
+  metadata.gz: 0ac7143c5a7730e13077c52448c2b90a7705959a4fe458e7c29e7782d7ee6df1
+  data.tar.gz: 23d3530348907214d341704fa7b098728fa8b677422c8f595e12ae33c2b88ff3
 SHA512:
-  metadata.gz: 81e9fe81a94ddd6d82ef044569b8d43ff5c663ca41cf2a59798660638e4f73cfc23f7aabf4eb3123757e3590d0d7524438f378c6b1bd447a2b39af95bb0089d5
-  data.tar.gz: e161ced4c4ab3634ae6a4fa6ebb0e95e970e1634b72a9d1b4ef5e1ad3800461170a2893bb89477c404f22b992813b7abf66abc083272e851c341affc699488fc
+  metadata.gz: 6395fec9ab3701df1b77f32bbe950feff8b918b9f79374b5c107d9e282bb5e9e5fdd91c152dc46526fc201910e69b7c57e8ee968d11e222be3cfdd346c6c7ab8
+  data.tar.gz: a55e0b100034203af05855e8881fbfa161d30d6043b3831080ebff8d0f9ca658ca74fd11e1d30aef6887b3108493e4f690f0e08133ca4cf3bca36399cbcacdc4

data/CHANGELOG.md

@@ -5,6 +5,39 @@ All notable changes to Leptris will be documented in this file.
 The format is based on [Keep a Changelog](https://keepachangelog.com/1.0.0/),
 and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
 
+## [1.9.174.6] - 2026-09-15
+
+### Changed
+
+- **C-yield traversal (TODO.perf/27)**: `traverse` and `visit`
+  no longer allocate an FFI::Function closure per call — the
+  ext's callbacks rb_yield directly, preserving post-order +
+  abort-at-self + stash-abort-raise (traverse) and the
+  (node, entering, depth) visit contract. Measured (load 34):
+  **traverse over 8k nodes 1663.5µs → 379.9µs (4.4x; 6.81x
+  Nokogiri); visit 2288.6µs → 545.4µs (4.2x)**.
+- **Address-based first-touch fills (TODO.perf/28)**:
+  text/comment/CDATA/PI content and `path` fill through address
+  faces (scope-eligible); the bulk attribute faces accept
+  scope-owned elements.
+
+## [1.9.174.5] - 2026-09-15
+
+### Changed
+
+- **Iterparse rides the bulk path (TODO.perf/25)**: scope-owned
+  elements materialize children through the one-pass C faces —
+  the IterationScope flows as the cache/version authority (its
+  cache resets per yield, so recycled pool addresses cannot
+  collide), with scope-aware wrapper stamps; pure address-based
+  reads (attribute/text/name) become scope-eligible. Mutations
+  on scope elements keep the Ruby path. Measured (load ~6): the
+  20k-record streaming walk 124.8ms → 100.5ms (~20%).
+- **Post-mutation memo seeding (TODO.perf/26)**: []= seeds the
+  attribute memo at the new version (partial memos extend,
+  full faces restart — the cold-[] rules); content= seeds
+  @content; the write-then-read cycle runs ~545ns combined.
+
 ## [1.9.174.4] - 2026-09-15
 
 ### Changed

data/TODO.perf/25-scope-owned-bulk-path.md

@@ -0,0 +1,39 @@
+# 25 — Scope-owned (iterparse) elements on the bulk/native read path
+
+Status: DONE (1.9.174.5)
+
+iterparse is moxml's streaming core, yet scope-owned elements
+ride the FFI read path (two-call fetch + per-child Ruby wrap
+frames; #4ca5b3c kept them off deliberately). The IterationScope
+exposes exactly the seams the C faces read — @wrapper_cache
+(reset per yield by new_subtree!, killing recycled-address
+entries) and @version (advances per yield and on mutation) — so
+the bulk children faces can take the SCOPE as their "document"
+argument safely, provided the constructed wrappers are stamped
+scope-aware: @structure_memoizable/@native_fast false,
+@pub_document nil.
+
+Pure address-based reads (fast_attribute / element text / name)
+touch no cache — scope elements can use them via a construction
+-time @addr_reads_fast that ignores scope ownership.
+
+Gates: iterparse children/element_children identical via the C
+path; identity holds within a yielded subtree and resets across
+yields (the new_subtree! discipline); document stays nil for
+scope yields; mutations on scope elements keep the Ruby path;
+suite both modes.
+
+## Outcome (1.9.174.5)
+
+The bulk children faces take the IterationScope itself as their
+"document" (its @wrapper_cache resets per yield via new_subtree!
+— recycled pool addresses cannot collide — and its @version
+advances per yield and on mutation); the C face stamps
+scope-aware (@structure_memoizable/@native_fast false,
+@pub_document nil) while document-owned stamps stay Qtrue.
+Pure address-based reads (fast_attribute / element text / name)
+switch to @addr_reads_fast — true for scope elements too (they
+touch no cache). Mutations on scope elements keep the Ruby path.
+Measured (load ~6): the 20k-record iterparse walk (attr + child
+name + content per record) 124.8ms (FFI) -> 100.5ms (native
+bulk) — ~20%.

data/TODO.perf/26-post-mutation-memo-seeding.md

@@ -0,0 +1,26 @@
+# 26 — Post-mutation memo seeding
+
+Status: DONE (1.9.174.5)
+
+Every mutation invalidates the read memos through the version
+bump, so a write-then-read cycle pays a full re-derivation even
+when the new value is known at write time. Seed it: Element#[]=
+re-stamps @attr_values at the new version and writes the value
+into it (a partial fill stays partial — same rules); content=
+stores the new text as the @content memo at the new version;
+name= already seeds @name.
+
+Gates: []=/content=/name= followed by reads answers from the
+seeded memo (spec-observable via identity for strings? — no,
+values only); mixed attribute sets then attributes() still
+rebuild the full face on demand; invalidation semantics
+unchanged for other mutators.
+
+## Outcome (1.9.174.5)
+
+Element#[]= seeds the version-stamped @attr_values at the new
+version (a just-invalidated valid partial memo extends in place;
+a full face or anything older restarts partial, mirroring the
+cold-[] rules); content= seeds @content; name= seeds @name
+(already). Measured: the []= + [] read-back cycle runs ~545ns
+(write + read combined).

data/TODO.perf/27-c-yield-traversal.md

@@ -0,0 +1,38 @@
+# 27 — C-yield traversal (traverse/visit without the FFI::Function closure)
+
+Status: DONE (1.9.174.6)
+
+Node#traverse and #visit allocate an FFI::Function closure per
+call (~1µs) and dispatch every visited node back through the
+Ruby closure — the dominant per-node cost of whole-subtree
+iteration, paid even on small subtrees where the walk itself is
+microseconds.
+
+Bind both to C callbacks that rb_yield directly: the ext's
+traverse face keeps the abort-at-self post-order contract and
+the exception discipline (rb_protect in the callback stashes,
+aborts the walk, re-raises after — exactly the Ruby version's
+contract); visit keeps its (node, entering, depth) callback
+shape with the same stash-and-raise treatment. Node wrapping per
+visit rides the shared binding construction (identity cache +
+kind dispatch), and the walk state lives on the C stack — no
+globals, nested traversals safe.
+
+Gates: post-order sequence identical (receiver last); the
+subtree boundary holds (abort-at-self); block exceptions
+re-raise after the walk stops; enum_for forms unchanged; suite
+both modes.
+
+## Outcome (1.9.174.6)
+
+Native.traverse_binding / visit_binding: C callbacks that
+rb_yield directly (block forwarded explicitly — rb_yield needs
+it on the C entry's frame). traverse keeps post-order +
+abort-at-self + stash-abort-raise (rb_protect in the callback,
+walk state on the C stack — nested traversals independent);
+visit keeps (node, entering, depth) with the engine's
+(user, node, ...) callback order. Two bugs found by the suite:
+the visit callback's argument order (user first — the segfault
+spec caught it) and the block forwarding. Measured (load 34):
+traverse over 8k nodes 1663.5µs (FFI closure) -> 379.9µs
+(4.4x; 6.81x Nokogiri); visit 2288.6µs -> 545.4µs (4.2x).

data/TODO.perf/28-address-fills.md

@@ -0,0 +1,25 @@
+# 28 — Address-based first-touch fills for the remaining reads
+
+Status: DONE (1.9.174.6)
+
+The memoized content readers still fill through FFI marshaling on
+first touch — Text/Comment/CDATA/PI content, and Node#path
+(owned-string read + free) — and the bulk attribute faces exclude
+scope-owned elements though they touch no document state. Add the
+trivial address faces (fast_text/comment/cdata content, pi
+target/data, node path with the free protocol) and switch the
+fills to @addr_reads_fast (scope-eligible); the attributes bulk
+path follows.
+
+Gates: identical values for every kind (UTF-8, escapes, nils);
+scope-owned fills answer the same; path memo unchanged; suite
+both modes.
+
+## Outcome (1.9.174.6)
+
+fast_text/comment/cdata content, fast_pi_target/data (the
+read_pi_data strip preserved), and fast_path (owned-string copy
++ engine free) — every first-touch content fill now rides the
+address faces via @addr_reads_fast (scope-eligible), and the
+bulk attribute faces accept scope-owned elements (they touch no
+document state; the scope's version stamps the memo).

data/ext/leptris/native/native.c

@@ -87,8 +87,18 @@ static node_str_fn f_comment_content, f_cdata_content, f_pi_target,
                    f_pi_data;
 typedef int (*set_str_fn)(void *, const char *);
 typedef int (*node_unlink_fn)(void *);
+typedef int (*traverse_cb_fn)(void *, void *);
+typedef int (*traverse_fn)(void *, int, traverse_cb_fn, void *);
+typedef void (*visit_cb_fn)(void *, void *, int, int);
+typedef void (*visit_fn)(void *, visit_cb_fn, void *);
+typedef void (*free_str_fn)(void *);
+typedef const char *(*node_xpath_fn)(void *);
 static set_str_fn f_elem_set_name, f_elem_set_text, f_text_set_content;
 static node_unlink_fn f_node_unlink;
+static traverse_fn f_node_traverse;
+static visit_fn f_node_visit;
+static free_str_fn f_free_str;
+static node_xpath_fn f_node_xpath;
 static set_root_fn f_set_root;
 static doc_free_fn f_doc_free;
 
@@ -105,6 +115,8 @@ static VALUE c_use_after_free_error;
 static VALUE native_cache_of(VALUE document);
 static void resolve_binding_classes(void);
 static VALUE binding_cache_of(VALUE document);
+static VALUE binding_klass_for(int kind);
+static VALUE c_iteration_scope;
 
 #define NT_ELEMENT 0
 
@@ -200,6 +212,10 @@ static void resolve_symbols(const char *lib_path)
     f_elem_set_text = (set_str_fn)lib_sym(h, "leptris_element_set_text");
     f_text_set_content = (set_str_fn)lib_sym(h, "leptris_text_node_set_content");
     f_node_unlink = (node_unlink_fn)lib_sym(h, "leptris_node_unlink");
+    f_node_traverse = (traverse_fn)lib_sym(h, "leptris_node_traverse");
+    f_node_visit = (visit_fn)lib_sym(h, "leptris_node_visit");
+    f_free_str = (free_str_fn)lib_sym(h, "leptris_free_string");
+    f_node_xpath = (node_xpath_fn)lib_sym(h, "leptris_node_get_xpath");
     f_set_root = (set_root_fn)lib_sym(h, "leptris_document_set_root");
     f_doc_free = (doc_free_fn)lib_sym(h, "leptris_document_free");
     if (!f_elem_name || !f_text_content || !f_attr ||
@@ -211,6 +227,8 @@ static void resolve_symbols(const char *lib_path)
         !f_comment_content || !f_cdata_content || !f_pi_target ||
         !f_pi_data || !f_elem_set_name || !f_elem_set_text ||
         !f_text_set_content || !f_node_unlink ||
+        !f_node_traverse || !f_node_visit || !f_free_str ||
+        !f_node_xpath ||
         !f_doc_free ||
         !f_elem_prefix || !f_xp_count || !f_xp_nodes_ex ||
         !f_xp_node_kind || !f_xp_node_name || !f_xp_node_value ||
@@ -707,6 +725,211 @@ static VALUE nf_unlink_binding_node(VALUE self, VALUE document,
     return INT2FIX(st);
 }
 
+/* ---- C-yield traversal (TODO.perf/27) ----------------------------
+ * Same contracts as the Ruby FFI::Function versions without the
+ * per-call closure: traverse is post-order with abort-at-self
+ * (the receiver is the LAST node of its subtree in post-order,
+ * so stopping at self bounds the walk exactly) and
+ * stash-abort-raise exception discipline; visit passes (node,
+ * entering, depth). Walk state lives on the caller's C stack —
+ * nested traversals are independent. */
+struct trav_state {
+    VALUE document;
+    VALUE err;
+    void *self_ptr;
+    int for_visit;
+};
+
+static VALUE trav_yield_one(VALUE arg)
+{
+    return rb_yield(arg);
+}
+
+static int trav_cb(void *node_ptr, void *user)
+{
+    struct trav_state *st = user;
+    VALUE cache, key, node;
+    int kind, state;
+
+    if (st->err != Qnil)
+        return 1;
+    kind = f_node_type(node_ptr);
+    cache = binding_cache_of(st->document);
+    key = ULL2NUM((uint64_t)(uintptr_t)node_ptr);
+    node = rb_hash_aref(cache, key);
+    if (NIL_P(node)) {
+        node = rb_obj_alloc(binding_klass_for(kind));
+        rb_iv_set(node, "@c_address", key);
+        rb_iv_set(node, "@document", st->document);
+        rb_iv_set(node, "@parent", Qnil);
+        if (rb_obj_class(st->document) == c_iteration_scope) {
+            rb_iv_set(node, "@structure_memoizable", Qfalse);
+            rb_iv_set(node, "@native_fast", Qfalse);
+            rb_iv_set(node, "@pub_document", Qnil);
+        } else {
+            rb_iv_set(node, "@structure_memoizable", Qtrue);
+            rb_iv_set(node, "@native_fast", Qtrue);
+            rb_iv_set(node, "@pub_document", st->document);
+        }
+        rb_iv_set(node, "@addr_reads_fast", Qtrue);
+        rb_iv_set(node, "@node_type", INT2FIX(kind));
+        rb_hash_aset(cache, key, node);
+    }
+    rb_protect(trav_yield_one, node, &state);
+    if (state) {
+        st->err = rb_errinfo();
+        return 1; /* abort the walk */
+    }
+    return node_ptr == st->self_ptr ? 1 : 0; /* abort-at-self */
+}
+
+static VALUE nf_traverse_binding(VALUE self, VALUE document,
+                                 VALUE addr)
+{
+    struct trav_state st;
+
+    (void)self;
+    resolve_binding_classes();
+    st.document = document;
+    st.err = Qnil;
+    st.self_ptr = (void *)(uintptr_t)NUM2ULL(addr);
+    st.for_visit = 0;
+    f_node_traverse(st.self_ptr, 1 /* TRAVERSE_POST_ORDER */,
+                    trav_cb, &st);
+    if (st.err != Qnil)
+        rb_exc_raise(st.err);
+    return Qnil;
+}
+
+struct visit_yield_args {
+    VALUE node, entering, depth;
+};
+
+static VALUE trav_yield_args(VALUE arg)
+{
+    struct visit_yield_args *a = (struct visit_yield_args *)arg;
+    return rb_yield_values(3, a->node, a->entering, a->depth);
+}
+
+/* Engine visit callback order is (user, node, entering, depth) —
+ * the Ruby closure's |_, node_ptr, entering, depth| mirrors it. */
+static void visit_cb(void *user, void *node_ptr, int entering,
+                     int depth)
+{
+    /* visit's engine signature cannot abort; exceptions are
+     * stashed and every later node is skipped — the raise
+     * happens after the walk, same observable outcome. */
+    struct trav_state *st = user;
+    struct visit_yield_args va;
+    VALUE cache, key, node;
+    int kind, state;
+
+    if (st->err != Qnil)
+        return;
+    kind = f_node_type(node_ptr);
+    cache = binding_cache_of(st->document);
+    key = ULL2NUM((uint64_t)(uintptr_t)node_ptr);
+    node = rb_hash_aref(cache, key);
+    if (NIL_P(node)) {
+        node = rb_obj_alloc(binding_klass_for(kind));
+        rb_iv_set(node, "@c_address", key);
+        rb_iv_set(node, "@document", st->document);
+        rb_iv_set(node, "@parent", Qnil);
+        if (rb_obj_class(st->document) == c_iteration_scope) {
+            rb_iv_set(node, "@structure_memoizable", Qfalse);
+            rb_iv_set(node, "@native_fast", Qfalse);
+            rb_iv_set(node, "@pub_document", Qnil);
+        } else {
+            rb_iv_set(node, "@structure_memoizable", Qtrue);
+            rb_iv_set(node, "@native_fast", Qtrue);
+            rb_iv_set(node, "@pub_document", st->document);
+        }
+        rb_iv_set(node, "@addr_reads_fast", Qtrue);
+        rb_iv_set(node, "@node_type", INT2FIX(kind));
+        rb_hash_aset(cache, key, node);
+    }
+    va.node = node;
+    va.entering = entering ? Qtrue : Qfalse;
+    va.depth = INT2NUM(depth);
+    rb_protect(trav_yield_args, (VALUE)&va, &state);
+    if (state)
+        st->err = rb_errinfo();
+}
+
+static VALUE nf_visit_binding(VALUE self, VALUE document, VALUE addr)
+{
+    struct trav_state st;
+
+    (void)self;
+    resolve_binding_classes();
+    st.document = document;
+    st.err = Qnil;
+    st.self_ptr = (void *)(uintptr_t)NUM2ULL(addr);
+    st.for_visit = 1;
+    f_node_visit(st.self_ptr, visit_cb, &st);
+    if (st.err != Qnil)
+        rb_exc_raise(st.err);
+    return Qnil;
+}
+
+/* ---- Address-based fills (TODO.perf/28) -------------------------
+ * Pure reads of node-local data — no cache, no version — the
+ * same shapes the inner_html pass uses. */
+static VALUE nf_fast_text_content(VALUE self, VALUE addr)
+{
+    const char *s;
+    (void)self;
+    s = f_text_content((void *)(uintptr_t)NUM2ULL(addr));
+    return s ? rb_utf8_str_new_cstr(s) : Qnil;
+}
+
+static VALUE nf_fast_comment_content(VALUE self, VALUE addr)
+{
+    const char *s;
+    (void)self;
+    s = f_comment_content((void *)(uintptr_t)NUM2ULL(addr));
+    return s ? rb_utf8_str_new_cstr(s) : Qnil;
+}
+
+static VALUE nf_fast_cdata_content(VALUE self, VALUE addr)
+{
+    const char *s;
+    (void)self;
+    s = f_cdata_content((void *)(uintptr_t)NUM2ULL(addr));
+    return s ? rb_utf8_str_new_cstr(s) : Qnil;
+}
+
+static VALUE nf_fast_pi_target(VALUE self, VALUE addr)
+{
+    const char *s;
+    (void)self;
+    s = f_pi_target((void *)(uintptr_t)NUM2ULL(addr));
+    return s ? rb_utf8_str_new_cstr(s) : Qnil;
+}
+
+static VALUE nf_fast_pi_data(VALUE self, VALUE addr)
+{
+    const char *s;
+    (void)self;
+    s = f_pi_data((void *)(uintptr_t)NUM2ULL(addr));
+    return s ? rb_utf8_str_new_cstr(s) : Qnil;
+}
+
+/* Owned string: copy + free through the engine's seam (the
+ * read_owned_string protocol). */
+static VALUE nf_fast_path(VALUE self, VALUE addr)
+{
+    const char *s;
+    VALUE out;
+    (void)self;
+    s = f_node_xpath((void *)(uintptr_t)NUM2ULL(addr));
+    if (!s)
+        return Qnil;
+    out = rb_utf8_str_new_cstr(s);
+    f_free_str((void *)s);
+    return out;
+}
+
 /* C-bound insert family (TODO.perf/14): prepend / insert-after /
  * insert-before share append's gates + predicate + bump shape.
  * mode: 1 prepend, 2 after, 3 before (anchor = receiver). */
@@ -874,6 +1097,7 @@ static VALUE nn_address(VALUE self)
 static VALUE c_b_element = Qundef, c_b_text, c_b_comment, c_b_cdata,
              c_b_pi, c_b_node, c_b_result_text, c_b_result_attr,
              c_b_attr, c_ffi_pointer, c_b_document, c_b_freed;
+static VALUE c_iteration_scope;
 static ID id_ptr_new;
 
 /* Binding classes resolve LAZILY: Init_native can run before the
@@ -895,6 +1119,7 @@ static void resolve_binding_classes(void)
     c_b_attr = rb_path2class("Leptris::XML::Attr");
     c_b_document = rb_path2class("Leptris::XML::Document");
     c_b_freed = rb_path2class("Leptris::XML::Document::Freed");
+    c_iteration_scope = rb_path2class("Leptris::XML::IterationScope");
     id_ptr_new = rb_intern("new");
     rb_gc_register_mark_object(c_b_element);
     rb_gc_register_mark_object(c_b_text);
@@ -908,6 +1133,7 @@ static void resolve_binding_classes(void)
     rb_gc_register_mark_object(c_b_attr);
     rb_gc_register_mark_object(c_b_document);
     rb_gc_register_mark_object(c_b_freed);
+    rb_gc_register_mark_object(c_iteration_scope);
 }
 
 static VALUE binding_klass_for(int kind)
@@ -951,9 +1177,20 @@ static VALUE bulk_children_impl(VALUE document, VALUE parent_addr,
             rb_iv_set(node, "@c_address", key);
             rb_iv_set(node, "@document", document);
             rb_iv_set(node, "@parent", Qnil);
-            rb_iv_set(node, "@structure_memoizable", Qtrue);
-            rb_iv_set(node, "@native_fast", Qtrue);
+            /* TODO.perf/25: scope-aware stamps — the IterationScope
+             * flows through here as "document" for iterparse
+             * elements; those stay memo-exempt and FFI-mutating,
+             * but address-based reads are cache-free and safe. */
+            if (rb_obj_class(document) == c_iteration_scope) {
+                rb_iv_set(node, "@structure_memoizable", Qfalse);
+                rb_iv_set(node, "@native_fast", Qfalse);
+                rb_iv_set(node, "@pub_document", Qnil);
+            } else {
+                rb_iv_set(node, "@structure_memoizable", Qtrue);
+                rb_iv_set(node, "@native_fast", Qtrue);
                 rb_iv_set(node, "@pub_document", document);
+            }
+            rb_iv_set(node, "@addr_reads_fast", Qtrue);
             rb_iv_set(node, "@node_type", INT2FIX(kinds[i]));
             rb_hash_aset(cache, key, node);
         }
@@ -989,6 +1226,8 @@ static VALUE m_native_sentinel; /* module object = fallback marker */
 static VALUE materialize_xp_entry(VALUE document, VALUE cache,
                                   void *result, void *ptr, int kind,
                                   int idx);
+static VALUE binding_klass_for(int kind);
+static VALUE c_iteration_scope;
 
 static VALUE bulk_xpath_array(VALUE document, void *result);
 
@@ -1697,6 +1936,22 @@ void Init_native(void)
                               nf_set_binding_text, 3);
     rb_define_module_function(m_native, "unlink_binding_node",
                               nf_unlink_binding_node, 2);
+    rb_define_module_function(m_native, "traverse_binding",
+                              nf_traverse_binding, 2);
+    rb_define_module_function(m_native, "visit_binding",
+                              nf_visit_binding, 2);
+    rb_define_module_function(m_native, "fast_text_content",
+                              nf_fast_text_content, 1);
+    rb_define_module_function(m_native, "fast_comment_content",
+                              nf_fast_comment_content, 1);
+    rb_define_module_function(m_native, "fast_cdata_content",
+                              nf_fast_cdata_content, 1);
+    rb_define_module_function(m_native, "fast_pi_target",
+                              nf_fast_pi_target, 1);
+    rb_define_module_function(m_native, "fast_pi_data",
+                              nf_fast_pi_data, 1);
+    rb_define_module_function(m_native, "fast_path",
+                              nf_fast_path, 1);
     rb_define_module_function(m_native, "fast_inner_xml",
                               nf_fast_inner_xml, 1);
     rb_define_module_function(m_native, "doc_handle_attach",

data/lib/leptris/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module Leptris
-  VERSION = "1.9.174.4"
+  VERSION = "1.9.174.6"
 end

data/lib/leptris/xml/cdata.rb

@@ -6,7 +6,11 @@ class Leptris::XML::CDATA < Leptris::XML::Text
   def content
     return @content if memo_hit?(@content_version)
     ensure_alive!
-    result = Leptris::XML::FFI.leptris_cdata_node_get_content(c_ptr)
+    result = if @addr_reads_fast
+               Leptris::XML::Native.fast_cdata_content(@c_address)
+             else
+               Leptris::XML::FFI.leptris_cdata_node_get_content(c_ptr)
+             end
     if @document
       @content = result
       @content_version = @document.version

data/lib/leptris/xml/comment.rb

@@ -6,7 +6,11 @@ class Leptris::XML::Comment < Leptris::XML::Node
   def content
     return @content if memo_hit?(@content_version)
     ensure_alive!
-    result = Leptris::XML::FFI.leptris_comment_node_get_content(c_ptr)
+    result = if @addr_reads_fast
+               Leptris::XML::Native.fast_comment_content(@c_address)
+             else
+               Leptris::XML::FFI.leptris_comment_node_get_content(c_ptr)
+             end
     if @document
       @content = result
       @content_version = @document.version

data/lib/leptris/xml/element.rb

@@ -15,7 +15,7 @@ class Leptris::XML::Element < Leptris::XML::Node
   def name
     return @name if @name
     ensure_alive!
-    @name = if native_fast?
+    @name = if @addr_reads_fast
               Leptris::XML::Native.fast_name(@c_address)
             else
               Leptris::XML::FFI.leptris_element_name(c_ptr)
@@ -40,7 +40,7 @@ class Leptris::XML::Element < Leptris::XML::Node
   def content
     return @content if memo_hit?(@content_version)
     ensure_alive!
-    result = if native_fast?
+    result = if @addr_reads_fast
                Leptris::XML::Native.fast_element_text(@c_address)
              else
                Leptris::XML::FFI.leptris_element_text(c_ptr)
@@ -57,11 +57,15 @@ class Leptris::XML::Element < Leptris::XML::Node
       Leptris::XML::FFI.check_status(
         Leptris::XML::Native.set_binding_text(
           @document, @c_address, new_content.to_s))
+      @content = new_content.to_s
+      @content_version = @document.version if @document
       return new_content
     end
     ensure_writable!
     Leptris::XML::FFI.check_status(
       Leptris::XML::FFI.leptris_element_set_text(c_ptr, new_content.to_s))
+    @content = new_content.to_s
+    @content_version = @document.version if @document
     new_content
   end
 
@@ -91,7 +95,7 @@ class Leptris::XML::Element < Leptris::XML::Node
         # engine call resolves it, then the name is cached.
         return v if !v.nil? || @attributes
         ensure_alive!
-        v = if native_fast?
+        v = if @addr_reads_fast
                  Leptris::XML::Native.fast_attribute(@c_address, name)
                else
                  Leptris::XML::FFI.leptris_element_attribute(c_ptr, name)
@@ -104,7 +108,7 @@ class Leptris::XML::Element < Leptris::XML::Node
       # post-mutation cold [] would stamp the new version onto a
       # stale @attributes hash and attributes would serve it.
       ensure_alive!
-      v = if native_fast?
+      v = if @addr_reads_fast
             Leptris::XML::Native.fast_attribute(@c_address, name)
           else
             Leptris::XML::FFI.leptris_element_attribute(c_ptr, name)
@@ -137,15 +141,39 @@ class Leptris::XML::Element < Leptris::XML::Node
       Leptris::XML::FFI.check_status(
         Leptris::XML::Native.set_binding_attribute(
           @document, @c_address, key.to_s, value.to_s))
+      seed_attribute_memo(key.to_s, value.to_s)
       return value
     end
     ensure_writable!
     Leptris::XML::FFI.check_status(
       Leptris::XML::FFI.leptris_element_set_attribute(c_ptr, key.to_s, value.to_s))
+    seed_attribute_memo(key.to_s, value.to_s)
     value
   end
   alias_method :set_attribute, :[]=
 
+  # TODO.perf/26: the write knows the new value — seed the
+  # version-stamped memo so a read-back skips re-derivation. A
+  # valid-and-just-invalidated partial memo extends in place; a
+  # full face (or anything older) restarts partial, mirroring the
+  # cold-[] rules.
+  def seed_attribute_memo(name, written)
+    return unless @document
+    values = @attr_values
+    if values && !@attributes &&
+       @attributes_version == @document.version - 1
+      values[name] = written
+    else
+      @attr_values = { name => written }
+      @attributes = nil
+      @attribute_nodes = nil
+      @keys = nil
+      @values = nil
+    end
+    @attributes_version = @document.version
+  end
+  private :seed_attribute_memo
+
   def key?(name)
     ensure_alive!
     # Versioned attribute memo first (TODO.perf/21): a hit proves
@@ -232,7 +260,9 @@ class Leptris::XML::Element < Leptris::XML::Node
     # Require the full Attr-hash face — a cold [] may have set the
     # version with only a partial @attr_values (ruby#150).
     return @attributes if @attributes && memo_hit?(@attributes_version)
-    if native_fast?
+    # TODO.perf/28: the bulk faces touch no document state —
+    # scope-eligible via @addr_reads_fast.
+    if @addr_reads_fast
       # TODO.perf/10: both memo faces in one C walk.
       result, values = Leptris::XML::Native.bulk_attr_faces(
         @c_address, self)

data/lib/leptris/xml/node.rb

@@ -36,6 +36,9 @@ class Leptris::XML::Node
     @structure_memoizable =
       !document.nil? && !document.is_a?(Leptris::XML::IterationScope)
     @pub_document = @structure_memoizable ? document : nil
+    # TODO.perf/25: pure address-based native reads touch no
+    # cache — safe for scope-owned (iterparse) elements too.
+    @addr_reads_fast = !!defined?(Leptris::XML::NATIVE_FAST)
     # NATIVE_FAST availability is fixed at load time before any
     # node exists, so the conjunction with document-ownership is
     # construct-time constant — the hot gates read one ivar.
@@ -164,6 +167,16 @@ class Leptris::XML::Node
     @native_fast
   end
 
+  # TODO.perf/25: the bulk children faces accept the
+  # IterationScope itself (its @wrapper_cache resets per yield —
+  # recycled addresses cannot collide — and its @version advances
+  # per yield and on mutation), so streaming children ride the
+  # one-pass materialization too. The scope-aware stamps live in
+  # the C face.
+  def native_bulk_children?
+    defined?(Leptris::XML::NATIVE_FAST)
+  end
+
   # Content-defined 64-bit Merkle digest of this subtree
   # (libleptris 1.9.99, #869): element name/prefix/resolved
   # namespace URI, attributes sorted and first-wins-deduplicated,
@@ -318,7 +331,7 @@ class Leptris::XML::Node
     # (leptris_node_children_ex), so no per-child get_type.
     return @children if memo_hit?(@children_version)
     ensure_alive!
-    if native_fast_children?
+    if native_bulk_children?
       # TODO.perf/01 tail: one C pass constructs every binding
       # wrapper (class dispatch + ivars + identity cache) — the
       # per-child Ruby wrap frames disappear.
@@ -419,7 +432,7 @@ class Leptris::XML::Node
     # children are never wrapped (nor their get_type paid — the
     # ELEMENT hint rides along). Other nodes keep the filter.
     result =
-      if native_fast_children?
+      if native_bulk_children?
         Leptris::XML::Native.bulk_element_children(@document, @c_address)
       elsif is_a?(Leptris::XML::Element)
         parent = as_element_or_self
@@ -479,6 +492,13 @@ class Leptris::XML::Node
   def visit(&block)
     return enum_for(:visit) unless block
     ensure_alive!
+    # TODO.perf/27: the ext's callback rb_yields directly — no
+    # FFI::Function closure per call.
+    if @addr_reads_fast
+      Leptris::XML::Native.visit_binding(@document, @c_address,
+                                          &block)
+      return self
+    end
     document = @document
     visitor = ::FFI::Function.new(
       :void, [:pointer, :pointer, :int, :int], blocking: true) do |_, node_ptr, entering, depth|
@@ -511,9 +531,18 @@ class Leptris::XML::Node
   # exception and returns non-zero, aborting the C walk — without
   # it the FFI dispatch silently swallowed the exception and the
   # walk continued with partially processed data.
-  def traverse
-    return enum_for(:traverse) unless block_given?
+  def traverse(&block)
+    return enum_for(:traverse) unless block
     ensure_alive!
+    # TODO.perf/27: post-order + abort-at-self + stash-abort-raise
+    # all preserved in the C callback (walk state on its stack).
+    # The block forwards explicitly — rb_yield needs it on the C
+    # entry's own frame.
+    if @addr_reads_fast
+      Leptris::XML::Native.traverse_binding(@document, @c_address,
+                                             &block)
+      return self
+    end
     error = nil
     self_address = @c_address
     callback = ::FFI::Function.new(:int, [:pointer, :pointer], blocking: true) do |node_ptr, _|
@@ -534,8 +563,12 @@ class Leptris::XML::Node
   def path
     return @path if memo_hit?(@path_version)
     ensure_alive!
-    str_ptr = Leptris::XML::FFI.leptris_node_get_xpath(c_ptr)
-    result = str_ptr.null? ? nil : Leptris::XML::FFI.read_owned_string(str_ptr)
+    result = if @addr_reads_fast
+               Leptris::XML::Native.fast_path(@c_address)
+             else
+               str_ptr = Leptris::XML::FFI.leptris_node_get_xpath(c_ptr)
+               str_ptr.null? ? nil : Leptris::XML::FFI.read_owned_string(str_ptr)
+             end
     if @document
       @path = result
       @path_version = @document.version

data/lib/leptris/xml/processing_instruction.rb

@@ -4,7 +4,11 @@ class Leptris::XML::ProcessingInstruction < Leptris::XML::Node
   def name
     return @name if memo_hit?(@name_version)
     ensure_alive!
-    result = Leptris::XML::FFI.leptris_pi_node_get_target(c_ptr)
+    result = if @addr_reads_fast
+               Leptris::XML::Native.fast_pi_target(@c_address)
+             else
+               Leptris::XML::FFI.leptris_pi_node_get_target(c_ptr)
+             end
     if @document
       @name = result
       @name_version = @document.version
@@ -16,8 +20,12 @@ class Leptris::XML::ProcessingInstruction < Leptris::XML::Node
   def content
     return @content if memo_hit?(@content_version)
     ensure_alive!
-    result = Leptris::XML::FFI.read_pi_data(
-      Leptris::XML::FFI.leptris_pi_node_get_data(c_ptr)).to_s
+    raw = if @addr_reads_fast
+            Leptris::XML::Native.fast_pi_data(@c_address)
+          else
+            Leptris::XML::FFI.leptris_pi_node_get_data(c_ptr)
+          end
+    result = Leptris::XML::FFI.read_pi_data(raw).to_s
     if @document
       @content = result
       @content_version = @document.version

data/lib/leptris/xml/text.rb

@@ -6,7 +6,11 @@ class Leptris::XML::Text < Leptris::XML::Node
   def content
     return @content if memo_hit?(@content_version)
     ensure_alive!
-    result = Leptris::XML::FFI.leptris_text_node_get_content(c_ptr)
+    result = if @addr_reads_fast
+               Leptris::XML::Native.fast_text_content(@c_address)
+             else
+               Leptris::XML::FFI.leptris_text_node_get_content(c_ptr)
+             end
     if @document
       @content = result
       @content_version = @document.version

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: leptris
 version: !ruby/object:Gem::Version
-  version: 1.9.174.4
+  version: 1.9.174.6
 platform: ruby
 authors:
 - Ribose Inc.
@@ -93,6 +93,10 @@ files:
 - TODO.perf/22-eager-nodeset-materialization.md
 - TODO.perf/23-cbound-value-mutations.md
 - TODO.perf/24-immutable-read-lanes.md
+- TODO.perf/25-scope-owned-bulk-path.md
+- TODO.perf/26-post-mutation-memo-seeding.md
+- TODO.perf/27-c-yield-traversal.md
+- TODO.perf/28-address-fills.md
 - TODO.restructure/01-constraint-compliance-audit.md
 - TODO.restructure/02-deep-copy-seam.md
 - TODO.restructure/03-evaluation-context-seam.md