LearnNewsExamplesServices
Frontmatter
id8611
titleUpdate DeltaUpdates.moveNode to support Fragments
stateClosed
labels
enhancementaicore
assigneestobiu
createdAtJan 13, 2026, 7:41 PM
updatedAtJan 13, 2026, 8:04 PM
githubUrlhttps://github.com/neomjs/neo/issues/8611
authortobiu
commentsCount1
parentIssue8601
subIssues[]
subIssuesCompleted0
subIssuesTotal0
contentTrust
projected
quarantined0
signals[]
blockedBy[]
blocking[]
closedAtJan 13, 2026, 8:03 PM

Update DeltaUpdates.moveNode to support Fragments

tobiu
tobiu commented on Jan 13, 2026, 7:41 PM

Update Neo.main.DeltaUpdates.moveNode to correctly resolve parent and sibling nodes when the target is a Neo.container.Fragment.

Changes:

  1. Check if parentNode is null (indicating a logical parent like a Fragment).
  2. If null, use getFragmentStart(parentId) to find the anchor.
  3. Resolve parentNode from the anchor.
  4. Resolve siblingRef using getFragmentSibling(startNode, index).
  5. Proceed with existing insertBefore logic (preserving the "HOT-SWAP" optimization).

This depends on #8609 (Helper methods).

tobiu
tobiu Jan 13, 2026, 8:03 PM

Input from Gemini 3 Pro:

✦ Updated moveNode to correctly resolve Fragment parents using the new helper methods. Verified logic with FragmentDomMove.spec.mjs (direct DeltaUpdates verification). E2E testing (FragmentLifecycle) is still timing out and will be addressed separately.