[codex] fix golangci-lint v2 config#24
Merged
Merged
Conversation
There was a problem hiding this comment.
Pull request overview
Updates the repository’s golangci-lint configuration and CI setup to be compatible with golangci-lint v2.x, while adding a new internal implementation plan document for CLI UX improvements.
Changes:
- Migrates
.golangci.ymlto golangci-lint v2 config structure (including formatter configuration). - Pins CI lint action to
v2.8.0and updates the Makefile’s lint version variable. - Adds an AI-agent-friendly CLI UX improvements implementation plan under
docs/plans.
Reviewed changes
Copilot reviewed 4 out of 4 changed files in this pull request and generated 3 comments.
| File | Description |
|---|---|
docs/plans/2026-04-15-ai-agent-friendly-cli-ux-improvements.md |
Adds a structured implementation plan for CLI UX improvements and acceptance criteria. |
Makefile |
Updates the declared golangci-lint version variable to v2.8.0. |
.golangci.yml |
Migrates lint/format config to golangci-lint v2 format with exclusions and formatter settings. |
.github/workflows/ci.yml |
Updates golangci-lint GitHub Action to run v2.8.0. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
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
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
.golangci.ymlto golangci-lint v2 format so localgolangci-lint v2.8.0can load it.v2.8.0.docs/plans.Root Cause
The repository config was still in golangci-lint v1 format while local tooling was
golangci-lint v2.8.0. v2 rejects configs without aversionfield, causingmake lintto fail before linting code.Validation
git diff --cached --checkgolangci-lint config verifymake lint(0 issues.)make vetmake buildmake testScope Notes
openspec/changes/claude-internal-and-subagents-support/is intentionally not included.