[PROTOBUS] Move commitSearch to protobus (RooCodeInc#3229)

* commitSearch protobus migration

* rename

* one small change to comments

* moved GitCommmit proto mapping to proto-conversions

Author: canvrno

.changeset/funny-numbers-trade.md

@@ -0,0 +1,5 @@
+---
+"claude-dev": patch
+---
+
+searchCommits protobus migration

proto/file.proto

@@ -10,7 +10,7 @@ import "common.proto";
 service FileService {
   // Opens a file in the editor
   rpc openFile(StringRequest) returns (Empty);
-  
+ 
   // Opens an image in the system viewer
   rpc openImage(StringRequest) returns (Empty);
 
@@ -19,6 +19,23 @@ service FileService {
 
   // Creates a rule file from either global or workspace rules directory
   rpc createRuleFile(RuleFileRequest) returns (RuleFile);
+  
+  // Search git commits in the workspace
+  rpc searchCommits(StringRequest) returns (GitCommits);
+}
+
+// Response for searchCommits
+message GitCommits {
+  repeated GitCommit commits = 1;
+}
+
+// Represents a Git commit
+message GitCommit {
+  string hash = 1;
+  string short_hash = 2;
+  string subject = 3;
+  string author = 4;
+  string date = 5;
 }
 
 // Unified request for all rule file operations

src/core/controller/file/methods.ts

@@ -7,6 +7,7 @@ import { createRuleFile } from "./createRuleFile"
 import { deleteRuleFile } from "./deleteRuleFile"
 import { openFile } from "./openFile"
 import { openImage } from "./openImage"
+import { searchCommits } from "./searchCommits"
 
 // Register all file service methods
 export function registerAllMethods(): void {
@@ -15,4 +16,5 @@ export function registerAllMethods(): void {
 	registerMethod("deleteRuleFile", deleteRuleFile)
 	registerMethod("openFile", openFile)
 	registerMethod("openImage", openImage)
+	registerMethod("searchCommits", searchCommits)
 }

src/core/controller/file/searchCommits.ts

@@ -0,0 +1,31 @@
+import { Controller } from ".."
+import { GitCommits } from "@shared/proto/file"
+import { StringRequest } from "@shared/proto/common"
+import { searchCommits as searchCommitsUtil } from "@utils/git"
+import { getWorkspacePath } from "@utils/path"
+import { FileMethodHandler } from "./index"
+import { convertGitCommitsToProtoGitCommits } from "@shared/proto-conversions/file/git-commit-conversion"
+
+/**
+ * Searches for git commits in the workspace repository
+ * @param controller The controller instance
+ * @param request The request message containing the search query in the 'value' field
+ * @returns GitCommits containing the matching commits
+ */
+export const searchCommits: FileMethodHandler = async (controller: Controller, request: StringRequest): Promise<GitCommits> => {
+	const cwd = getWorkspacePath()
+	if (!cwd) {
+		return GitCommits.create({ commits: [] })
+	}
+
+	try {
+		const commits = await searchCommitsUtil(request.value || "", cwd)
+
+		const protoCommits = convertGitCommitsToProtoGitCommits(commits)
+
+		return GitCommits.create({ commits: protoCommits })
+	} catch (error) {
+		console.error(`Error searching commits: ${JSON.stringify(error)}`)
+		return GitCommits.create({ commits: [] })
+	}
+}

src/core/controller/index.ts

@@ -538,21 +538,6 @@ export class Controller {
 				this.mcpHub?.sendLatestMcpServers()
 				break
 			}
-			case "searchCommits": {
-				const cwd = vscode.workspace.workspaceFolders?.map((folder) => folder.uri.fsPath).at(0)
-				if (cwd) {
-					try {
-						const commits = await searchCommits(message.text || "", cwd)
-						await this.postMessageToWebview({
-							type: "commitSearchResults",
-							commits,
-						})
-					} catch (error) {
-						console.error(`Error searching commits: ${JSON.stringify(error)}`)
-					}
-				}
-				break
-			}
 			case "openExtensionSettings": {
 				const settingsFilter = message.text || ""
 				await vscode.commands.executeCommand(

src/shared/proto-conversions/file/git-commit-conversion.ts

@@ -0,0 +1,28 @@
+import { GitCommit as ProtoGitCommit } from "@shared/proto/file"
+import { GitCommit } from "@utils/git"
+
+/**
+ * Converts domain GitCommit objects to proto GitCommit objects
+ */
+export function convertGitCommitsToProtoGitCommits(commits: GitCommit[]): ProtoGitCommit[] {
+	return commits.map((commit) => ({
+		hash: commit.hash,
+		shortHash: commit.shortHash,
+		subject: commit.subject,
+		author: commit.author,
+		date: commit.date,
+	}))
+}
+
+/**
+ * Converts proto GitCommit objects to domain GitCommit objects
+ */
+export function convertProtoGitCommitsToGitCommits(protoCommits: ProtoGitCommit[]): GitCommit[] {
+	return protoCommits.map((protoCommit) => ({
+		hash: protoCommit.hash,
+		shortHash: protoCommit.shortHash,
+		subject: protoCommit.subject,
+		author: protoCommit.author,
+		date: protoCommit.date,
+	}))
+}