fix: make parentId param optional in actions.add() (#635)

prevwong · May 13, 2024 · b25469e · b25469e
1 parent fcc71d8
commit b25469e
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 7 deletions.
diff --git a/.changeset/sixty-rice-build.md b/.changeset/sixty-rice-build.md
@@ -0,0 +1,5 @@
+---
+'@craftjs/core': patch
+---
+
+Make parentId param optional in actions.add()
diff --git a/packages/core/src/editor/actions.ts b/packages/core/src/editor/actions.ts
@@ -84,12 +84,7 @@ const Methods = (
 
     iterateChildren(tree.rootNodeId, parentId);
 
-    if (!parentId) {
-      invariant(
-        tree.rootNodeId === ROOT_NODE,
-        'Cannot add non-root Node without a parent'
-      );
-
+    if (!parentId && tree.rootNodeId === ROOT_NODE) {
       return;
     }
 
@@ -180,7 +175,7 @@ const Methods = (
      * @param parentId
      * @param index
      */
-    add(nodeToAdd: Node | Node[], parentId: NodeId, index?: number) {
+    add(nodeToAdd: Node | Node[], parentId?: NodeId, index?: number) {
       // TODO: Deprecate adding array of Nodes to keep implementation simpler
       let nodes = [nodeToAdd];
       if (Array.isArray(nodeToAdd)) {