fix: remove unused function and fix empty branch

- Remove unused isMemberTypesNever function
- Refactor empty branch in api.go to avoid staticcheck warning

Author: ScriptedAlchemy

cmd/rslint/api.go

@@ -216,12 +216,9 @@ func (h *IPCHandler) HandleLint(req api.LintRequest) (*api.LintResponse, error)
 			// Try both short name and full @typescript-eslint/ prefixed name
 			var option interface{}
 			var found bool
-			if option, found = req.RuleOptions[r.Name]; found {
-				// Found with short name (e.g., "member-ordering")
-				// option and found are set correctly by the if condition
-			} else if option, found = req.RuleOptions["@typescript-eslint/"+r.Name]; found {
-				// Found with full name (e.g., "@typescript-eslint/member-ordering")
-				// option and found are set correctly by the if condition
+			if option, found = req.RuleOptions[r.Name]; !found {
+				// Try with full @typescript-eslint/ prefix
+				option, found = req.RuleOptions["@typescript-eslint/"+r.Name]
 			}
 
 			if found {

internal/rules/member_ordering/member_ordering.go

@@ -858,16 +858,6 @@ func normalizeOrderForCompare(order Order) Order {
 	}
 }
 
-// isMemberTypesNever returns true if the memberTypes config is set to "never"
-func isMemberTypesNever(cfg *OrderConfig) bool {
-	if cfg == nil {
-		return false
-	}
-	if str, ok := cfg.MemberTypes.(string); ok && str == "never" {
-		return true
-	}
-	return false
-}
 
 func checkRequiredOrder(ctx rule.RuleContext, members []*ast.Node, optionalityOrder OptionalityOrder) bool {
 	switchIndex := -1