Skip to content

Add parallel execution and progress file configuration schemas#280

Merged
subsy merged 1 commit intomainfrom
claude/investigate-issue-278-IRYgv
Feb 4, 2026
Merged

Add parallel execution and progress file configuration schemas#280
subsy merged 1 commit intomainfrom
claude/investigate-issue-278-IRYgv

Conversation

@subsy
Copy link
Copy Markdown
Owner

@subsy subsy commented Feb 4, 2026

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 selection
      • maxWorkers: Integer between 1-32 for concurrent worker limit
      • worktreeDir: Directory path for git worktrees
      • directMerge: Boolean flag for direct branch merging
    • progressFile: New optional string field in StoredConfigSchema for progress tracking
  • Updated StoredConfigSchema:

    • Added progressFile field for cross-iteration context persistence
    • Added parallel field with full ParallelConfigSchema support
  • Comprehensive Test Coverage:

    • ParallelModeSchema tests: valid modes, invalid input rejection
    • ParallelConfigSchema tests: full/partial configs, optional fields, bounds validation, type validation
    • StoredConfigSchema tests: progressFile field, parallel configuration (full/partial), combined usage, validation of nested constraints

Implementation Details

  • All new fields are optional to maintain backward compatibility
  • maxWorkers enforces strict integer bounds (1-32) to prevent resource exhaustion
  • Parallel configuration is fully optional and can be partially specified
  • Tests validate both happy paths and error cases for robust schema enforcement

https://claude.ai/code/session_01PWHJ8gPeEp6FckvPj7L4Rp

Summary by CodeRabbit

Release Notes

  • New Features

    • Added parallel execution configuration with selectable modes ('auto', 'always', 'never').
    • Configurable worker thread limits (1–32 workers maximum).
    • Added progress file support for cross-iteration context tracking.
  • Tests

    • Comprehensive test coverage for new configuration schemas and validation rules.

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
@vercel
Copy link
Copy Markdown

vercel bot commented Feb 4, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

1 Skipped Deployment
Project Deployment Actions Updated (UTC)
ralph-tui Ignored Ignored Feb 4, 2026 5:59pm

Request Review

@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai bot commented Feb 4, 2026

Walkthrough

The 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

Cohort / File(s) Summary
Schema Definitions
src/config/schema.ts
Introduced ParallelModeSchema (enum: 'auto', 'always', 'never') and ParallelConfigSchema (object with mode, maxWorkers [1–32], worktreeDir, directMerge). Extended StoredConfigSchema with optional progressFile (string) and parallel (ParallelConfigSchema) fields.
Test Coverage
src/config/schema.test.ts
Added comprehensive tests for ParallelModeSchema and ParallelConfigSchema validation; extended StoredConfigSchema tests to cover progressFile field and nested parallel configuration with constraints and partial configuration support.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title accurately summarises the main changes: addition of schemas for parallel execution configuration and progress file tracking, matching the primary objectives of the PR.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing touches
  • 📝 Generate docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch claude/investigate-issue-278-IRYgv

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.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@codecov
Copy link
Copy Markdown

codecov bot commented Feb 4, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 43.70%. Comparing base (dda82e5) to head (57ca874).
⚠️ Report is 2 commits behind head on main.

Additional details and impacted files

Impacted file tree graph

@@            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              
Files with missing lines Coverage Δ
src/config/schema.ts 95.16% <100.00%> (+0.37%) ⬆️
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@subsy subsy merged commit df431d9 into main Feb 4, 2026
9 checks passed
@subsy subsy deleted the claude/investigate-issue-278-IRYgv branch February 4, 2026 18:02
sakaman pushed a commit to sakaman/ralph-tui that referenced this pull request Feb 15, 2026
…IRYgv

Add parallel execution and progress file configuration schemas
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants