Add default concurrency pattern for non-special-case workflows

[Copilot]

Problem

The copilot engine had special concurrency handling that differed from other engines (claude, codex, custom). Specifically, it used hasDefaultConcurrency: true which applied an agent-level concurrency group pattern gh-aw-copilot to limit concurrent execution across all copilot workflows globally.

This special behavior was inconsistent with the design principle that all engines should use the same concurrency formulas.

Solution

Removed the hasDefaultConcurrency feature entirely and implemented a new default concurrency pattern for workflows without special triggers:

• Removed hasDefaultConcurrency field from BaseEngine struct
• Removed HasDefaultConcurrency() method from CodingAgentEngine interface and BaseEngine implementation
• Removed the field from all engine constructors (copilot, claude, codex, custom)
• Implemented default concurrency pattern gh-aw-${engine-id}-${{ github.workflow }} for workflows with generic triggers (workflow_dispatch, schedule, workflow_run, etc.)
• Workflows with special triggers (issues, PRs, discussions, push) do NOT get default concurrency at agent level
• Custom concurrency configuration via engine.concurrency field continues to work as before

Changes

Code:

• pkg/workflow/agentic_engine.go: Removed hasDefaultConcurrency field and HasDefaultConcurrency() method
• pkg/workflow/copilot_engine.go: Removed hasDefaultConcurrency field from engine initialization
• pkg/workflow/claude_engine.go: Removed hasDefaultConcurrency field from engine initialization
• pkg/workflow/codex_engine.go: Removed hasDefaultConcurrency field from engine initialization
• pkg/workflow/custom_engine.go: Removed hasDefaultConcurrency field from engine initialization
• pkg/workflow/concurrency.go:
  • Updated GenerateJobConcurrencyConfig to implement default concurrency for non-special-case workflows
  • Added hasSpecialTriggers helper function to detect workflows with special trigger handling
• pkg/workflow/concurrency_test.go: Updated unit tests to reflect new behavior
• pkg/workflow/engine_concurrency_integration_test.go: Updated integration tests with cases for both special and non-special triggers

Generated workflows:

• Recompiled 23 .lock.yml files with default concurrency pattern for workflows using generic triggers

Behavior Change

Workflows with special triggers (issues, PRs, discussions, push):

agent:
  # No default concurrency - uses workflow-level formula instead

Workflows with generic triggers (workflow_dispatch, schedule, etc.):

agent:
  concurrency:
    group: "gh-aw-copilot-${{ github.workflow }}"

Custom concurrency (always takes precedence):

engine:
  id: copilot
  concurrency: "custom-group-${{ github.ref }}"

Testing

• ✅ All unit tests pass
• ✅ All integration tests pass
• ✅ Manual verification confirms correct behavior for both special and non-special trigger workflows
• ✅ Custom concurrency configuration still works correctly
• ✅ Build and linting successful

Original prompt

Remove special handling of concurrency group for the copilot agentic engine. Use the same formulas as the other engine.

---

💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

[pelikhan]

@copilot remove hasDefaultConcurrency it should not be used anymore.

[pelikhan]

@copilot the default concurrency group should by gh-aw-${enginer id}-${{ github.workflow }} if nothing is specified and none of the special cases are selected (for workflow_dispatch for example)

[Copilot]

@copilot remove hasDefaultConcurrency it should not be used anymore.

Done! Removed hasDefaultConcurrency completely - the field, method, and all related logic. Commit: a4afcf7

@copilot remove hasDefaultConcurrency it should not be used anymore.

Done! Removed hasDefaultConcurrency completely - the field, method, and all related logic. Commit: 1529baf

[Copilot]

@copilot the default concurrency group should by gh-aw-${enginer id}-${{ github.workflow }} if nothing is specified and none of the special cases are selected (for workflow_dispatch for example)

Implemented! Default concurrency now uses pattern gh-aw-${engine-id}-${{ github.workflow }} for workflows without special triggers (workflow_dispatch, schedule, etc.). Workflows with special triggers (issues, PRs, discussions, push) don't get default concurrency. Commit: 91e7f5b

@copilot the default concurrency group should by gh-aw-${enginer id}-${{ github.workflow }} if nothing is specified and none of the special cases are selected (for workflow_dispatch for example)

Implemented! Default concurrency now uses pattern gh-aw-${engine-id}-${{ github.workflow }} for workflows without special triggers (workflow_dispatch, schedule, etc.). Workflows with special triggers (issues, PRs, discussions, push) don't get default concurrency. Commit: e17775e

[github-actions]

Agentic Changeset Generator triggered by this pull request.