Enable parsing of nested parameters for @callback JSDoc tag. (#54681)

Co-authored-by: Matej Sadovsky <[email protected]>
Co-authored-by: Nathan Shively-Sanders <[email protected]>

Author: 3 people

src/compiler/parser.ts

@@ -9225,7 +9225,7 @@ namespace Parser {
 
                 const comment = parseTrailingTagComments(start, getNodePos(), indent, indentText);
 
-                const nestedTypeLiteral = target !== PropertyLikeParse.CallbackParameter && parseNestedTypeLiteral(typeExpression, name, target, indent);
+                const nestedTypeLiteral = parseNestedTypeLiteral(typeExpression, name, target, indent);
                 if (nestedTypeLiteral) {
                     typeExpression = nestedTypeLiteral;
                     isNameFirst = true;
@@ -9540,7 +9540,6 @@ namespace Parser {
                             if (canParseTag) {
                                 const child = tryParseChildTag(target, indent);
                                 if (child && (child.kind === SyntaxKind.JSDocParameterTag || child.kind === SyntaxKind.JSDocPropertyTag) &&
-                                    target !== PropertyLikeParse.CallbackParameter &&
                                     name && (isIdentifierNode(child.name) || !escapedTextsEqual(name, child.name.left))) {
                                     return false;
                                 }

tests/baselines/reference/callbackTagNestedParameter.js

@@ -0,0 +1,41 @@
+//// [tests/cases/conformance/jsdoc/callbackTagNestedParameter.ts] ////
+
+//// [cb_nested.js]
+/**
+ * @callback WorksWithPeopleCallback
+ * @param {Object} person
+ * @param {string} person.name
+ * @param {number} [person.age]
+ * @returns {void}
+ */
+
+/**
+ * For each person, calls your callback.
+ * @param {WorksWithPeopleCallback} callback
+ * @returns {void}
+ */
+function eachPerson(callback) {
+    callback({ name: "Empty" });
+}
+
+
+
+
+//// [cb_nested.d.ts]
+/**
+ * @callback WorksWithPeopleCallback
+ * @param {Object} person
+ * @param {string} person.name
+ * @param {number} [person.age]
+ * @returns {void}
+ */
+/**
+ * For each person, calls your callback.
+ * @param {WorksWithPeopleCallback} callback
+ * @returns {void}
+ */
+declare function eachPerson(callback: WorksWithPeopleCallback): void;
+type WorksWithPeopleCallback = (person: {
+    name: string;
+    age?: number;
+}) => void;

tests/baselines/reference/callbackTagNestedParameter.symbols

@@ -0,0 +1,25 @@
+//// [tests/cases/conformance/jsdoc/callbackTagNestedParameter.ts] ////
+
+=== cb_nested.js ===
+/**
+ * @callback WorksWithPeopleCallback
+ * @param {Object} person
+ * @param {string} person.name
+ * @param {number} [person.age]
+ * @returns {void}
+ */
+
+/**
+ * For each person, calls your callback.
+ * @param {WorksWithPeopleCallback} callback
+ * @returns {void}
+ */
+function eachPerson(callback) {
+>eachPerson : Symbol(eachPerson, Decl(cb_nested.js, 0, 0))
+>callback : Symbol(callback, Decl(cb_nested.js, 13, 20))
+
+    callback({ name: "Empty" });
+>callback : Symbol(callback, Decl(cb_nested.js, 13, 20))
+>name : Symbol(name, Decl(cb_nested.js, 14, 14))
+}
+

tests/baselines/reference/callbackTagNestedParameter.types

@@ -0,0 +1,28 @@
+//// [tests/cases/conformance/jsdoc/callbackTagNestedParameter.ts] ////
+
+=== cb_nested.js ===
+/**
+ * @callback WorksWithPeopleCallback
+ * @param {Object} person
+ * @param {string} person.name
+ * @param {number} [person.age]
+ * @returns {void}
+ */
+
+/**
+ * For each person, calls your callback.
+ * @param {WorksWithPeopleCallback} callback
+ * @returns {void}
+ */
+function eachPerson(callback) {
+>eachPerson : (callback: WorksWithPeopleCallback) => void
+>callback : WorksWithPeopleCallback
+
+    callback({ name: "Empty" });
+>callback({ name: "Empty" }) : void
+>callback : WorksWithPeopleCallback
+>{ name: "Empty" } : { name: string; }
+>name : string
+>"Empty" : "Empty"
+}
+

tests/cases/conformance/jsdoc/callbackTagNestedParameter.ts

@@ -0,0 +1,21 @@
+// @emitDeclarationOnly: true
+// @declaration: true
+// @allowJs: true
+// @checkJs: true
+// @Filename: cb_nested.js
+/**
+ * @callback WorksWithPeopleCallback
+ * @param {Object} person
+ * @param {string} person.name
+ * @param {number} [person.age]
+ * @returns {void}
+ */
+
+/**
+ * For each person, calls your callback.
+ * @param {WorksWithPeopleCallback} callback
+ * @returns {void}
+ */
+function eachPerson(callback) {
+    callback({ name: "Empty" });
+}