From a48fe732ce14327dc5aa8fbeca00f5904fb76b01 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Martynas=20Jusevi=C4=8Dius?= <martynas@atomgraph.com>
Date: Fri, 11 Sep 2026 15:12:11 +0200
Subject: [PATCH] Node identity and parentage are XPath, not DOM calls

Nine reads reached through ixsl:call/ixsl:get for facts the data model
already exposes, so the module asked JavaScript a question XPath answers.
isSameNode($a, $b) is `$a is $b`; parentNode is `..`; parentElement, which
is null when the parent is not an element, is exactly `parent::*`; and a
Range container's nodeType = 3 test is `instance of text()`. The event
targets that carried two of the identity tests take `as="node()?"` so the
comparison has a node to compare - the same typing relatedTarget already
had in navigate.xsl, and the reason `is` was reachable there and not here.

Two constructions follow: Object() through ixsl:call is ixsl:new('Object',
[]), the extension that exists for it; and typeahead's three-line scroll
options object - allocate, set `block`, pass - is one XDM map literal, the
form navigate.xsl was already passing to the same scrollIntoView on the
line above its own call.

Nothing moves in the DOM layer proper. Building detached subtrees through
rdfae:element and splicing them with insertNode/appendChild/before stays:
xsl:result-document emits at a location and returns nothing to hold, and
the insertion model hands the constructed node onward to be spliced, to
receive the caret and to seed a Range. That is an architecture, not a
workaround, and converting it would be a redesign of the editing core.

make sef compiles both entries; make test passes 52; make test-browser
reports 532 assertions true with the two pre-existing canonicalSource
paste-probe values unchanged.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01BSRUJBNKKjdgJeP3qRZTPd
---
 src/annotate.xsl  | 14 +++++++-------
 src/edit.xsl      |  7 ++++---
 src/index.xsl     |  2 +-
 src/navigate.xsl  |  2 +-
 src/typeahead.xsl |  4 +---
 5 files changed, 14 insertions(+), 15 deletions(-)

diff --git a/src/annotate.xsl b/src/annotate.xsl
index 6d40458..ca6e509 100644
--- a/src/annotate.xsl
+++ b/src/annotate.xsl
@@ -85,11 +85,10 @@ version="3.0">
     <xsl:function name="rdfae:selection-valid" as="xs:boolean">
         <xsl:param name="range"/>
 
-        <xsl:variable name="start" select="ixsl:get($range, 'startContainer')"/>
-        <xsl:variable name="end" select="ixsl:get($range, 'endContainer')"/>
-        <xsl:sequence select="ixsl:call($start, 'isSameNode', [ $end ])
-            or (ixsl:get($start, 'nodeType') = 3 and ixsl:get($end, 'nodeType') = 3
-                and ixsl:call(ixsl:get($start, 'parentNode'), 'isSameNode', [ ixsl:get($end, 'parentNode') ]))"/>
+        <xsl:variable name="start" select="ixsl:get($range, 'startContainer')" as="node()?"/>
+        <xsl:variable name="end" select="ixsl:get($range, 'endContainer')" as="node()?"/>
+        <xsl:sequence select="$start is $end
+            or ($start instance of text() and $end instance of text() and $start/.. is $end/..)"/>
     </xsl:function>
 
     <!-- the single write path for RDFa attributes, shared by create and edit. No modes:
@@ -187,7 +186,7 @@ version="3.0">
     <xsl:template name="rdfae:unwrap-element">
         <xsl:param name="element" as="element()"/>
 
-        <xsl:variable name="parent" select="ixsl:get($element, 'parentNode')"/>
+        <xsl:variable name="parent" select="$element/.." as="node()"/>
         <xsl:for-each select="1 to xs:integer(ixsl:get($element, 'childNodes.length'))">
             <xsl:sequence select="ixsl:call($parent, 'insertBefore', [ ixsl:get($element, 'firstChild'), $element ])[current-date() lt xs:date('2000-01-01')]"/>
         </xsl:for-each>
@@ -351,7 +350,8 @@ version="3.0">
 
     <!-- clicking the backdrop (not the content) closes the modal -->
     <xsl:template match="div[@id = 'output-modal']" mode="ixsl:onclick">
-        <xsl:if test="ixsl:call(ixsl:get(ixsl:event(), 'target'), 'isSameNode', [ . ])">
+        <xsl:variable name="target" select="ixsl:get(ixsl:event(), 'target')" as="node()?"/>
+        <xsl:if test="$target is .">
             <ixsl:set-style name="display" select="'none'"/>
         </xsl:if>
     </xsl:template>
diff --git a/src/edit.xsl b/src/edit.xsl
index d4c9019..b838e05 100644
--- a/src/edit.xsl
+++ b/src/edit.xsl
@@ -896,7 +896,8 @@ version="3.0">
         <xsl:variable name="key" as="xs:string" select="string(ixsl:get($event, 'key'))"/>
         <xsl:variable name="chord" as="xs:boolean"
             select="(ixsl:get($event, 'ctrlKey') or ixsl:get($event, 'metaKey')) and not(ixsl:get($event, 'altKey'))"/>
-        <xsl:if test="ixsl:call(ixsl:get($event, 'target'), 'isSameNode', [ . ])">
+        <xsl:variable name="target" select="ixsl:get($event, 'target')" as="node()?"/>
+        <xsl:if test="$target is .">
             <xsl:choose>
                 <xsl:when test="$chord and lower-case($key) = 'z' and not(ixsl:get($event, 'shiftKey'))">
                     <xsl:sequence select="ixsl:call($event, 'preventDefault', [])[current-date() lt xs:date('2000-01-01')]"/>
@@ -1193,7 +1194,7 @@ version="3.0">
 
         <xsl:sequence select="ixsl:call($range, 'insertNode', [ $node ])[current-date() lt xs:date('2000-01-01')]"/>
         <xsl:call-template name="rdfae:place-caret">
-            <xsl:with-param name="node" select="ixsl:get($node, 'parentNode')"/>
+            <xsl:with-param name="node" select="$node/.."/>
             <xsl:with-param name="offset" select="count($node/preceding-sibling::node()) + 1"/>
         </xsl:call-template>
     </xsl:template>
@@ -1710,7 +1711,7 @@ version="3.0">
                 <xsl:sequence select="ixsl:call($range, 'insertNode', [ $fragment ])[current-date() lt xs:date('2000-01-01')]"/>
                 <xsl:for-each select="$last">
                     <xsl:call-template name="rdfae:place-caret">
-                        <xsl:with-param name="node" select="ixsl:get(., 'parentNode')"/>
+                        <xsl:with-param name="node" select=".."/>
                         <xsl:with-param name="offset" select="count(preceding-sibling::node()) + 1"/>
                     </xsl:call-template>
                 </xsl:for-each>
diff --git a/src/index.xsl b/src/index.xsl
index ae7e71e..31fe9cd 100644
--- a/src/index.xsl
+++ b/src/index.xsl
@@ -60,7 +60,7 @@ version="3.0">
          LinkedDataHub's window.LinkedDataHub); reached everywhere via rdfae:editor-state().
          Hosts with their own initial template call this from there instead of main -->
     <xsl:template name="rdfae:init-state">
-        <xsl:variable name="state" select="ixsl:call(ixsl:window(), 'Object', [])"/>
+        <xsl:variable name="state" select="ixsl:new('Object', [])"/>
         <ixsl:set-property name="rdfaEditor" select="$state" object="ixsl:window()"/>
         <xsl:for-each select="('editingSpan', 'range', 'activeBlock', 'draggedBlock',
                 'editRange', 'editingLink', 'insertHost', 'lastUndoHost',
diff --git a/src/navigate.xsl b/src/navigate.xsl
index 980228b..1349354 100644
--- a/src/navigate.xsl
+++ b/src/navigate.xsl
@@ -613,7 +613,7 @@ version="3.0">
                             <xsl:sequence select="ixsl:call(rdfae:selection(),
                                 'setBaseAndExtent', [ $node, $position - 1, $node,
                                     $position - 1 + string-length($query) ])[current-date() lt xs:date('2000-01-01')]"/>
-                            <xsl:sequence select="ixsl:call(ixsl:get($node, 'parentElement'), 'scrollIntoView',
+                            <xsl:sequence select="ixsl:call($node/parent::*, 'scrollIntoView',
                                 [ map{ 'block': 'center' } ])[current-date() lt xs:date('2000-01-01')]"/>
                             <ixsl:set-property name="findNode" select="$node" object="rdfae:editor-state()"/>
                             <ixsl:set-property name="findOffset"
diff --git a/src/typeahead.xsl b/src/typeahead.xsl
index a94ff5d..0331b5f 100644
--- a/src/typeahead.xsl
+++ b/src/typeahead.xsl
@@ -321,9 +321,7 @@ version="3.0">
                 </xsl:for-each>
                 <xsl:for-each select="$items[$next]">
                     <ixsl:set-attribute name="aria-selected" select="'true'"/>
-                    <xsl:variable name="opts" select="ixsl:call(ixsl:window(), 'Object', [])"/>
-                    <ixsl:set-property name="block" select="'nearest'" object="$opts"/>
-                    <xsl:sequence select="ixsl:call(., 'scrollIntoView', [ $opts ])[current-date() lt xs:date('2000-01-01')]"/>
+                    <xsl:sequence select="ixsl:call(., 'scrollIntoView', [ map{ 'block': 'nearest' } ])[current-date() lt xs:date('2000-01-01')]"/>
                 </xsl:for-each>
             </xsl:when>
             <xsl:otherwise/>