Linter fixes

.golangci.yml

@@ -0,0 +1,8 @@
+linters:
+  # https://golangci-lint.run/usage/linters/#enabled-by-default
+  enable:
+    - goconst
+    - gocritic
+    - gosec
+    - nilnil
+    - revive

config.go

@@ -60,7 +60,7 @@ func ReadConfig() (*Config, error) {
 			return nil, err
 		}
 
-		err = os.WriteFile(fileName, defaultMarshalled, 0644)
+		err = os.WriteFile(fileName, defaultMarshalled, 0600)
 		if err != nil {
 			return nil, err
 		}

main.go

@@ -9,13 +9,15 @@ import (
 func main() {
 	conn, err := dbus.ConnectSessionBus()
 	if err != nil {
-		log.Fatalf("failed to connect to session bus: %v", err)
+		log.Printf("failed to connect to session bus: %v", err)
+		return
 	}
 	defer conn.Close()
 
 	config, err := ReadConfig()
 	if err != nil {
-		log.Fatalf("error reading config: %v", err)
+		log.Printf("error reading config: %v", err)
+		return
 	}
 
 	RunMainLoop(conn, config)

mpris.go

@@ -93,9 +93,8 @@ func getProperty[E any](obj dbus.BusObject, property string) (*E, error) {
 
 	if parsedValue, ok := value.Value().(E); ok {
 		return &parsedValue, nil
-	} else {
-		return nil, errors.New("failed to read property from DBus object")
 	}
+	return nil, errors.New("failed to read property from DBus object")
 }
 
 func getMapEntry[E any](metadata map[string]dbus.Variant, key string) (*E, error) {
@@ -106,7 +105,6 @@ func getMapEntry[E any](metadata map[string]dbus.Variant, key string) (*E, error
 
 	if parsedValue, ok := value.Value().(E); ok {
 		return &parsedValue, nil
-	} else {
-		return nil, fmt.Errorf("invalid data type for map entry %s", key)
 	}
+	return nil, fmt.Errorf("invalid data type for map entry %s", key)
 }

provider_file.go

@@ -9,7 +9,7 @@ import (
 	"time"
 )
 
-func (f *FileConfig) NowPlaying(n NowPlaying) {
+func (f *FileConfig) NowPlaying(_ NowPlaying) {
 
 }