Skip to content

fix(workflows): isolate concurrency for each pull request - #688

Merged
efraespada merged 1 commit into
developfrom
bugfix/687-fixworkflows-isolate-pr-concurrency-by-pull-request-identity
Oct 24, 2026
Merged

efraespada merged 1 commit into
developfrom
bugfix/687-fixworkflows-isolate-pr-concurrency-by-pull-request-identity

Conversation

@efraespada

@efraespada efraespada commented Oct 24, 2026 •

Copy link
Copy Markdown
Member

Pull Request analysis and review-state runs now use each PR’s number and lane for concurrency identity. This prevents PRs that share a head branch from canceling one another while preserving replacement of superseded runs for the same PR.

What changed

  • Updated shipped and setup workflow templates to key PR analysis and review-state concurrency groups by repository, PR number, and lane.
  • Added workflow contract cases for shared-branch isolation, stable PR identity, distinct lanes, and rejection of the old branch-based groups.
  • Updated the reconciliation specification, catalog, and workflow documentation with the new identity and consumer update guidance.

Validation

  • Six regression cases pass and fail against the previous implementation.
  • Workflow contract: 302/302.
  • The author reports build, 712 suites (11,991 cases passing; 35 platform omissions), lint, typecheck, specification and documentation checks, and package/smoke checks succeeded.
  • The author reports current-head checks succeeded and no findings remain pending.

Closes #687

…ty: scope review concurrency to pull request identity
@docs-page

docs-page Bot commented Oct 24, 2026

Copy link
Copy Markdown

To preview the documentation for this pull request, visit the following URL:

docs.page/vypdev/copilot~688

Documentation is deployed and generated using docs.page

@vypbot vypbot added size: XL Indicates that the issue or task is extra large in scope and complexity. 85% Progress: 85% state:ai-processing labels Oct 24, 2026
@vypbot
vypbot self-requested a review October 24, 2026 19:02
@vypbot vypbot added this to vypdev Oct 24, 2026
@vypbot vypbot moved this to In Progress in vypdev Oct 24, 2026
@vypbot
vypbot changed the base branch from develop to master October 24, 2026 19:03
@efraespada efraespada linked an issue Oct 24, 2026 that may be closed by this pull request
1 task done
@vypbot
vypbot changed the base branch from master to develop October 24, 2026 19:03
@vypbot

vypbot commented Oct 24, 2026

Copy link
Copy Markdown
Contributor

Bugbot: review complete

Current status: No active findings on 6244e4e.

Pull request · Verified commit · Workflow run

@codecov-commenter

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 94.14%. Comparing base (274db40) to head (6244e4e).
⚠️ Report is 1647 commits behind head on develop.

Additional details and impacted files
@@             Coverage Diff             @@
##           develop     #688      +/-   ##
===========================================
+ Coverage    90.94%   94.14%   +3.19%     
===========================================
  Files          135      974     +839     
  Lines         6251    31653   +25402     
  Branches      1338     9296    +7958     
===========================================
+ Hits          5685    29799   +24114     
- Misses         172      707     +535     
- Partials       394     1147     +753     

see 1041 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@vypbot

vypbot commented Oct 24, 2026

Copy link
Copy Markdown
Contributor

Copilot approval · Human review needed

Status: A protected workflow, policy, or trust-boundary file changed.
Completed: 1/1 observed checks succeeded; Bugbot complete.
Action required: Ask a human reviewer to decide.
Impact: No new approval is confirmed for this revision.
See checks

Technical evidence

Revision: 6244e4e17c9c; base: 0c25bfa36050; reason: protected-path.

@efraespada
efraespada merged commit 4dfe768 into develop Oct 24, 2026
12 checks passed
@github-project-automation github-project-automation Bot moved this from In Progress to Done in vypdev Oct 24, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

85% Progress: 85% size: XL Indicates that the issue or task is extra large in scope and complexity. state:verified

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

🐛🧑‍💻 - fixworkflows isolate PR concurrency by pull request identity

3 participants