Update with log/slog logging
Change-Id: Ie86d7082bb50576fdfbd898c44bd2905e389c399
diff --git a/server/subtasks/service_test.go b/server/subtasks/service_test.go
index 0a8e473..08dd72a 100644
--- a/server/subtasks/service_test.go
+++ b/server/subtasks/service_test.go
@@ -81,7 +81,7 @@
mockProvider := NewMockLLMProvider([]string{})
agentRoles := []string{"backend", "frontend", "qa"}
- service := NewSubtaskService(mockProvider, nil, agentRoles, nil, "example", "repo", nil)
+ service := NewSubtaskService(mockProvider, nil, agentRoles, nil, "example", "repo", nil, nil)
if service == nil {
t.Fatal("NewSubtaskService returned nil")
@@ -107,7 +107,7 @@
mockProvider := NewMockLLMProvider([]string{decisionResponse})
agentRoles := []string{"backend", "frontend", "qa"}
- service := NewSubtaskService(mockProvider, nil, agentRoles, nil, "example", "repo", nil)
+ service := NewSubtaskService(mockProvider, nil, agentRoles, nil, "example", "repo", nil, nil)
// Test the parseSubtaskDecision method directly since ShouldGenerateSubtasks is used by manager
decision, err := service.parseSubtaskDecision(decisionResponse)
@@ -166,7 +166,7 @@
mockProvider := NewMockLLMProvider([]string{jsonResponse})
agentRoles := []string{"backend", "frontend", "qa", "ceo"} // Include CEO for agent creation
- service := NewSubtaskService(mockProvider, nil, agentRoles, nil, "example", "repo", nil)
+ service := NewSubtaskService(mockProvider, nil, agentRoles, nil, "example", "repo", nil, nil)
task := &tm.Task{
ID: "test-task-123",
@@ -267,7 +267,7 @@
mockProvider := NewMockLLMProvider([]string{invalidResponse})
agentRoles := []string{"backend", "frontend"}
- service := NewSubtaskService(mockProvider, nil, agentRoles, nil, "example", "repo", nil)
+ service := NewSubtaskService(mockProvider, nil, agentRoles, nil, "example", "repo", nil, nil)
task := &tm.Task{
ID: "test-task-123",
@@ -303,7 +303,7 @@
mockProvider := NewMockLLMProvider([]string{jsonResponse})
agentRoles := []string{"backend", "frontend"}
- service := NewSubtaskService(mockProvider, nil, agentRoles, nil, "example", "repo", nil)
+ service := NewSubtaskService(mockProvider, nil, agentRoles, nil, "example", "repo", nil, nil)
task := &tm.Task{
ID: "test-task-123",
@@ -323,7 +323,7 @@
func TestGenerateSubtaskPR(t *testing.T) {
mockProvider := NewMockLLMProvider([]string{})
- service := NewSubtaskService(mockProvider, nil, []string{"backend"}, nil, "example", "repo", nil)
+ service := NewSubtaskService(mockProvider, nil, []string{"backend"}, nil, "example", "repo", nil, nil)
analysis := &tm.SubtaskAnalysis{
ParentTaskID: "task-123",
@@ -357,7 +357,7 @@
func TestBuildSubtaskAnalysisPrompt(t *testing.T) {
mockProvider := NewMockLLMProvider([]string{})
agentRoles := []string{"backend", "frontend", "qa"}
- service := NewSubtaskService(mockProvider, nil, agentRoles, nil, "example", "repo", nil)
+ service := NewSubtaskService(mockProvider, nil, agentRoles, nil, "example", "repo", nil, nil)
task := &tm.Task{
Title: "Build authentication system",
@@ -388,7 +388,7 @@
func TestGetSubtaskAnalysisSystemPrompt(t *testing.T) {
mockProvider := NewMockLLMProvider([]string{})
agentRoles := []string{"backend", "frontend", "qa", "devops"}
- service := NewSubtaskService(mockProvider, nil, agentRoles, nil, "example", "repo", nil)
+ service := NewSubtaskService(mockProvider, nil, agentRoles, nil, "example", "repo", nil, nil)
systemPrompt := service.getSubtaskAnalysisSystemPrompt()
@@ -412,7 +412,7 @@
func TestIsValidAgentRole(t *testing.T) {
mockProvider := NewMockLLMProvider([]string{})
agentRoles := []string{"backend", "frontend", "qa"}
- service := NewSubtaskService(mockProvider, nil, agentRoles, nil, "example", "repo", nil)
+ service := NewSubtaskService(mockProvider, nil, agentRoles, nil, "example", "repo", nil, nil)
if !service.isValidAgentRole("backend") {
t.Error("'backend' should be a valid agent role")
@@ -433,7 +433,7 @@
func TestParseSubtaskAnalysis_Priority(t *testing.T) {
mockProvider := NewMockLLMProvider([]string{})
- service := NewSubtaskService(mockProvider, nil, []string{"backend"}, nil, "example", "repo", nil)
+ service := NewSubtaskService(mockProvider, nil, []string{"backend"}, nil, "example", "repo", nil, nil)
tests := []struct {
input string
@@ -485,7 +485,7 @@
func TestGenerateSubtaskFile(t *testing.T) {
mockProvider := NewMockLLMProvider([]string{})
- service := NewSubtaskService(mockProvider, nil, []string{"backend"}, nil, "example", "repo", nil)
+ service := NewSubtaskService(mockProvider, nil, []string{"backend"}, nil, "example", "repo", nil, nil)
subtask := tm.SubtaskProposal{
Title: "Implement API endpoints",
@@ -552,7 +552,7 @@
func TestUpdateParentTaskAsCompleted(t *testing.T) {
mockProvider := NewMockLLMProvider([]string{})
- service := NewSubtaskService(mockProvider, nil, []string{"backend"}, nil, "example", "repo", nil)
+ service := NewSubtaskService(mockProvider, nil, []string{"backend"}, nil, "example", "repo", nil, nil)
// Create a temporary task file for testing
taskContent := `---
@@ -639,7 +639,7 @@
func TestClose(t *testing.T) {
mockProvider := NewMockLLMProvider([]string{})
- service := NewSubtaskService(mockProvider, nil, []string{"backend"}, nil, "example", "repo", nil)
+ service := NewSubtaskService(mockProvider, nil, []string{"backend"}, nil, "example", "repo", nil, nil)
err := service.Close()
if err != nil {
@@ -666,7 +666,7 @@
}`
mockProvider := NewMockLLMProvider([]string{jsonResponse})
- service := NewSubtaskService(mockProvider, nil, []string{"backend", "frontend"}, nil, "example", "repo", nil)
+ service := NewSubtaskService(mockProvider, nil, []string{"backend", "frontend"}, nil, "example", "repo", nil, nil)
task := &tm.Task{
ID: "benchmark-task",