fixes

Author: samjtro

.golangci.yml

@@ -6,15 +6,10 @@ run:
   timeout: 5m
   issues-exit-code: 1
   tests: true
-  skip-dirs:
-    - vendor
-    - testdata
-    - examples
-  skip-files:
-    - ".*_test.go"
 
 output:
-  format: colored-line-number
+  formats:
+    - format: colored-line-number
   print-issued-lines: true
   print-linter-name: true
   unique-by-line: true
@@ -27,7 +22,7 @@ linters:
     - dupl
     - errcheck
     - exhaustive
-    - exportloopref
+    - copyloopvar
     - gci
     - gochecknoinits
     - gocognit
@@ -57,17 +52,12 @@ linters:
     - whitespace
 
   disable:
-    - deadcode
     - exhaustruct
     - funlen
     - gochecknoglobals
     - godox
-    - gomnd
     - lll
     - nlreturn
-    - scopelint
-    - structcheck
-    - varcheck
     - wsl
 
 linters-settings:
@@ -119,7 +109,6 @@ linters-settings:
     confidence: medium
 
   govet:
-    check-shadowing: true
     enable-all: true
 
   misspell:
@@ -157,7 +146,6 @@ linters-settings:
       - name: errorf
 
   stylecheck:
-    go: "1.23"
     checks: ["all", "-ST1000", "-ST1003", "-ST1016", "-ST1020", "-ST1021", "-ST1022"]
     dot-import-whitelist:
       - fmt
@@ -170,7 +158,16 @@ linters-settings:
     check-exported: false
 
 issues:
+  exclude-dirs:
+    - vendor
+    - testdata
+    - examples
+  exclude-files:
+    - ".*_test.go"
   exclude-rules:
+    - path: sharedlib/
+      linters:
+        - gci
     - path: _test\.go
       linters:
         - gocyclo

cmd/example/main.go

@@ -1,3 +1,4 @@
+// Package main implements the example CLI application.
 package main
 
 import (
@@ -8,8 +9,11 @@ import (
 )
 
 var (
-	Version   = "dev"
+	// Version is the application version.
+	Version = "dev"
+	// GitCommit is the git commit hash.
 	GitCommit = "unknown"
+	// BuildDate is the build date.
 	BuildDate = "unknown"
 )
 
@@ -51,4 +55,4 @@ func printVersion() {
 	fmt.Printf("Version:    %s\n", Version)
 	fmt.Printf("Git Commit: %s\n", GitCommit)
 	fmt.Printf("Build Date: %s\n", BuildDate)
-}
+}

internal/version/version.go

@@ -47,4 +47,4 @@ func String() string {
 		info.GoVersion,
 		info.Platform,
 	)
-}
+}

pkg/example/example.go

@@ -50,10 +50,10 @@ func (c *Client) Process(input string) (string, error) {
 	}
 
 	result := fmt.Sprintf("Processed by %s: %s", c.config.Name, input)
-	
+
 	if c.config.Verbose {
 		fmt.Printf("[DEBUG] %s\n", result)
 	}
 
 	return result, nil
-}
+}

sharedlib/sharedlib.go

@@ -1,7 +1,9 @@
+// Package main provides a shared library for signing and verification.
 package main
 
 // #include <stdlib.h>
 import "C"
+
 import (
 	"crypto/sha256"
 	"encoding/hex"
@@ -10,52 +12,62 @@ import (
 	"unsafe"
 )
 
+// GetVersion returns the version string.
+//
 //export GetVersion
 func GetVersion() *C.char {
 	return C.CString("1.0.0")
 }
 
+// GetPlatform returns the platform string.
+//
 //export GetPlatform
 func GetPlatform() *C.char {
 	return C.CString(fmt.Sprintf("%s/%s", runtime.GOOS, runtime.GOARCH))
 }
 
+// Sign creates a signature for the given data.
+//
 //export Sign
 func Sign(data *C.char) *C.char {
 	if data == nil {
 		return C.CString("")
 	}
-	
+
 	goData := C.GoString(data)
 	hash := sha256.Sum256([]byte(goData))
 	signature := hex.EncodeToString(hash[:])
-	
+
 	return C.CString(signature)
 }
 
+// Verify checks if the signature matches the data.
+//
 //export Verify
-func Verify(data *C.char, signature *C.char) C.int {
+func Verify(data, signature *C.char) C.int {
 	if data == nil || signature == nil {
 		return 0
 	}
-	
+
 	goData := C.GoString(data)
 	goSignature := C.GoString(signature)
-	
+
 	hash := sha256.Sum256([]byte(goData))
 	expectedSignature := hex.EncodeToString(hash[:])
-	
+
 	if expectedSignature == goSignature {
 		return 1
 	}
 	return 0
 }
 
+// FreeString frees a C string allocated by the library.
+//
 //export FreeString
 func FreeString(str *C.char) {
 	C.free(unsafe.Pointer(str))
 }
 
 func main() {
 	// Required for building shared library
-}
+}