Merged
Conversation
🦋 Changeset detectedLatest commit: ce64079 The changes in this PR will be included in the next version bump. This PR includes changesets to release 1 package
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
![kilo-code-bot[bot]](https://avatars.githubusercontent.com/in/2193792?s=60&v=4){kind=small}
Contributor
Code Review SummaryStatus: 1 Issue Found | Recommendation: Address before merge Overview
Issue Details (click to expand)WARNING
Other Observations (not in diff)No additional issues found outside the diff. Files Reviewed (11 files)
|
Contributor
Code Review SummaryStatus: No Issues Found | Recommendation: Merge This PR adds post-completion suggestions to start a code review after
Previously flagged issues from earlier review rounds have been addressed:
Files Reviewed (33 files)
|
alex-alecu
force-pushed
the
feat/suggest-review-after-code
branch
from
ea76066 to
ce64079
Compare
marius-kilocode
approved these changes
Feb 20, 2026
This was referenced Feb 20, 2026
Merged
This was referenced Feb 25, 2026
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
2 participants
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.
Context
Suggest code review mode after implementation.
This feature will be ported to the new CLI & VS Code too. It provides great opportunity for the next few weeks to increase week-over-week users for code review NOW.
Initially I had implemented "Clear context and start review" , but the review mode always starts in a new session. No point in developing more for this version since it will be sunset soon.
Implementation
Flow
After a coding task completes (
AttemptCompletionTool.ts:32-41),getCompletionSuggestions()checks the task mode. Ifcodeororchestrator, it returns a single suggestion:{ answer: "Start code review", mode: "review" }.The suggestion is serialized (
AttemptCompletionTool.ts:186-188) asJSON.stringify({ suggest: [...] })and passed as the text payload of thecompletion_resultask.ChatRow parses and renders it (
ChatRow.tsx:462-468). ThecompletionSuggestionsmemo extracts thesuggestarray from the JSON text. If suggestions exist, aFollowUpSuggestcomponent renders clickable buttons (ChatRow.tsx:1819-1830). A guard at line 1832 prevents raw JSON from being rendered as markdown.When the user clicks "Start code review" (
ChatView.tsx:1383-1397):askResponse: "yesButtonClicked"to close the pendingcompletion_resultask (which causes theAttemptCompletionToolto return cleanly)setMode("review")to update local UI state{ type: "mode", text: "review", reviewScope: "uncommitted" }to the backendBackend handles the mode switch (
webviewMessageHandler.ts:1847-1852→ClineProvider.handleModeSwitch:1420-1509):reviewScope: "uncommitted"is provided, it callshandleReviewScopeSelected("uncommitted")directly, skipping the scope selection dialoghandleReviewScopeSelectedcallscreateTask()with a review prompt, which implicitly destroys the old task viaremoveClineFromStack()Edge cases handled
alwaysAllowModeSwitch: Silently ignored, thecompletion_resultask remains open for manual interactionScreenshots
How to Test
Implement something in code mode and look for the suggestions 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.