build(deps): bump go-simpler.org/sloglint from 0.1.2 to 0.2.0 (#4166)

Author: tmzane

.golangci.reference.yml

@@ -1824,9 +1824,16 @@ linters-settings:
     # Enforce using attributes only (incompatible with kv-only).
     # Default: false
     attr-only: true
+    # Enforce using methods that accept a context.
+    # Default: false
+    context-only: true
     # Enforce using constants instead of raw keys.
     # Default: false
     no-raw-keys: true
+    # Enforce a single key naming convention.
+    # Values: snake, kebab, camel, pascal
+    # Default: ""
+    key-naming-case: snake
     # Enforce putting arguments on separate lines.
     # Default: false
     args-on-sep-lines: true

go.mod

@@ -120,7 +120,7 @@ require (
 	github.com/yeya24/promlinter v0.2.0
 	github.com/ykadowak/zerologlint v0.1.3
 	gitlab.com/bosi/decorder v0.4.1
-	go-simpler.org/sloglint v0.1.2
+	go-simpler.org/sloglint v0.2.0
 	go.tmz.dev/musttag v0.7.2
 	golang.org/x/exp v0.0.0-20230510235704-dd950f8aeaea
 	golang.org/x/tools v0.14.0

go.sum

@@ -119,7 +119,9 @@ var defaultLintersSettings = LintersSettings{
 	SlogLint: SlogLintSettings{
 		KVOnly:         false,
 		AttrOnly:       false,
+		ContextOnly:    false,
 		NoRawKeys:      false,
+		KeyNamingCase:  "",
 		ArgsOnSepLines: false,
 	},
 	TagAlign: TagAlignSettings{
@@ -734,10 +736,12 @@ type RowsErrCheckSettings struct {
 }
 
 type SlogLintSettings struct {
-	KVOnly         bool `mapstructure:"kv-only"`
-	AttrOnly       bool `mapstructure:"attr-only"`
-	NoRawKeys      bool `mapstructure:"no-raw-keys"`
-	ArgsOnSepLines bool `mapstructure:"args-on-sep-lines"`
+	KVOnly         bool   `mapstructure:"kv-only"`
+	AttrOnly       bool   `mapstructure:"attr-only"`
+	ContextOnly    bool   `mapstructure:"context-only"`
+	NoRawKeys      bool   `mapstructure:"no-raw-keys"`
+	KeyNamingCase  string `mapstructure:"key-naming-case"`
+	ArgsOnSepLines bool   `mapstructure:"args-on-sep-lines"`
 }
 
 type StaticCheckSettings struct {

pkg/config/linters_settings.go

@@ -14,7 +14,9 @@ func NewSlogLint(settings *config.SlogLintSettings) *goanalysis.Linter {
 		opts = &sloglint.Options{
 			KVOnly:         settings.KVOnly,
 			AttrOnly:       settings.AttrOnly,
+			ContextOnly:    settings.ContextOnly,
 			NoRawKeys:      settings.NoRawKeys,
+			KeyNamingCase:  settings.KeyNamingCase,
 			ArgsOnSepLines: settings.ArgsOnSepLines,
 		}
 	}

pkg/golinters/sloglint.go

@@ -0,0 +1,3 @@
+linters-settings:
+  sloglint:
+    context-only: true

test/testdata/configs/sloglint_context_only.yml

@@ -0,0 +1,3 @@
+linters-settings:
+  sloglint:
+    key-naming-case: snake

test/testdata/configs/sloglint_key_naming_case.yml

@@ -0,0 +1,16 @@
+//go:build go1.21
+
+//golangcitest:args -Esloglint
+//golangcitest:config_path testdata/configs/sloglint_context_only.yml
+package testdata
+
+import (
+	"context"
+	"log/slog"
+)
+
+func test() {
+	slog.InfoContext(context.Background(), "msg")
+
+	slog.Info("msg") // want `methods without a context should not be used`
+}

test/testdata/sloglint_context_only.go

@@ -0,0 +1,24 @@
+//go:build go1.21
+
+//golangcitest:args -Esloglint
+//golangcitest:config_path testdata/configs/sloglint_key_naming_case.yml
+package testdata
+
+import "log/slog"
+
+const (
+	snakeKey = "foo_bar"
+	kebabKey = "foo-bar"
+)
+
+func test() {
+	slog.Info("msg", "foo_bar", 1)
+	slog.Info("msg", snakeKey, 1)
+	slog.Info("msg", slog.Int("foo_bar", 1))
+	slog.Info("msg", slog.Int(snakeKey, 1))
+
+	slog.Info("msg", "foo-bar", 1)           // want `keys should be written in snake_case`
+	slog.Info("msg", kebabKey, 1)            // want `keys should be written in snake_case`
+	slog.Info("msg", slog.Int("foo-bar", 1)) // want `keys should be written in snake_case`
+	slog.Info("msg", slog.Int(kebabKey, 1))  // want `keys should be written in snake_case`
+}