Commit Graph
5 Commits
Author SHA1 Message Date
jason.woltje 7bc37fc513 Merge branch 'develop' into feature/229-performance-testing 2026-02-05 19:33:06 +00:00
Jason WoltjeandClaude Opus 4.5 c68b541b6f fix(#226): Remediate code review findings for E2E tests
- Fix CRITICAL: Remove unused imports (Test, TestingModule, CleanupService)
- Fix CRITICAL: Remove unused mockValkeyService declaration
- Fix IMPORTANT: Rename misleading test describe/names to match actual behavior
- Fix IMPORTANT: Verify spawned agents exist before kill-all assertion

Co-Authored-By: Claude Opus 4.5 <[email protected]>
2026-02-05 13:26:21 -06:00
Jason WoltjeandClaude Opus 4.5 0796cbc744 fix(#229): Remediate code review findings for performance tests
- Fix CRITICAL: Increase single-spawn threshold from 10ms to 50ms (CI flakiness)
- Fix CRITICAL: Replace no-op validation test with real backoff scale tests
- Fix IMPORTANT: Add warmup iterations before all timed measurements
- Fix IMPORTANT: Increase scan position ratio tolerance to 10x for sub-ms noise
- Refactored queue perf tests to use actual service methods (calculateBackoffDelay)
- Helper function to reduce spawn request duplication

Co-Authored-By: Claude Opus 4.5 <[email protected]>
2026-02-05 13:23:19 -06:00
Jason WoltjeandClaude Opus 4.5 b93f4c59ce test(#229): Add performance test suite for orchestrator
Add 14 performance benchmarks across 3 test files:
- Spawner throughput: single/sequential/concurrent spawn latency,
  session lookup, list performance, memory efficiency
- Queue service: backoff calculation throughput, validation perf
- Secret scanner: content scanning throughput, pattern scalability

Adds test:perf script to package.json.

Fixes #229

Co-Authored-By: Claude Opus 4.5 <[email protected]>
2026-02-05 12:52:30 -06:00
Jason WoltjeandClaude Opus 4.5 c8c81fc437 test(#226,#227,#228): Add E2E integration tests for agent orchestration
Add comprehensive E2E test suites covering:
- Full agent lifecycle (spawn → running → completed/failed) - 7 tests
- Killswitch emergency stop mechanism (single/all/partial) - 5 tests
- Concurrent agent spawning and isolation - 5 tests

Includes vitest config for integration test runner with 30s timeout.

Fixes #226
Fixes #227
Fixes #228

Co-Authored-By: Claude Opus 4.5 <[email protected]>
2026-02-05 12:46:44 -06:00