Refactoring due to aggressive linting; add golanglint-ci github action

.github/workflows/golangci-lint.yml

@@ -0,0 +1,41 @@
+name: golangci-lint
+on:
+  push:
+    branches:
+      - '**' # Run on all branches
+  pull_request:
+
+permissions:
+  contents: read
+  # Optional: allow read access to pull request. Use with `only-new-issues` option.
+  # pull-requests: read
+
+jobs:
+  golangci:
+    name: lint
+    runs-on: ubuntu-latest
+    steps:
+      - uses: actions/checkout@v4
+      - uses: actions/setup-go@v5
+        with:
+          go-version-file: go.mod
+          check-latest: true
+          go-version: '1.22'
+      - name: golangci-lint
+        uses: golangci/golangci-lint-action@v5
+        with:
+          version: v1.57
+          skip-cache: true
+          args: --timeout=5m
+
+  test:
+    runs-on: ubuntu-latest
+    steps:
+      - uses: actions/checkout@v4
+      - uses: actions/setup-go@v5
+        with:
+          go-version-file: go.mod
+          check-latest: true
+          go-version: '1.22'
+      - name: Test all
+        run: go test ./...

README.md

@@ -52,7 +52,5 @@ pre-commit run --all-files
 
 #### Go-licenses pre-commit hook
 
-Windows users will not be able to run the `go-licenses` hook as yet - @momer will be sending through a 
-PR to that project to resolve the issue, which is to do with OS-agnostic filepath support!
-
-
+Windows users: Ensure that you have `C:\Program Files\Git\usr\bin` added 
+to your `PATH`!

bonsai/bonsai.go

@@ -1,2 +1,6 @@
 // Package bonsai wraps the Bonsai.io HTTP API to create a Go API Client.
 package bonsai
+
+const (
+	Float64Epsilon = 1e-9
+)

bonsai/bonsai_test.go

@@ -2,10 +2,9 @@ package bonsai_test
 
 import (
 	"fmt"
+	"log"
 	"log/slog"
 	"os"
-
-	_ "github.com/stretchr/testify/require"
 )
 
 func init() {
@@ -18,9 +17,12 @@ func initLogger() {
 
 	logHandler := slog.NewTextHandler(os.Stdout, &slog.HandlerOptions{
 		AddSource: true,
-		ReplaceAttr: func(groups []string, a slog.Attr) slog.Attr {
+		ReplaceAttr: func(_ []string, a slog.Attr) slog.Attr {
 			if a.Key == slog.SourceKey {
-				src := a.Value.Any().(*slog.Source)
+				src, ok := a.Value.Any().(*slog.Source)
+				if !ok {
+					log.Fatalf("sourceKey attr is not a Source: %v", a.Value)
+				}
 				// Ruby on Rails-ish formatting
 				a.Value = slog.StringValue(fmt.Sprintf("%s:%d:in '%s'", src.File, src.Line, src.Function))
 			}