Pluto 1382 change tools configs location

.codacy/codacy.yaml

@@ -2,7 +2,7 @@ runtimes:
     - [email protected]
     - [email protected]
 tools:
-    - eslint@9.3.0
+    - eslint@8.57.0
     - [email protected]
     - [email protected]
     - [email protected]

.gitignore

@@ -24,4 +24,8 @@ go.work.sum
 .idea/
 .vscode/
 
+# Codacy CLI 
 cli-v2
+.codacy/cli-config.yaml
+.codacy/tools-configs
+.codacy/tools-configs/*

cmd/analyze.go

@@ -20,7 +20,6 @@ var outputFormat string
 var sarifPath string
 var commitUuid string
 var projectToken string
-var pmdRulesetFile string
 
 type Sarif struct {
 	Runs []struct {
@@ -96,7 +95,6 @@ func init() {
 	analyzeCmd.Flags().StringVarP(&toolToAnalyze, "tool", "t", "", "Which tool to run analysis with")
 	analyzeCmd.Flags().StringVar(&outputFormat, "format", "", "Output format (use 'sarif' for SARIF format)")
 	analyzeCmd.Flags().BoolVar(&autoFix, "fix", false, "Apply auto fix to your issues when available")
-	analyzeCmd.Flags().StringVar(&pmdRulesetFile, "rulesets", "", "Path to PMD ruleset file")
 	rootCmd.AddCommand(analyzeCmd)
 }
 
@@ -209,7 +207,7 @@ func runPmdAnalysis(workDirectory string, pathsToCheck []string, outputFile stri
 	pmd := config.Config.Tools()["pmd"]
 	pmdBinary := pmd.Binaries["pmd"]
 
-	err := tools.RunPmd(workDirectory, pmdBinary, pathsToCheck, outputFile, outputFormat, pmdRulesetFile)
+	err := tools.RunPmd(workDirectory, pmdBinary, pathsToCheck, outputFile, outputFormat, "")
 	if err != nil {
 		log.Fatalf("Error running PMD: %v", err)
 	}

cmd/init.go

@@ -10,6 +10,7 @@
 	"log"
 	"net/http"
 	"os"
+	"path/filepath"
 	"time"
 
 	"github.com/spf13/cobra"
@@ -142,6 +143,13 @@
 	fmt.Println("Building repository configuration files ...")
 	fmt.Println("Fetching repository configuration from codacy ...")
 
+	toolsConfigDir := config.Config.ToolsConfigDirectory()
+
+	// Create tools-configs directory if it doesn't exist
+	if err := os.MkdirAll(toolsConfigDir, 0777); err != nil {
+		return fmt.Errorf("failed to create tools-configs directory: %w", err)
+	}
+
 	// API call to fetch settings
 	url := CodacyApiBase + "/2.0/project/analysis/configuration"
 
@@ -198,7 +206,7 @@
 		eslintDomainConfiguration := convertAPIToolConfigurationToDomain(*eslintApiConfiguration)
 		eslintConfigurationString := tools.CreateEslintConfig(eslintDomainConfiguration)
 
-		eslintConfigFile, err := os.Create("eslint.config.mjs")
+		eslintConfigFile, err := os.Create(filepath.Join(toolsConfigDir, "eslint.config.mjs"))
 		if err != nil {
 			return fmt.Errorf("failed to create eslint config file: %v", err)
 		}
@@ -210,7 +218,7 @@
 		}
 		fmt.Println("ESLint configuration created based on Codacy settings")
 	} else {
-		err = createDefaultEslintConfigFile()
+		err = createDefaultEslintConfigFile(toolsConfigDir)
 		if err != nil {
 			return fmt.Errorf("failed to create default ESLint config: %v", err)
 		}
@@ -220,13 +228,13 @@
 	// Trivy configuration
 	trivyApiConfiguration := extractTrivyConfiguration(apiToolConfigurations)
 	if trivyApiConfiguration != nil {
-		err = createTrivyConfigFile(*trivyApiConfiguration)
+		err = createTrivyConfigFile(*trivyApiConfiguration, toolsConfigDir)
 		if err != nil {
 			return fmt.Errorf("failed to create Trivy config: %v", err)
 		}
 		fmt.Println("Trivy configuration created based on Codacy settings")
 	} else {
-		err = createDefaultTrivyConfigFile()
+		err = createDefaultTrivyConfigFile(toolsConfigDir)
 		if err != nil {
 			return fmt.Errorf("failed to create default Trivy config: %v", err)
 		}
@@ -236,13 +244,13 @@
 	// PMD configuration
 	pmdApiConfiguration := extractPMDConfiguration(apiToolConfigurations)
 	if pmdApiConfiguration != nil {
-		err = createPMDConfigFile(*pmdApiConfiguration)
+		err = createPMDConfigFile(*pmdApiConfiguration, toolsConfigDir)
 		if err != nil {
 			return fmt.Errorf("failed to create PMD config: %v", err)
 		}
 		fmt.Println("PMD configuration created based on Codacy settings")
 	} else {
-		err = createDefaultPMDConfigFile()
+		err = createDefaultPMDConfigFile(toolsConfigDir)
 		if err != nil {
 			return fmt.Errorf("failed to create default PMD config: %v", err)
 		}
@@ -318,16 +326,16 @@
 	return nil
 }
 
-func createPMDConfigFile(config CodacyToolConfiguration) error {
+func createPMDConfigFile(config CodacyToolConfiguration, toolsConfigDir string) error {
 	pmdDomainConfiguration := convertAPIToolConfigurationToDomain(config)
 	pmdConfigurationString := tools.CreatePmdConfig(pmdDomainConfiguration)
-	return os.WriteFile("pmd-ruleset.xml", []byte(pmdConfigurationString), 0644)
+	return os.WriteFile(filepath.Join(toolsConfigDir, "pmd-ruleset.xml"), []byte(pmdConfigurationString), 0644)
 }
 
-func createDefaultPMDConfigFile() error {
+func createDefaultPMDConfigFile(toolsConfigDir string) error {
 	emptyConfig := tools.ToolConfiguration{}
 	content := tools.CreatePmdConfig(emptyConfig)
-	return os.WriteFile("pmd-ruleset.xml", []byte(content), 0644)
+	return os.WriteFile(filepath.Join(toolsConfigDir, "pmd-ruleset.xml"), []byte(content), 0644)
 }
 
 type CodacyToolConfiguration struct {
@@ -347,15 +355,15 @@
 }
 
 // createTrivyConfigFile creates a trivy.yaml configuration file based on the API configuration
-func createTrivyConfigFile(config CodacyToolConfiguration) error {
+func createTrivyConfigFile(config CodacyToolConfiguration, toolsConfigDir string) error {
 	// Convert CodacyToolConfiguration to tools.ToolConfiguration
 	trivyDomainConfiguration := convertAPIToolConfigurationForTrivy(config)
 
 	// Use the shared CreateTrivyConfig function to generate the config content
 	trivyConfigurationString := tools.CreateTrivyConfig(trivyDomainConfiguration)
 
 	// Write to file
-	return os.WriteFile("trivy.yaml", []byte(trivyConfigurationString), 0644)
+	return os.WriteFile(filepath.Join(toolsConfigDir, "trivy.yaml"), []byte(trivyConfigurationString), 0644)
 }
 
 // convertAPIToolConfigurationForTrivy converts API tool configuration to domain model for Trivy
@@ -399,21 +407,21 @@
 }
 
 // createDefaultTrivyConfigFile creates a default trivy.yaml configuration file
-func createDefaultTrivyConfigFile() error {
+func createDefaultTrivyConfigFile(toolsConfigDir string) error {
 	// Use empty tool configuration to get default settings
 	emptyConfig := tools.ToolConfiguration{}
 	content := tools.CreateTrivyConfig(emptyConfig)
 
 	// Write to file
-	return os.WriteFile("trivy.yaml", []byte(content), 0644)
+	return os.WriteFile(filepath.Join(toolsConfigDir, "trivy.yaml"), []byte(content), 0644)
 }
 
 // createDefaultEslintConfigFile creates a default eslint.config.mjs configuration file
-func createDefaultEslintConfigFile() error {
+func createDefaultEslintConfigFile(toolsConfigDir string) error {
 	// Use empty tool configuration to get default settings
 	emptyConfig := tools.ToolConfiguration{}
 	content := tools.CreateEslintConfig(emptyConfig)
 
 	// Write to file
-	return os.WriteFile("eslint.config.mjs", []byte(content), 0644)
+	return os.WriteFile(filepath.Join(toolsConfigDir, "eslint.config.mjs"), []byte(content), 0644)
 }

config/config.go

@@ -15,6 +15,7 @@
 	runtimesDirectory    string
 	toolsDirectory       string
 	localCodacyDirectory string
+	toolsConfigDirectory string
 	projectConfigFile    string
 	cliConfigFile        string
 
@@ -88,11 +89,16 @@
 	return nil
 }
 
+func (c *ConfigType) ToolsConfigDirectory() string {
+	return c.toolsConfigDirectory
+}
+
 func (c *ConfigType) setupCodacyPaths() {
 	c.globalCacheDirectory = filepath.Join(c.homePath, ".cache", "codacy")
 	c.runtimesDirectory = filepath.Join(c.globalCacheDirectory, "runtimes")
 	c.toolsDirectory = filepath.Join(c.globalCacheDirectory, "tools")
 	c.localCodacyDirectory = ".codacy"
+	c.toolsConfigDirectory = filepath.Join(c.localCodacyDirectory, "tools-configs")
 
 	c.projectConfigFile = filepath.Join(c.localCodacyDirectory, "codacy.yaml")
 	c.cliConfigFile = filepath.Join(c.localCodacyDirectory, "cli-config.yaml")

tools/eslintConfigCreator.go

@@ -26,7 +26,6 @@ func CreateEslintConfig(configuration ToolConfiguration) string {
 
 	for _, patternConfiguration := range configuration.PatternsConfiguration {
 		rule := strings.TrimPrefix(patternConfiguration.PatternId, "ESLint8_")
-		fmt.Println("Rule:", rule)
 
 		const tempstring = "TEMPORARYSTRING"
 		rule = strings.ReplaceAll(rule, "__", tempstring)

tools/eslintRunner.go

@@ -14,6 +14,15 @@ func RunEslint(repositoryToAnalyseDirectory string, eslintInstallationDirectory
 	eslintJsPath := filepath.Join(eslintInstallationNodeModules, ".bin", "eslint")
 
 	cmd := exec.Command(nodeBinary, eslintJsPath)
+
+	// For Eslint compatibility with version 8.
+	// https://eslint.org/docs/v8.x/use/configure/configuration-files-new
+	cmd.Env = append(cmd.Env, "ESLINT_USE_FLAT_CONFIG=true")
+
+	// Add config file from tools-configs directory
+	configFile := filepath.Join(".codacy", "tools-configs", "eslint.config.mjs")
+	cmd.Args = append(cmd.Args, "-c", configFile)
+
 	if autoFix {
 		cmd.Args = append(cmd.Args, "--fix")
 	}

tools/pmdRunner.go

@@ -3,39 +3,52 @@
 import (
 	"os"
 	"os/exec"
+	"path/filepath"
 	"strings"
 )
 
 // RunPmd executes PMD static code analyzer with the specified options
+//
+// Parameters:
+//   - repositoryToAnalyseDirectory: The root directory of the repository to analyze
+//   - pmdBinary: Path to the PMD executable
+//   - pathsToCheck: List of specific paths to analyze, if empty analyzes whole repository
+//   - outputFile: Path where analysis results should be written
+//   - outputFormat: Format for the output (e.g. "sarif")
+//   - rulesetFile: Path to custom ruleset XML file, if empty uses default ruleset
+//
+// Returns:
+//   - error: nil if analysis succeeds or violations found, error otherwise
 func RunPmd(repositoryToAnalyseDirectory string, pmdBinary string, pathsToCheck []string, outputFile string, outputFormat string, rulesetFile string) error {
-	cmdArgs := []string{"pmd"}
+	cmd := exec.Command(pmdBinary, "pmd")
+
+	// Add config file from tools-configs directory if not specified
+	if rulesetFile == "" {
+		configFile := filepath.Join(".codacy", "tools-configs", "pmd-ruleset.xml")
+		cmd.Args = append(cmd.Args, "-R", configFile)
+	} else {
+		cmd.Args = append(cmd.Args, "-R", rulesetFile)
+	}
 
 	// Add source directories (comma-separated list for PMD)
 	if len(pathsToCheck) > 0 {
 		dirArg := strings.Join(pathsToCheck, ",")
-		cmdArgs = append(cmdArgs, "-d", dirArg)
+		cmd.Args = append(cmd.Args, "-d", dirArg)
 	} else {
 		// Fall back to whole repo if no specific paths given
-		cmdArgs = append(cmdArgs, "-d", repositoryToAnalyseDirectory)
-	}
-
-	// Add ruleset
-	if rulesetFile != "" {
-		cmdArgs = append(cmdArgs, "-R", rulesetFile)
+		cmd.Args = append(cmd.Args, "-d", repositoryToAnalyseDirectory)
 	}
 
 	// Format
 	if outputFormat != "" {
-		cmdArgs = append(cmdArgs, "-f", outputFormat)
+		cmd.Args = append(cmd.Args, "-f", outputFormat)
 	}
 
 	// Output file
 	if outputFile != "" {
-		cmdArgs = append(cmdArgs, "-r", outputFile)
+		cmd.Args = append(cmd.Args, "-r", outputFile)
 	}
 
-	cmd := exec.Command(pmdBinary, cmdArgs...)
-
 	cmd.Dir = repositoryToAnalyseDirectory
 	cmd.Stderr = os.Stderr
 	cmd.Stdout = os.Stdout

tools/testdata/repositories/test1/expected.sarif

@@ -14,7 +14,7 @@
       "artifacts": [
         {
           "location": {
-            "uri": "testdata/repositories/test1/src/eslint.config.mjs"
+            "uri": "testdata/repositories/test1/src/.codacy/tools-configs/eslint.config.mjs"
           }
         },
         {

tools/trivyRunner.go

@@ -3,12 +3,17 @@ package tools
 import (
 	"os"
 	"os/exec"
+	"path/filepath"
 )
 
 // RunTrivy executes Trivy vulnerability scanner with the specified options
 func RunTrivy(repositoryToAnalyseDirectory string, trivyBinary string, pathsToCheck []string, outputFile string, outputFormat string) error {
 	cmd := exec.Command(trivyBinary, "fs")
 
+	// Add config file from tools-configs directory
+	configFile := filepath.Join(".codacy", "tools-configs", "trivy.yaml")
+	cmd.Args = append(cmd.Args, "--config", configFile)
+
 	// Add format options
 	if outputFile != "" {
 		cmd.Args = append(cmd.Args, "--output", outputFile)