fix(ci): exclude errcheck in tests and use settings for review permissions

errcheck in test files is noisy for test setup code. Replace invalid
allowed_tools input with settings parameter for claude-code-action.
The review auth failure on this PR is expected since the workflow file
differs from main — will resolve once merged.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

Author: jim80net

.github/workflows/claude-code-review.yml

@@ -40,7 +40,20 @@ jobs:
           plugins: 'code-review@claude-code-plugins'
           prompt: '/code-review:code-review ${{ github.repository }}/pull/${{ github.event.pull_request.number }}'
           show_full_output: true
-          allowed_tools: 'Bash(gh:*),Bash(git diff:*),Bash(git log:*),Bash(git show:*),Read,Glob,Grep'
+          settings: |
+            {
+              "permissions": {
+                "allow": [
+                  "Bash(gh:*)",
+                  "Bash(git diff:*)",
+                  "Bash(git log:*)",
+                  "Bash(git show:*)",
+                  "Read",
+                  "Glob",
+                  "Grep"
+                ]
+              }
+            }
           # See https://github.com/anthropics/claude-code-action/blob/main/docs/usage.md
           # or https://code.claude.com/docs/en/cli-reference for available options
 

.golangci.yml

@@ -10,3 +10,9 @@ linters:
     - unused
     - gosimple
     - ineffassign
+
+issues:
+  exclude-rules:
+    - path: _test\.go
+      linters:
+        - errcheck