@portabletext/editor 1.44.16 → 1.45.1

package/lib/selectors/index.d.cts

@@ -59,7 +59,7 @@ declare type AbstractBehaviorEvent =
   | {
       type: StrictExtract<AbstractBehaviorEventType, 'decorator.toggle'>
       decorator: string
-      offsets?: {
+      at?: {
         anchor: BlockOffset
         focus: BlockOffset
       }
@@ -5380,7 +5380,7 @@ declare type SyntheticBehaviorEvent =
   | {
       type: StrictExtract<SyntheticBehaviorEventType, 'decorator.add'>
       decorator: string
-      offsets?: {
+      at?: {
         anchor: BlockOffset
         focus: BlockOffset
       }
@@ -5391,7 +5391,7 @@ declare type SyntheticBehaviorEvent =
     }
   | {
       type: StrictExtract<SyntheticBehaviorEventType, 'delete'>
-      selection: NonNullable<EditorSelection>
+      at: NonNullable<EditorSelection>
     }
   | {
       type: StrictExtract<SyntheticBehaviorEventType, 'delete.backward'>
@@ -5407,8 +5407,10 @@ declare type SyntheticBehaviorEvent =
     }
   | {
       type: StrictExtract<SyntheticBehaviorEventType, 'delete.text'>
-      anchor: BlockOffset
-      focus: BlockOffset
+      at: {
+        anchor: BlockOffset
+        focus: BlockOffset
+      }
     }
   | {
       type: StrictExtract<SyntheticBehaviorEventType, 'focus'>
@@ -5456,7 +5458,7 @@ declare type SyntheticBehaviorEvent =
     }
   | {
       type: StrictExtract<SyntheticBehaviorEventType, 'select'>
-      selection: EditorSelection
+      at: EditorSelection
     }
   | {
       type: StrictExtract<SyntheticBehaviorEventType, 'split.block'>

package/lib/selectors/index.d.ts

@@ -59,7 +59,7 @@ declare type AbstractBehaviorEvent =
   | {
       type: StrictExtract<AbstractBehaviorEventType, 'decorator.toggle'>
       decorator: string
-      offsets?: {
+      at?: {
         anchor: BlockOffset
         focus: BlockOffset
       }
@@ -5380,7 +5380,7 @@ declare type SyntheticBehaviorEvent =
   | {
       type: StrictExtract<SyntheticBehaviorEventType, 'decorator.add'>
       decorator: string
-      offsets?: {
+      at?: {
         anchor: BlockOffset
         focus: BlockOffset
       }
@@ -5391,7 +5391,7 @@ declare type SyntheticBehaviorEvent =
     }
   | {
       type: StrictExtract<SyntheticBehaviorEventType, 'delete'>
-      selection: NonNullable<EditorSelection>
+      at: NonNullable<EditorSelection>
     }
   | {
       type: StrictExtract<SyntheticBehaviorEventType, 'delete.backward'>
@@ -5407,8 +5407,10 @@ declare type SyntheticBehaviorEvent =
     }
   | {
       type: StrictExtract<SyntheticBehaviorEventType, 'delete.text'>
-      anchor: BlockOffset
-      focus: BlockOffset
+      at: {
+        anchor: BlockOffset
+        focus: BlockOffset
+      }
     }
   | {
       type: StrictExtract<SyntheticBehaviorEventType, 'focus'>
@@ -5456,7 +5458,7 @@ declare type SyntheticBehaviorEvent =
     }
   | {
       type: StrictExtract<SyntheticBehaviorEventType, 'select'>
-      selection: EditorSelection
+      at: EditorSelection
     }
   | {
       type: StrictExtract<SyntheticBehaviorEventType, 'split.block'>

package/lib/utils/index.d.cts

@@ -59,7 +59,7 @@ declare type AbstractBehaviorEvent =
   | {
       type: StrictExtract<AbstractBehaviorEventType, 'decorator.toggle'>
       decorator: string
-      offsets?: {
+      at?: {
         anchor: BlockOffset
         focus: BlockOffset
       }
@@ -5214,7 +5214,7 @@ declare type SyntheticBehaviorEvent =
   | {
       type: StrictExtract<SyntheticBehaviorEventType, 'decorator.add'>
       decorator: string
-      offsets?: {
+      at?: {
         anchor: BlockOffset
         focus: BlockOffset
       }
@@ -5225,7 +5225,7 @@ declare type SyntheticBehaviorEvent =
     }
   | {
       type: StrictExtract<SyntheticBehaviorEventType, 'delete'>
-      selection: NonNullable<EditorSelection>
+      at: NonNullable<EditorSelection>
     }
   | {
       type: StrictExtract<SyntheticBehaviorEventType, 'delete.backward'>
@@ -5241,8 +5241,10 @@ declare type SyntheticBehaviorEvent =
     }
   | {
       type: StrictExtract<SyntheticBehaviorEventType, 'delete.text'>
-      anchor: BlockOffset
-      focus: BlockOffset
+      at: {
+        anchor: BlockOffset
+        focus: BlockOffset
+      }
     }
   | {
       type: StrictExtract<SyntheticBehaviorEventType, 'focus'>
@@ -5290,7 +5292,7 @@ declare type SyntheticBehaviorEvent =
     }
   | {
       type: StrictExtract<SyntheticBehaviorEventType, 'select'>
-      selection: EditorSelection
+      at: EditorSelection
     }
   | {
       type: StrictExtract<SyntheticBehaviorEventType, 'split.block'>

package/lib/utils/index.d.ts

@@ -59,7 +59,7 @@ declare type AbstractBehaviorEvent =
   | {
       type: StrictExtract<AbstractBehaviorEventType, 'decorator.toggle'>
       decorator: string
-      offsets?: {
+      at?: {
         anchor: BlockOffset
         focus: BlockOffset
       }
@@ -5214,7 +5214,7 @@ declare type SyntheticBehaviorEvent =
   | {
       type: StrictExtract<SyntheticBehaviorEventType, 'decorator.add'>
       decorator: string
-      offsets?: {
+      at?: {
         anchor: BlockOffset
         focus: BlockOffset
       }
@@ -5225,7 +5225,7 @@ declare type SyntheticBehaviorEvent =
     }
   | {
       type: StrictExtract<SyntheticBehaviorEventType, 'delete'>
-      selection: NonNullable<EditorSelection>
+      at: NonNullable<EditorSelection>
     }
   | {
       type: StrictExtract<SyntheticBehaviorEventType, 'delete.backward'>
@@ -5241,8 +5241,10 @@ declare type SyntheticBehaviorEvent =
     }
   | {
       type: StrictExtract<SyntheticBehaviorEventType, 'delete.text'>
-      anchor: BlockOffset
-      focus: BlockOffset
+      at: {
+        anchor: BlockOffset
+        focus: BlockOffset
+      }
     }
   | {
       type: StrictExtract<SyntheticBehaviorEventType, 'focus'>
@@ -5290,7 +5292,7 @@ declare type SyntheticBehaviorEvent =
     }
   | {
       type: StrictExtract<SyntheticBehaviorEventType, 'select'>
-      selection: EditorSelection
+      at: EditorSelection
     }
   | {
       type: StrictExtract<SyntheticBehaviorEventType, 'split.block'>

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@portabletext/editor",
-  "version": "1.44.16",
+  "version": "1.45.1",
   "description": "Portable Text Editor made in React",
   "keywords": [
     "sanity",
@@ -79,8 +79,8 @@
     "slate-react": "0.112.1",
     "use-effect-event": "^1.0.2",
     "xstate": "^5.19.2",
-    "@portabletext/patches": "1.1.3",
-    "@portabletext/block-tools": "1.1.16"
+    "@portabletext/block-tools": "1.1.16",
+    "@portabletext/patches": "1.1.3"
   },
   "devDependencies": {
     "@portabletext/toolkit": "^2.0.17",

package/src/behavior-actions/behavior.action.annotation.add.ts

@@ -1,5 +1,6 @@
 import type {Path} from '@sanity/types'
 import {Editor, Node, Range, Text, Transforms} from 'slate'
+import {parseAnnotation} from '../internal-utils/parse-blocks'
 import type {BehaviorActionImplementation} from './behavior.actions'
 
 /**
@@ -24,6 +25,21 @@ export const addAnnotationActionImplementation: BehaviorActionImplementation<
   'annotation.add',
   AddedAnnotationPaths | undefined
 > = ({context, action}) => {
+  const parsedAnnotation = parseAnnotation({
+    annotation: {
+      _type: action.annotation.name,
+      ...action.annotation.value,
+    },
+    context,
+    options: {refreshKeys: false},
+  })
+
+  if (!parsedAnnotation) {
+    throw new Error(
+      `Failed to parse annotation ${JSON.stringify(action.annotation)}`,
+    )
+  }
+
   const editor = action.editor
 
   if (!editor.selection || Range.isCollapsed(editor.selection)) {
@@ -41,6 +57,8 @@ export const addAnnotationActionImplementation: BehaviorActionImplementation<
     reverse: Range.isBackward(editor.selection),
   })
 
+  let blockIndex = 0
+
   for (const [block, blockPath] of selectedBlocks) {
     if (block.children.length === 0) {
       continue
@@ -50,11 +68,13 @@ export const addAnnotationActionImplementation: BehaviorActionImplementation<
       continue
     }
 
-    const annotationKey = context.keyGenerator()
+    // Make sure we don't generate more keys than needed
+    const annotationKey =
+      blockIndex === 0 ? parsedAnnotation._key : context.keyGenerator()
     const markDefs = block.markDefs ?? []
     const existingMarkDef = markDefs.find(
       (markDef) =>
-        markDef._type === action.annotation.name &&
+        markDef._type === parsedAnnotation._type &&
         markDef._key === annotationKey,
     )
 
@@ -65,9 +85,8 @@ export const addAnnotationActionImplementation: BehaviorActionImplementation<
           markDefs: [
             ...markDefs,
             {
-              _type: action.annotation.name,
+              ...parsedAnnotation,
               _key: annotationKey,
-              ...action.annotation.value,
             },
           ],
         },
@@ -100,7 +119,7 @@ export const addAnnotationActionImplementation: BehaviorActionImplementation<
       const existingSameTypeAnnotations = marks.filter((mark) =>
         markDefs.some(
           (markDef) =>
-            markDef._key === mark && markDef._type === action.annotation.name,
+            markDef._key === mark && markDef._type === parsedAnnotation._type,
         ),
       )
 
@@ -119,6 +138,8 @@ export const addAnnotationActionImplementation: BehaviorActionImplementation<
 
       spanPath = [{_key: block._key}, 'children', {_key: span._key}]
     }
+
+    blockIndex++
   }
 
   if (markDefPath && spanPath) {

package/src/behavior-actions/behavior.action.decorator.add.ts

@@ -18,17 +18,17 @@ export const decoratorAddActionImplementation: BehaviorActionImplementation<
     KEY_TO_VALUE_ELEMENT.get(editor),
   )
 
-  const manualAnchor = action.offsets?.anchor
+  const manualAnchor = action.at?.anchor
     ? utils.blockOffsetToSpanSelectionPoint({
         value,
-        blockOffset: action.offsets.anchor,
+        blockOffset: action.at.anchor,
         direction: 'backward',
       })
     : undefined
-  const manualFocus = action.offsets?.focus
+  const manualFocus = action.at?.focus
     ? utils.blockOffsetToSpanSelectionPoint({
         value,
-        blockOffset: action.offsets.focus,
+        blockOffset: action.at.focus,
         direction: 'forward',
       })
     : undefined

package/src/behavior-actions/behavior.action.delete.text.ts

@@ -17,10 +17,7 @@ export const deleteTextActionImplementation: BehaviorActionImplementation<
 
   const selection = utils.blockOffsetsToSelection({
     value,
-    offsets: {
-      anchor: action.anchor,
-      focus: action.focus,
-    },
+    offsets: action.at,
   })
 
   if (!selection) {

package/src/behavior-actions/behavior.action.delete.ts

@@ -4,11 +4,11 @@ import type {BehaviorActionImplementation} from './behavior.actions'
 export const deleteActionImplementation: BehaviorActionImplementation<
   'delete'
 > = ({action}) => {
-  const range = toSlateRange(action.selection, action.editor)
+  const range = toSlateRange(action.at, action.editor)
 
   if (!range) {
     throw new Error(
-      `Failed to get Slate Range for selection ${JSON.stringify(action.selection)}`,
+      `Failed to get Slate Range for selection ${JSON.stringify(action.at)}`,
     )
   }
 

package/src/behavior-actions/behavior.action.insert-inline-object.ts

@@ -1,17 +1,24 @@
 import {Editor, Transforms, type Element} from 'slate'
+import {parseInlineObject} from '../internal-utils/parse-blocks'
 import {toSlateValue} from '../internal-utils/values'
 import type {BehaviorActionImplementation} from './behavior.actions'
 
 export const insertInlineObjectActionImplementation: BehaviorActionImplementation<
   'insert.inline object'
 > = ({context, action}) => {
-  if (
-    !context.schema.inlineObjects.some(
-      (inlineObject) => inlineObject.name === action.inlineObject.name,
+  const parsedInlineObject = parseInlineObject({
+    context,
+    inlineObject: {
+      _type: action.inlineObject.name,
+      ...(action.inlineObject.value ?? {}),
+    },
+    options: {refreshKeys: false},
+  })
+
+  if (!parsedInlineObject) {
+    throw new Error(
+      `Failed to parse inline object ${JSON.stringify(action.inlineObject)}`,
     )
-  ) {
-    console.error('Unable to insert unknown inline object')
-    return
   }
 
   if (!action.editor.selection) {
@@ -36,13 +43,7 @@ export const insertInlineObjectActionImplementation: BehaviorActionImplementatio
       {
         _type: context.schema.block.name,
         _key: context.keyGenerator(),
-        children: [
-          {
-            _type: action.inlineObject.name,
-            _key: context.keyGenerator(),
-            ...(action.inlineObject.value ?? {}),
-          },
-        ],
+        children: [parsedInlineObject],
       },
     ],
     {schemaTypes: context.schema},

package/src/behavior-actions/behavior.action.select.ts

@@ -5,7 +5,7 @@ import type {BehaviorActionImplementation} from './behavior.actions'
 export const selectActionImplementation: BehaviorActionImplementation<
   'select'
 > = ({action}) => {
-  const newSelection = toSlateRange(action.selection, action.editor)
+  const newSelection = toSlateRange(action.at, action.editor)
 
   if (newSelection) {
     Transforms.select(action.editor, newSelection)

package/src/behaviors/behavior.abstract.decorator.ts

@@ -16,10 +16,10 @@ export const abstractDecoratorBehaviors = [
   defineBehavior({
     on: 'decorator.toggle',
     guard: ({snapshot, event}) => {
-      const manualSelection = event.offsets
+      const manualSelection = event.at
         ? blockOffsetsToSelection({
             value: snapshot.context.value,
-            offsets: event.offsets,
+            offsets: event.at,
           })
         : null
 

package/src/behaviors/behavior.abstract.select.ts

@@ -29,7 +29,7 @@ export const abstractSelectBehaviors = [
       (_, {selection}) => [
         raise({
           type: 'select',
-          selection,
+          at: selection,
         }),
       ],
     ],
@@ -54,7 +54,7 @@ export const abstractSelectBehaviors = [
       (_, {selection}) => [
         raise({
           type: 'select',
-          selection,
+          at: selection,
         }),
       ],
     ],

package/src/behaviors/behavior.core.annotations.ts

@@ -28,7 +28,7 @@ const addAnnotationOnCollapsedSelection = defineBehavior({
   },
   actions: [
     ({event}, {caretWordSelection}) => [
-      raise({type: 'select', selection: caretWordSelection}),
+      raise({type: 'select', at: caretWordSelection}),
       raise({type: 'annotation.add', annotation: event.annotation}),
     ],
   ],

package/src/behaviors/behavior.core.block-objects.ts

@@ -132,7 +132,7 @@ const clickingAboveLonelyBlockObject = defineBehavior({
     ({snapshot, event}) => [
       raise({
         type: 'select',
-        selection: event.position.selection,
+        at: event.position.selection,
       }),
       raise({
         type: 'insert.block',
@@ -186,7 +186,7 @@ const clickingBelowLonelyBlockObject = defineBehavior({
     ({snapshot, event}) => [
       raise({
         type: 'select',
-        selection: event.position.selection,
+        at: event.position.selection,
       }),
       raise({
         type: 'insert.block',
@@ -228,7 +228,7 @@ const deletingEmptyTextBlockAfterBlockObject = defineBehavior({
       }),
       raise({
         type: 'select',
-        selection: {
+        at: {
           anchor: {path: previousBlock.path, offset: 0},
           focus: {path: previousBlock.path, offset: 0},
         },
@@ -265,7 +265,7 @@ const deletingEmptyTextBlockBeforeBlockObject = defineBehavior({
       }),
       raise({
         type: 'select',
-        selection: {
+        at: {
           anchor: {path: nextBlock.path, offset: 0},
           focus: {path: nextBlock.path, offset: 0},
         },

package/src/behaviors/behavior.decorator-pair.ts

@@ -165,7 +165,7 @@ export function createDecoratorPairBehavior(config: {
         {
           type: 'decorator.add',
           decorator,
-          offsets: {
+          at: {
             anchor: prefixOffsets.focus,
             focus: suffixOffsets.anchor,
           },
@@ -173,12 +173,12 @@ export function createDecoratorPairBehavior(config: {
         // Delete the suffix
         {
           type: 'delete.text',
-          ...suffixOffsets,
+          at: suffixOffsets,
         },
         // Delete the prefix
         {
           type: 'delete.text',
-          ...prefixOffsets,
+          at: prefixOffsets,
         },
         // Toggle the decorator off so the next inserted text isn't emphasized
         {

package/src/behaviors/behavior.default.ts

@@ -149,7 +149,7 @@ export const defaultBehaviors = [
         }),
         raise({
           type: 'delete',
-          selection,
+          at: selection,
         }),
       ],
     ],
@@ -220,7 +220,7 @@ export const defaultBehaviors = [
       ({event}) => [
         raise({
           type: 'select',
-          selection: event.position.selection,
+          at: event.position.selection,
         }),
         raise({
           type: 'deserialize',
@@ -293,7 +293,7 @@ export const defaultBehaviors = [
           : [
               raise({
                 type: 'delete',
-                selection: dragOrigin.selection,
+                at: dragOrigin.selection,
               }),
             ]),
         raise({
@@ -414,7 +414,7 @@ export const defaultBehaviors = [
       ({event}, {selection}) => [
         raise({
           type: 'delete',
-          selection,
+          at: selection,
         }),
         raise({
           type: 'deserialize',

package/src/behaviors/behavior.emoji-picker.ts

@@ -131,13 +131,15 @@ export function createEmojiPickerBehaviors<TEmojiMatch>(
           },
           {
             type: 'delete.text',
-            anchor: {
-              path: params.focusBlock.path,
-              offset: params.textBeforeLength - params.emojiStringLength,
-            },
-            focus: {
-              path: params.focusBlock.path,
-              offset: params.textBeforeLength,
+            at: {
+              anchor: {
+                path: params.focusBlock.path,
+                offset: params.textBeforeLength - params.emojiStringLength,
+              },
+              focus: {
+                path: params.focusBlock.path,
+                offset: params.textBeforeLength,
+              },
             },
           },
           {
@@ -225,13 +227,15 @@ export function createEmojiPickerBehaviors<TEmojiMatch>(
               },
               {
                 type: 'delete.text',
-                anchor: {
-                  path: params.focusBlock.path,
-                  offset: params.textBeforeLength - params.emojiStringLength,
-                },
-                focus: {
-                  path: params.focusBlock.path,
-                  offset: params.textBeforeLength,
+                at: {
+                  anchor: {
+                    path: params.focusBlock.path,
+                    offset: params.textBeforeLength - params.emojiStringLength,
+                  },
+                  focus: {
+                    path: params.focusBlock.path,
+                    offset: params.textBeforeLength,
+                  },
                 },
               },
               {

package/src/behaviors/behavior.markdown.ts

@@ -137,13 +137,15 @@ export function createMarkdownBehaviors(config: MarkdownBehaviorsConfig) {
         },
         {
           type: 'delete.text',
-          anchor: {
-            path: focusTextBlock.path,
-            offset: 0,
-          },
-          focus: {
-            path: focusTextBlock.path,
-            offset: 2,
+          at: {
+            anchor: {
+              path: focusTextBlock.path,
+              offset: 0,
+            },
+            focus: {
+              path: focusTextBlock.path,
+              offset: 2,
+            },
           },
         },
       ],
@@ -213,7 +215,7 @@ export function createMarkdownBehaviors(config: MarkdownBehaviorsConfig) {
         },
         {
           type: 'delete.text',
-          ...hrBlockOffsets,
+          at: hrBlockOffsets,
         },
       ],
     ],
@@ -222,7 +224,7 @@ export function createMarkdownBehaviors(config: MarkdownBehaviorsConfig) {
     on: 'clipboard.paste',
     guard: ({snapshot, event}) => {
       const text = event.originEvent.dataTransfer.getData('text/plain')
-      const hrRegExp = /^(---)$|(___)$|(\*\*\*)$/gm
+      const hrRegExp = /^(---)$|(___)$|(\*\*\*)$/
       const hrCharacters = text.match(hrRegExp)?.[0]
       const hrObject = config.horizontalRuleObject?.(snapshot.context)
       const focusBlock = selectors.getFocusBlock(snapshot)
@@ -348,13 +350,15 @@ export function createMarkdownBehaviors(config: MarkdownBehaviorsConfig) {
         },
         {
           type: 'delete.text',
-          anchor: {
-            path: focusTextBlock.path,
-            offset: 0,
-          },
-          focus: {
-            path: focusTextBlock.path,
-            offset: level + 1,
+          at: {
+            anchor: {
+              path: focusTextBlock.path,
+              offset: 0,
+            },
+            focus: {
+              path: focusTextBlock.path,
+              offset: level + 1,
+            },
           },
         },
       ],
@@ -485,13 +489,15 @@ export function createMarkdownBehaviors(config: MarkdownBehaviorsConfig) {
         },
         {
           type: 'delete.text',
-          anchor: {
-            path: focusTextBlock.path,
-            offset: 0,
-          },
-          focus: {
-            path: focusTextBlock.path,
-            offset: listItemLength + 1,
+          at: {
+            anchor: {
+              path: focusTextBlock.path,
+              offset: 0,
+            },
+            focus: {
+              path: focusTextBlock.path,
+              offset: listItemLength + 1,
+            },
           },
         },
       ],