UAT: RouteConfig.buffer_size defaults to 1 instead of spec-required 10 #1434

New issue

Open

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

freemo commented

2026-04-02 17:51:51 +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 buffer_size field in RouteConfig for stream routes.

Expected Behavior (from spec)

The specification explicitly states in the Stream Routes YAML example:

buffer_size: 10            # Stream buffer size (default: 10)

And in the route field reference table:

buffer_size: 10  # Stream buffer size (optional, default: 10)

Actual Behavior

RouteConfig.buffer_size defaults to 1 in the implementation:

# src/cleveragents/reactive/route.py
buffer_size: int = 1  # Should be 10

Steps to Reproduce

from cleveragents.reactive.route import RouteConfig
fields = RouteConfig.model_fields
print(fields['buffer_size'].default)  # 1 — should be 10

Code Location

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

buffer_size: int = 1

Should be:

buffer_size: int = 10

Severity

Low — incorrect default may cause stream backpressure issues in stream routes that don't explicitly set buffer_size.

Metadata

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

Subtasks

Change buffer_size: int = 1 to buffer_size: int = 10 in src/cleveragents/reactive/route.py
Tests (Behave): Add/update scenario verifying RouteConfig.buffer_size defaults to 10
Tests (Robot): Add/update integration test verifying stream routes use buffer size of 10 by default
Audit any existing tests that may rely on the incorrect 1 default and update them
Verify coverage >= 97% via nox -s coverage_report
Run nox (all default sessions), fix any errors

Definition of Done

RouteConfig.buffer_size defaults to 10
Stream routes that do not explicitly set buffer_size use a buffer of 10 by default
No existing tests rely on the incorrect 1 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 `buffer_size` field in `RouteConfig` for stream routes. ### Expected Behavior (from spec) The specification explicitly states in the Stream Routes YAML example: ```yaml buffer_size: 10 # Stream buffer size (default: 10) ``` And in the route field reference table: ``` buffer_size: 10 # Stream buffer size (optional, default: 10) ``` ### Actual Behavior `RouteConfig.buffer_size` defaults to `1` in the implementation: ```python # src/cleveragents/reactive/route.py buffer_size: int = 1 # Should be 10 ``` ### Steps to Reproduce ```python from cleveragents.reactive.route import RouteConfig fields = RouteConfig.model_fields print(fields['buffer_size'].default) # 1 — should be 10 ``` ### Code Location `src/cleveragents/reactive/route.py`, `RouteConfig` class: ```python buffer_size: int = 1 ``` Should be: ```python buffer_size: int = 10 ``` ### Severity Low — incorrect default may cause stream backpressure issues in stream routes that don't explicitly set buffer_size. --- ## Metadata - **Branch**: `bugfix/route-config-buffer-size-default` - **Commit Message**: `fix(reactive): set RouteConfig.buffer_size default to 10 per spec` - **Milestone**: v3.3.0 - **Parent Epic**: *(needs manual linking — Actor Graphs epic)* ## Subtasks - [ ] Change `buffer_size: int = 1` to `buffer_size: int = 10` in `src/cleveragents/reactive/route.py` - [ ] Tests (Behave): Add/update scenario verifying `RouteConfig.buffer_size` defaults to `10` - [ ] Tests (Robot): Add/update integration test verifying stream routes use buffer size of 10 by default - [ ] Audit any existing tests that may rely on the incorrect `1` default and update them - [ ] Verify coverage >= 97% via `nox -s coverage_report` - [ ] Run `nox` (all default sessions), fix any errors ## Definition of Done - [ ] `RouteConfig.buffer_size` defaults to `10` - [ ] Stream routes that do not explicitly set `buffer_size` use a buffer of 10 by default - [ ] No existing tests rely on the incorrect `1` default - [ ] All nox stages pass - [ ] Coverage >= 97%

Rows
Columns