Skip to content

Comments

feat: add Kilo agent (kilo CLI)#314

Open
mariusvniekerk wants to merge 5 commits intoroborev-dev:mainfrom
mariusvniekerk:add-kilo-cli
Open

feat: add Kilo agent (kilo CLI)#314
mariusvniekerk wants to merge 5 commits intoroborev-dev:mainfrom
mariusvniekerk:add-kilo-cli

Conversation

@mariusvniekerk
Copy link
Contributor

Summary

  • Implements the Agent interface for Kilo CLI (kilo run)
  • Prompt piped via stdin, --model for model selection, --auto for agentic mode, --variant for reasoning levels
  • Reuses filterOpencodeToolCallLines for output cleanup (kilo shares opencode's output format)
  • Implementation intentionally cloned from opencode.go rather than abstracted, since kilo may diverge in the future
  • Adds kilo to auto-discovery fallback order and help/error strings

Test plan

  • go test ./internal/agent/ -run TestKilo passes (13 tests)
  • go test ./... green (2973 tests)
  • roborev review --local --agent kilo --wait HEAD produces a review

🤖 Generated with Claude Code

mariusvniekerk and others added 5 commits February 20, 2026 22:40
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@roborev-ci
Copy link

roborev-ci bot commented Feb 21, 2026

roborev: Combined Review (40b3bf4f)

Verdict: One medium-severity test reliability issue; no security vulnerabilities identified.

Medium

  1. Parallel test mutates shared/global test state (flaky/racy)
    • Locations: internal/agent/kilo_test.go:111, internal/agent/kilo_test.go:113, internal/agent/kilo_test.go:116
    • Finding: TestKiloAgenticAutoFlag is marked t.Parallel() while using withUnsafeAgents(...), which mutates shared/global agent safety state. This can cause nondeterministic failures (and potentially env/state conflicts) when tests run concurrently.
    • Suggested fix: Remove t.Parallel() from tests that call withUnsafeAgents(...), or refactor test setup to avoid global mutation in parallel execution.

Synthesized from 4 reviews (agents: codex, gemini | types: security, default)

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.

1 participant