Skip to content

ci(fork): split server tests from workspace tests - #130

Merged
incognitojam merged 1 commit into
mainfrom
t3code/speed-up-vite-ci-tests
Aug 14, 2026
Merged

ci(fork): split server tests from workspace tests#130
incognitojam merged 1 commit into
mainfrom
t3code/speed-up-vite-ci-tests

Conversation

@incognitojam

Copy link
Copy Markdown
Collaborator

Note

TL;DR: Run the serial server suite and the remaining workspace suites on separate runners so their CI time overlaps.

The Fork Test job currently waits for every dependency package suite before starting the server suite. Since server test files deliberately run serially, those two long phases add together on the critical path.

This splits the job into server and workspace jobs. The workspace job explicitly selects all 13 non-server test packages and retains the isolated image-compression test, Electron setup, and Rust resource-monitor tests.

Verification:

  • actionlint .github/workflows/fork-ci.yml
  • Confirmed the workspace selector resolves to exactly 13 non-server packages
  • Confirmed the server selector resolves only to apps/server

Written by an agent (T3 Code, gpt-5.6-sol).

@incognitojam
incognitojam enabled auto-merge (squash) August 14, 2026 13:17
@incognitojam
incognitojam disabled auto-merge August 14, 2026 13:25
@incognitojam
incognitojam merged commit c391928 into main Aug 14, 2026
9 checks passed
@incognitojam
incognitojam deleted the t3code/speed-up-vite-ci-tests branch August 14, 2026 13:25
yngatech-nightly Bot pushed a commit that referenced this pull request Aug 14, 2026
> [!NOTE]
> TL;DR: Run the serial server suite and the remaining workspace suites
on separate runners so their CI time overlaps.

The Fork Test job currently waits for every dependency package suite
before starting the server suite. Since server test files deliberately
run serially, those two long phases add together on the critical path.

This splits the job into server and workspace jobs. The workspace job
explicitly selects all 13 non-server test packages and retains the
isolated image-compression test, Electron setup, and Rust
resource-monitor tests.

Verification:
- actionlint .github/workflows/fork-ci.yml
- Confirmed the workspace selector resolves to exactly 13 non-server
packages
- Confirmed the server selector resolves only to apps/server

---
Written by an agent (T3 Code, gpt-5.6-sol).
incognitojam added a commit that referenced this pull request Aug 15, 2026
> [!NOTE]
> TL;DR: Run the serial server suite and the remaining workspace suites
on separate runners so their CI time overlaps.

The Fork Test job currently waits for every dependency package suite
before starting the server suite. Since server test files deliberately
run serially, those two long phases add together on the critical path.

This splits the job into server and workspace jobs. The workspace job
explicitly selects all 13 non-server test packages and retains the
isolated image-compression test, Electron setup, and Rust
resource-monitor tests.

Verification:
- actionlint .github/workflows/fork-ci.yml
- Confirmed the workspace selector resolves to exactly 13 non-server
packages
- Confirmed the server selector resolves only to apps/server

---
Written by an agent (T3 Code, gpt-5.6-sol).
yngatech-nightly Bot pushed a commit that referenced this pull request Aug 18, 2026
> [!NOTE]
> TL;DR: Run the serial server suite and the remaining workspace suites
on separate runners so their CI time overlaps.

The Fork Test job currently waits for every dependency package suite
before starting the server suite. Since server test files deliberately
run serially, those two long phases add together on the critical path.

This splits the job into server and workspace jobs. The workspace job
explicitly selects all 13 non-server test packages and retains the
isolated image-compression test, Electron setup, and Rust
resource-monitor tests.

Verification:
- actionlint .github/workflows/fork-ci.yml
- Confirmed the workspace selector resolves to exactly 13 non-server
packages
- Confirmed the server selector resolves only to apps/server

---
Written by an agent (T3 Code, gpt-5.6-sol).
yngatech-nightly Bot pushed a commit that referenced this pull request Aug 18, 2026
> [!NOTE]
> TL;DR: Run the serial server suite and the remaining workspace suites
on separate runners so their CI time overlaps.

The Fork Test job currently waits for every dependency package suite
before starting the server suite. Since server test files deliberately
run serially, those two long phases add together on the critical path.

This splits the job into server and workspace jobs. The workspace job
explicitly selects all 13 non-server test packages and retains the
isolated image-compression test, Electron setup, and Rust
resource-monitor tests.

Verification:
- actionlint .github/workflows/fork-ci.yml
- Confirmed the workspace selector resolves to exactly 13 non-server
packages
- Confirmed the server selector resolves only to apps/server

---
Written by an agent (T3 Code, gpt-5.6-sol).
yngatech-nightly Bot pushed a commit that referenced this pull request Aug 18, 2026
> [!NOTE]
> TL;DR: Run the serial server suite and the remaining workspace suites
on separate runners so their CI time overlaps.

The Fork Test job currently waits for every dependency package suite
before starting the server suite. Since server test files deliberately
run serially, those two long phases add together on the critical path.

This splits the job into server and workspace jobs. The workspace job
explicitly selects all 13 non-server test packages and retains the
isolated image-compression test, Electron setup, and Rust
resource-monitor tests.

Verification:
- actionlint .github/workflows/fork-ci.yml
- Confirmed the workspace selector resolves to exactly 13 non-server
packages
- Confirmed the server selector resolves only to apps/server

---
Written by an agent (T3 Code, gpt-5.6-sol).
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant