build(build): add concurrency controls to orchestrator workflows by WilliamBerryiii · Pull Request #275 · microsoft/hve-core

WilliamBerryiii · 2026-01-24T18:30:09Z

Description

Adds workflow-level concurrency controls to all three orchestrator workflows. This prevents unnecessary parallel runs when multiple commits are pushed in quick succession and reduces GitHub Actions resource consumption.

Changes:

Added concurrency block to pr-validation.yml with cancel-in-progress: true to cancel stale PR validation runs when new commits arrive
Added concurrency block to main.yml with cancel-in-progress: false to prevent cancellation of main branch builds
Added concurrency block to weekly-security-maintenance.yml with cancel-in-progress: false for scheduled job protection
Updated .cspell/general-technical.txt with missing dictionary terms (carryforward, SCHED, SECCHECK) that were causing pre-existing spell-check failures

Related Issue(s)

Fixes #274

Type of Change

Select all that apply:

Code & Documentation:

Bug fix (non-breaking change fixing an issue)
New feature (non-breaking change adding functionality)
Breaking change (fix or feature causing existing functionality to change)
Documentation update

Infrastructure & Configuration:

AI Artifacts:

Reviewed contribution with prompt-builder agent and addressed all feedback
Copilot instructions (.github/instructions/*.instructions.md)
Copilot prompt (.github/prompts/*.prompt.md)
Copilot agent (.github/agents/*.agent.md)

Note for AI Artifact Contributors:

Agents: Research, indexing/referencing other project (using standard VS Code GitHub Copilot/MCP tools), planning, and general implementation agents likely already exist. Review .github/agents/ before creating new ones.

Model Versions: Only contributions targeting the latest Anthropic and OpenAI models will be accepted. Older model versions (e.g., GPT-3.5, Claude 3) will be rejected.

See Agents Not Accepted and Model Version Requirements.

Other:

Script/automation (.ps1, .sh, .py)
Other (please describe):

Sample Prompts (for AI Artifact Contributions)

N/A - This PR does not include AI artifacts.

Testing

YAML lint validation passes (npm run lint:yaml)
Spell check passes (npm run spell-check)

Checklist

Required Checks

Documentation is updated (if applicable)
Files follow existing naming conventions
Changes are backwards compatible (if applicable)
Tests added for new functionality (if applicable)

AI Artifact Contributions

N/A

Required Automated Checks

The following validation commands must pass before merging:

Markdown linting: npm run lint:md
Spell checking: npm run spell-check
Frontmatter validation: npm run lint:frontmatter
Link validation: npm run lint:md-links
PowerShell analysis: npm run lint:ps

Security Considerations

This PR does not contain any sensitive or NDA information
Any new dependencies have been reviewed for security issues
Security-related scripts follow the principle of least privilege

Additional Notes

Concurrency Configuration Patterns

Workflow	Group Key	Cancel In-Progress	Rationale
`pr-validation.yml`	`workflow-head_ref`	`true`	Cancel stale PR runs when new commits arrive
`main.yml`	`workflow-ref`	`false`	Protect main branch builds from cancellation
`weekly-security-maintenance.yml`	`workflow`	`false`	Single weekly job, no ref variation needed

Dictionary Updates

Fixed pre-existing spell-check gaps that passed CI due to timing:

carryforward - Codecov configuration term (PR build(build): add Pester code coverage to main branch CI #250)
SCHED, SECCHECK - Mermaid diagram labels (PR docs(architecture): add architecture documentation and value proposition #252)

⚡ - Generated by Copilot

- Add workflow-level concurrency to pr-validation.yml with cancel-in-progress - Add workflow-level concurrency to main.yml without cancellation - Add workflow-level concurrency to weekly-security-maintenance.yml - Update dictionary with missing terms (carryforward, SCHED, SECCHECK) Fixes #274 ⚡ - Generated by Copilot

github-actions · 2026-01-24T18:30:20Z

Dependency Review

✅ No vulnerabilities or license issues or OpenSSF Scorecard issues found.

Scanned Files

None

codecov-commenter · 2026-01-24T18:31:21Z

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 29.44%. Comparing base (ccbdfa3) to head (428d5e1).

Additional details and impacted files

@@            Coverage Diff             @@
##             main     #275      +/-   ##
==========================================
+ Coverage   29.40%   29.44%   +0.03%     
==========================================
  Files          14       14              
  Lines        2785     2785              
==========================================
+ Hits          819      820       +1     
+ Misses       1966     1965       -1

Flag	Coverage Δ
pester	`29.44% <ø> (+0.03%)`	⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.
see 1 file with indirect coverage changes

🚀 New features to boost your workflow:

❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Copilot

Pull request overview

Adds GitHub Actions workflow-level concurrency controls to reduce redundant parallel orchestrator runs and lower Actions resource usage, plus fixes pre-existing spell-check dictionary gaps.

Changes:

Added concurrency blocks to PR validation, main CI, and weekly security maintenance workflows with appropriate cancel/queue behavior.
Updated .cspell dictionary with missing technical terms to eliminate existing spell-check failures.

Reviewed changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated 1 comment.

File	Description
.github/workflows/pr-validation.yml	Adds workflow concurrency with cancellation of superseded PR runs.
.github/workflows/main.yml	Adds workflow concurrency to serialize main builds without cancellation.
.github/workflows/weekly-security-maintenance.yml	Adds workflow concurrency to prevent overlapping weekly security runs.
.cspell/general-technical.txt	Adds missing Codecov and Mermaid diagram terms to the spell-check dictionary.

.github/workflows/pr-validation.yml

…sions - Change group key from github.head_ref to github.event.pull_request.number - Prevents unrelated fork PRs with same branch name from cancelling each other 🔧 - Generated by Copilot

Copilot

Pull request overview

Copilot reviewed 4 out of 4 changed files in this pull request and generated no new comments.

WilliamBerryiii requested a review from a team as a code owner January 24, 2026 18:30

Copilot AI review requested due to automatic review settings January 24, 2026 18:30

Copilot started reviewing on behalf of WilliamBerryiii January 24, 2026 18:30 View session

Copilot AI reviewed Jan 24, 2026

View reviewed changes

.github/workflows/pr-validation.yml Show resolved Hide resolved

fix(build): use PR number for concurrency group to prevent fork colli…

f16c918

…sions - Change group key from github.head_ref to github.event.pull_request.number - Prevents unrelated fork PRs with same branch name from cancelling each other 🔧 - Generated by Copilot

agreaves-ms approved these changes Jan 24, 2026

View reviewed changes

Merge branch 'main' into build/workflow-concurrency-controls

428d5e1

Copilot AI review requested due to automatic review settings January 25, 2026 00:31

Copilot started reviewing on behalf of WilliamBerryiii January 25, 2026 00:32 View session

WilliamBerryiii merged commit c3d2fe9 into main Jan 25, 2026
21 checks passed

WilliamBerryiii deleted the build/workflow-concurrency-controls branch January 25, 2026 00:33

Copilot AI reviewed Jan 25, 2026

View reviewed changes

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

build(build): add concurrency controls to orchestrator workflows#275

build(build): add concurrency controls to orchestrator workflows#275
WilliamBerryiii merged 3 commits intomainfrom
build/workflow-concurrency-controls

WilliamBerryiii commented Jan 24, 2026

Uh oh!

github-actions bot commented Jan 24, 2026 •

edited

Loading

Uh oh!

codecov-commenter commented Jan 24, 2026 •

edited

Loading

Uh oh!

Copilot AI left a comment

Uh oh!

Uh oh!

Uh oh!

Copilot AI left a comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants

Conversation

WilliamBerryiii commented Jan 24, 2026

Description

Related Issue(s)

Type of Change

Sample Prompts (for AI Artifact Contributions)

Testing

Checklist

Required Checks

AI Artifact Contributions

Required Automated Checks

Security Considerations

Additional Notes

Concurrency Configuration Patterns

Dictionary Updates

Uh oh!

github-actions bot commented Jan 24, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Dependency Review

Scanned Files

Uh oh!

codecov-commenter commented Jan 24, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Codecov Report

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Reviewed changes

Uh oh!

Uh oh!

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants

github-actions bot commented Jan 24, 2026 •

edited

Loading

codecov-commenter commented Jan 24, 2026 •

edited

Loading