Use positive IsIdentifier check instead of negative IsBindingPattern check

Following the TypeScript reference more precisely - check if param.Name() is an
Identifier rather than checking if it's NOT a binding pattern. This is more
explicit and matches the pattern in utilities.ts:4703 where it checks
`p.name.kind === SyntaxKind.Identifier`.

Co-authored-by: jakebailey <[email protected]>

Author: Copilot

internal/checker/checker.go

@@ -30398,8 +30398,8 @@ func (c *Checker) getSymbolAtLocation(node *ast.Node, ignoreErrors bool) *ast.Sy
 		} else if ast.IsJSDocParameterTag(parent) && parent.Name() == node {
 			if fn := ast.GetNodeAtPosition(ast.GetSourceFileOfNode(node), node.Pos(), false); fn != nil && ast.IsFunctionLike(fn) {
 				for _, param := range fn.Parameters() {
-					// Skip binding patterns - they don't have a simple name to compare
-					if !ast.IsBindingPattern(param.Name()) && param.Name().Text() == node.Text() {
+					// Only compare against identifier names, not binding patterns
+					if ast.IsIdentifier(param.Name()) && param.Name().Text() == node.Text() {
 						return c.getSymbolOfNode(param)
 					}
 				}