Errors detected in agent logs - continuing workflow step (not failing for now)

Line 4333: Each child in a list should have a unique "key" prop. (Pattern: Copilot CLI timestamped ERROR messages, Raw log: 2025-11-13T02:41:31.724Z [ERROR] Each child in a list should have a unique "key" prop.)

Line 4419: true,\\n\\t\\t\\terrorMsg: \\\"must be a relative path\\\",\\n\\t\\t},\\n\\t\\t{\\n\\t\\t\\tname: \\\"relative path allowed\\\",\\n\\t\\t\\tworkflowDir: \\\"custom/workflows\\\",\\n\\t\\t\\texpectError: false,\\n\\t\\t},\\n\\t\\t{\\n\\t\\t\\tname: \\\"default empty path\\\",\\n\\t\\t\\tworkflowDir: \\\"\\\",\\n\\t\\t\\texpectError: false,\\n\\t\\t},\\n\\t}\\n\\n\\tfor _, tt := range tests {\\n\\t\\tt.Run(tt.name, func(t *testing.T) {\\n\\t\\t\\tconfig := CompileConfig{\\n\\t\\t\\t\\tWorkflowDir: tt.workflowDir,\\n\\t\\t\\t}\\n\\n\\t\\t\\t// Use fast validation function instead of full compilation\\n\\t\\t\\terr := validateCompileConfig(config)\\n\\n\\t\\t\\tif tt.expectError {\\n\\t\\t\\t\\tif err == nil {\\n\\t\\t\\t\\t\\tt.Error(\\\"Expected error but got nil\\\")\\n\\t\\t\\t\\t} else if !strings.Contains(err.Error(), tt.errorMsg) {\\n\\t\\t\\t\\t\\tt.Errorf(\\\"Expected error containing %q, got %q\\\", tt.errorMsg, err.Error())\\n\\t\\t\\t\\t}\\n\\t\\t\\t} else if err != nil {\\n\\t\\t\\t\\tt.Errorf(\\\"Expected no error but got: %v\\\", err)\\n\\t\\t\\t}\\n\\t\\t})\\n\\t}\\n}\", \"old_str\": \"// TestCompileWorkflows_PurgeValidation tests purge flag validation\\nfunc TestCompileWorkflows_PurgeValidation(t *testing.T) {\\n\\tconfig := CompileConfig{\\n\\t\\tPurge: true,\\n\\t\\tMarkdownFiles: []string{\\\"test.md\\\"},\\n\\t}\\n\\n\\t_, err := CompileWorkflows(config)\\n\\n\\tif err == nil {\\n\\t\\tt.Error(\\\"Expected error when using purge with specific files, got nil\\\")\\n\\t}\\n\\n\\tif !strings.Contains(err.Error(), \\\"can only be used when compiling all markdown files\\\") {\\n\\t\\tt.Errorf(\\\"Expected error about purge flag, got: %v\\\", err)\\n\\t}\\n}\\n\\n// TestCompileWorkflows_WorkflowDirValidation tests workflow directory validation\\nfunc TestCompileWorkflows_WorkflowDirValidation(t *testing.T) {\\n\\ttests := []struct {\\n\\t\\tname string\\n\\t\\tworkflowDir string\\n\\t\\texpectError bool\\n\\t\\terrorMsg string\\n\\t}{\\n\\t\\t{\\n\\t\\t\\tname: \\\"absolute path not allowed\\\",\\n\\t\\t\\tworkflowDir: \\\"/absolute/path\\\",\\n\\t\\t\\texpectError: true,\\n\\t\\t\\terrorMsg: \\\"must be a relative path\\\",\\n\\t\\t},\\n\\t\\t{\\n\\t\\t\\tname: \\\"relative path allowed\\\",\\n\\t\\t\\tworkflowDir: \\\"custom/workflows\\\",\\n\\t\\t\\texpectError: false,\\n\\t\\t},\\n\\t\\t{\\n\\t\\t\\tname: \\\"default empty path\\\",\\n\\t\\t\\tworkflowDir: \\\"\\\",\\n\\t\\t\\texpectError: false,\\n\\t\\t},\\n\\t}\\n\\n\\tfor _, tt := range tests {\\n\\t\\tt.Run(tt.name, func(t *testing.T) {\\n\\t\\t\\tconfig := CompileConfig{\\n\\t\\t\\t\\tWorkflowDir: tt.workflowDir,\\n\\t\\t\\t}\\n\\n\\t\\t\\t_, err := CompileWorkflows(config)\\n\\n\\t\\t\\tif tt.expectError {\\n\\t\\t\\t\\tif err == nil {\\n\\t\\t\\t\\t\\tt.Error(\\\"Expected error but got nil\\\")\\n\\t\\t\\t\\t} else if !strings.Contains(err.Error(), tt.errorMsg) {\\n\\t\\t\\t\\t\\tt.Errorf(\\\"Expected error containing %q, got %q\\\", tt.errorMsg, err.Error())\\n\\t\\t\\t\\t}\\n\\t\\t\\t}\\n\\t\\t})\\n\\t}\", \"path\": \"/home/runner/work/gh-aw/gh-aw/pkg/cli/compile_command_test.go\"}" (Pattern: Generic ERROR messages, Raw log: "arguments": "{\"new_str\": \"// TestCompileWorkflows_PurgeValidation tests purge flag validation\\n// Uses the fast val...)

Line 4368: true,\\n\\t\\t\\terrorMsg: \\\"cannot be used with specific workflow files\\\",\\n\\t\\t},\\n\\t\\t{\\n\\t\\t\\tname: \\\"dependabot with custom workflow dir\\\",\\n\\t\\t\\tconfig: CompileConfig{\\n\\t\\t\\t\\tDependabot: true,\\n\\t\\t\\t\\tWorkflowDir: \\\"custom/workflows\\\",\\n\\t\\t\\t},\\n\\t\\t\\texpectError: true,\\n\\t\\t\\terrorMsg: \\\"cannot be used with custom --workflows-dir\\\",\\n\\t\\t},\\n\\t\\t{\\n\\t\\t\\tname: \\\"dependabot with default settings\\\",\\n\\t\\t\\tconfig: CompileConfig{\\n\\t\\t\\t\\tDependabot: true,\\n\\t\\t\\t\\tWorkflowDir: \\\"\\\",\\n\\t\\t\\t},\\n\\t\\t\\texpectError: false,\\n\\t\\t},\\n\\t}\\n\\n\\tfor _, tt := range tests {\\n\\t\\tt.Run(tt.name, func(t *testing.T) {\\n\\t\\t\\t_, err := CompileWorkflows(tt.config)\\n\\n\\t\\t\\tif tt.expectError {\\n\\t\\t\\t\\tif err == nil {\\n\\t\\t\\t\\t\\tt.Error(\\\"Expected error but got nil\\\")\\n\\t\\t\\t\\t} else if !strings.Contains(err.Error(), tt.errorMsg) {\\n\\t\\t\\t\\t\\tt.Errorf(\\\"Expected error containing %q, got %q\\\", tt.errorMsg, err.Error())\\n\\t\\t\\t\\t}\\n\\t\\t\\t}\\n\\t\\t})\\n\\t}\\n}\", \"new_str\": \"// TestCompileWorkflows_DependabotValidation tests dependabot flag validation\\n// Uses the fast validateCompileConfig function instead of full compilation\\nfunc TestCompileWorkflows_DependabotValidation(t *testing.T) {\\n\\ttests := []struct {\\n\\t\\tname string\\n\\t\\tconfig CompileConfig\\n\\t\\texpectError bool\\n\\t\\terrorMsg string\\n\\t}{\\n\\t\\t{\\n\\t\\t\\tname: \\\"dependabot with specific files\\\",\\n\\t\\t\\tconfig: CompileConfig{\\n\\t\\t\\t\\tDependabot: true,\\n\\t\\t\\t\\tMarkdownFiles: []string{\\\"test.md\\\"},\\n\\t\\t\\t},\\n\\t\\t\\texpectError: true,\\n\\t\\t\\terrorMsg: \\\"cannot be used with specific workflow files\\\",\\n\\t\\t},\\n\\t\\t{\\n\\t\\t\\tname: \\\"dependabot with custom workflow dir\\\",\\n\\t\\t\\tconfig: CompileConfig{\\n\\t\\t\\t\\tDependabot: true,\\n\\t\\t\\t\\tWorkflowDir: \\\"custom/workflows\\\",\\n\\t\\t\\t},\\n\\t\\t\\texpectError: true,\\n\\t\\t\\terrorMsg: \\\"cannot be used with custom --workflows-dir\\\",\\n\\t\\t},\\n\\t\\t{\\n\\t\\t\\tname: \\\"dependabot with default settings\\\",\\n\\t\\t\\tconfig: CompileConfig{\\n\\t\\t\\t\\tDependabot: true,\\n\\t\\t\\t\\tWorkflowDir: \\\"\\\",\\n\\t\\t\\t},\\n\\t\\t\\texpectError: false,\\n\\t\\t},\\n\\t}\\n\\n\\tfor _, tt := range tests {\\n\\t\\tt.Run(tt.name, func(t *testing.T) {\\n\\t\\t\\t// Use fast validation function instead of full compilation\\n\\t\\t\\terr := validateCompileConfig(tt.config)\\n\\n\\t\\t\\tif tt.expectError {\\n\\t\\t\\t\\tif err == nil {\\n\\t\\t\\t\\t\\tt.Error(\\\"Expected error but got nil\\\")\\n\\t\\t\\t\\t} else if !strings.Contains(err.Error(), tt.errorMsg) {\\n\\t\\t\\t\\t\\tt.Errorf(\\\"Expected error containing %q, got %q\\\", tt.errorMsg, err.Error())\\n\\t\\t\\t\\t}\\n\\t\\t\\t} else if err != nil {\\n\\t\\t\\t\\tt.Errorf(\\\"Expected no error but got: %v\\\", err)\\n\\t\\t\\t}\\n\\t\\t})\\n\\t}\\n}\"}" (Pattern: Generic ERROR messages, Raw log: "arguments": "{\"path\": \"/home/runner/work/gh-aw/gh-aw/pkg/cli/compile_command_test.go\", \"old_str\": \"// TestCompil...)

- Line 3: Too many items of type 'add_comment'. Maximum allowed: 1.