mirror of
https://github.com/pacnpal/Roo-Code.git
synced 2025-12-20 12:21:13 -05:00
Revert some whitespace changes
This commit is contained in:
@@ -390,7 +390,8 @@ export class Cline {
|
|||||||
async sayAndCreateMissingParamError(toolName: ToolUseName, paramName: string, relPath?: string) {
|
async sayAndCreateMissingParamError(toolName: ToolUseName, paramName: string, relPath?: string) {
|
||||||
await this.say(
|
await this.say(
|
||||||
"error",
|
"error",
|
||||||
`Cline tried to use ${toolName}${relPath ? ` for '${relPath.toPosix()}'` : ""
|
`Cline tried to use ${toolName}${
|
||||||
|
relPath ? ` for '${relPath.toPosix()}'` : ""
|
||||||
} without value for required parameter '${paramName}'. Retrying...`,
|
} without value for required parameter '${paramName}'. Retrying...`,
|
||||||
)
|
)
|
||||||
return formatResponse.toolError(formatResponse.missingToolParameterError(paramName))
|
return formatResponse.toolError(formatResponse.missingToolParameterError(paramName))
|
||||||
@@ -625,7 +626,8 @@ export class Cline {
|
|||||||
newUserContent.push({
|
newUserContent.push({
|
||||||
type: "text",
|
type: "text",
|
||||||
text:
|
text:
|
||||||
`[TASK RESUMPTION] This task was interrupted ${agoText}. It may or may not be complete, so please reassess the task context. Be aware that the project state may have changed since then. The current working directory is now '${cwd.toPosix()}'. If the task has not been completed, retry the last step before interruption and proceed with completing the task.\n\nNote: If you previously attempted a tool use that the user did not provide a result for, you should assume the tool use was not successful and assess whether you should retry. If the last tool was a browser_action, the browser has been closed and you must launch a new browser if needed.${wasRecent
|
`[TASK RESUMPTION] This task was interrupted ${agoText}. It may or may not be complete, so please reassess the task context. Be aware that the project state may have changed since then. The current working directory is now '${cwd.toPosix()}'. If the task has not been completed, retry the last step before interruption and proceed with completing the task.\n\nNote: If you previously attempted a tool use that the user did not provide a result for, you should assume the tool use was not successful and assess whether you should retry. If the last tool was a browser_action, the browser has been closed and you must launch a new browser if needed.${
|
||||||
|
wasRecent
|
||||||
? "\n\nIMPORTANT: If the last tool use was a write_to_file that was interrupted, the file was reverted back to its original state before the interrupted edit, and you do NOT need to re-read the file as you already have its up-to-date contents."
|
? "\n\nIMPORTANT: If the last tool use was a write_to_file that was interrupted, the file was reverted back to its original state before the interrupted edit, and you do NOT need to re-read the file as you already have its up-to-date contents."
|
||||||
: ""
|
: ""
|
||||||
}` +
|
}` +
|
||||||
@@ -741,7 +743,8 @@ export class Cline {
|
|||||||
return [
|
return [
|
||||||
true,
|
true,
|
||||||
formatResponse.toolResult(
|
formatResponse.toolResult(
|
||||||
`Command is still running in the user's terminal.${result.length > 0 ? `\nHere's the output so far:\n${result}` : ""
|
`Command is still running in the user's terminal.${
|
||||||
|
result.length > 0 ? `\nHere's the output so far:\n${result}` : ""
|
||||||
}\n\nThe user provided the following feedback:\n<feedback>\n${userFeedback.text}\n</feedback>`,
|
}\n\nThe user provided the following feedback:\n<feedback>\n${userFeedback.text}\n</feedback>`,
|
||||||
userFeedback.images,
|
userFeedback.images,
|
||||||
),
|
),
|
||||||
@@ -753,7 +756,8 @@ export class Cline {
|
|||||||
} else {
|
} else {
|
||||||
return [
|
return [
|
||||||
false,
|
false,
|
||||||
`Command is still running in the user's terminal.${result.length > 0 ? `\nHere's the output so far:\n${result}` : ""
|
`Command is still running in the user's terminal.${
|
||||||
|
result.length > 0 ? `\nHere's the output so far:\n${result}` : ""
|
||||||
}\n\nYou will be updated on the terminal status and new output in the future.`,
|
}\n\nYou will be updated on the terminal status and new output in the future.`,
|
||||||
]
|
]
|
||||||
}
|
}
|
||||||
@@ -931,7 +935,8 @@ export class Cline {
|
|||||||
case "apply_diff":
|
case "apply_diff":
|
||||||
return `[${block.name} for '${block.params.path}']`
|
return `[${block.name} for '${block.params.path}']`
|
||||||
case "search_files":
|
case "search_files":
|
||||||
return `[${block.name} for '${block.params.regex}'${block.params.file_pattern ? ` in '${block.params.file_pattern}'` : ""
|
return `[${block.name} for '${block.params.regex}'${
|
||||||
|
block.params.file_pattern ? ` in '${block.params.file_pattern}'` : ""
|
||||||
}]`
|
}]`
|
||||||
case "list_files":
|
case "list_files":
|
||||||
return `[${block.name} for '${block.params.path}']`
|
return `[${block.name} for '${block.params.path}']`
|
||||||
@@ -1118,7 +1123,7 @@ export class Cline {
|
|||||||
if (block.partial) {
|
if (block.partial) {
|
||||||
// update gui message
|
// update gui message
|
||||||
const partialMessage = JSON.stringify(sharedMessageProps)
|
const partialMessage = JSON.stringify(sharedMessageProps)
|
||||||
await this.ask("tool", partialMessage, block.partial).catch(() => { })
|
await this.ask("tool", partialMessage, block.partial).catch(() => {})
|
||||||
// update editor
|
// update editor
|
||||||
if (!this.diffViewProvider.isEditing) {
|
if (!this.diffViewProvider.isEditing) {
|
||||||
// open the editor and prepare to stream content in
|
// open the editor and prepare to stream content in
|
||||||
@@ -1154,7 +1159,7 @@ export class Cline {
|
|||||||
if (!this.diffViewProvider.isEditing) {
|
if (!this.diffViewProvider.isEditing) {
|
||||||
// show gui message before showing edit animation
|
// show gui message before showing edit animation
|
||||||
const partialMessage = JSON.stringify(sharedMessageProps)
|
const partialMessage = JSON.stringify(sharedMessageProps)
|
||||||
await this.ask("tool", partialMessage, true).catch(() => { }) // sending true for partial even though it's not a partial, this shows the edit row before the content is streamed into the editor
|
await this.ask("tool", partialMessage, true).catch(() => {}) // sending true for partial even though it's not a partial, this shows the edit row before the content is streamed into the editor
|
||||||
await this.diffViewProvider.open(relPath)
|
await this.diffViewProvider.open(relPath)
|
||||||
}
|
}
|
||||||
await this.diffViewProvider.update(everyLineHasLineNumbers(newContent) ? stripLineNumbers(newContent) : newContent, true)
|
await this.diffViewProvider.update(everyLineHasLineNumbers(newContent) ? stripLineNumbers(newContent) : newContent, true)
|
||||||
@@ -1252,7 +1257,7 @@ export class Cline {
|
|||||||
if (block.partial) {
|
if (block.partial) {
|
||||||
// update gui message
|
// update gui message
|
||||||
const partialMessage = JSON.stringify(sharedMessageProps)
|
const partialMessage = JSON.stringify(sharedMessageProps)
|
||||||
await this.ask("tool", partialMessage, block.partial).catch(() => { })
|
await this.ask("tool", partialMessage, block.partial).catch(() => {})
|
||||||
break
|
break
|
||||||
} else {
|
} else {
|
||||||
if (!relPath) {
|
if (!relPath) {
|
||||||
@@ -1367,7 +1372,7 @@ export class Cline {
|
|||||||
...sharedMessageProps,
|
...sharedMessageProps,
|
||||||
content: undefined,
|
content: undefined,
|
||||||
} satisfies ClineSayTool)
|
} satisfies ClineSayTool)
|
||||||
await this.ask("tool", partialMessage, block.partial).catch(() => { })
|
await this.ask("tool", partialMessage, block.partial).catch(() => {})
|
||||||
break
|
break
|
||||||
} else {
|
} else {
|
||||||
if (!relPath) {
|
if (!relPath) {
|
||||||
@@ -1409,7 +1414,7 @@ export class Cline {
|
|||||||
...sharedMessageProps,
|
...sharedMessageProps,
|
||||||
content: "",
|
content: "",
|
||||||
} satisfies ClineSayTool)
|
} satisfies ClineSayTool)
|
||||||
await this.ask("tool", partialMessage, block.partial).catch(() => { })
|
await this.ask("tool", partialMessage, block.partial).catch(() => {})
|
||||||
break
|
break
|
||||||
} else {
|
} else {
|
||||||
if (!relDirPath) {
|
if (!relDirPath) {
|
||||||
@@ -1449,7 +1454,7 @@ export class Cline {
|
|||||||
...sharedMessageProps,
|
...sharedMessageProps,
|
||||||
content: "",
|
content: "",
|
||||||
} satisfies ClineSayTool)
|
} satisfies ClineSayTool)
|
||||||
await this.ask("tool", partialMessage, block.partial).catch(() => { })
|
await this.ask("tool", partialMessage, block.partial).catch(() => {})
|
||||||
break
|
break
|
||||||
} else {
|
} else {
|
||||||
if (!relDirPath) {
|
if (!relDirPath) {
|
||||||
@@ -1494,7 +1499,7 @@ export class Cline {
|
|||||||
...sharedMessageProps,
|
...sharedMessageProps,
|
||||||
content: "",
|
content: "",
|
||||||
} satisfies ClineSayTool)
|
} satisfies ClineSayTool)
|
||||||
await this.ask("tool", partialMessage, block.partial).catch(() => { })
|
await this.ask("tool", partialMessage, block.partial).catch(() => {})
|
||||||
break
|
break
|
||||||
} else {
|
} else {
|
||||||
if (!relDirPath) {
|
if (!relDirPath) {
|
||||||
@@ -1549,7 +1554,7 @@ export class Cline {
|
|||||||
"browser_action_launch",
|
"browser_action_launch",
|
||||||
removeClosingTag("url", url),
|
removeClosingTag("url", url),
|
||||||
block.partial
|
block.partial
|
||||||
).catch(() => { })
|
).catch(() => {})
|
||||||
} else {
|
} else {
|
||||||
await this.say(
|
await this.say(
|
||||||
"browser_action",
|
"browser_action",
|
||||||
@@ -1649,7 +1654,8 @@ export class Cline {
|
|||||||
await this.say("browser_action_result", JSON.stringify(browserActionResult))
|
await this.say("browser_action_result", JSON.stringify(browserActionResult))
|
||||||
pushToolResult(
|
pushToolResult(
|
||||||
formatResponse.toolResult(
|
formatResponse.toolResult(
|
||||||
`The browser action has been executed. The console logs and screenshot have been captured for your analysis.\n\nConsole logs:\n${browserActionResult.logs || "(No new logs)"
|
`The browser action has been executed. The console logs and screenshot have been captured for your analysis.\n\nConsole logs:\n${
|
||||||
|
browserActionResult.logs || "(No new logs)"
|
||||||
}\n\n(REMEMBER: if you need to proceed to using non-\`browser_action\` tools or launch a new browser, you MUST first close this browser. For example, if after analyzing the logs and screenshot you need to edit a file, you must first close the browser before you can use the write_to_file tool.)`,
|
}\n\n(REMEMBER: if you need to proceed to using non-\`browser_action\` tools or launch a new browser, you MUST first close this browser. For example, if after analyzing the logs and screenshot you need to edit a file, you must first close the browser before you can use the write_to_file tool.)`,
|
||||||
browserActionResult.screenshot ? [browserActionResult.screenshot] : [],
|
browserActionResult.screenshot ? [browserActionResult.screenshot] : [],
|
||||||
),
|
),
|
||||||
@@ -1676,7 +1682,7 @@ export class Cline {
|
|||||||
try {
|
try {
|
||||||
if (block.partial) {
|
if (block.partial) {
|
||||||
await this.ask("command", removeClosingTag("command", command), block.partial).catch(
|
await this.ask("command", removeClosingTag("command", command), block.partial).catch(
|
||||||
() => { }
|
() => {}
|
||||||
)
|
)
|
||||||
break
|
break
|
||||||
} else {
|
} else {
|
||||||
@@ -1717,7 +1723,7 @@ export class Cline {
|
|||||||
toolName: removeClosingTag("tool_name", tool_name),
|
toolName: removeClosingTag("tool_name", tool_name),
|
||||||
arguments: removeClosingTag("arguments", mcp_arguments),
|
arguments: removeClosingTag("arguments", mcp_arguments),
|
||||||
} satisfies ClineAskUseMcpServer)
|
} satisfies ClineAskUseMcpServer)
|
||||||
await this.ask("use_mcp_server", partialMessage, block.partial).catch(() => { })
|
await this.ask("use_mcp_server", partialMessage, block.partial).catch(() => {})
|
||||||
break
|
break
|
||||||
} else {
|
} else {
|
||||||
if (!server_name) {
|
if (!server_name) {
|
||||||
@@ -1810,7 +1816,7 @@ export class Cline {
|
|||||||
serverName: removeClosingTag("server_name", server_name),
|
serverName: removeClosingTag("server_name", server_name),
|
||||||
uri: removeClosingTag("uri", uri),
|
uri: removeClosingTag("uri", uri),
|
||||||
} satisfies ClineAskUseMcpServer)
|
} satisfies ClineAskUseMcpServer)
|
||||||
await this.ask("use_mcp_server", partialMessage, block.partial).catch(() => { })
|
await this.ask("use_mcp_server", partialMessage, block.partial).catch(() => {})
|
||||||
break
|
break
|
||||||
} else {
|
} else {
|
||||||
if (!server_name) {
|
if (!server_name) {
|
||||||
@@ -1866,7 +1872,7 @@ export class Cline {
|
|||||||
try {
|
try {
|
||||||
if (block.partial) {
|
if (block.partial) {
|
||||||
await this.ask("followup", removeClosingTag("question", question), block.partial).catch(
|
await this.ask("followup", removeClosingTag("question", question), block.partial).catch(
|
||||||
() => { },
|
() => {},
|
||||||
)
|
)
|
||||||
break
|
break
|
||||||
} else {
|
} else {
|
||||||
@@ -1925,7 +1931,7 @@ export class Cline {
|
|||||||
"command",
|
"command",
|
||||||
removeClosingTag("command", command),
|
removeClosingTag("command", command),
|
||||||
block.partial,
|
block.partial,
|
||||||
).catch(() => { })
|
).catch(() => {})
|
||||||
} else {
|
} else {
|
||||||
// last message is completion_result
|
// last message is completion_result
|
||||||
// we have command string, which means we have the result as well, so finish it (doesnt have to exist yet)
|
// we have command string, which means we have the result as well, so finish it (doesnt have to exist yet)
|
||||||
@@ -1939,7 +1945,7 @@ export class Cline {
|
|||||||
"command",
|
"command",
|
||||||
removeClosingTag("command", command),
|
removeClosingTag("command", command),
|
||||||
block.partial,
|
block.partial,
|
||||||
).catch(() => { })
|
).catch(() => {})
|
||||||
}
|
}
|
||||||
} else {
|
} else {
|
||||||
// no command, still outputting partial result
|
// no command, still outputting partial result
|
||||||
@@ -2165,7 +2171,8 @@ export class Cline {
|
|||||||
type: "text",
|
type: "text",
|
||||||
text:
|
text:
|
||||||
assistantMessage +
|
assistantMessage +
|
||||||
`\n\n[${cancelReason === "streaming_failed"
|
`\n\n[${
|
||||||
|
cancelReason === "streaming_failed"
|
||||||
? "Response interrupted by API Error"
|
? "Response interrupted by API Error"
|
||||||
: "Response interrupted by user"
|
: "Response interrupted by user"
|
||||||
}]`,
|
}]`,
|
||||||
@@ -2421,7 +2428,7 @@ export class Cline {
|
|||||||
await pWaitFor(() => busyTerminals.every((t) => !this.terminalManager.isProcessHot(t.id)), {
|
await pWaitFor(() => busyTerminals.every((t) => !this.terminalManager.isProcessHot(t.id)), {
|
||||||
interval: 100,
|
interval: 100,
|
||||||
timeout: 15_000,
|
timeout: 15_000,
|
||||||
}).catch(() => { })
|
}).catch(() => {})
|
||||||
}
|
}
|
||||||
|
|
||||||
// we want to get diagnostics AFTER terminal cools down for a few reasons: terminal could be scaffolding a project, dev servers (compilers like webpack) will first re-compile and then send diagnostics, etc
|
// we want to get diagnostics AFTER terminal cools down for a few reasons: terminal could be scaffolding a project, dev servers (compilers like webpack) will first re-compile and then send diagnostics, etc
|
||||||
|
|||||||
@@ -146,8 +146,8 @@ jest.mock('../../../integrations/misc/extract-text', () => ({
|
|||||||
|
|
||||||
// Spy on console.error and console.log to suppress expected messages
|
// Spy on console.error and console.log to suppress expected messages
|
||||||
beforeAll(() => {
|
beforeAll(() => {
|
||||||
jest.spyOn(console, 'error').mockImplementation(() => { })
|
jest.spyOn(console, 'error').mockImplementation(() => {})
|
||||||
jest.spyOn(console, 'log').mockImplementation(() => { })
|
jest.spyOn(console, 'log').mockImplementation(() => {})
|
||||||
})
|
})
|
||||||
|
|
||||||
afterAll(() => {
|
afterAll(() => {
|
||||||
@@ -263,8 +263,7 @@ describe('ClineProvider', () => {
|
|||||||
browserViewportSize: "900x600",
|
browserViewportSize: "900x600",
|
||||||
fuzzyMatchThreshold: 1.0,
|
fuzzyMatchThreshold: 1.0,
|
||||||
mcpEnabled: true,
|
mcpEnabled: true,
|
||||||
alwaysApproveResubmit: false,
|
requestDelaySeconds: 5
|
||||||
requestDelaySeconds: 5,
|
|
||||||
}
|
}
|
||||||
|
|
||||||
const message: ExtensionMessage = {
|
const message: ExtensionMessage = {
|
||||||
|
|||||||
@@ -1,4 +1,4 @@
|
|||||||
import { ApiConfiguration } from "./api"
|
import { ApiConfiguration, ApiProvider } from "./api"
|
||||||
|
|
||||||
export type AudioType = "notification" | "celebration" | "progress_loop"
|
export type AudioType = "notification" | "celebration" | "progress_loop"
|
||||||
|
|
||||||
|
|||||||
Reference in New Issue
Block a user