Comparing changes

- Added Stage#id with SecureRandom.hex(8) for unique identification - Created NameRegistry for centralized stage management - All changes are purely additive (non-breaking) - Tests: 203 examples, 1 failure (pre-existing: 31_configurable_pipeline.rb), 13 pending

- Enhanced find_stage to work with both names (backward compatible) and IDs - Tries name lookup first, then falls back to ID lookup - All changes are backward compatible - Tests: 368 examples, 1 failure (pre-existing: 31_configurable_pipeline.rb), 4 pending

- Added normalize_identifier helper method (currently no-op) - Sets up infrastructure for future ID-based internal operations - All changes are purely additive and backward compatible - Tests: 61 examples, 1 failure (pre-existing: 31_configurable_pipeline.rb), 1 pending

- Stage.initialize now supports both old keyword and new positional signatures - New style: Stage.new(pipeline, name, block, options) - Old style: Stage.new(name: :foo, block: proc {}, options: {}) - still works - Added pipeline attribute to Stage for future use - All existing tests pass - fully backward compatible - Tests: 198 examples, 1 failure (pre-existing: 31_configurable_pipeline.rb), 4 pending

- Added merge_nested_pipeline_into_dag method to Pipeline - Recursively builds and merges nested pipeline DAGs into parent - Enables future direct parentâ†’nested stage routing - Method exists but is not activated yet (purely infrastructure) - All existing tests pass - no behavior changes - Tests: 102 examples, 1 failure (pre-existing: 31_configurable_pipeline.rb), 2 pending

- Pipeline.initialize now supports both old and new signatures - New style: Pipeline.new(task, name, config, ...) - Old style: Pipeline.new(name, config, ...) - still works - Added task attribute to Pipeline for parent-child relationship - All existing tests pass - fully backward compatible - Tests: 140 examples, 1 failure (pre-existing), 2 pending

- Added registry (NameRegistry) to Task initialization - Stages auto-register with registry when created via new signature - Registration happens in Stage#initialize if pipeline.task.registry exists - Old-style stages (without pipeline) skip registration gracefully - All existing tests pass - backward compatible - Tests: 506 examples, 1 failure (pre-existing), 5 pending

@stages

- Added @stages_by_id hash alongside @stages for ID-based lookups - Populate @stages_by_id whenever stages are added to pipeline - Updated find_stage to use @stages_by_id (faster than scanning) - All stage addition points updated: add_stage, routers, entrance/exit, nested - Maintains full backward compatibility - @stages still works by name - Tests: 506 examples, 1 failure (pre-existing), 5 pending

@stages

- Added @stages_by_id hash for parallel ID-based lookups - Added stage_id to StageContext (alongside stage_name for compatibility) - Updated Worker to populate both stage_name and stage_id - Fixed worker_spec doubles to include id attribute - find_stage now uses @stages_by_id for O(1) ID lookups - All stage additions populate both @stages and @stages_by_id - Tests: 506 examples, 1 failure (pre-existing), 5 pending

- Added display_name method that returns name if available, otherwise ID - Useful for logging and debugging - Sets up for future where stages may not have names - Tests: 506 examples, 1 failure (pre-existing: 31_configurable_pipeline.rb), 5 pending

- Created INCREMENTAL_REFACTOR_SUMMARY.md documenting all changes - 8 commits, 14 steps completed successfully - All infrastructure for ID-based operations in place - Full backward compatibility maintained - Tests: 506 examples, 1 failure (pre-existing), 5 pending

- Added auto-generation of stage names using _xxxxxxxx format (8 char hex) - Ensures all stages have names for routing support - Updated display_name to always return the name (now guaranteed non-nil) - Tests: 506 examples, 1 failure (pre-existing), 5 pending

@stages

- @stages is now an Array of Stage objects (not a hash) - @stages_by_name is a Hash for name â†’ stage lookups - Updated all @stages accesses in Pipeline and Task - Updated iteration patterns from .each |name, stage| to .each |stage| - Updated find_stage to use @stages_by_name - Tests: 506 examples, 22 failures, 5 pending (down from 68!)

Navigation Menu

Repository navigation

Comparing changes

Open a pull request

Uh oh!

Commits on Nov 1, 2025

This comparison is taking too long to generate.

Uh oh!