Skip to content

Commit c6e91c1

Browse files
committed
log supported tools as before
1 parent 5f17a60 commit c6e91c1

File tree

2 files changed

+26
-4
lines changed

2 files changed

+26
-4
lines changed

domain/tool.go

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -34,6 +34,7 @@ type ToolInfo struct {
3434
Priority int // lower means newer
3535
}
3636

37+
// SupportedToolsMetadata group same family tools by name
3738
var SupportedToolsMetadata = map[string]ToolInfo{
3839
ESLint9: {Name: "eslint", Priority: 0},
3940
ESLint: {Name: "eslint", Priority: 1},

tools/getTools.go

Lines changed: 25 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,11 @@ package tools
33
import (
44
codacyClient "codacy/cli-v2/codacy-client"
55
"codacy/cli-v2/domain"
6+
"codacy/cli-v2/utils/logger"
67
"fmt"
8+
"strings"
9+
10+
"github.com/sirupsen/logrus"
711
)
812

913
func enrichToolsWithVersion(tools []domain.Tool) ([]domain.Tool, error) {
@@ -34,18 +38,35 @@ func GetRepositoryTools(initFlags domain.InitFlags) ([]domain.Tool, error) {
3438
return nil, err
3539
}
3640

37-
enabledTools := make([]domain.Tool, 0)
38-
seen := map[string]bool{}
41+
// Log supported tool UUIDs and names
42+
supportedTools := make([]map[string]string, 0, len(domain.SupportedToolsMetadata))
43+
for uuid, meta := range domain.SupportedToolsMetadata {
44+
supportedTools = append(supportedTools, map[string]string{
45+
"uuid": uuid,
46+
"name": meta.Name,
47+
})
48+
}
49+
logger.Info("Supported tools:", logrus.Fields{
50+
"supported_tools": supportedTools,
51+
})
52+
53+
var enabledTools []domain.Tool
54+
var unsupportedTools []string
3955

4056
for _, tool := range tools {
4157
if tool.Settings.Enabled {
42-
if _, ok := domain.SupportedToolsMetadata[tool.Uuid]; ok && !seen[tool.Uuid] {
58+
if _, supported := domain.SupportedToolsMetadata[tool.Uuid]; supported {
4359
enabledTools = append(enabledTools, tool)
44-
seen[tool.Uuid] = true
60+
} else {
61+
unsupportedTools = append(unsupportedTools, tool.Name)
4562
}
4663
}
4764
}
4865

66+
if len(unsupportedTools) > 0 {
67+
fmt.Printf("Warning: Some tools are not supported: %s\n", strings.Join(unsupportedTools, ", "))
68+
}
69+
4970
return enrichToolsWithVersion(enabledTools)
5071
}
5172

0 commit comments

Comments
 (0)