Skip to content

Add CI and lint GitHub Actions workflows#2

Merged
alishakawaguchi merged 2 commits intomainfrom
gh-actions
Mar 24, 2026
Merged

Add CI and lint GitHub Actions workflows#2
alishakawaguchi merged 2 commits intomainfrom
gh-actions

Conversation

@alishakawaguchi
Copy link
Contributor

@alishakawaguchi alishakawaguchi commented Mar 24, 2026

Summary

  • Add CI workflow (ci.yml): unit tests with race detector + build verification on PRs and pushes to main
  • Add lint workflow (lint.yml): gofmt formatting check + golangci-lint v2 with inline PR annotations (per-module)
  • Add .golangci.yaml: curated linter set adapted from devenv/cli with appropriate exclusions for CLI tools and e2e test infra
  • Add .gitignore: ignore .entire/ runtime data and built binaries
  • Fix existing lint issues: unused types, unchecked json.Marshal errors, formatting alignment, dead code removal

Test plan

  • gofmt -l . returns empty (all files formatted)
  • golangci-lint run ./... passes in agents/entire-agent-kiro (0 issues)
  • golangci-lint run ./... passes in e2e (0 issues)
  • go test -race -count=1 ./... passes in agents/entire-agent-kiro (2/2 packages)
  • go build -o /dev/null ./cmd/entire-agent-kiro succeeds

🤖 Generated with Claude Code


Note

Low Risk
Low risk: mostly adds GitHub Actions workflows and lint configuration; code changes are small, lint-driven cleanup (error handling and dead code removal) with minimal behavior impact.

Overview
Adds GitHub Actions automation: a CI workflow to run go test -race and a build check for agents/entire-agent-kiro, plus a Lint workflow that enforces gofmt and runs golangci-lint for both agents/entire-agent-kiro and e2e.

Introduces a repo-wide .golangci.yaml and updates .gitignore to exclude .entire/ runtime data and the built entire-agent-kiro binary. To satisfy the new linting, it also tightens error handling around json.Marshal, removes unused execution-index types, and makes minor cleanup/refactors (e.g., stubSessionID constant reuse and simpler session file path building).

Written by Cursor Bugbot for commit 05e2acb. Configure here.

alishakawaguchi and others added 2 commits March 24, 2026 09:07
Add CI workflow (unit tests with race detector + build verification) and
lint workflow (gofmt check + golangci-lint with inline PR annotations)
triggered on PRs and pushes to main. Add .golangci.yaml v2 config with
curated linter set adapted from devenv/cli. Fix existing lint issues:
unused types, unchecked json.Marshal errors, formatting, and minor
code quality improvements. Add .gitignore for runtime artifacts.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Entire-Checkpoint: 8e58bb2aa1e6
CI golangci-lint v2.11.3 flagged G703 (path traversal via taint) for
controlled file paths and goconst for "darwin" in runtime.GOOS switches.
Exclude G703 (paths are constructed from controlled inputs) and raise
goconst min-occurrences to 5 to avoid flagging idiomatic patterns.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Entire-Checkpoint: 93daed613361
@alishakawaguchi alishakawaguchi self-assigned this Mar 24, 2026
@alishakawaguchi alishakawaguchi marked this pull request as ready for review March 24, 2026 16:51
@alishakawaguchi alishakawaguchi merged commit f51fba5 into main Mar 24, 2026
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

1 participant