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

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

Hide file tree

Showing 2 changed files with 11 additions and 11 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
@@ -38,7 +38,7 @@ const Methods = (
   /** Helper functions */
   const addNodeTreeToParent = (
     tree: NodeTree,
-    parentId?: NodeId,
+    parentId: NodeId,
     addNodeType?:
       | {
           type: 'child';
@@ -84,15 +84,6 @@ const Methods = (
 
     iterateChildren(tree.rootNodeId, parentId);
 
-    if (!parentId) {
-      invariant(
-        tree.rootNodeId === ROOT_NODE,
-        'Cannot add non-root Node without a parent'
-      );
-
-      return;
-    }
-
     const parent = getParentAndValidate(parentId);
 
     if (addNodeType.type === 'child') {
@@ -180,7 +171,11 @@ const Methods = (
      * @param parentId
      * @param index
      */
-    add(nodeToAdd: Node | Node[], parentId: NodeId, index?: number) {
+    add(
+      nodeToAdd: Node | Node[],
+      parentId: NodeId = ROOT_NODE,
+      index?: number
+    ) {
       // TODO: Deprecate adding array of Nodes to keep implementation simpler
       let nodes = [nodeToAdd];
       if (Array.isArray(nodeToAdd)) {