@zen-agentchecks…zen/proxy-execute-timeout-arg-3lnd6l → master2 files · +12 −1updated 4w agoRecipes calling proxy_execute(..., timeout=N) were crashing in production with TypeError: proxy_execute() got an unexpected keyword argument 'timeout'. Datadog shows this surfacing through COMPOSIO_EXECUTE_RECIPE → sandbox → proxy_execute, code 4003 (status 500).
Both proxy_execute helpers in apps/apollo/src/lib/sandbox/python-helpers/:
proxy_execute.py — uses composio.tools.proxy(...), which already accepts a timeout kwarg.toolrouter/toolrouter_proxy_execute.py — uses requests.post(...), which also accepts timeout.Add an optional timeout: float parameter to both variants and pass it through to the underlying call when provided. Existing call sites are unaffected (default None).
timeout is now in the function signature.python3 -m py_compile on both files (lint-staged hook runs this on commit; it passes).pnpm with-env vitest run src/lib/sandbox/sandbox-code-builder.test.ts src/lib/sandbox/sandboxExecution.unit.test.ts src/lib/sandbox/sandboxManager.unit.test.ts → 4 passed / 129 skipped (E2B integration tests gated by ENABLE_WORKBENCH_TESTS=true, expected).pnpm lint for apollo → 1 unrelated pre-existing warning, no new issues.The change is additive (new optional kwarg, default None, behavior unchanged when omitted), so no blast-radius callers need updating.
Origin: cron-dd-errors / zen-cron-456487c9b63e Triggered by: Cron: Datadog Error Polling cron-dd-errors | Source: unknown Session: https://zen-api-production-4c98.up.railway.app/dashboard/#/chat/zen-cron-456487c9b63e
Based on git blame analysis of 2 file(s):
| Contributor | Contribution | Files |
|---|---|---|
| abir-taheer | 86% | 2 |
| Karan Vaidya | 11% | 2 |
| Zen | 3% | 2 |
| Dhawal Upadhyay | 1% | 1 |
Recommend Zen — made a recent edit today and can quickly validate the latest changes. Also add abir-taheer as a second reviewer since they’re the primary contributor and will have the deepest context on the code.
🤖 Based on git blame with recency weighting (recent edits count more).
:white_check_mark: All modified and coverable lines are covered by tests.
| Flag | Coverage Δ | |
|---|---|---|
| e2e-tests | 6.06% <ø> (ø) | |
| self-hosted-tests | 5.60% <ø> (ø) | |
| thermos-unit-tests | ? | |
| unit-tests | 58.73% <ø> (-0.01%) | :arrow_down: |
Flags with carried forward coverage won't be shown. Click here to find out more. see 266 files with indirect coverage changes
