fix(48540): Extract to typedef from (invalid) type with comments in JS file causes assertion failure (#48545)

* fix(48540): Remove comments from jsdoc union type expression

* fix(48540) - Remove comment for top level import and add extract typedef tests

* fix(48540) - Remove comments from jsdoc's descendant

* fix(48540) - Using no nested comments instead of traversing

Author: Minh Quy

src/services/refactors/extractType.ts

@@ -229,6 +229,8 @@ namespace ts.refactor {
     function doTypedefChange(changes: textChanges.ChangeTracker, file: SourceFile, name: string, info: ExtractInfo) {
         const { firstStatement, selection, typeParameters } = info;
 
+        setEmitFlags(selection, EmitFlags.NoComments | EmitFlags.NoNestedComments);
+
         const node = factory.createJSDocTypedefTag(
             factory.createIdentifier("typedef"),
             factory.createJSDocTypeExpression(selection),

tests/cases/fourslash/refactorExtractType_js_TypeLiteral_CommentAfterProperty.ts

@@ -0,0 +1,26 @@
+/// <reference path='fourslash.ts' />
+
+// @allowJs: true
+// @Filename: a.js
+////type Foo = /*a*/[|{
+////    oops: string;
+////    /**
+////     *
+////     */
+////}|]/*b*/;
+
+goTo.file('a.js')
+goTo.select("a", "b");
+edit.applyRefactor({
+    refactorName: "Extract type",
+    actionName: "Extract to typedef",
+    actionDescription: "Extract to typedef",
+    newContent:
+`/**
+ * @typedef {{
+    oops: string;
+}} /*RENAME*/NewType
+ */
+
+type Foo = NewType;`,
+});

tests/cases/fourslash/refactorExtractType_js_TypeLiteral_CommentBeforeProperty.ts

@@ -0,0 +1,26 @@
+/// <reference path='fourslash.ts' />
+
+// @allowJs: true
+// @Filename: a.js
+////type Foo = /*a*/[|{
+////    /**
+////     *
+////     */
+////    oops: string;
+////}|]/*b*/;
+
+goTo.file('a.js')
+goTo.select("a", "b");
+edit.applyRefactor({
+    refactorName: "Extract type",
+    actionName: "Extract to typedef",
+    actionDescription: "Extract to typedef",
+    newContent:
+`/**
+ * @typedef {{
+    oops: string;
+}} /*RENAME*/NewType
+ */
+
+type Foo = NewType;`,
+});

tests/cases/fourslash/refactorExtractType_js_Union_CommentAfterMember.ts

@@ -0,0 +1,19 @@
+/// <reference path='fourslash.ts' />
+
+// @allowJs: true
+// @Filename: a.js
+////type Bar = /*a*/[|string | boolean /* oops */ |]/*b*/;
+
+goTo.file('a.js')
+goTo.select("a", "b");
+edit.applyRefactor({
+    refactorName: "Extract type",
+    actionName: "Extract to typedef",
+    actionDescription: "Extract to typedef",
+    newContent:
+`/**
+ * @typedef {string | boolean} /*RENAME*/NewType
+ */
+
+type Bar = NewType /* oops */ ;`,
+});

tests/cases/fourslash/refactorExtractType_js_Union_CommentBeforeMember.ts

@@ -0,0 +1,19 @@
+/// <reference path='fourslash.ts' />
+
+// @allowJs: true
+// @Filename: a.js
+////type Bar = /*a*/[|string | /* oops */ boolean|]/*b*/;
+
+goTo.file('a.js')
+goTo.select("a", "b");
+edit.applyRefactor({
+    refactorName: "Extract type",
+    actionName: "Extract to typedef",
+    actionDescription: "Extract to typedef",
+    newContent:
+`/**
+ * @typedef {string | boolean} /*RENAME*/NewType
+ */
+
+type Bar = NewType;`,
+});