Deprecate WorkerPool v1 #39

New issue

Closed

opened 2026-02-15 03:50:47 +03:00 by NiXTheDev · 0 comments

NiXTheDev commented

2026-02-15 03:50:47 +03:00

(Migrated from github.com)

Description

Deprecate and remove WorkerPool v1 implementation, making WorkerPoolV2 the standard worker pool.

Background

WorkerPool v1 was the original implementation but lacks:

Dynamic scaling capabilities
Health monitoring integration
Idle worker management
Proper error recovery

WorkerPoolV2 provides all these features and is already feature-complete.

Implementation Plan

Verify WorkerPoolV2 parity
- Ensure WorkerPoolV2 implements all methods from IWorkerPool
- Test coverage for all edge cases
- Verify graceful shutdown behavior matches v1
Remove WorkerPool v1 files
- Delete
- Remove v1-specific tests
Rename WorkerPoolV2
- Rename →
- Update class name from to
- Update all imports across codebase
- Rename test file →
Update configuration
- Remove env var
- Make WorkerPoolV2 behavior the default
- Update .env.example
- Update README documentation
Clean up references
- Remove conditional logic checking
- Update type imports
- Clean up dead code in index.ts

Acceptance Criteria

WorkerPool v1 completely removed
WorkerPoolV2 renamed to WorkerPool
All tests pass (166)
Zero lint/type errors
Documentation updated
Configuration simplified

Part of Epic #38

## Description Deprecate and remove WorkerPool v1 implementation, making WorkerPoolV2 the standard worker pool. ## Background WorkerPool v1 was the original implementation but lacks: - Dynamic scaling capabilities - Health monitoring integration - Idle worker management - Proper error recovery WorkerPoolV2 provides all these features and is already feature-complete. ## Implementation Plan 1. **Verify WorkerPoolV2 parity** - Ensure WorkerPoolV2 implements all methods from IWorkerPool - Test coverage for all edge cases - Verify graceful shutdown behavior matches v1 2. **Remove WorkerPool v1 files** - Delete - Remove v1-specific tests 3. **Rename WorkerPoolV2** - Rename → - Update class name from to - Update all imports across codebase - Rename test file → 4. **Update configuration** - Remove env var - Make WorkerPoolV2 behavior the default - Update .env.example - Update README documentation 5. **Clean up references** - Remove conditional logic checking - Update type imports - Clean up dead code in index.ts ## Acceptance Criteria - [ ] WorkerPool v1 completely removed - [ ] WorkerPoolV2 renamed to WorkerPool - [ ] All tests pass (166) - [ ] Zero lint/type errors - [ ] Documentation updated - [ ] Configuration simplified ## Related Part of Epic #38