cleveragents/cleveragents-core

Fork 3

[Bug Hunt Cycle 2 Batch 3] CRITICAL: Container timeout buffer logic causes premature timeouts #7068

New issue

Open

opened 2026-04-10 07:28:54 +00:00 by HAL9000 · 1 comment

HAL9000 commented

2026-04-10 07:28:54 +00:00

Owner

Bug Report: Container Timeout Buffer — Premature Timeout for Short Durations

Severity Assessment

Impact: Tools with short timeouts (≤5 seconds) become unreliable, always timing out after 1 second
Likelihood: High - affects all short-duration tools in container environments
Priority: Critical

Location

File: src/cleveragents/tool/container_executor.py
Function: ContainerToolExecutor._build_exec_command()
Lines: 448

Description

The container timeout buffer logic uses container_timeout = max(timeout - 5, 1) which causes tools with timeouts ≤5 seconds to always get a 1-second container timeout, regardless of the intended timeout value.

Evidence

# Line 448:
container_timeout = max(timeout - 5, 1)

# Test cases:
# timeout=2 -> container_timeout = max(2-5, 1) = 1 second
# timeout=5 -> container_timeout = max(5-5, 1) = 1 second  
# timeout=3 -> container_timeout = max(3-5, 1) = 1 second

Expected Behavior

Short timeouts should be respected proportionally. A 2-second timeout should not become a 1-second timeout.

Actual Behavior

All timeouts ≤5 seconds become 1-second container timeouts, causing tools to timeout prematurely.

Reproduction Details

Create a tool with 2-second timeout
Execute it in container environment
Tool times out after 1 second instead of ~2 seconds
Affects scenarios: quick validation tools, simple file operations, lightweight commands

Suggested Fix

Use proportional buffer logic:

container_timeout = max(int(timeout * 0.9), timeout - 2, 1)

TDD Note

After this bug issue is verified, a corresponding Type/Testing issue will be created for TDD. The test will use tags: @tdd_issue, @tdd_issue_, and @tdd_expected_fail to prove the bug exists before fixing it.

Automated by CleverAgents Bot
Supervisor: Bug Hunting | Agent: bug-hunter
Worker Type: Module Scanner

## Bug Report: Container Timeout Buffer — Premature Timeout for Short Durations ### Severity Assessment - **Impact**: Tools with short timeouts (≤5 seconds) become unreliable, always timing out after 1 second - **Likelihood**: High - affects all short-duration tools in container environments - **Priority**: Critical ### Location - **File**: `src/cleveragents/tool/container_executor.py` - **Function**: `ContainerToolExecutor._build_exec_command()` - **Lines**: 448 ### Description The container timeout buffer logic uses `container_timeout = max(timeout - 5, 1)` which causes tools with timeouts ≤5 seconds to always get a 1-second container timeout, regardless of the intended timeout value. ### Evidence ```python # Line 448: container_timeout = max(timeout - 5, 1) # Test cases: # timeout=2 -> container_timeout = max(2-5, 1) = 1 second # timeout=5 -> container_timeout = max(5-5, 1) = 1 second # timeout=3 -> container_timeout = max(3-5, 1) = 1 second ``` ### Expected Behavior Short timeouts should be respected proportionally. A 2-second timeout should not become a 1-second timeout. ### Actual Behavior All timeouts ≤5 seconds become 1-second container timeouts, causing tools to timeout prematurely. ### Reproduction Details 1. Create a tool with 2-second timeout 2. Execute it in container environment 3. Tool times out after 1 second instead of ~2 seconds 4. Affects scenarios: quick validation tools, simple file operations, lightweight commands ### Suggested Fix Use proportional buffer logic: ```python container_timeout = max(int(timeout * 0.9), timeout - 2, 1) ``` ### Category boundary ### TDD Note After this bug issue is verified, a corresponding Type/Testing issue will be created for TDD. The test will use tags: @tdd_issue, @tdd_issue_<this-issue-number>, and @tdd_expected_fail to prove the bug exists before fixing it. --- **Automated by CleverAgents Bot** Supervisor: Bug Hunting | Agent: bug-hunter **Worker Type**: Module Scanner

HAL9000 referenced this issue

2026-04-10 07:30:47 +00:00

[AUTO-BUG-POOL] Status: Bug Detection Report (Cycle 2) #7023

HAL9000 added the

State

Unverified

label

2026-04-10 18:57:47 +00:00

freemo added the

labels

2026-04-12 18:37:33 +00:00

HAL9000 added

and removed

labels

2026-04-14 06:52:50 +00:00

HAL9000 commented

2026-04-14 06:52:50 +00:00

Author

Owner

✅ Verified — Critical bug: container timeout buffer logic causes premature timeouts. MoSCoW: Must-have. Priority: Critical.

Automated by CleverAgents Bot
Supervisor: Project Owner | Agent: project-owner-pool-supervisor

✅ **Verified** — Critical bug: container timeout buffer logic causes premature timeouts. MoSCoW: Must-have. Priority: Critical. --- **Automated by CleverAgents Bot** Supervisor: Project Owner | Agent: project-owner-pool-supervisor