fix(slate-dom): propagate suppressThrow to toSlateNode in toSlatePoint

[pangoyal-gong]

Problem

toSlatePoint accepts a suppressThrow option that is supposed to prevent it from throwing when a DOM node cannot be resolved to a Slate point. However, the implementation only suppresses errors from findPath — errors thrown by toSlateNode (called on the line immediately before) are not guarded.

This means that callers passing suppressThrow: true can still receive an unhandled Error: Cannot resolve a Slate node from DOM node exception thrown by toSlateNode, defeating the purpose of the option.

Affected code path

In packages/slate-dom/src/plugin/dom-editor.ts, the toSlatePoint implementation:

// BEFORE (buggy)
const slateNode = DOMEditor.toSlateNode(editor, textNode!)  // throws unconditionally
let path
try {
  path = DOMEditor.findPath(editor, slateNode)  // only this is guarded
} catch (e) {
  if (suppressThrow) {
    return null
  }
  throw e
}

A common trigger: the document-level selectionchange listener in slate-react calls toSlatePoint with suppressThrow: true, but receives the error anyway because toSlateNode is not guarded. This produces high-volume Sentry noise for applications using Slate in a React portal or with multiple editors on the page.

Relationship to PR #6004

PR #6004 added suppressThrow handling to findPath inside toSlatePoint. This fix applies the same pattern one step earlier, to the toSlateNode call that precedes it.

Fix

Wrap the toSlateNode call in the same try/catch pattern already used for findPath:

// AFTER (fixed)
let slateNode
try {
  slateNode = DOMEditor.toSlateNode(editor, textNode!)
} catch (e) {
  if (suppressThrow) {
    return null
  }
  throw e
}
let path
try {
  path = DOMEditor.findPath(editor, slateNode)
} catch (e) {
  if (suppressThrow) {
    return null
  }
  throw e
}

Testing

The change is minimal and mirrors the existing pattern in the same function. The behavior when suppressThrow is false (or omitted) is unchanged — errors still propagate as before.

[changeset-bot]

🦋 Changeset detected

Latest commit: 5a47415

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package

Name	Type
slate-dom	Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

[dylans]

@pangoyal-gong needs a quick prettier run please on the changeset, likely needs a single-quote change and a shorter line.

[dylans]

Forgot this still needs prettier changes to land it please.

[pangoyal-gong]

@dylans pushed.