UAT: RouteConfig.parallel_execution defaults to True instead of spec-required False #1433

New issue

Open

opened 2026-04-02 17:51:16 +00:00 by freemo · 0 comments

freemo commented

2026-04-02 17:51:16 +00:00

Owner

Bug Report

Feature Area: Actor Graphs (v3.3.0)
Tested by: UAT instance uat-worker-actor-graphs-1

What Was Tested

Default value of parallel_execution field in RouteConfig for graph routes.

Expected Behavior (from spec)

The specification explicitly states in the Graph Routes YAML example:

parallel_execution: false  # Allow parallel node execution (default: false)

Actual Behavior

RouteConfig.parallel_execution defaults to True in the implementation:

# src/cleveragents/reactive/route.py
parallel_execution: bool = True  # Should be False

Steps to Reproduce

from cleveragents.reactive.route import RouteConfig
fields = RouteConfig.model_fields
print(fields['parallel_execution'].default)  # True — should be False

Code Location

src/cleveragents/reactive/route.py, RouteConfig class:

parallel_execution: bool = True

Should be:

parallel_execution: bool = False

Severity

Medium — incorrect default could cause unexpected parallel execution behavior in graph routes that don't explicitly set this field.

Metadata

Branch: bugfix/route-config-parallel-execution-default
Commit Message: fix(reactive): set RouteConfig.parallel_execution default to False per spec
Milestone: v3.3.0
Parent Epic: (needs manual linking — Actor Graphs epic)

Subtasks

Change parallel_execution: bool = True to parallel_execution: bool = False in src/cleveragents/reactive/route.py
Tests (Behave): Add/update scenario verifying RouteConfig.parallel_execution defaults to False
Tests (Robot): Add/update integration test verifying graph routes default to sequential execution
Audit any existing tests that may rely on the incorrect True default and update them
Verify coverage >= 97% via nox -s coverage_report
Run nox (all default sessions), fix any errors

Definition of Done

RouteConfig.parallel_execution defaults to False
Graph routes that do not explicitly set parallel_execution execute sequentially by default
No existing tests rely on the incorrect True default
All nox stages pass
Coverage >= 97%

## Bug Report **Feature Area:** Actor Graphs (v3.3.0) **Tested by:** UAT instance uat-worker-actor-graphs-1 ### What Was Tested Default value of `parallel_execution` field in `RouteConfig` for graph routes. ### Expected Behavior (from spec) The specification explicitly states in the Graph Routes YAML example: ```yaml parallel_execution: false # Allow parallel node execution (default: false) ``` ### Actual Behavior `RouteConfig.parallel_execution` defaults to `True` in the implementation: ```python # src/cleveragents/reactive/route.py parallel_execution: bool = True # Should be False ``` ### Steps to Reproduce ```python from cleveragents.reactive.route import RouteConfig fields = RouteConfig.model_fields print(fields['parallel_execution'].default) # True — should be False ``` ### Code Location `src/cleveragents/reactive/route.py`, `RouteConfig` class: ```python parallel_execution: bool = True ``` Should be: ```python parallel_execution: bool = False ``` ### Severity Medium — incorrect default could cause unexpected parallel execution behavior in graph routes that don't explicitly set this field. --- ## Metadata - **Branch**: `bugfix/route-config-parallel-execution-default` - **Commit Message**: `fix(reactive): set RouteConfig.parallel_execution default to False per spec` - **Milestone**: v3.3.0 - **Parent Epic**: *(needs manual linking — Actor Graphs epic)* ## Subtasks - [ ] Change `parallel_execution: bool = True` to `parallel_execution: bool = False` in `src/cleveragents/reactive/route.py` - [ ] Tests (Behave): Add/update scenario verifying `RouteConfig.parallel_execution` defaults to `False` - [ ] Tests (Robot): Add/update integration test verifying graph routes default to sequential execution - [ ] Audit any existing tests that may rely on the incorrect `True` default and update them - [ ] Verify coverage >= 97% via `nox -s coverage_report` - [ ] Run `nox` (all default sessions), fix any errors ## Definition of Done - [ ] `RouteConfig.parallel_execution` defaults to `False` - [ ] Graph routes that do not explicitly set `parallel_execution` execute sequentially by default - [ ] No existing tests rely on the incorrect `True` default - [ ] All nox stages pass - [ ] Coverage >= 97%

Rows
Columns