feat: add golangci-lint v2 configuration and CI workflow #10
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Add comprehensive linting setup with best practices and security scanning: - Add .golangci.yml with golangci-lint v2 configuration - Enable 34 linters including gosec for security scanning - Configure appropriate thresholds for complexity checks - Set up import grouping with local package separation - Add exclusions for test files and generated mocks - Fix 100+ linter issues across the codebase - Address errcheck violations for unchecked errors - Fix govet shadow warnings for variable shadowing - Update interfaces to use pointer types for large structs - Correct import grouping (stdlib → third-party → local) - Tighten file/directory permissions (0755→0750, 0644→0600) - Add nolint comments with explanations for intentional exceptions - Replace fmt.Errorf with errors.New where no formatting needed - Update to Go 1.22+ integer range loops - Add justfile with development recipes - lint: run golangci-lint - fmt: format code with gofmt and goimports - test: run all tests - check: run all three in sequence - Add GitHub Actions workflow (.github/workflows/cli.yml) - Triggers on Go file changes in PRs and master pushes - Runs format checking, linting, and tests with race detection - Uploads coverage report as artifact 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
- Move exclusion rules from issues.exclude-rules to linters.exclusions.rules - Remove deprecated sloglint.context-only option - Change goimports.local-prefixes from string to array - Add built-in exclusion presets (comments, std-error-handling) 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
jmgilman
added a commit
that referenced
this pull request
Jan 3, 2026
Current behavior: N/A - new feature New behavior: Adds comprehensive linting setup with 34 enabled linters including security scanning, format checking, and automated CI workflow that runs on Go file changes with coverage reporting Closes: #10
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Summary
Changes
New Files
.golangci.yml- golangci-lint v2 configuration with 34 linters enabledjustfile- Development recipes (just check,just lint,just fmt,just test).github/workflows/cli.yml- CI workflow for Go code quality checksLinter Fixes
Security Linters Enabled
gosec- Go security checkerbodyclose- HTTP response body leak detectionnoctx- HTTP request context enforcementTest plan
golangci-lint runpasses with 0 issuesgo test ./...passes🤖 Generated with Claude Code