Preserve original comments on params in dts generated by pseudo type node builder under removeComments: false

internal/checker/pseudotypenodebuilder.go

@@ -258,15 +258,17 @@ func (b *NodeBuilderImpl) pseudoParameterToNode(p *pseudochecker.PseudoParameter
 	if p.Optional {
 		questionMark = b.f.NewToken(ast.KindQuestionToken)
 	}
-	return b.f.NewParameterDeclaration(
+	parameter := b.f.NewParameterDeclaration(
 		nil,
 		dotDotDot,
 		// matches strada behavior of always reserializing param names from scratch
-		b.parameterToParameterDeclarationName(p.Name.Parent.Symbol(), p.Name.Parent),
+		b.parameterToParameterDeclarationName(p.Declaration.Symbol, p.Declaration.AsNode()),
 		questionMark,
 		b.pseudoTypeToNode(p.Type),
 		nil,
 	)
+	b.setCommentRange(parameter, p.Declaration.AsNode())
+	return parameter
 }
 
 // see `typeNodeIsEquivalentToType` in strada, but applied more broadly here, so is setup to handle more equivalences - strada only used it via

internal/pseudochecker/lookup.go

@@ -659,11 +659,13 @@ func (ch *PseudoChecker) cloneParameters(nodes *ast.NodeList) []*PseudoParameter
 	}
 	result := make([]*PseudoParameter, 0, len(nodes.Nodes))
 	for _, e := range nodes.Nodes {
+		parameter := e.AsParameterDeclaration()
 		result = append(result, NewPseudoParameter(
-			e.AsParameterDeclaration().DotDotDotToken != nil,
+			parameter,
+			parameter.DotDotDotToken != nil,
 			e.Name(),
-			e.AsParameterDeclaration().QuestionToken != nil || e.AsParameterDeclaration().Initializer != nil,
-			ch.typeFromParameter(e.AsParameterDeclaration()),
+			parameter.QuestionToken != nil || parameter.Initializer != nil,
+			ch.typeFromParameter(parameter),
 		))
 	}
 	return result

internal/pseudochecker/type.go

@@ -159,14 +159,15 @@ func (t *PseudoType) AsPseudoTypeUnion() *PseudoTypeUnion {
 }
 
 type PseudoParameter struct {
-	Rest     bool
-	Name     *ast.Node
-	Optional bool
-	Type     *PseudoType
+	Declaration *ast.ParameterDeclaration
+	Rest        bool
+	Name        *ast.Node
+	Optional    bool
+	Type        *PseudoType
 }
 
-func NewPseudoParameter(isRest bool, name *ast.Node, isOptional bool, t *PseudoType) *PseudoParameter {
-	return &PseudoParameter{Rest: isRest, Name: name, Optional: isOptional, Type: t}
+func NewPseudoParameter(declaration *ast.ParameterDeclaration, isRest bool, name *ast.Node, isOptional bool, t *PseudoType) *PseudoParameter {
+	return &PseudoParameter{Declaration: declaration, Rest: isRest, Name: name, Optional: isOptional, Type: t}
 }
 
 // PseudoTypeSingleCallSignature represents an object type with a single call signature, like an arrow or function expression

testdata/baselines/reference/submodule/compiler/commentsFunction(target=es2015).js

@@ -112,9 +112,9 @@ b: number): void;
 /** fooFunc
  * comment
  */
-declare var fooFunc: (b: string) => string;
-declare var lambdaFoo: (a: number, b: number) => number;
-declare var lambddaNoVarComment: (a: number, b: number) => number;
+declare var fooFunc: (/** fooFunctionValue param */ b: string) => string;
+declare var lambdaFoo: (/**param a*/ a: number, /**param b*/ b: number) => number;
+declare var lambddaNoVarComment: (/**param a*/ a: number, /**param b*/ b: number) => number;
 declare function blah(a: string): void;
 declare function blah2(a: string): void;
 declare function blah3(a: string): void;