fix: only lint opened file (#151)

Author: hardfist

cmd/rslint/lsp.go

@@ -426,11 +426,13 @@ func runLintWithPrograms(uri lsproto.DocumentUri, programs []*compiler.Program,
 		defer diagnosticsLock.Unlock()
 		diagnostics = append(diagnostics, d)
 	}
+	filename := uriToPath(string(uri))
+
 	// Run linter with all programs using rule registry
 	_, err := linter.RunLinter(
 		programs,
 		false, // Don't use single-threaded mode for LSP
-		nil,
+		[]string{filename},
 		func(sourceFile *ast.SourceFile) []linter.ConfiguredRule {
 			activeRules := config.GlobalRuleRegistry.GetEnabledRules(rslintConfig, sourceFile.FileName())
 			return activeRules

internal/linter/linter.go

@@ -21,7 +21,7 @@ type ConfiguredRule struct {
 
 // when allowedFiles is passed as nil which means all files are allowed
 // when allowedFiles is passed as slice, only files in the slice are allowed
-func RunLinter(programs []*compiler.Program, singleThreaded bool, allowFiles []*ast.SourceFile, getRulesForFile func(sourceFile *ast.SourceFile) []ConfiguredRule, onDiagnostic func(diagnostic rule.RuleDiagnostic)) (int32, error) {
+func RunLinter(programs []*compiler.Program, singleThreaded bool, allowFiles []string, getRulesForFile func(sourceFile *ast.SourceFile) []ConfiguredRule, onDiagnostic func(diagnostic rule.RuleDiagnostic)) (int32, error) {
 
 	wg := core.NewWorkGroup(singleThreaded)
 
@@ -33,7 +33,6 @@ func RunLinter(programs []*compiler.Program, singleThreaded bool, allowFiles []*
 
 			wg.Queue(func() {
 				for _, file := range program.GetSourceFiles() {
-
 					p := string(file.Path())
 					// skip lint node_modules and bundled files
 					// FIXME: we may have better api to tell whether a file is a bundled file or not
@@ -43,8 +42,9 @@ func RunLinter(programs []*compiler.Program, singleThreaded bool, allowFiles []*
 					// only lint allowedFiles if allowedFiles is not empty
 					if allowFiles != nil {
 						found := false
-						for _, f := range allowFiles {
-							if f.Path() == file.Path() {
+						for _, filePath := range allowFiles {
+
+							if filePath == file.FileName() {
 								found = true
 								break
 							}

internal/rule_tester/rule_tester.go

@@ -76,12 +76,13 @@ func RunRuleTester(rootDir string, tsconfigPath string, t *testing.T, r *rule.Ru
 		program, err := utils.CreateProgram(true, fs, rootDir, tsconfigPath, host)
 		assert.NilError(t, err, "couldn't create program. code: "+code)
 
-		files := []*ast.SourceFile{program.GetSourceFile(fileName)}
+		sourceFile := program.GetSourceFile(fileName)
+		allowedFiles := []string{string(sourceFile.Path())}
 
 		_, err = linter.RunLinter(
 			[]*compiler.Program{program},
 			true,
-			files,
+			allowedFiles,
 			func(sourceFile *ast.SourceFile) []linter.ConfiguredRule {
 				return []linter.ConfiguredRule{
 					{