Fix panic by checking for binding patterns before calling Text()

The correct fix is to check if a parameter name is a binding pattern before
calling Text() on it, rather than making Text() handle binding patterns.
This follows the TypeScript reference implementation which checks
!isBindingPattern(param.name) before accessing the name's text.

Reverted the previous change to ast.go and fixed the caller in checker.go instead.

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

Author: Copilot

internal/ast/ast.go

@@ -334,10 +334,6 @@ func (n *Node) Text() string {
 		return strings.Join(n.AsJSDocLinkCode().text, "")
 	case KindJSDocLinkPlain:
 		return strings.Join(n.AsJSDocLinkPlain().text, "")
-	case KindObjectBindingPattern, KindArrayBindingPattern:
-		// Binding patterns don't have a simple text representation
-		// Return empty string to avoid panics in comparisons
-		return ""
 	}
 	panic(fmt.Sprintf("Unhandled case in Node.Text: %T", n.data))
 }

internal/checker/checker.go

@@ -30398,7 +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() {
-					if param.Name().Text() == node.Text() {
+					// Skip binding patterns - they don't have a simple name to compare
+					if !ast.IsBindingPattern(param.Name()) && param.Name().Text() == node.Text() {
 						return c.getSymbolOfNode(param)
 					}
 				}