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

.golangci.reference.yml

@@ -1819,9 +1819,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

pkg/config/linters_settings.go

@@ -116,7 +116,9 @@ var defaultLintersSettings = LintersSettings{
  SlogLint: SlogLintSettings{
  KVOnly: false,
  AttrOnly: false,
+ ContextOnly: false,
  NoRawKeys: false,
+ KeyNamingCase: "",
  ArgsOnSepLines: false,
  },
  TagAlign: TagAlignSettings{
@@ -725,10 +727,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/golinters/sloglint.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,
  }
  }

test/testdata/configs/sloglint_context_only.yml

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

test/testdata/configs/sloglint_key_naming_case.yml

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

test/testdata/sloglint_context_only.go

@@ -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_key_naming_case.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`
+}