Skip to content

Commit 85eef29

Browse files
committed
🤖 Update analysis doc - clarify --silent is intentional
Document that --silent is necessary for log hygiene with 64 parallel workers, and can be bypassed for debugging via workflow_dispatch.
1 parent a2c9695 commit 85eef29

File tree

1 file changed

+5
-4
lines changed

1 file changed

+5
-4
lines changed

CI_SPEED_ANALYSIS.md

Lines changed: 5 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -36,15 +36,16 @@ Total: 228s
3636
- Jest config: `maxWorkers=200%` (64 parallel workers)
3737
- Test files: 17 files in `tests/ipcMain/` + `tests/runtime/`
3838
- Flags: `--coverage --maxWorkers=200% --silent`
39+
- Note: `--silent` is necessary to prevent overwhelming log output from 64 parallel workers
3940

4041
#### Potential Issues
4142
1. **Over-parallelization**: 64 workers on 32 cores may cause contention
4243
- Each test spawns child processes, file I/O, git operations
4344
- May be better to use fewer workers (100% or 150%)
4445

45-
2. **Silent mode hides timing data**: Can't see which tests are slow
46-
- Remove `--silent` temporarily to profile
47-
- Add `--verbose` to see timing per test
46+
2. **Silent mode needed for log hygiene**: 17 test files × 64 workers = overwhelming output
47+
- Silent mode suppresses per-test spam while keeping suite status, timing, and failures
48+
- Can use workflow_dispatch with test_filter to run specific tests verbosely for debugging
4849

4950
3. **Coverage overhead**: Jest coverage adds significant overhead
5051
- Consider running coverage only on specific files
@@ -55,9 +56,9 @@ Total: 228s
5556
- Check if any tests can be simplified
5657

5758
#### Action Items
58-
- [ ] Profile tests without `--silent` to identify slow tests
5959
- [ ] Experiment with `maxWorkers=100%` vs `200%` vs `150%`
6060
- [ ] Measure overhead of `--coverage` flag
61+
- [ ] Use workflow_dispatch with test_filter to profile specific slow tests
6162
- [ ] Review slowest test files for optimization opportunities
6263
- [ ] Check if test files can share more setup code
6364

0 commit comments

Comments
 (0)