feat: implement no-unnecessary-condition TypeScript rule

internal/config/config.go

@@ -32,6 +32,7 @@ import (
 	"github.com/web-infra-dev/rslint/internal/plugins/typescript/rules/no_redundant_type_constituents"
 	"github.com/web-infra-dev/rslint/internal/plugins/typescript/rules/no_require_imports"
 	"github.com/web-infra-dev/rslint/internal/plugins/typescript/rules/no_unnecessary_boolean_literal_compare"
+	"github.com/web-infra-dev/rslint/internal/plugins/typescript/rules/no_unnecessary_condition"
 	"github.com/web-infra-dev/rslint/internal/plugins/typescript/rules/no_unnecessary_template_expression"
 	"github.com/web-infra-dev/rslint/internal/plugins/typescript/rules/no_unnecessary_type_arguments"
 	"github.com/web-infra-dev/rslint/internal/plugins/typescript/rules/no_unnecessary_type_assertion"
@@ -354,6 +355,7 @@ func registerAllTypeScriptEslintPluginRules() {
 	GlobalRuleRegistry.Register("@typescript-eslint/no-redundant-type-constituents", no_redundant_type_constituents.NoRedundantTypeConstituentsRule)
 	GlobalRuleRegistry.Register("@typescript-eslint/no-require-imports", no_require_imports.NoRequireImportsRule)
 	GlobalRuleRegistry.Register("@typescript-eslint/no-unnecessary-boolean-literal-compare", no_unnecessary_boolean_literal_compare.NoUnnecessaryBooleanLiteralCompareRule)
+	GlobalRuleRegistry.Register("@typescript-eslint/no-unnecessary-condition", no_unnecessary_condition.NoUnnecessaryConditionRule)
 	GlobalRuleRegistry.Register("@typescript-eslint/no-unnecessary-template-expression", no_unnecessary_template_expression.NoUnnecessaryTemplateExpressionRule)
 	GlobalRuleRegistry.Register("@typescript-eslint/no-unnecessary-type-arguments", no_unnecessary_type_arguments.NoUnnecessaryTypeArgumentsRule)
 	GlobalRuleRegistry.Register("@typescript-eslint/no-unnecessary-type-assertion", no_unnecessary_type_assertion.NoUnnecessaryTypeAssertionRule)

internal/plugins/import/plugin.go

@@ -1,2 +1,3 @@
 package import_plugin
-const PLUGIN_NAME = "eslint-plugin-import"
+
+const PLUGIN_NAME = "eslint-plugin-import"

internal/plugins/import/rules/no_self_import/no_self_import_test.go

@@ -3,9 +3,9 @@ package no_self_import_test
 import (
 	"testing"
 
+	"github.com/web-infra-dev/rslint/internal/plugins/import/fixtures"
 	"github.com/web-infra-dev/rslint/internal/plugins/import/rules/no_self_import"
 	"github.com/web-infra-dev/rslint/internal/rule_tester"
-	"github.com/web-infra-dev/rslint/internal/plugins/import/fixtures"
 )
 
 func TestNoSelfImportRule(t *testing.T) {

internal/plugins/typescript/rules/no_duplicate_type_constituents/no_duplicate_type_constituents_test.go

@@ -150,7 +150,7 @@ type T = Record<string, A | B>;
 		},
 	}, []rule_tester.InvalidTestCase{
 		{
-			Only: true,
+			Only:   true,
 			Code:   "type T = 1 | 1;",
 			Output: []string{"type T = 1  ;"},
 			Errors: []rule_tester.InvalidTestCaseError{

internal/plugins/typescript/rules/no_explicit_any/no_explicit_any.go

@@ -73,7 +73,7 @@ func isAnyInRestParameter(node *ast.Node) bool {
 	// Check if the any keyword is inside a rest parameter with array type
 	// We need to check if the any is part of an array type in a rest parameter
 	// Valid patterns to ignore: ...args: any[], ...args: readonly any[], ...args: Array<any>, ...args: ReadonlyArray<any>
-	
+
 	// First check if we're inside an ArrayType
 	inArrayType := false
 	for p := node.Parent; p != nil; p = p.Parent {
@@ -92,11 +92,11 @@ func isAnyInRestParameter(node *ast.Node) bool {
 			}
 		}
 	}
-	
+
 	if !inArrayType {
 		return false
 	}
-	
+
 	// Then check if we're in a rest parameter
 	for p := node.Parent; p != nil; p = p.Parent {
 		if p.Kind == ast.KindParameter {