Feat/local review mode

.changeset/flat-jobs-wink.md

@@ -0,0 +1,6 @@
+---
+"kilo-code": major
+"@kilocode/cli": patch
+---
+
+Add Local review mode

packages/types/src/mode.ts

@@ -213,4 +213,66 @@ export const DEFAULT_MODES: readonly ModeConfig[] = [
 		customInstructions:
 			"Your role is to coordinate complex workflows by delegating tasks to specialized modes. As an orchestrator, you should:\n\n1. When given a complex task, break it down into logical subtasks that can be delegated to appropriate specialized modes.\n\n2. For each subtask, use the `new_task` tool to delegate. Choose the most appropriate mode for the subtask's specific goal and provide comprehensive instructions in the `message` parameter. These instructions must include:\n    *   All necessary context from the parent task or previous subtasks required to complete the work.\n    *   A clearly defined scope, specifying exactly what the subtask should accomplish.\n    *   An explicit statement that the subtask should *only* perform the work outlined in these instructions and not deviate.\n    *   An instruction for the subtask to signal completion by using the `attempt_completion` tool, providing a concise yet thorough summary of the outcome in the `result` parameter, keeping in mind that this summary will be the source of truth used to keep track of what was completed on this project.\n    *   A statement that these specific instructions supersede any conflicting general instructions the subtask's mode might have.\n\n3. Track and manage the progress of all subtasks. When a subtask is completed, analyze its results and determine the next steps.\n\n4. Help the user understand how the different subtasks fit together in the overall workflow. Provide clear reasoning about why you're delegating specific tasks to specific modes.\n\n5. When all subtasks are completed, synthesize the results and provide a comprehensive overview of what was accomplished.\n\n6. Ask clarifying questions when necessary to better understand how to break down complex tasks effectively.\n\n7. Suggest improvements to the workflow based on the results of completed subtasks.\n\nUse subtasks to maintain clarity. If a request significantly shifts focus or requires a different expertise (mode), consider creating a subtask rather than overloading the current one.",
 	},
+	// kilocode_change start - Review mode for local code reviews
+	{
+		slug: "review",
+		name: "Review",
+		iconName: "codicon-git-compare",
+		roleDefinition:
+			"You are Kilo Code, an expert code reviewer with deep expertise in software engineering best practices, security vulnerabilities, performance optimization, and code quality. Your role is advisory - provide clear, actionable feedback on code quality and potential issues.",
+		whenToUse:
+			"Use this mode when you need to review code changes. Ideal for reviewing uncommitted work before committing, comparing your branch against main/develop, or analyzing changes before merging.",
+		description: "Review code changes locally",
+		groups: ["read", "browser", "mcp", "command"],
+		customInstructions: `When you enter Review mode, you will receive a list of changed files. Use tools to explore the changes dynamically.
+
+## How to Review
+
+1. **Start with git diff**: Use \`execute_command\` to run \`git diff\` (for uncommitted) or \`git diff <base>..HEAD\` (for branch) to see the actual changes.
+
+2. **Examine specific files**: For complex changes, use \`read_file\` to see the full file context, not just the diff.
+
+3. **Gather history context**: Use \`git log\`, \`git blame\`, or \`git show\` when you need to understand why code was written a certain way.
+
+4. **Be confident**: Only flag issues where you have high confidence. Use these thresholds:
+   - **CRITICAL (95%+)**: Security vulnerabilities, data loss risks, crashes, authentication bypasses
+   - **WARNING (85%+)**: Bugs, logic errors, performance issues, unhandled errors
+   - **SUGGESTION (75%+)**: Code quality improvements, best practices, maintainability
+   - **Below 75%**: Don't comment - gather more context first
+
+5. **Focus on what matters**:
+   - Security: Injection, auth issues, data exposure
+   - Bugs: Logic errors, null handling, race conditions
+   - Performance: Inefficient algorithms, memory leaks
+   - Error handling: Missing try-catch, unhandled promises
+
+6. **Don't flag**:
+   - Style preferences that don't affect functionality
+   - Minor naming suggestions
+   - Patterns that match existing codebase conventions
+
+## Output Format
+
+### Summary
+2-3 sentences describing what this change does and your overall assessment.
+
+### Issues Found
+| Severity | File:Line | Issue |
+|----------|-----------|-------|
+| CRITICAL | path/file.ts:42 | Brief description |
+| WARNING | path/file.ts:78 | Brief description |
+
+If no issues: "No issues found."
+
+### Detailed Findings
+For each issue:
+- **File:** \`path/to/file.ts:line\`
+- **Confidence:** X%
+- **Problem:** What's wrong and why it matters
+- **Suggestion:** Recommended fix with code snippet
+
+### Recommendation
+One of: **APPROVE** | **APPROVE WITH SUGGESTIONS** | **NEEDS CHANGES** | **NEEDS DISCUSSION**`,
+	},
+	// kilocode_change end
 ] as const

packages/types/src/vscode-extension-host.ts

@@ -256,7 +256,8 @@ export interface ExtensionMessage {
 		| "customToolsResult"
 		| "modes"
 		| "taskWithAggregatedCosts"
-		| "skillsData" // kilocode_change: Skills data response
+		| "skillsData"
+		| "askReviewScope" // kilocode_change: Skills data response
 	text?: string
 	// kilocode_change start
 	completionRequestId?: string // Correlation ID from request
@@ -442,6 +443,23 @@ export interface ExtensionMessage {
 		childrenCost: number
 	}
 	historyItem?: HistoryItem
+	// kilocode_change start: Review mode
+	reviewScopeInfo?: {
+		uncommitted: {
+			available: boolean
+			fileCount: number
+			filePreview?: string[]
+		}
+		branch: {
+			available: boolean
+			currentBranch: string
+			baseBranch: string
+			fileCount: number
+			filePreview?: string[]
+		}
+		error?: string
+	}
+	// kilocode_change end: Review mode
 }
 
 export type ExtensionState = Pick<
@@ -929,7 +947,8 @@ export interface WebviewMessage {
 		| "requestModes"
 		| "switchMode"
 		| "debugSetting"
-		| "refreshSkills" // kilocode_change: Request skills data refresh
+		| "refreshSkills"
+		| "reviewScopeSelected" // kilocode_change: Request skills data refresh
 	text?: string
 	suggestionLength?: number // kilocode_change: Length of accepted suggestion for telemetry
 	completionRequestId?: string // kilocode_change
@@ -1049,6 +1068,9 @@ export interface WebviewMessage {
 		codebaseIndexOpenRouterApiKey?: string
 	}
 	updatedSettings?: RooCodeSettings
+	// kilocode_change start: Review mode
+	reviewScope?: "uncommitted" | "branch"
+	// kilocode_change end: Review mode
 }
 
 // kilocode_change: Create discriminated union for type-safe messages

src/core/webview/ClineProvider.ts

@@ -1493,6 +1493,12 @@ export class ClineProvider
 		}
 
 		await this.postStateToWebview()
+
+		// kilocode_change start: Review mode scope selection
+		if (newMode === "review") {
+			await this.triggerReviewScopeSelection()
+		}
+		// kilocode_change end
 	}
 
 	// Provider Profile Management
@@ -3379,6 +3385,64 @@ export class ClineProvider
 		await this.setValues({ mode })
 	}
 
+	// kilocode_change start: Review mode
+	/**
+	 * Triggers the review scope selection UI
+	 * Called when user enters review mode
+	 */
+	public async triggerReviewScopeSelection(): Promise<void> {
+		try {
+			const cwd = getWorkspacePath()
+			if (!cwd) {
+				this.log("Cannot start review: no workspace folder open")
+				return
+			}
+
+			const { ReviewService } = await import("../../services/review")
+			const reviewService = new ReviewService({ cwd })
+			const scopeInfo = await reviewService.getScopeInfo()
+
+			await this.postMessageToWebview({
+				type: "askReviewScope",
+				reviewScopeInfo: scopeInfo,
+			})
+		} catch (error) {
+			this.log(
+				`Error triggering review scope selection: ${error instanceof Error ? error.message : String(error)}`,
+			)
+		}
+	}
+
+	/**
+	 * Handles the user's review scope selection
+	 * Gets lightweight summary and starts the review task
+	 * The agent will dynamically explore changes using tools
+	 */
+	public async handleReviewScopeSelected(scope: "uncommitted" | "branch"): Promise<void> {
+		try {
+			const cwd = getWorkspacePath()
+			if (!cwd) {
+				this.log("Cannot start review: no workspace folder open")
+				vscode.window.showErrorMessage("Cannot start review: no workspace folder open")
+				return
+			}
+
+			const { ReviewService, buildReviewPrompt } = await import("../../services/review")
+			const reviewService = new ReviewService({ cwd })
+
+			// Get lightweight summary - agent will explore details with tools
+			const summary = await reviewService.getReviewSummary(scope)
+
+			// Build the review prompt and start the task
+			// Let the agent handle cases with no changes - it can explain and offer alternatives
+			const reviewPrompt = buildReviewPrompt(summary)
+			await this.createTask(reviewPrompt)
+		} catch (error) {
+			this.log(`Error handling review scope selection: ${error instanceof Error ? error.message : String(error)}`)
+		}
+	}
+	// kilocode_change end: Review mode
+
 	// Provider Profiles
 
 	public async getProviderProfiles(): Promise<{ name: string; provider?: string }[]> {

src/core/webview/webviewMessageHandler.ts

@@ -4594,6 +4594,16 @@ export const webviewMessageHandler = async (
 			break
 		}
 
+		// kilocode_change start: Review mode scope selection
+		case "reviewScopeSelected": {
+			const scope = message.reviewScope
+			if (scope === "uncommitted" || scope === "branch") {
+				await provider.handleReviewScopeSelected(scope)
+			}
+			break
+		}
+		// kilocode_change end
+
 		default: {
 			// console.log(`Unhandled message type: ${message.type}`)
 			//

src/services/review/ReviewService.ts

@@ -0,0 +1,168 @@
+// kilocode_change - new file
+
+/**
+ * Review Service
+ *
+ * Lightweight service for gathering review scope information.
+ * The actual review is done dynamically by the agent using tools.
+ */
+
+import type { ReviewSummary, ReviewScopeInfo, ReviewServiceOptions, FileSummary, FileStatus } from "./types"
+import {
+	getCurrentBranch,
+	detectBaseBranch,
+	hasUncommittedChanges,
+	getUncommittedFiles,
+	getBranchFilesChanged,
+	isOnBaseBranch,
+	type GitFileChange,
+} from "../../utils/git"
+
+/**
+ * Converts git status code to FileStatus
+ */
+function gitStatusToFileStatus(status: string): FileStatus {
+	const upper = status.toUpperCase()
+	if (upper === "A" || upper === "?") return "A"
+	if (upper === "M") return "M"
+	if (upper === "D") return "D"
+	if (upper === "R") return "R"
+	if (upper === "C") return "C"
+	if (upper === "U") return "U"
+	return "M" // Default to modified
+}
+
+/**
+ * Converts GitFileChange array to FileSummary array
+ */
+function toFileSummaries(files: GitFileChange[]): FileSummary[] {
+	return files.map((file) => ({
+		path: file.path,
+		status: gitStatusToFileStatus(file.status),
+		oldPath: file.oldPath,
+	}))
+}
+
+/**
+ * ReviewService - Provides review scope information for the UI
+ * and lightweight summaries for the agent to start reviews
+ */
+export class ReviewService {
+	private cwd: string
+
+	constructor(options: ReviewServiceOptions) {
+		this.cwd = options.cwd
+	}
+
+	/**
+	 * Gets information about available review scopes
+	 * Used by UI to show preview before user selects
+	 */
+	async getScopeInfo(): Promise<ReviewScopeInfo> {
+		try {
+			// Get current branch
+			const currentBranch = (await getCurrentBranch(this.cwd)) || "HEAD"
+
+			// Check uncommitted changes
+			const hasUncommitted = await hasUncommittedChanges(this.cwd)
+			const uncommittedFiles = hasUncommitted ? await getUncommittedFiles(this.cwd) : []
+
+			// Check branch diff - available as long as not on base branch
+			const onBaseBranch = await isOnBaseBranch(this.cwd)
+			const baseBranch = await detectBaseBranch(this.cwd)
+			const branchFiles = !onBaseBranch ? await getBranchFilesChanged(this.cwd, baseBranch) : []
+
+			const result = {
+				uncommitted: {
+					available: hasUncommitted,
+					fileCount: uncommittedFiles.length,
+					filePreview: uncommittedFiles.slice(0, 5).map((f) => f.path),
+				},
+				branch: {
+					available: !onBaseBranch,
+					currentBranch,
+					baseBranch,
+					fileCount: branchFiles.length,
+					filePreview: branchFiles.slice(0, 5).map((f) => f.path),
+				},
+			}
+
+			return result
+		} catch (error) {
+			console.error("Error getting scope info:", error)
+			return {
+				uncommitted: {
+					available: false,
+					fileCount: 0,
+				},
+				branch: {
+					available: false,
+					currentBranch: "unknown",
+					baseBranch: "main",
+					fileCount: 0,
+				},
+				error: error instanceof Error ? error.message : String(error),
+			}
+		}
+	}
+
+	/**
+	 * Gets a lightweight review summary for the selected scope
+	 * This provides minimal context - the agent will explore details with tools
+	 */
+	async getReviewSummary(scope: "uncommitted" | "branch"): Promise<ReviewSummary> {
+		try {
+			const currentBranch = (await getCurrentBranch(this.cwd)) || "HEAD"
+
+			if (scope === "uncommitted") {
+				return this.getUncommittedSummary(currentBranch)
+			} else {
+				return this.getBranchSummary(currentBranch)
+			}
+		} catch (error) {
+			console.error(`Error getting review summary for ${scope}:`, error)
+			return {
+				scope,
+				currentBranch: "unknown",
+				files: [],
+				totalFiles: 0,
+				hasChanges: false,
+				error: error instanceof Error ? error.message : String(error),
+			}
+		}
+	}
+
+	/**
+	 * Gets summary for uncommitted changes
+	 */
+	private async getUncommittedSummary(currentBranch: string): Promise<ReviewSummary> {
+		const gitFiles = await getUncommittedFiles(this.cwd)
+		const files = toFileSummaries(gitFiles)
+
+		return {
+			scope: "uncommitted",
+			currentBranch,
+			files,
+			totalFiles: files.length,
+			hasChanges: files.length > 0,
+		}
+	}
+
+	/**
+	 * Gets summary for branch comparison
+	 */
+	private async getBranchSummary(currentBranch: string): Promise<ReviewSummary> {
+		const baseBranch = await detectBaseBranch(this.cwd)
+		const gitFiles = await getBranchFilesChanged(this.cwd, baseBranch)
+		const files = toFileSummaries(gitFiles)
+
+		return {
+			scope: "branch",
+			currentBranch,
+			baseBranch,
+			files,
+			totalFiles: files.length,
+			hasChanges: files.length > 0,
+		}
+	}
+}