fix: improve task reporting tool description

kylecarbs · kylecarbs · commit 88bf0a1bc400 · 2025-05-29T19:11:20.000-04:00
diff --git a/cli/exp_mcp.go b/cli/exp_mcp.go
@@ -188,18 +188,13 @@ func (*RootCmd) mcpConfigureClaudeCode() *serpent.Command {
 				reportTaskPrompt = defaultReportTaskPrompt
 			}
 
-			// If a user overrides the coder prompt, we don't want to append
-			// the report task prompt, as it then becomes the responsibility
-			// of the user.
-			actualCoderPrompt := defaultCoderPrompt
+			// The Coder Prompt just allows users to extend our
 			if coderPrompt != "" {
-				actualCoderPrompt = coderPrompt
-			} else if reportTaskPrompt != "" {
-				actualCoderPrompt += "\n\n" + reportTaskPrompt
+				reportTaskPrompt += "\n\n" + coderPrompt
 			}
 
 			// We also write the system prompt to the CLAUDE.md file.
-			if err := injectClaudeMD(fs, actualCoderPrompt, systemPrompt, claudeMDPath); err != nil {
+			if err := injectClaudeMD(fs, reportTaskPrompt, systemPrompt, claudeMDPath); err != nil {
 				return xerrors.Errorf("failed to modify CLAUDE.md: %w", err)
 			}
 			cliui.Infof(inv.Stderr, "Wrote CLAUDE.md to %s", claudeMDPath)
@@ -648,25 +643,9 @@ func configureClaude(fs afero.Fs, cfg ClaudeConfig) error {
 }
 
 var (
-	defaultCoderPrompt = `You are a helpful Coding assistant. Aim to autonomously investigate
-and solve issues the user gives you and test your work, whenever possible.
-Avoid shortcuts like mocking tests. When you get stuck, you can ask the user
-but opt for autonomy.`
-
-	defaultReportTaskPrompt = `YOU MUST REPORT ALL TASKS TO CODER.
-When reporting tasks, you MUST follow these EXACT instructions:
-- IMMEDIATELY report status after receiving ANY user message.
-- Be granular. If you are investigating with multiple steps, report each step to coder.
-
-Task state MUST be one of the following:
-- Use "state": "working" when actively processing WITHOUT needing additional user input.
-- Use "state": "complete" only when finished with a task.
-- Use "state": "failure" when you need ANY user input, lack sufficient details, or encounter blockers.
-
-Task summaries MUST:
-- Include specifics about what you're doing.
-- Include clear and actionable steps for the user.
-- Be less than 160 characters in length.`
+	defaultCoderPrompt = ``
+
+	defaultReportTaskPrompt = `Respect the requirements of the "coder_report"task" tool. It is pertinent to provide a fantastic user-experience.`
 
 	// Define the guard strings
 	coderPromptStartGuard  = "<coder-prompt>"
diff --git a/codersdk/toolsdk/toolsdk.go b/codersdk/toolsdk/toolsdk.go
@@ -180,8 +180,28 @@ type ReportTaskArgs struct {
 
 var ReportTask = Tool[ReportTaskArgs, codersdk.Response]{
 	Tool: aisdk.Tool{
-		Name:        "coder_report_task",
-		Description: "Report progress on a user task in Coder.",
+		Name: "coder_report_task",
+		Description: `Report progress on your work.
+
+The user observes your work through a Task UI. To keep them updated
+on your progress, or if you need help - use this tool.
+
+Good Tasks
+- "Cloning the repository <repository-url>"
+- "Working on <feature-name>"
+- "Figuring our why <issue> is happening"
+
+Bad Tasks
+- "I'm working on it"
+- "I'm trying to fix it"
+- "I'm trying to implement <feature-name>"
+
+Use the "state" field to indicate your progress. Periodically report
+progress to keep the user updated. It is not possible to send too many updates!
+
+After you complete your work, ALWAYS send a "complete" or "failure" state. Only report
+these states if you are finished, not if you are working on it.
+`,
 		Schema: aisdk.Schema{
 			Properties: map[string]any{
 				"summary": map[string]any{
