Skip to content

Comments

fix: handle removed linters in golangci-lint v2 and add CLI convention filtering#55

Merged
ikjeong merged 4 commits intomainfrom
fix/golangci-lint
Dec 13, 2025
Merged

fix: handle removed linters in golangci-lint v2 and add CLI convention filtering#55
ikjeong merged 4 commits intomainfrom
fix/golangci-lint

Conversation

@ikjeong
Copy link
Contributor

@ikjeong ikjeong commented Dec 12, 2025

Summary

  • Update golangci-lint converter to support v2
  • Add filtering capabilities to CLI convention list command.

Changes

golangci-lint v2 Compatibility (internal/linter/golangcilint/converter.go)

  • Remove deprecated linters: stylecheck, gosimple, typecheck, exportloopref
  • Add new v2 linters: err113, errorlint, perfsprint, varnamelen, lll, godoclint, mnd
  • Remove swaggo from formatters list
  • Update LLM prompt to reflect v2 linter categories and settings

CLI Enhancement (internal/cmd/convention.go)

  • Add --category flag to filter conventions by category
  • Add --languages flag to filter conventions by programming languages
  • Align CLI functionality with MCP list_convention tool

- Remove unsupported linters: stylecheck, typecheck, gosimple, exportloopref
- Add new v2 linters: err113, errorlint, varnamelen, lll, godoclint, mnd, perfsprint
- Update LLM prompt with v2 migration notes and decision guide
@ikjeong ikjeong self-assigned this Dec 12, 2025
@sehwan505
Copy link
Contributor

LGTM

@ikjeong ikjeong changed the title fix: fix issues caused by removed or changed linters in golangci-lint v2 fix: handle removed linters in golangci-lint v2 and add CLI convention filtering Dec 13, 2025
@ikjeong ikjeong merged commit bf5b059 into main Dec 13, 2025
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants