Add parallel execution and progress file configuration schemas#280
Add parallel execution and progress file configuration schemas#280
Conversation
The StoredConfigSchema was missing two fields that are defined in the StoredConfig TypeScript interface: progressFile and parallel. Since the schema uses .strict() mode, any config files containing these fields would fail validation silently, causing the app to use empty defaults. This fixes issue #278 where users with progressFile or parallel config sections would experience the app returning to terminal after selecting a beads task list. Changes: - Add ParallelModeSchema for 'auto' | 'always' | 'never' values - Add ParallelConfigSchema with mode, maxWorkers, worktreeDir, directMerge - Add progressFile field to StoredConfigSchema - Add parallel field to StoredConfigSchema - Add comprehensive tests for new schema fields https://claude.ai/code/session_01PWHJ8gPeEp6FckvPj7L4Rp
|
The latest updates on your projects. Learn more about Vercel for GitHub. |
WalkthroughThe pull request adds two new configuration schemas for parallel execution—ParallelModeSchema and ParallelConfigSchema—and extends the existing StoredConfigSchema with optional progressFile and nested parallel configuration fields. Changes
Estimated code review effort🎯 2 (Simple) | ⏱️ ~10 minutes 🚥 Pre-merge checks | ✅ 3✅ Passed checks (3 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing touches
🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #280 +/- ##
==========================================
+ Coverage 43.68% 43.70% +0.01%
==========================================
Files 94 94
Lines 29012 29021 +9
==========================================
+ Hits 12675 12684 +9
Misses 16337 16337
🚀 New features to boost your workflow:
|
…IRYgv Add parallel execution and progress file configuration schemas
Summary
This PR adds schema definitions and comprehensive test coverage for two new configuration features: parallel execution settings and progress file tracking.
Key Changes
New Schemas Added:
ParallelModeSchema: Enum schema validating execution modes ('auto', 'always', 'never')ParallelConfigSchema: Object schema for parallel execution configuration with fields:mode: Execution mode selectionmaxWorkers: Integer between 1-32 for concurrent worker limitworktreeDir: Directory path for git worktreesdirectMerge: Boolean flag for direct branch mergingprogressFile: New optional string field inStoredConfigSchemafor progress trackingUpdated StoredConfigSchema:
progressFilefield for cross-iteration context persistenceparallelfield with fullParallelConfigSchemasupportComprehensive Test Coverage:
ParallelModeSchematests: valid modes, invalid input rejectionParallelConfigSchematests: full/partial configs, optional fields, bounds validation, type validationStoredConfigSchematests: progressFile field, parallel configuration (full/partial), combined usage, validation of nested constraintsImplementation Details
maxWorkersenforces strict integer bounds (1-32) to prevent resource exhaustionhttps://claude.ai/code/session_01PWHJ8gPeEp6FckvPj7L4Rp
Summary by CodeRabbit
Release Notes
New Features
Tests