fix: cloneNode(deep, withoutLoc) handles absent comments

This fragment (maybe all fragments?) throw during cloning, as
'comments' is unset here. Handle it being unset, by returning
undefined.

packages/babel-types/src/clone/cloneNode.ts

@@ -102,7 +102,9 @@ export default function cloneNode<T extends t.Node>(
   return newNode;
 }
 
-function cloneCommentsWithoutLoc<T extends t.Comment>(comments: T[]): T[] {
+function cloneCommentsWithoutLoc<T extends t.Comment>(
+  comments: ReadonlyArray<T>,
+): T[] {
   return comments.map(
     ({ type, value }) =>
       ({
@@ -113,6 +115,12 @@ function cloneCommentsWithoutLoc<T extends t.Comment>(comments: T[]): T[] {
   );
 }
 
-function maybeCloneComments(comments, deep, withoutLoc) {
-  return deep && withoutLoc ? cloneCommentsWithoutLoc(comments) : comments;
+function maybeCloneComments<T extends t.Comment>(
+  comments: ReadonlyArray<T> | null,
+  deep: boolean,
+  withoutLoc: boolean,
+): ReadonlyArray<T> | null {
+  return deep && withoutLoc && comments
+    ? cloneCommentsWithoutLoc(comments)
+    : comments;
 }

packages/babel-types/test/cloning.js

@@ -42,6 +42,14 @@ describe("cloneNode", function () {
     expect(t.isNodesEquivalent(node, cloned)).toBe(true);
   });
 
+  it("should handle deep cloning without loc of fragments", function () {
+    const program = "foo();";
+    const node = parse(program);
+    const cloned = t.cloneNode(node, /* deep */ true, /* withoutLoc */ true);
+    expect(node).not.toBe(cloned);
+    expect(t.isNodesEquivalent(node, cloned)).toBe(true);
+  });
+
   it("should handle missing array element", function () {
     const node = parse("[,0]");
     const cloned = t.cloneNode(node);