migrate project config to jig, add lint scripts, fix pre-existing lint issues

Author: Jason-Abbott

.jig.yaml

@@ -0,0 +1,69 @@
+citations:
+    - repo: yayolande/gota
+      branch: master
+      last_checked_sha: 46acdb27bc3801e1b13bad8809f708d96e00428c
+      last_checked_date: "2026-02-24T18:16:15Z"
+      paths:
+        high:
+            - analyzer.go
+            - parser/**
+            - lexer/**
+            - template.go
+        medium:
+            - go.mod
+            - go.sum
+            - '*_test.go'
+        low:
+            - .github/**
+            - README.md
+            - LICENSE
+    - repo: yayolande/go-template-lsp
+      branch: master
+      last_checked_sha: e79b308f1c0e1a0f7b8e4ae79d94eb3eaa631d4c
+      last_checked_date: "2026-02-24T18:16:15Z"
+      paths:
+        high:
+            - lsp/**
+            - main.go
+        medium:
+            - go.mod
+            - go.sum
+            - '*_test.go'
+        low:
+            - .github/**
+            - README.md
+            - LICENSE
+            - .gitignore
+todo:
+    default_status: draft
+    default_type: task
+    path: .issues
+    sync:
+        clickup:
+            custom_fields:
+                bean_id: a0b0c69a-d036-4031-9deb-4ecc4cfa3e55
+                created_at: d3adc5e1-eedf-46e0-b5d5-a4dce3a0666b
+            list_id: "901112937048"
+            status_mapping:
+                completed: completed
+                draft: not started
+                in-progress: in progress
+                ready: not started
+                review: review
+                scrapped: completed
+            type_mapping:
+                bug: 1001
+                epic: 1002
+                feature: 0
+                milestone: 1
+                task: 0
+companions:
+    zed: toba/gozer
+nope:
+    rules:
+        - name: git-no-verify
+          pattern: 'git\s+.*--no-verify'
+          message: "--no-verify is never allowed - fix the issue that's failing the hook instead"
+        - name: git-checkout-switch
+          pattern: 'git\s+(checkout|switch)\s+'
+          message: "git checkout/switch not allowed - branch changes require user approval"

.toba.yaml

@@ -1057,9 +1057,9 @@ func ProcessFormattingRequest(
 			responseData, _ := json.Marshal(res)
 			return responseData, fileName
 		}
-		fileContent, err = os.ReadFile(parsed.Path)
+		fileContent, err = os.ReadFile(parsed.Path) //nolint:gosec // path comes from editor URI
 		if err != nil {
-			slog.Warn("Cannot read file for formatting: " + err.Error())
+			slog.Warn("Cannot read file for formatting: " + err.Error()) //nolint:gosec // error context only
 			responseData, _ := json.Marshal(res)
 			return responseData, fileName
 		}

cmd/go-template-lsp/lsp/methods.go

@@ -317,7 +317,7 @@ func main() {
 
 	if scanner.Err() != nil {
 		msg := "error while closing LSP: " + scanner.Err().Error()
-		slog.Error(msg)
+		slog.Error(msg) //nolint:gosec // error context only
 		panic(msg)
 	}
 }
@@ -345,7 +345,7 @@ func insertTextDocumentToDiagnostic(
 
 	if len(textChangedNotification) >= 2 {
 		msg := "'textChangedNotification' channel size should never exceed 1"
-		slog.Error(msg,
+		slog.Error(msg, //nolint:gosec // internal diagnostic
 			slog.Group("error_details",
 				slog.String("uri_file_to_diagnostic", uri),
 				slog.Any("files_waiting_processing", mapToKeys(textFromClient)),

cmd/go-template-lsp/main.go

@@ -166,7 +166,7 @@ func TestTokenizeLine_Variables(t *testing.T) {
 			tokenID:  DotVariable,
 			tokenVal: ".",
 		},
-		{
+		{ //nolint:gosec // test data, not credentials
 			name:     "chained dot variable",
 			source:   "{{ .Field.SubField }}",
 			tokenID:  DotVariable,
@@ -1014,4 +1014,3 @@ func TestTokenize_FunctionWithKeywordPrefix(t *testing.T) {
 		})
 	}
 }
-

internal/template/lexer/lexer_test.go

@@ -24,10 +24,10 @@ var varMapStringAny map[string]any
 var varMapIntString map[int]string
 
 // Iterator types (Go 1.23+)
-var varIterSeq func(yield func(int) bool)                    // iter.Seq[int]
-var varIterSeq2 func(yield func(string, int) bool)           // iter.Seq2[string, int]
-var varIterSeqPerson func(yield func(*Person) bool)          // iter.Seq[*Person]
-var varIterSeq2StringAst func(yield func(string, Ast) bool)  // iter.Seq2[string, Ast]
+var varIterSeq func(yield func(int) bool)                   // iter.Seq[int]
+var varIterSeq2 func(yield func(string, int) bool)          // iter.Seq2[string, int]
+var varIterSeqPerson func(yield func(*Person) bool)         // iter.Seq[*Person]
+var varIterSeq2StringAst func(yield func(string, Ast) bool) // iter.Seq2[string, Ast]
 
 type Ast struct {
 	Kind int

internal/template/testdata/source_code.go

@@ -0,0 +1,13 @@
+#!/usr/bin/env bash
+# Run golangci-lint with auto-fixes, then report remaining issues.
+set -euo pipefail
+
+repo_root="$(git -C "$(dirname "$0")" rev-parse --show-toplevel)" || exit 1
+cd "${repo_root}"
+
+echo "Running golangci-lint with auto-fix..."
+golangci-lint run --fix ./... 2>&1 || true
+
+echo ""
+echo "Remaining issues (require manual intervention):"
+golangci-lint run ./... 2>&1 || true

scripts/lint.sh

@@ -0,0 +1,22 @@
+#!/usr/bin/env bash
+# Pre-commit hook: lint, format, and build.
+set -euo pipefail
+
+repo_root="$(git rev-parse --show-toplevel)"
+cd "${repo_root}"
+
+echo "==> Running golangci-lint..."
+bash scripts/lint.sh
+
+echo "==> Running go fmt..."
+unformatted=$(gofmt -l .)
+if [ -n "$unformatted" ]; then
+  echo "The following files need formatting:"
+  echo "$unformatted"
+  exit 1
+fi
+
+echo "==> Running go build..."
+go build ./...
+
+echo "All pre-commit checks passed."