svedit 0.6.4 → 0.7.1

package/dist/Transaction.svelte.js

@@ -18,7 +18,8 @@ import {
 	apply_op,
 	count_references_excluding_deleted,
 	validate_node,
-	get_active_annotation
+	get_active_annotation,
+	validate_selection
 } from './doc_utils.js';
 
 /**
@@ -145,7 +146,7 @@ export default class Transaction {
 	}
 
 	/**
-	 * Returns the annotation object that is currently "under the cursor".
+	 * Returns the annotation object that is currently "under the caret".
 	 * NOTE: Annotations in Svedit are exclusive, so there can only be one active_annotation
 	 *
 	 * @param {string} [annotation_type] Optional annotation type to filter by
@@ -336,6 +337,7 @@ export default class Transaction {
 		const previous_value = this.get(id);
 		if (!previous_value) {
 			console.warn(`Deletion of node ${id} skipped, as it does not exist.`);
+			return this;
 		}
 		// Get nodes referenced by this node BEFORE deleting it.
 		const referenced_nodes = this.get_referenced_nodes(id);
@@ -343,9 +345,7 @@ export default class Transaction {
 		this.ops.push(op);
 		this.inverse_ops.push(['create', previous_value]);
 		this._apply_op(op);
-		// Check if the nodes that were referenced by the deleted node are now orphaned
-		// NOTE: We don't do this yet, because we still have some manual child node removal code
-		// that causes errors. But we should soon implement this only here.
+		// Cascade delete any nodes that are now orphaned after removing this node
 		this._cascade_delete_unreferenced_nodes(referenced_nodes);
 		return this;
 	}
@@ -371,50 +371,7 @@ export default class Transaction {
 	 * @private
 	 */
 	_validate_selection(selection) {
-		if (!selection) return;
-
-		const selection_type = selection.type;
-		if (!['node', 'text', 'property'].includes(selection_type)) {
-			throw new Error(`Invalid selection type: ${selection_type}`);
-		}
-
-		if (selection_type === 'node') {
-			const node_array = this.get(selection.path);
-
-			if (!Array.isArray(node_array)) {
-				throw new Error('Node selection path must point to a node_array');
-			}
-
-			const max_offset = node_array.length;
-			if (selection.anchor_offset < 0 || selection.anchor_offset > max_offset) {
-				throw new Error(
-					`Node selection anchor_offset (${selection.anchor_offset}) is out of bounds. Max is ${max_offset}.`
-				);
-			}
-			if (selection.focus_offset < 0 || selection.focus_offset > max_offset) {
-				throw new Error(
-					`Node selection focus_offset (${selection.focus_offset}) is out of bounds. Max is ${max_offset}.`
-				);
-			}
-		} else if (selection_type === 'text') {
-			const annotated_text = this.get(selection.path);
-
-			if (!annotated_text || typeof annotated_text.text !== 'string') {
-				throw new Error('Text selection path must point to annotated_text');
-			}
-
-			const char_length = get_char_length(annotated_text.text);
-			if (selection.anchor_offset < 0 || selection.anchor_offset > char_length) {
-				throw new Error(
-					`Text selection anchor_offset (${selection.anchor_offset}) is out of bounds. Max is ${char_length}.`
-				);
-			}
-			if (selection.focus_offset < 0 || selection.focus_offset > char_length) {
-				throw new Error(
-					`Text selection focus_offset (${selection.focus_offset}) is out of bounds. Max is ${char_length}.`
-				);
-			}
-		}
+		validate_selection(selection, this);
 	}
 
 	/**
@@ -653,7 +610,7 @@ export default class Transaction {
 	insert_nodes(node_ids) {
 		if (this.selection.type !== 'node') return this;
 
-		// Unless cursor is collapsed, delete the selected nodes as a first step
+		// Unless caret is collapsed, delete the selected nodes as a first step
 		if (this.selection.anchor_offset !== this.selection.focus_offset) {
 			this.delete_selection();
 		}
@@ -661,14 +618,7 @@ export default class Transaction {
 		const path = this.selection.path;
 		const node_array = [...this.get(path)];
 
-		// Get the start and end indices for the selection
 		let start = Math.min(this.selection.anchor_offset, this.selection.focus_offset);
-		let end = Math.max(this.selection.anchor_offset, this.selection.focus_offset);
-
-		if (start !== end) {
-			// Remove the selected nodes from the node_array
-			node_array.splice(start, end - start);
-		}
 
 		// Insert the new nodes
 		node_array.splice(start, 0, ...node_ids);
@@ -744,16 +694,6 @@ export default class Transaction {
 					node_id
 				};
 			}
-
-			// Insertion point is inside the annotation - extend the annotation
-			if (annotation_start < range.start_offset && annotation_end > range.start_offset) {
-				return {
-					start_offset: annotation_start,
-					end_offset: annotation_end + delta,
-					node_id
-				};
-			}
-
 			return annotation;
 		});
 
@@ -771,7 +711,7 @@ export default class Transaction {
 		this.selection = new_selection;
 
 		// Now we apply annotations if there are any, but only if there's no active annotation
-		// at the current collapsed cursor
+		// at the current collapsed caret
 		if (!this.active_annotation() && annotations.length > 0) {
 			const new_annotations = annotations
 				.map((annotation) => {

package/dist/doc_utils.d.ts

@@ -114,6 +114,18 @@ export function get_active_annotation(schema: DocumentSchema, doc: Document, sel
  * @returns {number} The number of references excluding deleted nodes
  */
 export function count_references_excluding_deleted(schema: DocumentSchema, doc: Document, target_node_id: NodeId, nodes_to_delete: Record<NodeId, boolean>): number;
+/**
+ * Validates a selection against the current document state.
+ * Works with any object that implements get() and inspect() (Session or Transaction).
+ *
+ * @param {Selection} selection - The selection to validate
+ * @param {{ get: Function, inspect: Function }} session_or_transaction - A Session or Transaction instance
+ * @throws {Error} Throws if the selection is invalid
+ */
+export function validate_selection(selection: Selection, session_or_transaction: {
+    get: Function;
+    inspect: Function;
+}): void;
 import type { NodeSchema } from './types';
 import type { PrimitiveType } from './types';
 import type { NodeProperty } from './types';

package/dist/doc_utils.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"doc_utils.d.ts","sourceRoot":"","sources":["../src/lib/doc_utils.js"],"names":[],"mappings":"AAWA;;;;;;;GAOG;AACH,uCAJ0C,CAAC,SAA7B,MAAM,CAAC,MAAM,EAAE,UAAU,CAAE,UAC9B,CAAC,GACC,CAAC,CAIb;AAED;;;;GAIG;AACH,wCAHW,MAAM,GACJ,IAAI,IAAI,aAAa,CAejC;AAED;;;;GAIG;AACH,2DAHW,YAAY,GAAG,iBAAiB,GAC9B,MAAM,GAAG,IAAI,CAWzB;AAED;;;;GAIG;AACH,0DAHW,cAAc,QAmBxB;AAgDD;;;;;;GAMG;AACH,oCALW,GAAG,UACH,cAAc,cACd,MAAM,CAAC,MAAM,EAAE,GAAG,CAAC,QA6D7B;AAED;;;;;;;GAOG;AACH,4BALW,cAAc,OACd,QAAQ,QACR,YAAY,GAAC,MAAM,GACjB,GAAG,CA4Ef;AAED;;;;;;;GAOG;AACH,sCALW,MAAM,QACN,MAAM,YACN,MAAM,GACJ,MAAM,CAclB;AAED;;;;;;GAMG;AACH,6BAJW,MAAM,QACN,GAAG,GACD,OAAO,GAAC,MAAM,GAAC,YAAY,CAIvC;AAED;;;;;;;GAOG;AACH,gCALW,cAAc,OACd,QAAQ,QACR,YAAY,GACV;IAAC,IAAI,EAAE,UAAU,GAAC,MAAM,CAAC;IAAC,CAAC,GAAG,EAAE,MAAM,GAAG,GAAG,CAAA;CAAC,CAoBzD;AAED;;;;;;;GAOG;AACH,8BAJW,QAAQ,cAEN,QAAQ,CAkCpB;AAED;;;;;;;GAOG;AACH,yCALW,cAAc,OACd,QAAQ,WACR,MAAM,GACJ,MAAM,CAoBlB;AAED;;;;;;;;;;;;;GAaG;AACH,8CANW,cAAc,OACd,QAAQ,cACR,SAAS,oBACT,MAAM,GACJ,UAAU,GAAG,IAAI,CAwB7B;AAED;;;;;;;;GAQG;AACH,2DANW,cAAc,OACd,QAAQ,kBACR,MAAM,mBACN,MAAM,CAAC,MAAM,EAAE,OAAO,CAAC,GACrB,MAAM,CA0BlB;gCA3dqJ,SAAS;mCAAT,SAAS;kCAAT,SAAS;uCAAT,SAAS;oCAAT,SAAS;8BAAT,SAAS;kCAAT,SAAS;4BAAT,SAAS;+BAAT,SAAS;gCAAT,SAAS"}
+{"version":3,"file":"doc_utils.d.ts","sourceRoot":"","sources":["../src/lib/doc_utils.js"],"names":[],"mappings":"AAWA;;;;;;;GAOG;AACH,uCAJ0C,CAAC,SAA7B,MAAM,CAAC,MAAM,EAAE,UAAU,CAAE,UAC9B,CAAC,GACC,CAAC,CAIb;AAED;;;;GAIG;AACH,wCAHW,MAAM,GACJ,IAAI,IAAI,aAAa,CAejC;AAED;;;;GAIG;AACH,2DAHW,YAAY,GAAG,iBAAiB,GAC9B,MAAM,GAAG,IAAI,CAWzB;AAED;;;;GAIG;AACH,0DAHW,cAAc,QAmBxB;AAgDD;;;;;;GAMG;AACH,oCALW,GAAG,UACH,cAAc,cACd,MAAM,CAAC,MAAM,EAAE,GAAG,CAAC,QA6D7B;AAED;;;;;;;GAOG;AACH,4BALW,cAAc,OACd,QAAQ,QACR,YAAY,GAAC,MAAM,GACjB,GAAG,CA4Ef;AAED;;;;;;;GAOG;AACH,sCALW,MAAM,QACN,MAAM,YACN,MAAM,GACJ,MAAM,CAclB;AAED;;;;;;GAMG;AACH,6BAJW,MAAM,QACN,GAAG,GACD,OAAO,GAAC,MAAM,GAAC,YAAY,CAIvC;AAED;;;;;;;GAOG;AACH,gCALW,cAAc,OACd,QAAQ,QACR,YAAY,GACV;IAAC,IAAI,EAAE,UAAU,GAAC,MAAM,CAAC;IAAC,CAAC,GAAG,EAAE,MAAM,GAAG,GAAG,CAAA;CAAC,CAoBzD;AAED;;;;;;;GAOG;AACH,8BAJW,QAAQ,cAEN,QAAQ,CAkCpB;AAED;;;;;;;GAOG;AACH,yCALW,cAAc,OACd,QAAQ,WACR,MAAM,GACJ,MAAM,CAoBlB;AAED;;;;;;;;;;;;;GAaG;AACH,8CANW,cAAc,OACd,QAAQ,cACR,SAAS,oBACT,MAAM,GACJ,UAAU,GAAG,IAAI,CAwB7B;AAED;;;;;;;;GAQG;AACH,2DANW,cAAc,OACd,QAAQ,kBACR,MAAM,mBACN,MAAM,CAAC,MAAM,EAAE,OAAO,CAAC,GACrB,MAAM,CA0BlB;AAED;;;;;;;GAOG;AACH,8CAJW,SAAS,0BACT;IAAE,GAAG,WAAW;IAAC,OAAO,WAAU;CAAE,QAoD9C;gCAthBqJ,SAAS;mCAAT,SAAS;kCAAT,SAAS;uCAAT,SAAS;oCAAT,SAAS;8BAAT,SAAS;kCAAT,SAAS;4BAAT,SAAS;+BAAT,SAAS;gCAAT,SAAS"}

package/dist/doc_utils.js

@@ -7,7 +7,7 @@
  * @import { NodeId, DocumentPath, PrimitiveType, NodeProperty, NodeArrayProperty, NodeSchema, DocumentSchema, Selection, Annotation, Document } from './types'
  */
 
-import { get_selection_range } from './utils.js';
+import { get_selection_range, get_char_length } from './utils.js';
 
 /**
  * Identity function — keeps schema at runtime & makes IDE infer types.
@@ -119,7 +119,7 @@ function validate_primitive_value(type, value) {
 }
 
 /**
- * @param {String} id
+ * @param {string} id
  * @returns {boolean}
  */
 function is_id_valid(id) {
@@ -480,3 +480,62 @@ export function count_references_excluding_deleted(schema, doc, target_node_id,
 
 	return count;
 }
+
+/**
+ * Validates a selection against the current document state.
+ * Works with any object that implements get() and inspect() (Session or Transaction).
+ *
+ * @param {Selection} selection - The selection to validate
+ * @param {{ get: Function, inspect: Function }} session_or_transaction - A Session or Transaction instance
+ * @throws {Error} Throws if the selection is invalid
+ */
+export function validate_selection(selection, session_or_transaction) {
+	if (!selection) return;
+
+	const selection_type = selection.type;
+	if (!['node', 'text', 'property'].includes(selection_type)) {
+		throw new Error(`Invalid selection type: ${selection_type}`);
+	}
+
+	if (selection_type === 'node') {
+		const node_array = session_or_transaction.get(selection.path);
+
+		if (!Array.isArray(node_array)) {
+			throw new Error('Node selection path must point to a node_array');
+		}
+
+		const max_offset = node_array.length;
+		if (selection.anchor_offset < 0 || selection.anchor_offset > max_offset) {
+			throw new Error(
+				`Node selection anchor_offset (${selection.anchor_offset}) is out of bounds. Max is ${max_offset}.`
+			);
+		}
+		if (selection.focus_offset < 0 || selection.focus_offset > max_offset) {
+			throw new Error(
+				`Node selection focus_offset (${selection.focus_offset}) is out of bounds. Max is ${max_offset}.`
+			);
+		}
+	} else if (selection_type === 'text') {
+		const annotated_text = session_or_transaction.get(selection.path);
+
+		if (!annotated_text || typeof annotated_text.text !== 'string') {
+			throw new Error('Text selection path must point to annotated_text');
+		}
+
+		const char_length = get_char_length(annotated_text.text);
+		if (selection.anchor_offset < 0 || selection.anchor_offset > char_length) {
+			throw new Error(
+				`Text selection anchor_offset (${selection.anchor_offset}) is out of bounds. Max is ${char_length}.`
+			);
+		}
+		if (selection.focus_offset < 0 || selection.focus_offset > char_length) {
+			throw new Error(
+				`Text selection focus_offset (${selection.focus_offset}) is out of bounds. Max is ${char_length}.`
+			);
+		}
+	} else if (selection_type === 'property') {
+		if (!session_or_transaction.inspect(selection.path)) {
+			throw new Error(`Property selection path not found: ${selection.path.join('.')}`);
+		}
+	}
+}

package/dist/index.d.ts

@@ -3,6 +3,10 @@ export { default as AnnotatedTextProperty } from "./AnnotatedTextProperty.svelte
 export { default as CustomProperty } from "./CustomProperty.svelte";
 export { default as Node } from "./Node.svelte";
 export { default as NodeArrayProperty } from "./NodeArrayProperty.svelte";
+export { default as NodeGap } from "./NodeGap.svelte";
+export { default as NodeGapMarkers } from "./NodeGapMarkers.svelte";
+export { default as NodeCaret } from "./NodeCaret.svelte";
+export { default as NodeSelectionMarkers } from "./NodeSelectionMarkers.svelte";
 export { default as Session } from "./Session.svelte.js";
 export { default as Transaction } from "./Transaction.svelte.js";
 export { default as Command } from "./Command.svelte.js";

package/dist/index.js

@@ -4,6 +4,10 @@ export { default as AnnotatedTextProperty } from './AnnotatedTextProperty.svelte
 export { default as CustomProperty } from './CustomProperty.svelte';
 export { default as Node } from './Node.svelte';
 export { default as NodeArrayProperty } from './NodeArrayProperty.svelte';
+export { default as NodeGap } from './NodeGap.svelte';
+export { default as NodeGapMarkers } from './NodeGapMarkers.svelte';
+export { default as NodeCaret } from './NodeCaret.svelte';
+export { default as NodeSelectionMarkers } from './NodeSelectionMarkers.svelte';
 
 // Core classes and utilities
 export { default as Session } from './Session.svelte.js';

package/dist/node_gap_computation.svelte.d.ts

@@ -0,0 +1,5 @@
+/**
+ * @param {object} svedit - Svedit context (session, editable, doc, schema, etc.)
+ */
+export function create_gap_computation(svedit: object): void;
+//# sourceMappingURL=node_gap_computation.svelte.d.ts.map

package/dist/node_gap_computation.svelte.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"node_gap_computation.svelte.d.ts","sourceRoot":"","sources":["../src/lib/node_gap_computation.svelte.js"],"names":[],"mappings":"AAkWA;;GAEG;AACH,+CAFW,MAAM,QAyQhB"}