Skip to content

fix(tools): reject non-positive timeout decorator seconds#805

Open
Bartok9 wants to merge 1 commit into
RobotecAI:mainfrom
Bartok9:fix/tools-timeout-reject-nonpositive
Open

fix(tools): reject non-positive timeout decorator seconds#805
Bartok9 wants to merge 1 commit into
RobotecAI:mainfrom
Bartok9:fix/tools-timeout-reject-nonpositive

Conversation

@Bartok9

@Bartok9 Bartok9 commented Jul 16, 2026

Copy link
Copy Markdown

Summary

  • timeout / timeout_method accepted 0, negatives, and non-finite values
  • Normalize and validate at decorate time; raise ValueError if not positive finite
  • Existing RaiTimeoutError behavior for runtime timeouts unchanged

Test plan

  • Offline importlib validation of decorator configure failures
  • Extended tests/tools/test_timeout.py for 0 / -1 / NaN

Spec: aerial specs/rai/2026-07-16-2.md

Validate timeout/timeout_method durations are positive and finite at
decorate time so zero/negative/NaN configs fail closed with ValueError.
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.

1 participant