PROTO refactor reportbug (RooCodeInc#3485)

* protos report bug

* changeset

Author: 0xToshii

.changeset/rude-cameras-approve.md

@@ -0,0 +1,5 @@
+---
+"claude-dev": patch
+---
+
+move slash report bug to protos / grpc

proto/build-proto.js

@@ -29,6 +29,7 @@ const serviceNameMap = {
 	task: "cline.TaskService",
 	web: "cline.WebService",
 	models: "cline.ModelsService",
+	slash: "cline.SlashService",
 	// Add new services here - no other code changes needed!
 }
 const serviceDirs = Object.keys(serviceNameMap).map((serviceKey) => path.join(ROOT_DIR, "src", "core", "controller", serviceKey))

proto/slash.proto

@@ -0,0 +1,13 @@
+syntax = "proto3";
+
+package cline;
+option java_package = "bot.cline.proto";
+option java_multiple_files = true;
+
+import "common.proto";
+
+// SlashService provides methods for managing slash
+service SlashService {
+  // Sends button click message
+  rpc reportBug(StringRequest) returns (Empty);
+}

src/core/controller/grpc-service-config.ts

@@ -12,6 +12,7 @@ import { handleStateServiceRequest, handleStateServiceStreamingRequest } from ".
 import { handleTaskServiceRequest, handleTaskServiceStreamingRequest } from "./task/index"
 import { handleWebServiceRequest, handleWebServiceStreamingRequest } from "./web/index"
 import { handleModelsServiceRequest, handleModelsServiceStreamingRequest } from "./models/index"
+import { handleSlashServiceRequest, handleSlashServiceStreamingRequest } from "./slash/index"
 
 /**
  * Configuration for a service handler
@@ -67,4 +68,8 @@ export const serviceHandlers: Record<string, ServiceHandlerConfig> = {
 		requestHandler: handleModelsServiceRequest,
 		streamingHandler: handleModelsServiceStreamingRequest,
 	},
+	"cline.SlashService": {
+		requestHandler: handleSlashServiceRequest,
+		streamingHandler: handleSlashServiceStreamingRequest,
+	},
 }

src/core/controller/index.ts

@@ -274,9 +274,6 @@ export class Controller {
 			case "condense":
 				this.task?.handleWebviewAskResponse("yesButtonClicked")
 				break
-			case "reportBug":
-				this.task?.handleWebviewAskResponse("yesButtonClicked")
-				break
 			case "apiConfiguration":
 				if (message.apiConfiguration) {
 					await updateApiConfiguration(this.context, message.apiConfiguration)

src/core/controller/slash/index.ts

@@ -0,0 +1,22 @@
+// AUTO-GENERATED FILE - DO NOT MODIFY DIRECTLY
+// Generated by proto/build-proto.js
+
+import { createServiceRegistry, ServiceMethodHandler, StreamingMethodHandler } from "../grpc-service"
+import { StreamingResponseHandler } from "../grpc-handler"
+import { registerAllMethods } from "./methods"
+
+// Create slash service registry
+const slashService = createServiceRegistry("slash")
+
+// Export the method handler types and registration function
+export type SlashMethodHandler = ServiceMethodHandler
+export type SlashStreamingMethodHandler = StreamingMethodHandler
+export const registerMethod = slashService.registerMethod
+
+// Export the request handlers
+export const handleSlashServiceRequest = slashService.handleRequest
+export const handleSlashServiceStreamingRequest = slashService.handleStreamingRequest
+export const isStreamingMethod = slashService.isStreamingMethod
+
+// Register all slash methods
+registerAllMethods()

src/core/controller/slash/methods.ts

@@ -0,0 +1,12 @@
+// AUTO-GENERATED FILE - DO NOT MODIFY DIRECTLY
+// Generated by proto/build-proto.js
+
+// Import all method implementations
+import { registerMethod } from "./index"
+import { reportBug } from "./reportBug"
+
+// Register all slash service methods
+export function registerAllMethods(): void {
+	// Register each method with the registry
+	registerMethod("reportBug", reportBug)
+}

src/core/controller/slash/reportBug.ts

@@ -0,0 +1,10 @@
+import { Controller } from ".."
+import { StringRequest, Empty } from "../../../shared/proto/common"
+
+/**
+ * Report bug slash command logic
+ */
+export async function reportBug(controller: Controller, request: StringRequest): Promise<Empty> {
+	await controller.task?.handleWebviewAskResponse("yesButtonClicked")
+	return Empty.create()
+}

src/core/task/index.ts

@@ -3259,7 +3259,7 @@ export class Task {
 									await this.say("user_feedback", text ?? "", images)
 									pushToolResult(
 										formatResponse.toolResult(
-											`The user provided feedback on the Github issue generated:\n<feedback>\n${text}\n</feedback>`,
+											`The user did not submit the bug, and provided feedback on the Github issue generated instead:\n<feedback>\n${text}\n</feedback>`,
 											images,
 										),
 									)