fix: resolve rig-scoped mayor/deacon to town-level address (gt-te23)

[seanbearden]

Summary

• gt mail send gastown/mayor silently lost mail — bead created with assignee gastown/mayor but mayor inbox checks mayor/
• Fix: normalizeAddress now resolves <rig>/mayor → mayor/ and <rig>/deacon → deacon/ since these are town-level singletons
• Two mails from laser/crew/happy (hq-wisp-50yd2, hq-wisp-1be1q) were lost due to this bug

Changes

• internal/mail/types.go: normalizeAddress resolves 2-part addresses where the second part is mayor or deacon to their town-level canonical form
• internal/mail/types_test.go: Added test cases for gastown/mayor, gastown/deacon, laser/mayor, laser/deacon

Test plan

• go build passes
• go test ./internal/mail/ — all pass including new address resolution tests
• TestAddressToIdentity covers all new cases

🤖 Generated with Claude Code

[codecov-commenter]

❌ 5 Tests Failed:

Tests completed	Failed	Passed	Skipped
9323	5	9318	48

View the top 3 failed test(s) by shortest run time

github.com/steveyegge/gastown/internal/cmd::TestBuildRefineryPatrolVars_FullConfig

Stack Traces | 0s run time

=== RUN   TestBuildRefineryPatrolVars_FullConfig
    patrol_helpers_test.go:166: expected 5 vars, got 7: [target_branch=main integration_branch_refinery_enabled=true integration_branch_auto_land=false run_tests=true delete_merged_branches=true judgment_enabled=false review_depth=standard]
--- FAIL: TestBuildRefineryPatrolVars_FullConfig (0.00s)

github.com/steveyegge/gastown/internal/cmd::TestRunHooksSyncNonClaudeAgent

Stack Traces | 0.01s run time

=== RUN   TestRunHooksSyncNonClaudeAgent
Syncing hooks...
  ✓ mayor/.claude/settings.json (created)
  ✓ deacon/.claude/settings.json (created)
  ✓ myrig/crew/.claude/settings.json (created)
warning: role_agents[crew]=opencode - agent "opencode" binary "opencode" not found in PATH, falling back to default

Synced 3 targets (3 created, 0 updated, 0 unchanged)
    hooks_sync_test.go:366: opencode plugin not created in worktree alice
--- FAIL: TestRunHooksSyncNonClaudeAgent (0.01s)

github.com/steveyegge/gastown/internal/doctor::TestHooksSyncCheck_Fix_TemplateAgent

Stack Traces | 0.01s run time

=== RUN   TestHooksSyncCheck_Fix_TemplateAgent
warning: role_agents[crew]=opencode - agent "opencode" binary "opencode" not found in PATH, falling back to default
    hooks_sync_check_test.go:219: expected StatusWarning before fix, got OK
--- FAIL: TestHooksSyncCheck_Fix_TemplateAgent (0.01s)

github.com/steveyegge/gastown/internal/doctor::TestHooksSyncCheck_TemplateAgent_Missing

Stack Traces | 0.01s run time

=== RUN   TestHooksSyncCheck_TemplateAgent_Missing
warning: role_agents[crew]=opencode - agent "opencode" binary "opencode" not found in PATH, falling back to default
    hooks_sync_check_test.go:193: expected StatusWarning for missing template agent file, got OK: All 3 hook targets in sync
--- FAIL: TestHooksSyncCheck_TemplateAgent_Missing (0.01s)

github.com/steveyegge/gastown/internal/doctor::TestHooksSyncCheck_TemplateAgent_OutOfSync

Stack Traces | 0.02s run time

=== RUN   TestHooksSyncCheck_TemplateAgent_OutOfSync
warning: role_agents[crew]=opencode - agent "opencode" binary "opencode" not found in PATH, falling back to default
    hooks_sync_check_test.go:172: expected StatusWarning for out-of-sync template agent, got OK: All 3 hook targets in sync
--- FAIL: TestHooksSyncCheck_TemplateAgent_OutOfSync (0.02s)

To view more test analytics, go to the Test Analytics Dashboard
_{📋 Got 3 mins? Take this short survey to help us improve Test Analytics.}