Problem

pyproject.toml sets timeout = 30 globally for all tests (line 231). Despite this, 299 test files redundantly apply pytest.mark.timeout(30) at module or class level. These markers have no effect and add noise.

Proposed solution

1. Remove all redundant pytest.mark.timeout(30) markers across the 299 files in tests/
2. Keep intentional overrides -- 2 files use timeout(60) for e2e/docker tests (these are legitimate)
3. Add a pre-commit hook (or ruff rule) to prevent re-introduction of pytest.mark.timeout(30) since it matches the global default

Patterns to handle

• pytestmark = pytest.mark.timeout(30) -- remove entire line
• pytestmark = [pytest.mark.unit, pytest.mark.timeout(30)] -- simplify to pytestmark = pytest.mark.unit
• pytestmark = [pytest.mark.e2e, pytest.mark.timeout(30)] -- simplify to pytestmark = pytest.mark.e2e
• Similar list patterns with integration, etc.

Notes

• Global setting: pyproject.toml line 231: timeout = 30
• Non-default overrides (keep): tests/e2e/test_single_agent_e2e.py (60s), tests/integration/tools/test_docker_sandbox_integration.py (60s)