fix(task): declare create-only required fields

Author: GPT-5.2

internal/app/runtime_validation_test.go

@@ -60,6 +60,31 @@ func TestRuntimeCallEnforcesRequiredEnumBoundsAndOneOf(t *testing.T) {
 	}
 }
 
+func TestRuntimeCallEnforcesTaskCreateRequiredFields(t *testing.T) {
+	runtime := newRuntimeValidationTestRuntime(t)
+	valid := map[string]any{
+		"action":                "create",
+		"title":                 "schema contract",
+		"goal":                  "match runtime requirements",
+		"completion_conditions": []any{"task is created"},
+	}
+	for _, field := range []string{"title", "goal", "completion_conditions"} {
+		args := make(map[string]any, len(valid)-1)
+		for key, value := range valid {
+			if key != field {
+				args[key] = value
+			}
+		}
+		t.Run("missing_"+field, func(t *testing.T) {
+			assertInvalidToolArguments(t, runtime, "task_manage", args)
+		})
+	}
+
+	if _, err := runtime.Call(context.Background(), "task_manage", valid); err != nil {
+		t.Fatalf("schema-complete task create failed: %v", err)
+	}
+}
+
 func TestRuntimeCallRejectsNestedUnknownFields(t *testing.T) {
 	runtime := newRuntimeValidationTestRuntime(t)
 	assertInvalidToolArguments(t, runtime, "task_manage", map[string]any{

internal/tool/task/contract.go

@@ -13,9 +13,9 @@ func ManageInputSchema(cfg config.Config) map[string]any {
 	props := map[string]any{
 		"action":                map[string]any{"type": "string", "description": "Task lifecycle action. Use checkpoint to update live step progress.", "enum": []string{"create", "list", "get", "checkpoint", "block", "resume", "final_review", "complete"}},
 		"task_id":               stringProp("Persistent task id for get, checkpoint, block, resume, final_review, or complete."),
-		"title":                 stringProp("Short task title for create."),
-		"goal":                  stringProp("Fixed task goal for create."),
-		"completion_conditions": map[string]any{"type": "array", "minItems": 1, "items": map[string]any{"type": "string"}, "description": "Conditions that must be true before final_review can pass."},
+		"title":                 stringProp("Short task title. Required for action=create."),
+		"goal":                  stringProp("Fixed task goal. Required for action=create."),
+		"completion_conditions": map[string]any{"type": "array", "minItems": 1, "items": map[string]any{"type": "string"}, "description": "Conditions that must be true before final_review can pass. Required for action=create."},
 		"step_id":               stringProp("Task step id for a single-step checkpoint."),
 		"completed_step_ids":    map[string]any{"type": "array", "minItems": 1, "maxItems": 12, "uniqueItems": true, "items": map[string]any{"type": "string"}, "description": "Task step ids to mark completed in one atomic batch checkpoint."},
 		"current_step_id":       stringProp("Single task step id to mark in_progress in a batch checkpoint."),
@@ -33,7 +33,7 @@ func ManageInputSchema(cfg config.Config) map[string]any {
 			"type": "array", "maxItems": 12, "description": "Concrete task steps.",
 			"items": map[string]any{"type": "object", "additionalProperties": false, "required": []string{"id", "title"}, "properties": map[string]any{"id": stringProp("Stable step id."), "title": stringProp("Human-readable step title.")}},
 		}
-		return toolcontract.InputObject(props, "action")
+		return taskManageInputObject(props)
 	}
 
 	props["project"] = stringProp("Optional project identifier used to hard-scope Evolution guidance and evidence candidates. Omit only for global tasks.")
@@ -55,7 +55,19 @@ func ManageInputSchema(cfg config.Config) map[string]any {
 			},
 		},
 	}
-	return toolcontract.InputObject(props, "action")
+	return taskManageInputObject(props)
+}
+
+func taskManageInputObject(props map[string]any) map[string]any {
+	schema := toolcontract.InputObject(props, "action")
+	schema["allOf"] = []any{map[string]any{
+		"if": map[string]any{
+			"properties": map[string]any{"action": map[string]any{"const": "create"}},
+			"required":   []string{"action"},
+		},
+		"then": map[string]any{"required": []string{"title", "goal", "completion_conditions"}},
+	}}
+	return schema
 }
 
 func ManageOutputSchema(cfg config.Config) map[string]any {