Enforce canonical dispatch_repository safe-output key (remove dashed alias)

[Copilot]

dispatch_repository parsing had drifted from schema/docs by accepting an undocumented dashed alias (dispatch-repository). This change realigns runtime behavior with the declared contract by making underscore-only keying canonical.

• Compiler behavior

  • Removed fallback lookup for safe-outputs.dispatch-repository in parseDispatchRepositoryConfig.
  • Compiler now recognizes only safe-outputs.dispatch_repository.

• Workflow parsing coverage

  • Updated dispatch repository parser tests to assert dashed key rejection instead of acceptance.

• Schema-level regression guard

  • Added parser validation coverage to ensure safe-outputs.dispatch-repository fails schema/location validation.

// before: accepted both keys
if configData, exists = outputMap["dispatch_repository"]; !exists {
    if configData, exists = outputMap["dispatch-repository"]; !exists {
        return nil
    }
}

// after: underscore-only
if configData, exists = outputMap["dispatch_repository"]; !exists {
    return nil
}

[github-actions]

🤖 PR Triage — §28342769269

Field	Value
Category	bug
Risk	low
Score	40/100
Priority	medium
Action	defer
Status	Draft — awaiting promotion to ready

Score breakdown: Impact 20 · Urgency 8 · Quality 12

Rationale: Tiny targeted fix removing undocumented dashed alias from dispatch_repository parsing (+22/−9, 3 files). Enforces contract alignment between runtime and schema. Tests added. No CI yet. Currently draft — defer until promoted to ready.

i️ Promote to ready when complete and CI passes.

Generated by 🔧 PR Triage Agent · 107.8 AIC · ⌖ 10.9 AIC · ⊞ 5.4K · ◷

[github-actions]

Great work on this one! 🎉 The dispatch_repository schema-alignment fix is clean, well-scoped, and easy to reason about.

• Runtime change in dispatch_repository.go is minimal (1 addition, 5 deletions — surgical removal of the dashed fallback).
• Test coverage is solid: existing parser tests updated to assert rejection of the dashed key, plus a new schema-level regression guard added in schema_location_test.go.
• Description clearly explains the contract drift, the fix, and includes a readable before/after snippet.

This PR looks ready for review. 🟢

Generated by ✅ Contribution Check · 277.8 AIC · ⌖ 16.5 AIC · ⊞ 6K · ◷

[pelikhan]

@copilot align using dashes like other safe outputs

[Copilot]

@copilot align using dashes like other safe outputs

Implemented in e35808a (plus follow-ups f706b8f and 453e85c). dispatch-repository is now the canonical safe-output key (aligned with dashed naming), while dispatch_repository remains a backward-compatible alias.

[pelikhan]

@copilot create a codemod to rename to the proper syntax

[Copilot]

@copilot create a codemod to rename to the proper syntax

Implemented in b18f3a7. Added a gh aw fix codemod that renames safe-outputs.dispatch_repository to safe-outputs.dispatch-repository, registered it in the codemod registry, and added codemod/registry tests.