fix(#1220): add fix role to PerRepoDefaultRoles for dispatch loop

[fullsend-ai-coder]

The fix role was missing from PerRepoDefaultRoles(), causing the dispatch role-check to block fix agent runs after review requested changes. All other agent roles were already included. The fix role has full agent scaffolding and is ready for production use.

Co-Authored-By: Claude Opus 4.6 [email protected]

---

Closes #1220

Post-script verification

• Branch is not main/master (fullsend/code/issue-1220)
• Secret scan passed (gitleaks — ece4b4b71a4d2eb5563ac365f04a333b95722288..HEAD)
• Pre-commit hooks passed (authoritative run on runner)
• Tests ran inside sandbox

[github-actions]

Site preview

Preview: https://52b5920e-site.fullsend-ai.workers.dev

Commit: 2991723a42133836ae8f73cfde9691b91539be62

[fullsend-ai-review]

Review

Findings

No findings.

Previous run

Review

Findings

Low

• [test-coverage] internal/config/config_test.go — No dedicated test for PerRepoDefaultRoles(). Other role functions (ValidRoles, DefaultAgentRoles) have explicit tests, but PerRepoDefaultRoles() lacks one. The linked issue's triage comment proposed a test; adding one would prevent future regressions where a role is accidentally omitted again.
  Remediation: Add a TestPerRepoDefaultRolesIncludesFix test (or a broader test asserting all expected roles are present), similar to the pattern in TestValidRoles.

[fullsend-ai-review]

Review follow-ups

Created follow-up issues for actionable non-blocking review findings:

• #1226 — No dedicated test for PerRepoDefaultRoles(). Other role functions (ValidRoles, DefaultAgentRoles) have explicit tests, but PerRepoDefaultRoles() lacks one. Adding a test would prevent future regressions where a role is accidentally omitted.

[rh-hemartin]

/fs-fix

[rh-hemartin]

/fs-fix

[fullsend-ai-coder]

🔧 Fix agent — iteration 1 (human-triggered)

Addressed 1 review finding: added TestPerRepoDefaultRoles test for regression coverage of the PerRepoDefaultRoles() function.

Fixed (1):

1. No dedicated test for PerRepoDefaultRoles() (internal/config/config_test.go): Added TestPerRepoDefaultRoles test that verifies all 6 expected roles are present (triage, coder, review, fix, retro, prioritize) and that the fullsend dispatch role is excluded in per-repo mode. Follows the same pattern as TestValidRoles.

Tests: passed

_{Updated by fullsend fix agent}