Skip to content

[WIP] Update workflow to parallelize frontend validation from backend execution#989

Merged
dgee2 merged 17 commits into
mainfrom
codex/parallelize-frontend-validation
Apr 20, 2026
Merged

[WIP] Update workflow to parallelize frontend validation from backend execution#989
dgee2 merged 17 commits into
mainfrom
codex/parallelize-frontend-validation

Conversation

@Codex

@Codex Codex AI commented Apr 15, 2026

Copy link
Copy Markdown
Contributor

Thanks for asking me to work on this. I will get started on it and keep this PR's description up to date as I form a plan and make progress.


This section details on the original issue you should resolve

<issue_title>Phase 2: Parallelize frontend validation from backend test execution</issue_title>
<issue_description>## Summary
Restructure the workflow so frontend validation no longer waits behind the full backend lane.

Change

Create separate jobs for:

  • backend tests
  • frontend validation
  • optional small OpenAPI-producing artifact job if required by the frontend

Why

The main structural bottleneck is serialization. Frontend work currently trails the backend instead of running concurrently.

Acceptance criteria

  • Frontend runs in a separate job
  • Frontend depends only on the minimum backend artifact it needs
  • Total wall-clock time improves without reducing coverage
  • Workflow graph remains clear and maintainable</issue_description>

Comments on the Issue (you are @codex[agent] in this section)

Co-authored-by: dgee2 <5671841+dgee2@users.noreply.github.com>
@Codex Codex AI requested a review from dgee2 April 15, 2026 21:47
@dgee2 dgee2 marked this pull request as ready for review April 15, 2026 21:49
Copilot AI review requested due to automatic review settings April 15, 2026 21:49
Comment thread .github/workflows/main.yml Fixed
Comment thread .github/workflows/main.yml Fixed

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Restructures the CI workflow to decouple frontend validation from backend execution by introducing a dedicated OpenAPI artifact job that the frontend can depend on.

Changes:

  • Added an openapi job to build the API and upload an OpenAPI spec artifact.
  • Split CI into separate backend and frontend jobs, with frontend depending on openapi.
  • Updated the workflow graph so frontend validation can run in parallel with backend tests.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread .github/workflows/main.yml Outdated
Co-authored-by: dgee2 <5671841+dgee2@users.noreply.github.com>
Co-authored-by: dgee2 <5671841+dgee2@users.noreply.github.com>
Comment thread .github/workflows/main.yml Outdated
Co-authored-by: dgee2 <5671841+dgee2@users.noreply.github.com>

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

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


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread .github/workflows/main.yml
Comment thread .github/workflows/main.yml Outdated
Comment thread .github/workflows/main.yml Outdated
Comment thread .github/workflows/main.yml Outdated
Claude AI and others added 2 commits April 16, 2026 09:22
The --project flag is not supported by dotnet test. Use path argument directly.

This fixes the build failure in PR #989 where the backend tests job was failing with:
"Get projects properties with MSBuild didn't execute properly with exit code: 1"

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>

Co-authored-by: dgee2 <5671841+dgee2@users.noreply.github.com>
The backend-tests job downloads pre-built artifacts but still needs
NuGet package dependencies restored before running tests with --no-build.
This fixes the error where dotnet test couldn't locate required dependencies.

Co-authored-by: dgee2 <5671841+dgee2@users.noreply.github.com>
Comment thread .github/workflows/main.yml Outdated
Comment thread .github/workflows/main.yml Outdated
Comment thread .github/workflows/main.yml Fixed
Comment thread .github/workflows/main.yml Outdated
Comment thread .github/workflows/main.yml Fixed
@sonarqubecloud

Copy link
Copy Markdown

@dgee2 dgee2 enabled auto-merge (squash) April 20, 2026 07:52
@dgee2 dgee2 disabled auto-merge April 20, 2026 08:02
@dgee2 dgee2 merged commit 4dfd0e3 into main Apr 20, 2026
13 checks passed
@dgee2 dgee2 deleted the codex/parallelize-frontend-validation branch April 20, 2026 08:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Phase 2: Parallelize frontend validation from backend test execution

6 participants