Skip to content
This repository was archived by the owner on Feb 25, 2026. It is now read-only.

feat: Add one-click apply fixes suggestions to review mode#360

Merged
alex-alecu merged 22 commits intodevfrom
feat/review-apply-fixes
Feb 18, 2026
Merged

feat: Add one-click apply fixes suggestions to review mode#360
alex-alecu merged 22 commits intodevfrom
feat/review-apply-fixes

Conversation

@alex-alecu
Copy link
Contributor

Context

Add suggestions for implementing review findings. Follow-up from VS Code extension Kilo-Org/kilocode#5926

Implementation

Changed the prompt to recommend max 3 options based on the amount of review findings. Included options for orchestrator and debug where the issues are complex or need discussing first.

Screenshots

Screenshot 2026-02-17 at 10 11 17

How to Test

Run /local-review-uncommitted in the Kilo CLI and look for the actionable items at the end.

Get in Touch

We'd love to have a way to chat with you about your changes if necessary. If you're in the Kilo Code Discord, please share your handle here.

@alex-alecu alex-alecu self-assigned this Feb 17, 2026
@kilo-code-bot
Copy link
Contributor

kilo-code-bot bot commented Feb 17, 2026

Code Review Summary

Status: No New Issues Found | Recommendation: Address existing comments before merge

Overview

This PR adds mode-switching support to the question dock, allowing review question options to trigger agent mode changes (e.g., switching to "code" or "debug" mode after a review). The implementation spans the backend schema (Question.Option.mode field), the review prompt instructions, the SDK types, and the frontend UI flow.

The code is well-structured with proper abort controller cleanup, timeout handling, and error boundaries. The waitForIdle pattern using createRoot/createEffect correctly manages the reactive subscription lifecycle with a settle guard to prevent double-resolution.

No new issues were found beyond those already flagged in existing inline comments (15 active). The previously reported concerns about polling cancellation, TDZ in dispose(), and root leaks appear to have been addressed in the latest commit based on author responses.

Files Reviewed (7 files)
  • packages/app/src/components/question-dock.tsx — Added onModeAction prop, mode-aware option picking, mode badge UI
  • packages/app/src/pages/session.tsx — Added handleModeAction, waitForIdle, abort controller lifecycle
  • packages/app/src/pages/session/session-prompt-dock.tsx — Prop threading for onModeAction
  • packages/app/src/i18n/en.ts — New i18n strings for mode switching
  • packages/opencode/src/kilocode/review/review.ts — Updated review prompt with post-review workflow instructions
  • packages/opencode/src/question/index.ts — Added optional mode field to Question.Option schema
  • packages/sdk/js/src/v2/gen/types.gen.ts — Generated SDK type update for QuestionOption.mode
  • packages/sdk/openapi.json — OpenAPI schema update for mode field

@alex-alecu alex-alecu enabled auto-merge February 17, 2026 16:00
@alex-alecu alex-alecu force-pushed the feat/review-apply-fixes branch from b4dd43b to 9d43ad4 Compare February 18, 2026 10:06
@alex-alecu alex-alecu merged commit e8895d6 into dev Feb 18, 2026
7 checks passed
@alex-alecu alex-alecu deleted the feat/review-apply-fixes branch February 18, 2026 10:47
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants