address PR feedback

Author: Andarist

internal/ast/utilities.go

@@ -2847,13 +2847,11 @@ func GetLeftmostAccessExpression(expr *Node) *Node {
 func IsTypeOnlyImportDeclaration(node *Node) bool {
 	switch node.Kind {
 	case KindImportSpecifier:
-		return node.AsImportSpecifier().IsTypeOnly || node.Parent.Parent.AsImportClause().PhaseModifier == KindTypeKeyword
+		return node.IsTypeOnly() || node.Parent.Parent.IsTypeOnly()
 	case KindNamespaceImport:
-		return node.Parent.AsImportClause().PhaseModifier == KindTypeKeyword
-	case KindImportClause:
-		return node.AsImportClause().PhaseModifier == KindTypeKeyword
-	case KindImportEqualsDeclaration:
-		return node.AsImportEqualsDeclaration().IsTypeOnly
+		return node.Parent.IsTypeOnly()
+	case KindImportClause, KindImportEqualsDeclaration:
+		return node.IsTypeOnly()
 	}
 	return false
 }

internal/ls/autoimportfixes.go

@@ -89,7 +89,7 @@ func (ct *changeTracker) doAddExistingFix(
 					identifier = ct.NodeFactory.NewIdentifier(namedImport.propertyName).AsIdentifier().AsNode()
 				}
 				return ct.NodeFactory.NewImportSpecifier(
-					(importClause.PhaseModifier != ast.KindTypeKeyword || promoteFromTypeOnly) && shouldUseTypeOnly(namedImport.addAsTypeOnly, preferences),
+					(!importClause.IsTypeOnly() || promoteFromTypeOnly) && shouldUseTypeOnly(namedImport.addAsTypeOnly, preferences),
 					identifier,
 					ct.NodeFactory.NewIdentifier(namedImport.name),
 				)

internal/parser/parser.go

@@ -2112,9 +2112,7 @@ func (p *Parser) parseImportDeclarationOrImportEqualsDeclaration(pos int, hasJSD
 	} else if identifier != nil && identifier.AsIdentifier().Text == "defer" {
 		var shouldParseAsDeferModifier bool
 		if p.token == ast.KindFromKeyword {
-			shouldParseAsDeferModifier = !p.lookAhead(func(p *Parser) bool {
-				return p.nextTokenIsTokenStringLiteral()
-			})
+			shouldParseAsDeferModifier = !p.lookAhead((*Parser).nextTokenIsTokenStringLiteral)
 		} else {
 			shouldParseAsDeferModifier = p.token != ast.KindCommaToken && p.token != ast.KindEqualsToken
 		}