Remove possible allocation of NodeList by eliminating IfElse.

[DanielRosenwasser]

Just a drive-by change. I'm not sure if the compiler is "smart enough" to get rid of the allocation, but it felt better to be more explicit anyway.

[Copilot]

Pull Request Overview

This PR aims to optimize the allocation of the NodeList by eliminating the use of the IfElse construct.

• Removed the conditional allocation using core.IfElse and replaced it with a pre-declared variable (tagsNodeList).
• The change clarifies the code and potentially avoids unnecessary allocations.

Comments suppressed due to low confidence (1)

internal/parser/jsdoc.go:522

• Ensure that passing nil as tagsNodeList to NewJSDoc is handled correctly in all execution paths to prevent potential nil dereference issues.

var tagsNodeList *ast.NodeList

[jakebailey]

The compiler is smart enough, but it would be an illegal transform because it would change the runtime behavior