jwilger/auto_review
1
0
Fork 0
Code Issues 6 Pull requests 1 Releases Packages 1 Activity Actions

chore(kilo): relax agent step limits #37

Merged
jwilger merged 1 commit from chore/kilo-agent-step-limits into main 2026-05-02 12:32:12 -07:00
Conversation 0 Commits 1 Files changed 10 +9 -10
jwilger commented 2026-05-02 12:30:25 -07:00
Owner
Copy link

Summary

  • Remove the project-specific step cap from the primary auto_review implementer agent so long-running orchestration can continue across issue workflows.
  • Raise focused subagent step limits to 200 to avoid premature forced text-only responses while still keeping an explicit ceiling.

Verification

  • Kilo config validation hooks passed when the agent files were edited.
  • git diff --check -- .kilo/agent
## Summary - Remove the project-specific step cap from the primary auto_review implementer agent so long-running orchestration can continue across issue workflows. - Raise focused subagent step limits to 200 to avoid premature forced text-only responses while still keeping an explicit ceiling. ## Verification - Kilo config validation hooks passed when the agent files were edited. - git diff --check -- .kilo/agent
chore(kilo): relax agent step limits
All checks were successful
auto_review auto_review: no findings
CI / Nix flake check (pull_request) Successful in 5s
Details
26b3d1d226
auto-review reviewed 2026-05-02 12:30:43 -07:00
auto-review left a comment
Copy link

This PR relaxes the step limits for various agents in the .kilo configuration, allowing for longer execution times. While this change can enable more comprehensive reviews, it is important to ensure that it does not lead to performance issues or resource exhaustion.

Walkthrough

  • architecture-reviewer.md: Step limit increased from 20 to 200.
  • auto-review-rust-implementer.md: Step limit removed, which could lead to unbounded execution.
  • docs-operator-reviewer.md: Step limit increased from 15 to 200.
  • forgejo-feedback-processor.md: Step limit increased from 30 to 200.
  • rgr-diagnostic-implementer.md: Step limit increased from 20 to 200.
  • rgr-implementation-reviewer.md: Step limit increased from 20 to 200.
  • rgr-test-author.md: Step limit increased from 20 to 200.
  • rgr-test-reviewer.md: Step limit increased from 20 to 200.
  • security-reviewer.md: Step limit increased from 20 to 200.
  • test-coverage-reviewer.md: Step limit increased from 20 to 200.

Pre-merge checks

  • [~] CHANGELOG updated — diff is docs/config only — no CHANGELOG entry expected
  • [~] Tests touched — no source files in the diff
  • No new TODO/FIXME comments — no new TODO/FIXME markers
This PR relaxes the step limits for various agents in the `.kilo` configuration, allowing for longer execution times. While this change can enable more comprehensive reviews, it is important to ensure that it does not lead to performance issues or resource exhaustion. ## Walkthrough - **architecture-reviewer.md**: Step limit increased from 20 to 200. - **auto-review-rust-implementer.md**: Step limit removed, which could lead to unbounded execution. - **docs-operator-reviewer.md**: Step limit increased from 15 to 200. - **forgejo-feedback-processor.md**: Step limit increased from 30 to 200. - **rgr-diagnostic-implementer.md**: Step limit increased from 20 to 200. - **rgr-implementation-reviewer.md**: Step limit increased from 20 to 200. - **rgr-test-author.md**: Step limit increased from 20 to 200. - **rgr-test-reviewer.md**: Step limit increased from 20 to 200. - **security-reviewer.md**: Step limit increased from 20 to 200. - **test-coverage-reviewer.md**: Step limit increased from 20 to 200. ## Pre-merge checks - [~] CHANGELOG updated — diff is docs/config only — no CHANGELOG entry expected - [~] Tests touched — no source files in the diff - [x] No new TODO/FIXME comments — no new TODO/FIXME markers
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels
No items
No labels
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
2 participants
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
jwilger/auto_review!37
No description provided.