Skip to content

Commit 419cdda

Browse files
authored
add CoreManageTodoList tool to packagejson tools for testing (#548)
* add CoreManageTodoList tool to packagejson tools for testing * fix: update references from CoreTodoListTool to CoreManageTodoList
1 parent 796bedb commit 419cdda

File tree

4 files changed

+3
-3
lines changed

4 files changed

+3
-3
lines changed

src/extension/prompts/node/agent/agentInstructions.tsx

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -670,7 +670,7 @@ class NotebookInstructions extends PromptElement<DefaultAgentPromptProps> {
670670
class TodoListToolInstructions extends PromptElement<DefaultAgentPromptProps> {
671671
render() {
672672
return <Tag name='todoListToolInstructions'>
673-
Use the {ToolName.CoreTodoListTool} frequently to plan tasks throughout your coding session for task visibility and proper planning.<br />
673+
Use the {ToolName.CoreManageTodoList} frequently to plan tasks throughout your coding session for task visibility and proper planning.<br />
674674
When to use: complex multi-step work requiring planning and tracking, when user provides multiple tasks or requests (numbered/comma-separated), after receiving new instructions that require multiple steps, BEFORE starting work on any todo (mark as in-progress), IMMEDIATELY after completing each todo (mark completed individually), when breaking down larger tasks into smaller actionable steps, to give users visibility into your progress and planning.<br />
675675
When NOT to use: single, trivial tasks that can be completed in one step, purely conversational/informational requests, when just reading files or performing simple searches.<br />
676676
CRITICAL workflow to follow:<br />

src/extension/prompts/node/agent/agentPrompt.tsx

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -292,7 +292,7 @@ export class AgentUserMessage extends PromptElement<AgentUserMessageProps> {
292292
' (See <attachments> above for file contents. You may not need to search or read the file again.)'
293293
: '';
294294
const hasToolsToEditNotebook = hasCreateFileTool || hasEditNotebookTool || hasReplaceStringTool || hasApplyPatchTool || hasEditFileTool;
295-
const hasTodoTool = !!this.props.availableTools?.find(tool => tool.name === ToolName.CoreTodoListTool);
295+
const hasTodoTool = !!this.props.availableTools?.find(tool => tool.name === ToolName.CoreManageTodoList);
296296

297297
return (
298298
<>

src/extension/tools/common/toolNames.ts

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -51,7 +51,6 @@ export const enum ToolName {
5151
CoreRunTask = 'run_task',
5252
CoreGetTaskOutput = 'get_task_output',
5353
CoreRunTest = 'runTests',
54-
CoreTodoListTool = 'manage_todo_list',
5554
}
5655

5756
// When updating this, also update contributedToolNameToToolNames

src/extension/tools/node/test/testToolsService.ts

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -209,6 +209,7 @@ export function getPackagejsonToolsForTest() {
209209
tools.add(ToolName.CoreGetTaskOutput);
210210
tools.add(ToolName.CoreRunTask);
211211
tools.add(ToolName.CoreRunTest);
212+
tools.add(ToolName.CoreManageTodoList);
212213

213214
return tools;
214215
}

0 commit comments

Comments
 (0)