Refactor agent-session safe-output parsing to a single shared path

[Copilot]

parseAgentSessionConfig had two near-identical branches for create-agent-session and deprecated create-agent-task, creating drift risk for future field changes. This PR centralizes parsing while preserving key-precedence and deprecation behavior.

• What changed

  • Extracted shared parsing into parseAgentSessionConfigMap(configData any).
  • Kept key selection in parseAgentSessionConfig:
    • prefer create-agent-session
    • fallback to create-agent-task with existing warning

• Behavior preserved

  • base, target-repo, and base safe-output fields are parsed identically for both keys.
  • invalid target-repo still returns nil (validation path unchanged).
  • non-map / empty config still defaults max to 1.

• Coverage updates

  • Added focused tests for deprecated-key parsing parity.
  • Added a precedence test to ensure create-agent-session wins when both keys are present.

if configData, exists := outputMap["create-agent-session"]; exists {
    return c.parseAgentSessionConfigMap(configData)
}
if configData, exists := outputMap["create-agent-task"]; exists {
    createAgentSessionLog.Print("WARNING: Using deprecated 'create-agent-task' configuration. Please migrate to 'create-agent-session' using 'gh aw fix'")
    return c.parseAgentSessionConfigMap(configData)
}

• Fixes [duplicate-code] Deduplicate create-agent-session parsing for current and deprecated config keys #53837

[github-actions]

PR Triage

• Category: refactor · Risk: medium
• Score: 42/100 (impact 20, urgency 10, quality 12)
• Recommended action: batch_review
• Batch: B1-safeoutput-refactor (with Extract shared safe-output target parsing #53841, Refactor safe-output repo target accessors #53838)

Part of the same safe-output parsing refactor cluster. Recommend batch review with the other two PRs.

Generated by 🔧 PR Triage Agent · auto · 58.5 AIC · ⌖ 2.15 AIC · ⊞ 8.3K · ◷

[pelikhan]

@copilot Remove create-agent-task add codemod to migrate

[Copilot]

@copilot Remove create-agent-task add codemod to migrate

Implemented in ec94ca5. create-agent-task support was removed from runtime parsing/handler aliases and from the main workflow schema, while keeping codemod migration support (gh aw fix) to rewrite safe-outputs.create-agent-task to safe-outputs.create-agent-session.

[github-actions]

Triage: category=refactor · risk=medium · score=40/100 (impact 20/50, urgency 10/30, quality 10/20)
Recommended action: batch_review

Part of the B1-safeoutput-refactor batch (with #53841, #53838). Draft, no CI yet, no reviews. Undraft once ready and review together with the batch.

Generated by 🔧 PR Triage Agent · auto · 58.8 AIC · ⌖ 2.56 AIC · ⊞ 8.3K · ◷