Skip to content

Commit c6b689a

Browse files
committed
Remove unused ClineSay and -Ask type variants
1 parent 62a7bc7 commit c6b689a

File tree

6 files changed

+3
-23
lines changed

6 files changed

+3
-23
lines changed

src/exports/roo-code.d.ts

Lines changed: 0 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -339,12 +339,10 @@ type ClineMessage = {
339339
| "mistake_limit_reached"
340340
| "browser_action_launch"
341341
| "use_mcp_server"
342-
| "finishTask"
343342
)
344343
| undefined
345344
say?:
346345
| (
347-
| "task"
348346
| "error"
349347
| "api_req_started"
350348
| "api_req_finished"
@@ -357,15 +355,11 @@ type ClineMessage = {
357355
| "user_feedback"
358356
| "user_feedback_diff"
359357
| "command_output"
360-
| "tool"
361358
| "shell_integration_warning"
362359
| "browser_action"
363360
| "browser_action_result"
364-
| "command"
365361
| "mcp_server_request_started"
366362
| "mcp_server_response"
367-
| "new_task_started"
368-
| "new_task"
369363
| "checkpoint_saved"
370364
| "rooignore_error"
371365
)

src/exports/types.ts

Lines changed: 0 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -344,12 +344,10 @@ type ClineMessage = {
344344
| "mistake_limit_reached"
345345
| "browser_action_launch"
346346
| "use_mcp_server"
347-
| "finishTask"
348347
)
349348
| undefined
350349
say?:
351350
| (
352-
| "task"
353351
| "error"
354352
| "api_req_started"
355353
| "api_req_finished"
@@ -362,15 +360,11 @@ type ClineMessage = {
362360
| "user_feedback"
363361
| "user_feedback_diff"
364362
| "command_output"
365-
| "tool"
366363
| "shell_integration_warning"
367364
| "browser_action"
368365
| "browser_action_result"
369-
| "command"
370366
| "mcp_server_request_started"
371367
| "mcp_server_response"
372-
| "new_task_started"
373-
| "new_task"
374368
| "checkpoint_saved"
375369
| "rooignore_error"
376370
)

src/schemas/index.ts

Lines changed: 0 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -701,7 +701,6 @@ export const clineAsks = [
701701
"mistake_limit_reached",
702702
"browser_action_launch",
703703
"use_mcp_server",
704-
"finishTask",
705704
] as const
706705

707706
export const clineAskSchema = z.enum(clineAsks)
@@ -711,7 +710,6 @@ export type ClineAsk = z.infer<typeof clineAskSchema>
711710
// ClineSay
712711

713712
export const clineSays = [
714-
"task",
715713
"error",
716714
"api_req_started",
717715
"api_req_finished",
@@ -724,15 +722,11 @@ export const clineSays = [
724722
"user_feedback",
725723
"user_feedback_diff",
726724
"command_output",
727-
"tool",
728725
"shell_integration_warning",
729726
"browser_action",
730727
"browser_action_result",
731-
"command",
732728
"mcp_server_request_started",
733729
"mcp_server_response",
734-
"new_task_started",
735-
"new_task",
736730
"checkpoint_saved",
737731
"rooignore_error",
738732
] as const

webview-ui/src/__tests__/ContextWindowProgress.test.tsx

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -40,7 +40,7 @@ describe("ContextWindowProgress", () => {
4040
const defaultTask = {
4141
ts: Date.now(),
4242
type: "say" as const,
43-
say: "task" as const,
43+
say: "text" as const,
4444
text: "Test task",
4545
}
4646

webview-ui/src/components/chat/ChatRow.tsx

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -245,11 +245,11 @@ export const ChatRowContent = ({
245245
}
246246

247247
const tool = useMemo(() => {
248-
if (message.ask === "tool" || message.say === "tool") {
248+
if (message.ask === "tool") {
249249
return JSON.parse(message.text || "{}") as ClineSayTool
250250
}
251251
return null
252-
}, [message.ask, message.say, message.text])
252+
}, [message.ask, message.text])
253253

254254
const followUpData = useMemo(() => {
255255
if (message.type === "ask" && message.ask === "followup" && !message.partial) {

webview-ui/src/components/chat/ChatView.tsx

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -247,7 +247,6 @@ const ChatView = ({ isHidden, showAnnouncement, hideAnnouncement, showHistoryVie
247247
}
248248
break
249249
case "api_req_finished":
250-
case "task":
251250
case "error":
252251
case "text":
253252
case "browser_action":
@@ -256,7 +255,6 @@ const ChatView = ({ isHidden, showAnnouncement, hideAnnouncement, showHistoryVie
256255
case "mcp_server_request_started":
257256
case "mcp_server_response":
258257
case "completion_result":
259-
case "tool":
260258
break
261259
}
262260
break

0 commit comments

Comments
 (0)