fix(model-datastructure): some operations were not handled in OperationsCompressor

Which results in exceptions at runtime.

Author: slisson

model-datastructure/src/commonMain/kotlin/org/modelix/model/lazy/OperationsCompressor.kt

@@ -15,11 +15,14 @@ package org.modelix.model.lazy
 
 import org.modelix.model.operations.AddNewChildOp
 import org.modelix.model.operations.AddNewChildSubtreeOp
+import org.modelix.model.operations.AddNewChildrenOp
+import org.modelix.model.operations.BulkUpdateOp
 import org.modelix.model.operations.DeleteNodeOp
 import org.modelix.model.operations.IOperation
 import org.modelix.model.operations.MoveNodeOp
 import org.modelix.model.operations.NoOp
 import org.modelix.model.operations.RevertToOp
+import org.modelix.model.operations.SetConceptOp
 import org.modelix.model.operations.SetPropertyOp
 import org.modelix.model.operations.SetReferenceOp
 import org.modelix.model.operations.UndoOp
@@ -42,7 +45,7 @@ class OperationsCompressor(val resultTree: CLTree) {
 
         for (op in ops) {
             when (op) {
-                is UndoOp, is RevertToOp, is AddNewChildSubtreeOp, is DeleteNodeOp, is MoveNodeOp -> return ops
+                is UndoOp, is RevertToOp, is AddNewChildSubtreeOp, is DeleteNodeOp, is MoveNodeOp, is BulkUpdateOp, is AddNewChildrenOp -> return ops
                 is NoOp -> {}
                 is AddNewChildOp -> {
                     if (!createdNodes.contains(op.position.nodeId)) {
@@ -53,10 +56,12 @@ class OperationsCompressor(val resultTree: CLTree) {
                 is SetPropertyOp -> {
                     if (!createdNodes.contains(op.nodeId)) compressedOps += op
                 }
+                is SetConceptOp -> {
+                    if (!createdNodes.contains(op.nodeId)) compressedOps += op
+                }
                 is SetReferenceOp -> {
                     if (!createdNodes.contains(op.sourceId)) compressedOps += op
                 }
-                else -> throw RuntimeException("Unknown operation type: $op")
             }
         }
 

model-datastructure/src/commonMain/kotlin/org/modelix/model/operations/AbstractOperation.kt

@@ -19,7 +19,7 @@ import org.modelix.model.api.ITree
 import org.modelix.model.lazy.KVEntryReference
 import org.modelix.model.persistent.IKVValue
 
-abstract class AbstractOperation : IOperation {
+sealed class AbstractOperation : IOperation {
 
     abstract inner class Applied {
         override fun toString(): String {

model-datastructure/src/commonMain/kotlin/org/modelix/model/operations/IOperation.kt

@@ -21,7 +21,7 @@ import org.modelix.model.lazy.IDeserializingKeyValueStore
 import org.modelix.model.lazy.KVEntryReference
 import org.modelix.model.persistent.IKVValue
 
-interface IOperation {
+sealed interface IOperation {
     fun apply(transaction: IWriteTransaction, store: IDeserializingKeyValueStore): IAppliedOperation
     fun captureIntend(tree: ITree, store: IDeserializingKeyValueStore): IOperationIntend
     fun getReferencedEntries(): List<KVEntryReference<IKVValue>>