Split safe-output handler registry by category

[Copilot]

The 1,091-line safe-output handler registry combined 50 inline handlers in one map, making reviews and concurrent changes unnecessarily difficult.

• Registry organization

  • Split handlers into focused issue/discussion, pull request, automation, dispatch/reporting, and artifact/project registries.
  • Moved shared token resolution into a dedicated file.
  • Reduced the original registry file to category assembly.

• Regression coverage

  • Verify all 50 handler keys are assembled exactly once.
  • Detect missing, duplicated, or unexpected registrations.

• Fixes [deep-report] Split pkg/workflow/safe_outputs_handler_registry.go (1,091 lines, 49-entry closure map) by handler category #55470

---

Generated by 👨‍🍳 PR Sous Chef · pi · gpt54 · 10.3 AIC · ⌖ 9.22 AIC · ⊞ 6.2K · ◷
_{Comment /souschef to run again}

[Copilot]

Pull request overview

Splits the monolithic safe-output registry into focused category registries without changing handler behavior.

Changes:

• Organizes all 50 handlers across five category files.
• Extracts shared token-resolution helpers.
• Adds coverage for missing, duplicate, and unexpected registrations.

Show a summary per file

File	Description
pkg/workflow/safe_outputs_handler_registry.go	Assembles category registries.
pkg/workflow/safe_outputs_handler_tokens.go	Contains shared token helpers.
pkg/workflow/safe_outputs_handler_registry_test.go	Verifies complete, unique registration.
pkg/workflow/safe_outputs_handler_registry_issues.go	Registers issue and discussion handlers.
pkg/workflow/safe_outputs_handler_registry_pull_requests.go	Registers pull-request handlers.
pkg/workflow/safe_outputs_handler_registry_automation.go	Registers repository automation handlers.
pkg/workflow/safe_outputs_handler_registry_dispatch.go	Registers dispatch and reporting handlers.
pkg/workflow/safe_outputs_handler_registry_artifacts_projects.go	Registers artifact and project handlers.

Review details

💡 Add a code-review agent skill for context-aware, tailored reviews. Learn more in the docs.

• Files reviewed: 8/8 changed files
• Comments generated: 0
• Review effort level: Balanced

[gh-aw-bot]

@copilot Please review the latest PR state, refresh the branch if needed, and run the pr-finisher skill before handing back for maintainer review.

Generated by 👨‍🍳 PR Sous Chef · pi · gpt54 · 10.3 AIC · ⌖ 9.22 AIC · ⊞ 6.2K · ◷
_{Comment /souschef to run again}

[Copilot]

@copilot Please review the latest PR state, refresh the branch if needed, and run the pr-finisher skill before handing back for maintainer review.

Generated by [👨‍🍳 PR Sous Chef](https://...

Merged main and resolved conflicts — main had already merged an equivalent registry split (#55482) for the same underlying issue, so this branch now adopts main's version of the affected files and drops the redundant duplicate files this branch had added. Local make fmt/make lint/make test-unit pass (3 pre-existing lint warnings confirmed present on main itself, unrelated). See 7db22d6.

[github-actions]

🎉 This pull request is included in a new release.

Release: v0.87.5