diff --git a/internal/config/config.go b/internal/config/config.go index f12b586c9..c94e7c84e 100644 --- a/internal/config/config.go +++ b/internal/config/config.go @@ -73,7 +73,7 @@ func DefaultAgentRoles() []string { // The "fullsend" dispatch role is excluded because per-repo mode uses // the target repo's shim workflow for dispatch instead of a separate app. func PerRepoDefaultRoles() []string { - return []string{"triage", "coder", "review", "retro", "prioritize"} + return []string{"triage", "coder", "review", "fix", "retro", "prioritize"} } // NewOrgConfig creates a new OrgConfig with sensible defaults. diff --git a/internal/config/config_test.go b/internal/config/config_test.go index be975a3bf..8ebaf8ff9 100644 --- a/internal/config/config_test.go +++ b/internal/config/config_test.go @@ -20,6 +20,19 @@ func TestValidRoles(t *testing.T) { assert.Contains(t, roles, "prioritize") } +func TestPerRepoDefaultRoles(t *testing.T) { + roles := PerRepoDefaultRoles() + assert.Len(t, roles, 6) + assert.Contains(t, roles, "triage") + assert.Contains(t, roles, "coder") + assert.Contains(t, roles, "review") + assert.Contains(t, roles, "fix") + assert.Contains(t, roles, "retro") + assert.Contains(t, roles, "prioritize") + // "fullsend" dispatch role must be excluded in per-repo mode. + assert.NotContains(t, roles, "fullsend") +} + func TestNewOrgConfig(t *testing.T) { allRepos := []string{"repo-a", "repo-b", "repo-c"} enabledRepos := []string{"repo-a", "repo-c"}